Skip to content

Commit

Permalink
Disable new tool tests on Windows (#550)
Browse files Browse the repository at this point in the history
Because of weirdness of node shims, we don't support this at the moment.
Disable these tests.
  • Loading branch information
TylerJang27 authored Nov 6, 2023
1 parent eb7d34a commit 84323c3
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 0 deletions.
4 changes: 4 additions & 0 deletions tools/gulp/gulp.test.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import { makeToolTestConfig, toolTest } from "tests";
import { skipOS } from "tests/utils";

toolTest({
toolName: "gulp",
Expand All @@ -9,4 +10,7 @@ toolTest({
expectedOut: "Local version: 4.0.2",
}),
],
// On Windows, the shim is gulp.cmd, and we don't support platform-specific shims yet.
// To use on Windows, override the shim with gulp.cmd.
skipTestIf: skipOS(["win32"]),
});
4 changes: 4 additions & 0 deletions tools/tailwindcss/tailwindcss.test.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import { makeToolTestConfig, toolTest } from "tests";
import { skipOS } from "tests/utils";

toolTest({
toolName: "tailwindcss",
Expand All @@ -9,4 +10,7 @@ toolTest({
expectedOut: "tailwindcss v3.3.5",
}),
],
// On Windows, the shim is tailwindcss.cmd, and we don't support platform-specific shims yet.
// To use on Windows, override the shim with tailwindcss.cmd.
skipTestIf: skipOS(["win32"]),
});
4 changes: 4 additions & 0 deletions tools/webpack/webpack.test.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import { makeToolTestConfig, toolTest } from "tests";
import { skipOS } from "tests/utils";

toolTest({
toolName: "webpack",
Expand All @@ -9,4 +10,7 @@ toolTest({
expectedOut: "Binaries:",
}),
],
// On Windows, the shim is webpack.cmd, and we don't support platform-specific shims yet.
// To use on Windows, override the shim with webpack.cmd.
skipTestIf: skipOS(["win32"]),
});

0 comments on commit 84323c3

Please sign in to comment.