fix: lower max batch size from 2000 to 200 - #507
Merged
Conversation
TMS-196 lowers the bulk-results API cap to 200 results per request. - BatchConfig.set_size now rejects a size above 200; the message wording also matches the check, which previously said "less than 2000" while testing for "> 2000" - Both TestOps reporters clamp with min(200, ...) instead of min(2000, ...) - pytest and robotframework UPGRADE docs state the 1-200 range Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
gibiw
approved these changes
Aug 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Lowers the batch-size ceiling in
qase-python-commonsfrom 2000 to 200.What changed
qase-python-commons/src/qase/commons/models/config/batch.py—set_sizerejects a size above 200. The old message said "less than 2000" while the check tested> 2000; the new wording matches the check.qase-python-commons/src/qase/commons/reporters/testops.pyandtestops_multi.py— clamp withmin(200, ...).qase-pytest/docs/UPGRADE.mdandqase-robotframework/docs/UPGRADE.md— state the 1-200 range.The default stays 200, so no user on defaults is affected.
Tests
qase-python-commonssuite: 243 passed, 4 skipped.Why
TMS-196 lowers the bulk-results cap from 2000 to 200 results per request. Very large batches arrive as a single spike in the result-processing queue and delay processing for every customer, including runs that report a handful of results at a time. Fewer than 1% of batches exceed 200 today.
This PR removes the stale 2000 from this repository. Server-side enforcement lands separately in the
appandspecsrepositories.Flags for the reviewer
maxItems: 2000constraint and the "more than 2,000 results" endpoint description are generated from the v1 OpenAPI spec into every client (qase-api-client,qase-php-client, and the vendored clients in this and the other language repos). They change by regenerating from the spec, not by hand. Noteqase-javascriptcarries two generated trees (qase-api-client/andqaseio/src/generated/), so a partial regeneration leaves one stale.Full cross-repo inventory of every mention, including an org-wide sweep of all 163
qase-tmsrepositories, is available on request.