Skip to content

Commit

Permalink
Support / on windows for fixtures grep (#2049)
Browse files Browse the repository at this point in the history
Can now on windows use `/` in `testSubsetGrep.properties`

Fixes #1779

## Checklist

- [-] I have added
[tests](https://www.cursorless.org/docs/contributing/test-case-recorder/)
- [-] I have updated the
[docs](https://github.com/cursorless-dev/cursorless/tree/main/docs) and
[cheatsheet](https://github.com/cursorless-dev/cursorless/tree/main/cursorless-talon/src/cheatsheet)
- [-] I have not broken the cheatsheet
  • Loading branch information
AndreasArvidsson authored Dec 5, 2023
1 parent 6de055b commit a70c889
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion packages/common/src/testUtil/getFixturePaths.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,9 @@ export function getRecordedTestPaths() {
return walkFilesSync(directory)
.filter((p) => p.endsWith(".yml") || p.endsWith(".yaml"))
.map((p) => ({
name: path.relative(relativeDir, p.substring(0, p.lastIndexOf("."))),
name: path
.relative(relativeDir, p.substring(0, p.lastIndexOf(".")))
.replaceAll("\\", "/"),
path: p,
}));
}

0 comments on commit a70c889

Please sign in to comment.