test: skip symlink test on Windows (#19503)

fisker · mdjermanovic · web-flow · commit b9a5efa93704 · 2025-03-29T08:49:18.000+01:00
* test: allow skip symlink test on Windows

* test: skip on windows

* Update tests/lib/cli-engine/file-enumerator.js

Co-authored-by: Milos Djermanovic &lt;milos.djermanovic@gmail.com&gt;

* chore: restore files, I don't know what happened

* chore: restore files, I don't know what happened

* style: fix code style

---------

Co-authored-by: Milos Djermanovic &lt;milos.djermanovic@gmail.com&gt;
diff --git a/tests/lib/cli-engine/file-enumerator.js b/tests/lib/cli-engine/file-enumerator.js
@@ -763,64 +763,66 @@ describe("FileEnumerator", () => {
 			});
 		});
 
-		describe("if contains symbolic links", async () => {
-			const root = path.join(os.tmpdir(), "eslint/file-enumerator");
-			const files = {
-				"dir1/1.js": "",
-				"dir1/2.js": "",
-				"top-level.js": "",
-				".eslintrc.json": JSON.stringify({ rules: {} }),
-			};
-			const dir2 = path.join(root, "dir2");
-			const { prepare, cleanup } = createCustomTeardown({
-				cwd: root,
-				files,
-			});
+		if (process.platform !== "win32" || process.env.CI === "true") {
+			describe("if contains symbolic links", async () => {
+				const root = path.join(os.tmpdir(), "eslint/file-enumerator");
+				const files = {
+					"dir1/1.js": "",
+					"dir1/2.js": "",
+					"top-level.js": "",
+					".eslintrc.json": JSON.stringify({ rules: {} }),
+				};
+				const dir2 = path.join(root, "dir2");
+				const { prepare, cleanup } = createCustomTeardown({
+					cwd: root,
+					files,
+				});
+
+				beforeEach(async () => {
+					await prepare();
+					fs.mkdirSync(dir2);
+					fs.symlinkSync(
+						path.join(root, "top-level.js"),
+						path.join(dir2, "top.js"),
+						"file",
+					);
+					fs.symlinkSync(
+						path.join(root, "dir1"),
+						path.join(dir2, "nested"),
+						"dir",
+					);
+				});
 
-			beforeEach(async () => {
-				await prepare();
-				fs.mkdirSync(dir2);
-				fs.symlinkSync(
-					path.join(root, "top-level.js"),
-					path.join(dir2, "top.js"),
-					"file",
-				);
-				fs.symlinkSync(
-					path.join(root, "dir1"),
-					path.join(dir2, "nested"),
-					"dir",
-				);
-			});
+				afterEach(cleanup);
 
-			afterEach(cleanup);
+				it("should resolve", () => {
+					const enumerator = new FileEnumerator({ cwd: root });
+					const list = Array.from(
+						enumerator.iterateFiles(["dir2/**/*.js"]),
+					).map(({ filePath }) => filePath);
 
-			it("should resolve", () => {
-				const enumerator = new FileEnumerator({ cwd: root });
-				const list = Array.from(
-					enumerator.iterateFiles(["dir2/**/*.js"]),
-				).map(({ filePath }) => filePath);
-
-				assert.deepStrictEqual(list, [
-					path.join(dir2, "nested", "1.js"),
-					path.join(dir2, "nested", "2.js"),
-					path.join(dir2, "top.js"),
-				]);
-			});
+					assert.deepStrictEqual(list, [
+						path.join(dir2, "nested", "1.js"),
+						path.join(dir2, "nested", "2.js"),
+						path.join(dir2, "top.js"),
+					]);
+				});
 
-			it("should ignore broken links", () => {
-				fs.unlinkSync(path.join(root, "top-level.js"));
+				it("should ignore broken links", () => {
+					fs.unlinkSync(path.join(root, "top-level.js"));
 
-				const enumerator = new FileEnumerator({ cwd: root });
-				const list = Array.from(
-					enumerator.iterateFiles(["dir2/**/*.js"]),
-				).map(({ filePath }) => filePath);
+					const enumerator = new FileEnumerator({ cwd: root });
+					const list = Array.from(
+						enumerator.iterateFiles(["dir2/**/*.js"]),
+					).map(({ filePath }) => filePath);
 
-				assert.deepStrictEqual(list, [
-					path.join(dir2, "nested", "1.js"),
-					path.join(dir2, "nested", "2.js"),
-				]);
+					assert.deepStrictEqual(list, [
+						path.join(dir2, "nested", "1.js"),
+						path.join(dir2, "nested", "2.js"),
+					]);
+				});
 			});
-		});
+		}
 	});
 
 	// https://github.com/eslint/eslint/issues/13789
