From 06bd692e5c4a8f66960d3919e7087530b60c20dd Mon Sep 17 00:00:00 2001
From: Liam Murphy <43807659+Liamolucko@users.noreply.github.com>
Date: Tue, 26 Jan 2021 23:34:40 +1100
Subject: fix(std/node): Stop callbacks being called twice when callback throws
 error (#8867)

---
 std/node/_fs/_fs_unlink_test.ts | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

(limited to 'std/node/_fs/_fs_unlink_test.ts')

diff --git a/std/node/_fs/_fs_unlink_test.ts b/std/node/_fs/_fs_unlink_test.ts
index 46cde60b1..5021b1c38 100644
--- a/std/node/_fs/_fs_unlink_test.ts
+++ b/std/node/_fs/_fs_unlink_test.ts
@@ -1,5 +1,6 @@
 import { assertEquals, fail } from "../../testing/asserts.ts";
 import { existsSync } from "../../fs/mod.ts";
+import { assertCallbackErrorUncaught } from "../_utils.ts";
 import { unlink, unlinkSync } from "./_fs_unlink.ts";
 
 Deno.test({
@@ -12,8 +13,7 @@ Deno.test({
         resolve();
       });
     })
-      .then(() => assertEquals(existsSync(file), false))
-      .catch(() => fail())
+      .then(() => assertEquals(existsSync(file), false), () => fail())
       .finally(() => {
         if (existsSync(file)) Deno.removeSync(file);
       });
@@ -28,3 +28,12 @@ Deno.test({
     assertEquals(existsSync(file), false);
   },
 });
+
+Deno.test("[std/node/fs] unlink callback isn't called twice if error is thrown", async () => {
+  const tempFile = await Deno.makeTempFile();
+  const importUrl = new URL("./_fs_unlink.ts", import.meta.url);
+  await assertCallbackErrorUncaught({
+    prelude: `import { unlink } from ${JSON.stringify(importUrl)}`,
+    invocation: `unlink(${JSON.stringify(tempFile)}, `,
+  });
+});
-- 
cgit v1.2.3