fix: new middleware integration test (#5755)

tests/integration/server-hook/new-middleware/package.json

@@ -13,7 +13,6 @@
   },
   "dependencies": {
     "@modern-js/runtime": "workspace:*",
-    "axios": "^1.6.0",
     "react": "^18",
     "react-dom": "^18"
   },

tests/integration/server-hook/new-middleware/server/index.ts

@@ -21,13 +21,6 @@ function parseQuery(request: Request): URLSearchParams {
   return url.searchParams;
 }
 
-function getPathname(request: Request): string {
-  // eslint-disable-next-line node/no-unsupported-features/node-builtins, node/prefer-global/url
-  const url = new URL(request.url);
-
-  return url.pathname;
-}
-
 function auth(): UnstableMiddleware<Var> {
   function getUserInfo(req: Request) {
     const query = parseQuery(req);
@@ -42,8 +35,7 @@ function auth(): UnstableMiddleware<Var> {
 
   // eslint-disable-next-line consistent-return
   return async (c, next) => {
-    const pathname = getPathname(c.request);
-    if (pathname.startsWith('/login')) {
+    if (c.request.url.includes('/login')) {
       return next();
     }
 

tests/integration/server-hook/new-middleware/tests/index.test.ts

@@ -1,15 +1,24 @@
 import path from 'path';
-import axios from 'axios';
-import { launchApp, getPort, killApp } from '../../../../utils/modernTestUtils';
+import puppeteer, { Browser, Page } from 'puppeteer';
+import {
+  launchApp,
+  getPort,
+  killApp,
+  launchOptions,
+} from '../../../../utils/modernTestUtils';
 
 const appPath = path.resolve(__dirname, '../');
 
 describe('test new middleware run correctly', () => {
   let app: any;
   let port: number;
+  let page: Page;
+  let browser: Browser;
   beforeAll(async () => {
     jest.setTimeout(1000 * 60 * 2);
 
+    browser = await puppeteer.launch(launchOptions as any);
+    page = await browser.newPage();
     port = await getPort();
 
     app = await launchApp(appPath, port);
@@ -19,13 +28,16 @@ describe('test new middleware run correctly', () => {
     if (app) {
       await killApp(app);
     }
+    await page.close();
+    await browser.close();
   });
 
   test('should request "/" correctly', async () => {
     const url = `http://localhost:${port}`;
-    const res = await axios.get(url);
+    const res = await page.goto(url);
 
-    const { headers, data: body } = res;
+    const headers = res?.headers();
+    const body = await res?.text();
 
     expect(body).toMatch('Liming');
 
@@ -38,9 +50,13 @@ describe('test new middleware run correctly', () => {
 
   test('should redirect corretly', async () => {
     const url = `http://localhost:${port}/?unlogin=1`;
-    const res = await axios.get(url);
+    const res = await page.goto(url);
 
-    const { data: body, headers } = res;
+    const body = await res?.text();
+    const headers = res?.headers();
+    const chain = res?.request().redirectChain();
+
+    expect(chain?.length).toBe(1);
 
     expect(body).toMatch('Login');