Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test fixes for Python 3.13 #295

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Test fixes for Python 3.13 #295

wants to merge 2 commits into from

Conversation

mgorny
Copy link
Contributor

@mgorny mgorny commented Jul 12, 2024

Two changes to make the tests pass with Python 3.13:

  • update the regular expressions for changed help output
  • skip new tests that rely on exit_on_error=False semantic that isn't implemented in ConfigArgParse

Python 3.13 no longer repeats the placeholder for options with multiple
aliases in the help message.  For example, rather than:

    -c CONFIG_FILE, --config CONFIG_FILE

it now outputs:

    -c, --config CONFIG_FILE

Update the regular expressions to account for both possibilities.

Fixes bw2#294
Skip `exit_on_error*` tests from `test.test_argparse` to avoid test
failures on Python 3.13.  The `exit_on_error=False` semantics
is not supported by ConfigArgParse at the moment.
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