Merge pull request #4306 from cdr/jsjoeio-fix-broken-tests

fix: revert CI change + fix broken tests

Author: jsjoeio

.github/workflows/ci.yaml

@@ -54,6 +54,10 @@ jobs:
         run: yarn lint
         if: success()
 
+      - name: Run code-server unit tests
+        run: yarn test:unit
+        if: success()
+
       - name: Upload coverage report to Codecov
         run: yarn coverage
         if: success()
@@ -404,9 +408,6 @@ jobs:
           rm -r node_modules/playwright
           yarn install --check-files
 
-      - name: Run end-to-end tests
-        run: yarn test:unit
-
       - name: Run end-to-end tests
         run: yarn test:e2e
 

src/node/routes/index.ts

@@ -162,7 +162,7 @@ export const register = async (
   }
 
   server.on("close", () => {
-    vscode.vscodeServer.close()
+    vscode?.vscodeServer.close()
   })
 
   app.use(() => {

test/unit/node/routes/static.test.ts

@@ -7,7 +7,7 @@ import * as integration from "../../../utils/integration"
 
 const NOT_FOUND = {
   code: 404,
-  message: "not found",
+  message: "Not Found",
 }
 
 describe("/_static", () => {