Skip to content

[FIX]: COLUMN keeps first and last part of description - #786

Merged
authorjapps merged 2 commits into
masterfrom
txt_report_length
Aug 31, 2026
Merged

[FIX]: COLUMN keeps first and last part of description#786
authorjapps merged 2 commits into
masterfrom
txt_report_length

Conversation

@nirmalchandra

@nirmalchandra nirmalchandra commented Jul 25, 2026

Copy link
Copy Markdown
Collaborator

Fixed Which Issue?

BEFORE FIX:

= SCENARIO's only cut marker, ending lost

+--------------------------------------------------+---------------------------+------------------------+----------+------------+
| SCENARIO                                         | STEP                      | METHOD                 | RESULT   | DELAY (ms) |
+--------------------------------------------------+---------------------------+------------------------+----------+------------+
| GIVEN the very long scenario name that exceeds.. | get_user                  | GET                    | PASSED ✅|         42 |
| Verify search returns results                    | verify_search_results_are█ | GET                    | PASSED ✅|         88 |
| Update order shipping address                    | patch_address             | PATCH_WITH_CUSTOM_HEAD█ | FAILED ❌|        120 |
| Delete order cancels pending shipment and refu.. | check_order_status_update█ | DELETE_WITH_RETRY_AND_█ | PASSED ✅|         76 |
+-------------------------------------------------------------------------------------------------------------------------------+
| Total: 4  |  PASSED: 3  |  FAILED: 1  |  Min delay: 42 ms  |  Max delay: 120 ms                                               |
+-------------------------------------------------------------------------------------------------------------------------------+

AFTER FIX:

= SCENARIO's starting and ending preserved

+--------------------------------------------------+---------------------------+------------------------+----------+------------+
| SCENARIO                                         | STEP                      | METHOD                 | RESULT   | DELAY (ms) |
+--------------------------------------------------+---------------------------+------------------------+----------+------------+
| GIVEN the very long sce...imit set for the table | get_user                  | GET                    | PASSED ✅|         42 |
| Verify search returns results                    | verify_sear...vance_score | GET                    | PASSED ✅|         88 |
| Update order shipping address                    | patch_address             | PATCH_WITH...LONG_NAME | FAILED ❌|        120 |
| Delete order cancels pe...s customer immediately | check_order...elled_in_db | DELETE_WIT...D_BACKOFF | PASSED ✅|         76 |
+-------------------------------------------------------------------------------------------------------------------------------+
| Total: 4  |  PASSED: 3  |  FAILED: 1  |  Min delay: 42 ms  |  Max delay: 120 ms                                               |
+-------------------------------------------------------------------------------------------------------------------------------+

PR Branch
#ADD LINK TO THE PR BRANCH

Motivation and Context

Checklist:

  • 1. New Unit tests were added

    • 1.1 Covered in existing Unit tests
  • 2. Integration tests were added

    • 2.1 Covered in existing Integration tests
  • 3. Test names are meaningful

  • 3.1 Feature manually tested and outcome is successful

  • 4. PR doesn't break any of the earlier features for end users

    • 4.1 WARNING! This might break one or more earlier earlier features, hence left a comment tagging all reviewrs
  • 5. PR doesn't break the HTML report features directly

    • 5.1 Yes! I've manually run it locally and seen the HTML reports are generated perfectly fine
    • 5.2 Yes! I've opened the generated HTML reports from the /target folder and they look fine
  • 6. PR doesn't break any HTML report features indirectly

    • 6.1 I have not added or amended any dependencies in this PR
    • 6.2 I have double checked, the new dependency added or removed has not affected the report generation indirectly
    • 6.3 Yes! I've seen the Sample report screenshots here, and HTML report of the current PR looks simillar.
  • 7. Branch build passed in CI

  • 8. No 'package.*' in the imports

  • 9. Relevant DOcumentation page added or updated with clear instructions and examples for the end user

    • 9.1 Not applicable. This was only a code refactor change, no functional or behaviourial changes were introduced
  • 10. Http test added to http-testing-examples module(if applicable) ?

    • 10.1 Not applicable. The changes did not affect HTTP automation flow
  • 11. Kafka test added to kafka-testing-examples module(if applicable) ?

    • 11.1 Not applicable. The changes did not affect Kafka automation flow

@authorjapps
authorjapps merged commit c97d238 into master Aug 31, 2026
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants