test: remove unused skip logic in E2E tests

Author: zxch3n

packages/loro-websocket/tests/e2e-elo.test.ts

@@ -14,8 +14,6 @@ import {
 } from "loro-protocol";
 import { EloAdaptor } from "loro-adaptors/loro";
 
-let skip = false;
-
 // Make WebSocket available globally for the client
 Object.defineProperty(globalThis, "WebSocket", {
   value: WebSocket,
@@ -291,13 +289,10 @@ describe("E2E: %ELO decrypt failure and unknown key handling", () => {
   });
 
   afterAll(async () => {
-    if (!skip && server) {
-      await server.stop();
-    }
+    await server.stop();
   }, 15000);
 
   it("client reports decrypt error for unknown keyId", async () => {
-    if (skip) return;
     const client1 = new LoroWebsocketClient({ url: `ws://localhost:${port}` });
     const client2 = new LoroWebsocketClient({ url: `ws://localhost:${port}` });
     await Promise.all([client1.waitConnected(), client2.waitConnected()]);
@@ -371,7 +366,7 @@ async function waitForJoinOk(ws: WebSocket): Promise<void> {
           clearTimeout(t);
           resolve();
         }
-      } catch {}
+      } catch { }
     });
   });
 }
@@ -388,7 +383,7 @@ async function waitForAck(ws: WebSocket): Promise<Ack> {
           clearTimeout(t);
           resolve(msg);
         }
-      } catch {}
+      } catch { }
     });
   });
 }

packages/loro-websocket/tests/e2e-update-error.test.ts

@@ -23,7 +23,6 @@ Object.defineProperty(globalThis, "WebSocket", {
 describe("E2E: onUpdateError", () => {
   let server: WebSocketServer;
   let port: number;
-  let skip = false;
   let lastUpdates: Uint8Array[] = [];
 
   beforeAll(async () => {
@@ -39,13 +38,11 @@ describe("E2E: onUpdateError", () => {
         });
       });
     } catch (e) {
-      skip = true;
       console.warn("Skipping onUpdateError e2e: cannot start ws server", e);
     }
   });
 
   afterAll(async () => {
-    if (skip || !server) return;
     for (const client of server.clients) {
       try {
         client.terminate();
@@ -59,8 +56,6 @@ describe("E2E: onUpdateError", () => {
   });
 
   it("invokes adaptor onUpdateError with original batch", async () => {
-    if (skip) return;
-
     const errors: Array<{ updates: Uint8Array[]; code: number; reason?: string }> = [];
     const adaptor = new LoroAdaptor(undefined, {
       onUpdateError: (updates, code, reason) => {