Small changes based on code review

agama-project · May 27, 2024 · e9de732 · e9de732
1 parent 12ffebd
commit e9de732
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/web/src/client/http.js b/web/src/client/http.js
@@ -86,9 +86,9 @@ class WSClient {
   buildClient() {
     const client = new WebSocket(this.url);
     client.onopen = () => {
-      console.log("Connected websocket");
+      console.log("Websocket connected");
       this.reconnectAttempts = 0;
-      if (this.timeout !== undefined) clearTimeout(this.timeout);
+      clearTimeout(this.timeout);
 
       return this.dispatchOpenEvent();
     };
@@ -98,7 +98,7 @@ class WSClient {
     };
 
     client.onclose = () => {
-      console.log(`Closed WebSocket`);
+      console.log(`WebSocket closed`);
       this.dispatchCloseEvent();
       this.timeout = setTimeout(() => this.connect(this.reconnectAttempts + 1), ATTEMPT_INTERVAL);
     };

diff --git a/web/src/context/installer.test.jsx b/web/src/context/installer.test.jsx
@@ -27,7 +27,7 @@ import { InstallerClientProvider, useInstallerClientStatus } from "./installer";
 
 jest.mock("~/client");
 
-let onDisconnectFn = jest.fn();
+const onDisconnectFn = jest.fn();
 const isConnectedFn = jest.fn();
 
 // Helper component to check the client status.