fix hidden specs on linux

photostructure · Nov 30, 2024 · 2c5c970 · 2c5c970
1 parent fa03e7a
commit 2c5c970
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 9 deletions.
diff --git a/src/__tests__/hidden.test.ts b/src/__tests__/hidden.test.ts
@@ -1,3 +1,4 @@
+import { jest } from "@jest/globals";
 import { execSync } from "node:child_process";
 import fs from "node:fs/promises";
 import path from "node:path";
@@ -304,7 +305,9 @@ describe("hidden file tests", () => {
       });
 
       // Test "all" method
-      const allResult = await setHidden(testFile, true, "all");
+      const allResult = await setHidden(dotPrefixResult.pathname, true, "all");
+
+      expect(allResult.pathname).toEqual(dotPrefixResult.pathname);
 
       if (isWindows) {
         expect(allResult.actions).toEqual({
@@ -313,7 +316,7 @@ describe("hidden file tests", () => {
         });
       } else {
         expect(allResult.actions).toEqual({
-          dotPrefix: true,
+          dotPrefix: false,
           systemFlag: process.platform === "darwin",
         });
       }

diff --git a/src/hidden.ts b/src/hidden.ts
@@ -163,7 +163,7 @@ export type HideMethod = "dotPrefix" | "systemFlag" | "all" | "auto";
 
 export async function setHidden(
   pathname: string,
-  hidden: boolean,
+  hide: boolean,
   method: HideMethod,
   nativeFn: NativeBindingsFn,
 ) {
@@ -174,21 +174,21 @@ export async function setHidden(
     systemFlag: false,
   };
 
-  let handled = false;
+  let hidden = false;
 
   if (LocalSupport.dotPrefix && ["auto", "all", "dotPrefix"].includes(method)) {
-    if (isPosixHidden(pathname) !== hidden) {
-      pathname = await setHiddenPosix(pathname, hidden);
+    if (isPosixHidden(pathname) !== hide) {
+      pathname = await setHiddenPosix(pathname, hide);
       actions.dotPrefix = true;
     }
-    handled = true;
+    hidden = true;
   }
 
   if (
     LocalSupport.systemFlag &&
-    (["all", "systemFlag"].includes(method) || (!handled && method === "auto"))
+    (["all", "systemFlag"].includes(method) || (!hidden && method === "auto"))
   ) {
-    await (await nativeFn()).setHidden(pathname, hidden);
+    await (await nativeFn()).setHidden(pathname, hide);
     actions.systemFlag = true;
   }