fix(ext/node): pass normalized watchFile handler to StatWatcher (#22940)

Fixes #22939
denoland · Mar 16, 2024 · 9c5ddf7 · 9c5ddf7
1 parent ab67b4c
commit 9c5ddf7
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/ext/node/polyfills/_fs/_fs_watch.ts b/ext/node/polyfills/_fs/_fs_watch.ts
@@ -211,7 +211,7 @@ export function watchFile(
     statWatchers.set(watchPath, stat);
   }
 
-  stat.addListener("change", listener!);
+  stat.addListener("change", handler);
   return stat;
 }
 

diff --git a/tests/unit_node/_fs/_fs_watch_test.ts b/tests/unit_node/_fs/_fs_watch_test.ts
@@ -1,5 +1,5 @@
 // Copyright 2018-2024 the Deno authors. All rights reserved. MIT license.
-import { watch } from "node:fs";
+import { unwatchFile, watch, watchFile } from "node:fs";
 import { assertEquals } from "@std/assert/mod.ts";
 
 function wait(time: number) {
@@ -25,3 +25,16 @@ Deno.test({
     assertEquals(result.length >= 1, true);
   },
 });
+
+Deno.test({
+  name: "watching a file with options",
+  async fn() {
+    const file = Deno.makeTempFileSync();
+    watchFile(
+      file,
+      () => {},
+    );
+    await wait(100);
+    unwatchFile(file);
+  },
+});