Split tests into individual test cases #67
Merged
Conversation
This file contains 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
facebook-github-bot
added
the
CLA Signed
sbrugman
force-pushed
the
individual-test-cases
branch
5 times, most recently
from
fb945ae to
7677b9a
Compare
sbrugman
changed the title
sbrugman
force-pushed
the
individual-test-cases
branch
from
7677b9a to
b99c016
Compare
|
Resolved merge conflict. I'll start adding some lint rules once this one it merged. |
Few pathlib cleanups
sbrugman
force-pushed
the
individual-test-cases
branch
from
d4845d7 to
955670b
Compare
kit1980
approved these changes
Sep 10, 2024
|
Looks good to me, thanks! Please resolve conflicts before merging. |
sbrugman
added a commit
that referenced
this pull request
Sep 12, 2024
This PR updates the test fixtures to have the `.py` extension for codemod output files. This is helpful for writing new tests (I'll be adding some for the new rules) - Codemod output is now stored as `.out.py` instead of `.py.out`. This marks the files as Python source to IDEs, linters etc. For test discovery, the input files were renamed to `.in.py` from `.py`. Follow-up PR for #67. Will rebase this one once that PR is merged.
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.
metafunc.parametrizecan be used to create more granular test cases by dynamically generating test fixtures (i.e. arguments). This in our case is helpful for debugging failing test cases, by knowing directly which of the paths are failing.Before
After
Changes: