Skip to content

feat(qase-pytest: commons): fix str env var comparing to int - #423

Closed
meyafex wants to merge 2 commits into
qase-tms:mainfrom
meyafex:yanf/fix-batch-env
Closed

feat(qase-pytest: commons): fix str env var comparing to int#423
meyafex wants to merge 2 commits into
qase-tms:mainfrom
meyafex:yanf/fix-batch-env

Conversation

@meyafex

@meyafex meyafex commented Nov 9, 2025

Copy link
Copy Markdown

Note

BatchConfig.set_size now accepts strings (trimmed and converted to int) and size is typed as non-optional int defaulting to 200.

  • Commons / Config:
    • BatchConfig (qase-python-commons/src/qase/commons/models/config/batch.py):
      • size annotated as int (non-optional), initialized to 200 in __init__.
      • set_size now accepts Union[int, str]; trims and converts string inputs to int.
      • Validation logic updated: assigns size when size_int > 2000 or size_int == 0; raises ValueError("Batch size should be numeric value") for non-numeric input.

Written by Cursor Bugbot for commit 4019d56. This will update automatically on new commits. Configure here.

fixes #422

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Comment thread qase-python-commons/src/qase/commons/models/config/batch.py
Comment thread qase-python-commons/src/qase/commons/models/config/batch.py
Comment thread qase-python-commons/src/qase/commons/models/config/batch.py
@gibiw

gibiw commented Nov 10, 2025

Copy link
Copy Markdown
Contributor

The problem has been fixed as part of the PR #425

@gibiw gibiw closed this Nov 10, 2025
@meyafex

meyafex commented Nov 10, 2025

Copy link
Copy Markdown
Author

thanks

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.

qase-pytest: Failed to load config from env vars {e}

2 participants