-
-
Notifications
You must be signed in to change notification settings - Fork 30.1k
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
gh-106368: Increase Argument Clinic CLI test coverage #107156
gh-106368: Increase Argument Clinic CLI test coverage #107156
Conversation
Instead of hacking into the Clinic class, use the Argument Clinic tool to run the ClinicWholeFileTest test suite.
test_external: run clinic on clinic.test.c test_no_change: check that dest is left unchanged if content does not change
FWIW; this messes up test coverage locally; that's solvable (IIRC), though. |
The coveragerc file that's included in the sources disables coverage for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Better error messages :)
Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
Thanks for the review, Nikita! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! 👍
Thanks @erlend-aasland for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12. |
Thanks @erlend-aasland for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11. |
GH-107189 is a backport of this pull request to the 3.12 branch. |
GH-107190 is a backport of this pull request to the 3.11 branch. |
…ythonGH-107156) (python#107190) Instead of hacking into the Clinic class, use the Argument Clinic tool to run the ClinicExternalTest test suite. (cherry picked from commit 83a2837) Co-authored-by: Erlend E. Aasland <erlend@python.org> Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
…07156) Instead of hacking into the Clinic class, use the Argument Clinic tool to run the ClinicExternalTest test suite. Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
Instead of hacking into the Clinic class, use the Argument Clinic tool
to run the
ClinicWholeFileTestClinicExternalTest test suite.TODO:
test_external
, use-o
to redirect the output, and simply compare the redirected output withclinic.test.c
test_external
)-f
is given