From 0031c1acc04f62bed0d3953b44785828365b7c85 Mon Sep 17 00:00:00 2001
From: Robert <robertbrignull@github.com>
Date: Wed, 21 Jun 2023 16:02:57 +0100
Subject: [PATCH] Throw error instead of using fail method

---
 extensions/ql-vscode/test/unit-tests/command-lint.test.ts   | 2 +-
 .../ql-vscode/test/unit-tests/pure/helpers-pure.test.ts     | 2 +-
 .../cli-integration/debugger/debug-controller.ts            | 6 +-----
 3 files changed, 3 insertions(+), 7 deletions(-)

diff --git a/extensions/ql-vscode/test/unit-tests/command-lint.test.ts b/extensions/ql-vscode/test/unit-tests/command-lint.test.ts
index 41cec28bc..66709e6b0 100644
--- a/extensions/ql-vscode/test/unit-tests/command-lint.test.ts
+++ b/extensions/ql-vscode/test/unit-tests/command-lint.test.ts
@@ -49,7 +49,7 @@ describe("commands declared in package.json", () => {
       expect(title).toBeDefined();
       commandTitles[command] = title!;
     } else {
-      fail(`Unexpected command name ${command}`);
+      throw new Error(`Unexpected command name ${command}`);
     }
   });
 
diff --git a/extensions/ql-vscode/test/unit-tests/pure/helpers-pure.test.ts b/extensions/ql-vscode/test/unit-tests/pure/helpers-pure.test.ts
index 8c7bf2164..0c6aedf62 100644
--- a/extensions/ql-vscode/test/unit-tests/pure/helpers-pure.test.ts
+++ b/extensions/ql-vscode/test/unit-tests/pure/helpers-pure.test.ts
@@ -13,7 +13,7 @@ describe("helpers-pure", () => {
 
     try {
       await asyncFilter([1, 2, 3], rejects);
-      fail("Should have thrown");
+      throw new Error("Should have thrown");
     } catch (e) {
       expect(getErrorMessage(e)).toBe("opps");
     }
diff --git a/extensions/ql-vscode/test/vscode-tests/cli-integration/debugger/debug-controller.ts b/extensions/ql-vscode/test/vscode-tests/cli-integration/debugger/debug-controller.ts
index ccad6a896..7e4f40456 100644
--- a/extensions/ql-vscode/test/vscode-tests/cli-integration/debugger/debug-controller.ts
+++ b/extensions/ql-vscode/test/vscode-tests/cli-integration/debugger/debug-controller.ts
@@ -310,11 +310,7 @@ export class DebugController
    */
   private async nextEvent(): Promise<AnyDebugEvent> {
     if (this.resolver !== undefined) {
-      const error = new Error(
-        "Attempt to wait for multiple debugger events at once.",
-      );
-      fail(error);
-      throw error;
+      throw new Error("Attempt to wait for multiple debugger events at once.");
     } else {
       if (this.nextEventIndex < this.eventQueue.length) {
         // No need to wait.