Remove undefined fail call from tests (#1202)

Removes `fail` calls from tests, to use plain promise rejections instead.
safe-global · Feb 26, 2024 · 08181bf · 08181bf
1 parent b701175
commit 08181bf
Show file tree

Hide file tree

Showing 5 changed files with 83 additions and 63 deletions.
diff --git a/src/routes/owners/owners.controller.spec.ts b/src/routes/owners/owners.controller.spec.ts
@@ -238,7 +238,7 @@ describe('Owners Controller (Unit)', () => {
           }
 
           default: {
-            fail(`Unexpected URL: ${url}`);
+            return Promise.reject(`No matching rule for url: ${url}`);
           }
         }
       });
@@ -265,7 +265,7 @@ describe('Owners Controller (Unit)', () => {
           );
           return Promise.reject(error);
         }
-        fail(`Unexpected URL: ${url}`);
+        return Promise.reject(`No matching rule for url: ${url}`);
       });
 
       await request(app.getHttpServer())
@@ -322,7 +322,7 @@ describe('Owners Controller (Unit)', () => {
           }
 
           default: {
-            fail(`Unexpected URL: ${url}`);
+            return Promise.reject(`No matching rule for url: ${url}`);
           }
         }
       });