docs: add comment

Author: jsjoeio

test/unit/proxy.test.ts

@@ -154,48 +154,20 @@ describe("proxy", () => {
   })
 })
 
+// NOTE@jsjoeio
+// Both this test suite and the one above it are very similar
+// The main difference is this one uses http and node-fetch
+// and specifically tests the proxy in isolation vs. using
+// the httpserver abstraction we've built.
+//
+// Leaving this as a separate test suite for now because
+// we may consider refactoring the httpserver abstraction
+// in the future.
+//
+// If you're writing a test specifically for code in
+// src/node/proxy.ts, you should probably add it to
+// this test suite.
 describe("proxy (standalone)", () => {
-  const testServer = new httpserver.HttpServer()
-  const proxyServer = proxy
-  let expressServer: express.Express
-
-  beforeAll(async () => {
-    await testServer.listen((req, res) => {
-      proxyServer.web(req, res, { target: `http://localhost:${testServer.port()}` })
-      expressServer(req, res)
-    })
-  })
-
-  afterAll(async () => {
-    await testServer.close()
-    await proxyServer.close()
-  })
-
-  beforeEach(() => {
-    expressServer = express.default()
-  })
-
-  it("should handle errors", async () => {
-    // Notes: I think I'm close
-    // but I think this code is testing that our fake expressServer
-    // is returning a 500
-    // not the proxy
-    expressServer.post("/error", (req, res) => {
-      throw new Error("BROKEN")
-    })
-
-    const resp = await testServer.fetch(`/error`, {
-      method: "post",
-      body: JSON.stringify("coder is the best"),
-      headers: {
-        "Content-Type": "application/json",
-      },
-    })
-    expect(resp.status).toBe(500)
-  })
-})
-
-describe("code-server proxy", () => {
   const PORT = 9003
   const PROXY_PORT = 8003
   const URL = `http://localhost:${PORT}`