You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The reason will be displayed to describe this comment to others. Learn more.
Looks good. BTW, can you remove test_parameters_required_after_depr_star (line 1471)? I introduced an equal named, but slightly better test in #95151). Makes sense to bake it into this PR, rather than creating a new PR specifically for that removal.
Looks good. BTW, can you remove test_parameters_required_after_depr_star (line 1471)? I introduced an equal named, but slightly better test in #95151). Makes sense to bake it into this PR, rather than creating a new PR specifically for that removal.
Hmm, the new test that you added fails if I remove test_parameters_required_after_depr_star on line 1471. The new test method that you added is currently never run, because it is immediately overridden by the old test method later on in the class definition before unittest even tries to run any of the tests.
We really need to start running pyflakes on Tools/clinic/ and test_clinic.py in CI :)
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
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.
And fix a minor bug I discovered along the way.
This gets us up to 95% coverage.