Update test path matchers

test/e2e/routes.test.ts

@@ -18,22 +18,35 @@ describe("VS Code Routes", ["--disable-workspace-trust"], {}, async () => {
       const pathname = getMaybeProxiedPathname(url)
       expect(pathname).toBe(route)
 
-      // TODO@jsjoeio
-      // now that we are in a proper browser instead of scraping the HTML we
-      // could possibly intercept requests to make sure assets are loading from
-      // the right spot.
-      //
-      // Check that page loaded from correct route
-      const html = await codeServerPage.page.innerHTML("html")
+      // Check that assets are pointing to the right spot.  Some will be
+      // relative, without a leading dot (VS Code's assets).  Some will be
+      // relative with a leading dot (our assets).  Others will have been
+      // resolved against the origin.
+      const matchers: string[] = []
       switch (route) {
         case "/":
+          matchers.push(/src="\.\/manifest.json"/)
+          matchers.push(/src="\.\/_static\//)
+          matchers.push(/src="[a-z]+-[0-9a-z]+\/static\//)
+          matchers.push(/src="http:\/\/localhost:[0-9]+\/[a-z]+-[0-9a-z]+\/static\//)
         case "/vscode/":
-          expect(html).toMatch(/src="\.\/[a-z]+-[0-9a-z]+\/static\//)
+          matchers.push(/src="\.\/manifest.json"/)
+          matchers.push(/src="\.\/\.\.\/_static\//)
+          matchers.push(/src="[a-z]+-[0-9a-z]+\/static\//)
+          matchers.push(/src="http:\/\/localhost:[0-9]+\/vscode\/[a-z]+-[0-9a-z]+\/static\//)
           break
         case "/vscode":
-          expect(html).toMatch(/src="\.\/vscode\/[a-z]+-[0-9a-z]+\/static\//)
+          matchers.push(/src="\.\/_static\//)
+          matchers.push(/src="\.\/vscode\/manifest.json"/)
+          matchers.push(/src="vscode\/[a-z]+-[0-9a-z]+\/static\//)
+          matchers.push(/src="http:\/\/localhost:[0-9]+\/vscode\/[a-z]+-[0-9a-z]+\/static\//)
           break
       }
+      const elements = await codeServerPage.page.locator("[src]").all()
+      for (const element of elements) {
+        const src = await element.getAttribute("src")
+        expect(matchers.some((m) => m.test(src))).toBeTruthy()
+      }
     }
   })
 })