Skip to content

Conversation

@jdalton
Copy link
Contributor

@jdalton jdalton commented Nov 20, 2025

Fixes test failure in cmd-fix.integration.test.mts where git-based fixture cleanup was failing when no changes existed to revert.

Replaced git checkout/clean approach with tmpdir-based isolation helpers following the pattern used in optimize tests.


Note

Replaces git-based fixture cleanup in cmd-fix.integration.test.mts with tmpdir-based utilities and removes related lifecycle hooks.

  • Tests (src/commands/fix/cmd-fix.integration.test.mts)
    • Replace git-based fixture reset with tmpdir-based helpers: copyDirectory and createTempFixture.
    • Remove beforeAll/afterEach/afterAll cleanup hooks tied to git.
    • Keep existing flag, help, output, and error-handling test cases unchanged functionally.

Written by Cursor Bugbot for commit 823b345. Configure here.

Replaced git checkout/clean-based fixture cleanup with tmpdir-based
isolation helpers. The git-based cleanup was failing when no changes
existed to revert.
@jdalton jdalton merged commit 7b44151 into v1.x Nov 20, 2025
8 checks passed
@jdalton jdalton deleted the fix/test-fixture-cleanup branch November 20, 2025 16:31
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.

3 participants