Repository navigation
test: document rename-only Git path normalization - #67
Open
carlitogwapito wants to merge 1 commit into
Open
carlitogwapito wants to merge 1 commit into
carlitogwapito wants to merge 1 commit into
Conversation
This branch has not been deployed
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
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.
This adds a small, self-contained regression module for #43. No parser changes.
The two rename-only cases are marked
unittest.expectedFailurebecause the bug is still present on currentmain. One uses ordinary names; the other uses reala/andb/directories, so a future fix can't accidentally strip those as well. A passing unified-diff control checks that one outer prefix is removed while the real directories remain.I saw that #66 was closed during the author's cleanup. This doesn't resubmit that parser change; it's just an executable test case for the still-open issue.
Run from the repository root:
python -m unittest tests.test_rename_onlyFocused validation on Python 3.14.4: 3 tests run, 1 passed, 2 expected failures. Both expected failures are the wrong path pairs described above, not import/setup failures. Ruff checks and compilation passed. I didn't run the full suite, Black, hosted CI, or other Python/platform combinations.
When #43 is fixed, the two
expectedFailuredecorators can be removed once the assertions pass. I'd keep the nested-directory case to guard against stripping a real path component twice.