fixes

Author: wmertens

e2e/qwik-e2e/apps/qwikrouter-test/src/routes/issue4502/broken/index.tsx

@@ -1,6 +1,19 @@
-import type { RequestHandler } from '@qwik.dev/router';
+import { component$ } from '@qwik.dev/core';
+import { routeLoader$, type RequestHandler } from '@qwik.dev/router';
+
+// We need a routeLoader$ here to ensure the page is loaded via the loader URL (q-loader-*.json) which triggers the onRequest middleware.
+export const useForceLoadingThisIndex = routeLoader$(() => null);
 
 export const onRequest: RequestHandler<void> = async (onRequestArgs) => {
   const { redirect, url } = onRequestArgs;
   throw redirect(302, `${url.pathname}/route/`);
 };
+
+export default component$(() => {
+  useForceLoadingThisIndex();
+  return (
+    <div id="index">
+      You should be redirecting to <code>/broken/route</code>
+    </div>
+  );
+});

e2e/qwik-e2e/tests/qwikrouter/loaders.e2e.ts

@@ -60,9 +60,6 @@ test.describe('loaders', () => {
       await expect(title).toHaveText('Loaders - Qwik', { useInnerText: true });
       await expect(date).toHaveText('date: 2021-01-01T00:00:00.000Z');
       await expect(slow).toHaveText('slow: 123');
-      if (javaScriptEnabled) {
-        await expect(nestedName).toHaveText('name: stuff');
-      }
       await expect(nestedName).toHaveText('name: welcome');
       await expect(nestedDate).toHaveText('date: 2021-01-01T00:00:00.000Z');
       await expect(nestedDep).toHaveText('dep: 84');