Skip to content

TRCLI-266: Attachments not 'attached' to the test in the test run, for AutomationID (code first) tests#423

Open
acuanico-tr-galt wants to merge 2 commits into
release/1.14.3from
fix/TRCLI-266
Open

TRCLI-266: Attachments not 'attached' to the test in the test run, for AutomationID (code first) tests#423
acuanico-tr-galt wants to merge 2 commits into
release/1.14.3from
fix/TRCLI-266

Conversation

@acuanico-tr-galt
Copy link
Copy Markdown
Collaborator

@acuanico-tr-galt acuanico-tr-galt commented May 19, 2026

Issue being resolved: #421

Solution description

The as_completed() function returns futures in the order they complete, not in submission order, to resolve this we
store responses by request ID and reconstruct in original order.

Changes

Modified handle_futures() in api_request_handler.py to:

  • Store responses in a dictionary keyed by request body id()
  • Reconstruct responses in original request order before returning
  • This ensures zip() in result_handler.py pairs requests with their correct responses

We also optimized attachments uploading to use Threadpool executor, also updated progress tracking displayed in the standard output.

Potential impacts

None. Changes introduced are backwards compatible.

Steps to test

  1. Upload more than 500 test cases (preferably with attachments)
  2. There should be no mapping error once uploaded to TestRail.

PR Tasks

  • PR reference added to issue
  • README updated
  • Unit tests added/updated

…, also updated uploading attachments to use threadpool executor to parallelize and speed up uploading.
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.

1 participant