fix double `resolve()` in test (#1707)

also clean up `reject()` values

1 files changed, 6 insertions, 6 deletions

diff --git a/test/bun.js/websocket-server.test.ts b/test/bun.js/websocket-server.test.ts
index e25429b05..949b1fc6e 100644
--- a/test/bun.js/websocket-server.test.ts
+++ b/test/bun.js/websocket-server.test.ts
@@ -178,7 +178,7 @@ describe("websocket server", () => {
               headers: 1238 as any,
             })
           ) {
-            reject();
+            reject(new Error("should not upgrade"));
             return;
           }
         } catch (e) {
@@ -186,7 +186,7 @@ describe("websocket server", () => {
           return new Response("success");
         }
 
-        reject();
+        reject(new Error("should throw error"));
         return new Response("noooooo hello world");
       },
     });
@@ -446,7 +446,7 @@ describe("websocket server", () => {
         },
         fetch(req, server) {
           if (server.upgrade(req)) {
-            reject("should not upgrade");
+            reject(new Error("should not upgrade"));
           }
 
           return new Response("success");
@@ -468,12 +468,12 @@ describe("websocket server", () => {
         fetch(req, server) {
           try {
             server.upgrade(req);
-            reject("should not upgrade");
+            reject(new Error("should not upgrade"));
           } catch (e) {
-            resolve();
+            return new Response("success");
           }
 
-          return new Response("success");
+          return new Response("fail");
         },
       });