chore(qc): extra tests for redirects

Author: wmertens

starters/apps/qwikcity-test/src/routes/[email protected]

@@ -0,0 +1,7 @@
+import type { RequestEvent } from "@builder.io/qwik-city";
+
+export const onGet = (ev: RequestEvent) => {
+  if (ev.url.pathname === "/qwikcity-test/redirectme/") {
+    throw ev.redirect(301, "/qwikcity-test/");
+  }
+};

starters/e2e/qwikcity/nav.spec.ts

@@ -457,20 +457,20 @@ test.describe("actions", () => {
         "/qwikcity-test/issue7732/c/?redirected=true",
       );
     });
-    // TODO: Fix this test (currently not working because the action redirect adds a `/q-data.json` at the end of the path)
-    test.fixme(
-      "action with redirect without query params in a route with query param should redirect to route without query params",
-      async ({ page }) => {
-        await page.goto(
-          "/qwikcity-test/action-redirect-without-search-params/?test=test",
-        );
-        const button = page.locator("button");
-        await button.click();
-        await page.waitForURL(
-          "/qwikcity-test/action-redirect-without-search-params-target/",
-        );
-      },
-    );
+    test("action with redirect without query params in a route with query param should redirect to route without query params", async ({
+      page,
+    }) => {
+      await page.goto(
+        "/qwikcity-test/action-redirect-without-search-params/?test=test",
+      );
+      const button = page.locator("button");
+      await button.click();
+      await page.waitForURL(
+        "/qwikcity-test/action-redirect-without-search-params-target/",
+      );
+      const searchParams = new URL(page.url()).searchParams;
+      expect(searchParams.size).toBe(0);
+    });
     test("media in home page", async ({ page }) => {
       await page.goto("/qwikcity-test/");
 
@@ -521,6 +521,22 @@ test.describe("actions", () => {
 
       await expect(page.locator("#redirected-result")).toHaveText("true");
     });
+
+    test.only("server plugin q-data redirect from /redirectme to /", async ({
+      baseURL,
+    }) => {
+      const res = await fetch(
+        new URL("/qwikcity-test/redirectme/q-data.json", baseURL),
+        {
+          redirect: "manual",
+          headers: {
+            Accept: "application/json",
+          },
+        },
+      );
+      expect(res.status).toBe(301);
+      expect(res.headers.get("Location")).toBe("/qwikcity-test/q-data.json");
+    });
   }
 });