Skip to content

Commit 3f4517c

Browse files
Limit link-to-link test to Linux systems
Because on macOS we don't have a predictable binary to point to that we know Shescape knows (note that `/bin/sh` is not a symlink but not sufficient for identifying the shell).
1 parent 6a01a52 commit 3f4517c

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

test/_constants.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ export const echoScript = "test/_echo.js";
99

1010
export const isMacOS = os.platform() === "darwin";
1111
export const isWindows = os.platform() === "win32";
12+
export const isLinux = !isMacOS && !isWindows;
1213

1314
/* Illegal arguments */
1415
export const illegalArguments = [

test/e2e/_common.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ export function getTestShells() {
6161
}
6262
}
6363

64-
if (!constants.isWindows) {
64+
if (constants.isLinux) {
6565
const doubleLinkedShell = path.resolve(temp, "double-link", "link-to-link");
6666
systemShells.push(doubleLinkedShell);
6767
}

0 commit comments

Comments
 (0)