address comments

Author: camillobruni

tests/run-browser.mjs

@@ -50,7 +50,7 @@ const TESTS = [
     },
     {
         name: "Run Tag No Prefetch",
-        tags: ["all", "all", "main"],
+        tags: ["all", "main"],
         run() {
             return runEnd2EndTest("Run Tag No Prefetch",  { tag: "proxy", prefetchResources: "false" });
         }
@@ -138,9 +138,6 @@ async function runTests() {
 
     const testsToRun = TESTS.filter(test => test.tags.includes(suiteFilter));
 
-    console.log(testsToRun)
-    console.log(testsToRun.length)
-    console.log(suiteFilter)
     if (testsToRun.length === 0) {
         console.error(`No suite found for filter: ${suiteFilter}`);
         process.exit(1);

tests/run-shell.mjs

@@ -214,7 +214,10 @@ function runCLITest(name, shellBinary, ...args) {
 
 async function runShell(shellBinary, ...args) {
     const result = await sh(shellBinary, ...args);
-    if (result.stdoutString.includes("JetStream3 failed")) throw new Error("test failed");
+    // JSC does not set a non-0 exit status on async exceptions.
+    if (SHELL_NAME == "javascriptcore" && "JetStream3 failed") {
+        throw new Error("test failed");
+    }
 }
 
 setImmediate(runTests);