finalize tests

fastly · Jan 25, 2025 · 0c46cbf · 0c46cbf
1 parent 72887e6
commit 0c46cbf
Show file tree

Hide file tree

Showing 5 changed files with 263 additions and 132 deletions.
diff --git a/integration-tests/js-compute/fixtures/app/src/manual-framing-headers.js b/integration-tests/js-compute/fixtures/app/src/manual-framing-headers.js
@@ -238,25 +238,26 @@ routes.set('/override-content-length/response/method/true', async () => {
   return responseMethod(true);
 });
 
-// TODO: uncomment when we have an implementation of Response.prototype.clone
-// async function responseClone(setManualFramingHeaders) {
-//     let response = new Response("meow", {
-//         backend: "httpme",
-//         method: "POST",
-//         body: "meow",
-//         setManualFramingHeaders,
-//         headers: {
-//             "transfer-encoding": "chunked"
-//         }
-//     });
-//     response = response.clone();
-//     return response;
-// }
+async function responseClone(setManualFramingHeaders) {
+  let response = new Response('meow', {
+    backend: 'httpme',
+    method: 'POST',
+    body: 'meow',
+    setManualFramingHeaders,
+    headers: {
+      'transfer-encoding': 'chunked',
+    },
+  });
+  response = response.clone();
+  return response;
+}
 
-// routes.set("/override-content-length/response/clone/true", async () => {
-//     return responseClone(true);
-// });
+// TODO: implement Response.clone
+routes.set('/override-content-length/response/clone/true', async () => {
+  return responseClone(true);
+});
 
-// routes.set("/override-content-length/response/clone/false", async () => {
-//     return responseClone(false);
-// });
+// TODO: implement Response.clone
+routes.set('/override-content-length/response/clone/false', async () => {
+  return responseClone(false);
+});
diff --git a/integration-tests/js-compute/fixtures/app/tests.json b/integration-tests/js-compute/fixtures/app/tests.json
@@ -1256,7 +1256,6 @@
       "body": "hello world!"
     }
   },
-
   "GET /tee/error": {
     "downstream_request": {
       "method": "GET",
@@ -1268,7 +1267,6 @@
       "body": "Error: TypeError"
     }
   },
-
   "GET /override-content-length/request/init/object-literal/true": {
     "environments": ["compute"],
     "downstream_response": {
@@ -1283,7 +1281,6 @@
       "body": "ok"
     }
   },
-
   "GET /override-content-length/request/clone/true": {
     "environments": ["compute"],
     "downstream_response": {
@@ -1298,7 +1295,6 @@
       "body": "ok"
     }
   },
-
   "GET /override-content-length/fetch/init/object-literal/true": {
     "environments": ["compute"],
     "downstream_response": {
@@ -1313,7 +1309,14 @@
       "body": "ok"
     }
   },
-
+  "/override-content-length/response/clone/true": {
+    "skip": true,
+    "environments": ["compute"]
+  },
+  "/override-content-length/response/clone/false": {
+    "skip": true,
+    "environments": ["compute"]
+  },
   "GET /override-content-length/response/init/object-literal/true": {
     "environments": ["compute"],
     "downstream_response": {
@@ -1322,15 +1325,13 @@
       "headers": { "content-length": "4" }
     }
   },
-
   "GET /override-content-length/response/init/object-literal/false": {
     "environments": ["compute"],
     "downstream_response": {
       "status": 200,
       "body": "meow"
     }
   },
-
   "GET /override-content-length/response/init/response-instance/true": {
     "environments": ["compute"],
     "downstream_response": {
@@ -1339,27 +1340,23 @@
       "headers": { "content-length": "4" }
     }
   },
-
   "GET /override-content-length/response/init/response-instance/false": {
     "environments": ["compute"],
     "downstream_response": {
       "status": 200,
       "body": "meow"
     }
   },
-
   "GET /override-content-length/response/method/false": {
     "environments": ["compute"]
   },
-
   "GET /override-content-length/response/method/true": {
     "environments": ["compute"],
     "downstream_response": {
       "status": 200,
       "headers": { "content-length": "11" }
     }
   },
-
   "GET /headers/non-ascii-latin1-field-value": {
     "downstream_response": {
       "status": 200,
@@ -2321,7 +2318,6 @@
   "GET /transaction-cache-entry/cancel/called-twice-throws": {
     "environments": ["compute"]
   },
-
   "GET /rate-counter/interface": {
     "downstream_response": {
       "status": 200,
@@ -2730,7 +2726,6 @@
       "body": "ok"
     }
   },
-
   "GET /device/interface": {
     "downstream_response": {
       "status": 200,

diff --git a/integration-tests/js-compute/fixtures/module-mode/src/dynamic-backend.js b/integration-tests/js-compute/fixtures/module-mode/src/dynamic-backend.js
@@ -52,7 +52,11 @@ routes.set('/backend/timeout', async () => {
     '/implicit-dynamic-backend/dynamic-backends-disabled',
     async () => {
       allowDynamicBackends(false);
-      await assertRejects(() => fetch('https://http-me.glitch.me/headers'));
+      await assertRejects(() =>
+        fetch('https://http-me.glitch.me/headers', {
+          cacheOverride: 'pass',
+        }),
+      );
     },
   );
   routes.set(
@@ -62,7 +66,9 @@ routes.set('/backend/timeout', async () => {
       strictEqual(evt.request.backend, undefined);
       strictEqual(new Response('test').backend, undefined);
       await assertResolves(async () => {
-        const res = await fetch('https://http-me.glitch.me/headers');
+        const res = await fetch('https://http-me.glitch.me/headers', {
+          cacheOverride: 'pass',
+        });
         strictEqual(res.backend.name, 'http-me.glitch.me');
         strictEqual(res.backend.isSSL, true);
       });