test: fix ci (#10586)

* test: fix ci * chore: fix test
umijs · Feb 23, 2023 · 0be5d13 · 0be5d13
1 parent c7a6b86
commit 0be5d13
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/examples/test-test/package.json b/examples/test-test/package.json
@@ -3,6 +3,7 @@
   "private": true,
   "scripts": {
     "dev": "umi dev",
+    "setup": "umi setup",
     "start": "npm run dev",
     "test": "jest --no-cache --coverage"
   },

diff --git a/packages/preset-umi/src/features/tmpFiles/routes.test.ts b/packages/preset-umi/src/features/tmpFiles/routes.test.ts
@@ -46,6 +46,7 @@ test('getRoutes', async () => {
           },
         ],
       },
+      userConfig: {},
       applyPlugins(opts: any) {
         return opts?.initialValue;
       },

diff --git a/packages/preset-umi/src/features/tmpFiles/tmpFiles.ts b/packages/preset-umi/src/features/tmpFiles/tmpFiles.ts
@@ -380,7 +380,12 @@ export default function EmptyRoute() {
         '...$1',
       );
       // import: route props
-      headerImports.push(`import routeProps from './routeProps.js';`);
+      // why has this branch? since test env don't build routeProps.js
+      if (process.env.NODE_ENV === 'test') {
+        headerImports.push(`import routeProps from './routeProps';`);
+      } else {
+        headerImports.push(`import routeProps from './routeProps.js';`);
+      }
       // prevent override internal route props
       headerImports.push(`
 if (process.env.NODE_ENV === 'development') {
-Original file line number
+Diff line change
@@ Expand Up / @@ -46,6 +46,7 @@ test('getRoutes', async () => { @@
               },
             ],
           },
+          userConfig: {},
           applyPlugins(opts: any) {
             return opts?.initialValue;
           },
@@ Expand Down @@