Skip to content

fix: resolve statistics keys with the simulated path instead of the host path - #1162

Merged
vbreuss merged 1 commit into
mainfrom
fix/path-statistics-host-cwd
Oct 29, 2026
Merged

vbreuss merged 1 commit into
mainfrom
fix/path-statistics-host-cwd

Conversation

@vbreuss

@vbreuss vbreuss commented Oct 29, 2026

Copy link
Copy Markdown
Member

When simulating another operating system, the host Path produced wrong keys and fell back to the process's current directory, which made watcher tests fail on macOS whenever a real file system test had left it pointing at a deleted directory. The directory cleaner now also resets the current directory when a test moved it into a subdirectory, so it no longer leaves the process in a deleted directory.

@vbreuss vbreuss self-assigned this Oct 29, 2026
Copilot AI balanced review requested due to automatic review settings October 29, 2026 22:22
@vbreuss vbreuss added the bug Something isn't working label Oct 29, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

…ost path

When simulating another operating system, the host Path produced wrong keys and fell back to the process's current directory, which made watcher tests fail on macOS whenever a real file system test had left it pointing at a deleted directory. The directory cleaner now also resets the current directory when a test moved it into a subdirectory, so it no longer leaves the process in a deleted directory.
@vbreuss
vbreuss force-pushed the fix/path-statistics-host-cwd branch from 63f9774 to 55edb7e Compare October 29, 2026 22:36
@vbreuss
vbreuss enabled auto-merge (squash) October 29, 2026 22:48
@sonarqubecloud

Copy link
Copy Markdown

@vbreuss
vbreuss merged commit 56bfe05 into main Oct 29, 2026
15 checks passed
@vbreuss
vbreuss deleted the fix/path-statistics-host-cwd branch October 29, 2026 23:20
@github-actions

Copy link
Copy Markdown

This is addressed in release v7.2.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working state: released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants