chore: fix ci

umijs · May 11, 2023 · 0dfecfc · 0dfecfc
1 parent fb09425
commit 0dfecfc
Showing 1 changed file with 12 additions and 18 deletions.
diff --git a/tests/doctor.test.ts b/tests/doctor.test.ts
@@ -30,32 +30,30 @@ test('doctor: warn checkups', async () => {
   });
 
   // DUP_IN_PEER_DEPS
-  expect(console.log).toHaveBeenCalledWith(
+  expect(logSpy).toHaveBeenCalledWith(
     expect.stringContaining('peerDependency'),
   );
 
   // EFFECTS_IN_SIDE_EFFECTS
-  expect(console.log).toHaveBeenCalledWith(
+  expect(logSpy).toHaveBeenCalledWith(
     expect.stringContaining('sideEffect syntax'),
   );
 
   // PREFER_BABEL_RUNTIME
-  expect(console.log).toHaveBeenCalledWith(
+  expect(logSpy).toHaveBeenCalledWith(
     expect.stringContaining('@babel/runtime'),
   );
 
   // PREFER_PACK_FILES
-  expect(console.log).toHaveBeenCalledWith(
+  expect(logSpy).toHaveBeenCalledWith(
     expect.stringContaining('No `files` field'),
   );
 
   // PREFER_NO_CSS_MODULES
-  expect(console.log).toHaveBeenCalledWith(
-    expect.stringContaining('CSS Modules'),
-  );
+  expect(logSpy).toHaveBeenCalledWith(expect.stringContaining('CSS Modules'));
 
   // PREFER_PEER_DEPS
-  expect(console.log).toHaveBeenCalledWith(
+  expect(logSpy).toHaveBeenCalledWith(
     expect.stringContaining('multi-instance risk'),
   );
 });
@@ -67,30 +65,28 @@ test('doctor: error checkups', async () => {
   });
 
   // PHANTOM_DEPS
-  expect(console.log).toHaveBeenCalledWith(
+  expect(logSpy).toHaveBeenCalledWith(
     expect.stringContaining('Source depends on'),
   );
 
   // PHANTOM_DEPS no standard library
-  expect(console.log).toHaveBeenCalledWith(
+  expect(logSpy).toHaveBeenCalledWith(
     expect.not.stringContaining('child_process'),
   );
 
   // EFFECTS_IN_SIDE_EFFECTS
-  expect(console.log).toHaveBeenCalledWith(
-    expect.stringContaining('styles lost'),
-  );
+  expect(logSpy).toHaveBeenCalledWith(expect.stringContaining('styles lost'));
 
   // PACK_FILES_MISSING
-  expect(console.log).toHaveBeenCalledWith(
+  expect(logSpy).toHaveBeenCalledWith(
     expect.stringContaining('will not be published'),
   );
 
   // CASE_SENSITIVE_PATHS
   // why only win32 and drawin?
   // because Windows and macOS are case-insensitive by default
   if (['win32', 'drawin'].includes(process.platform)) {
-    expect(console.log).toHaveBeenCalledWith(
+    expect(logSpy).toHaveBeenCalledWith(
       expect.stringContaining('the corresponding path'),
     );
   }
@@ -110,7 +106,5 @@ test('doctor: health', async () => {
     args: { _: ['doctor'], $0: 'node' },
   });
 
-  expect(console.log).toHaveBeenCalledWith(
-    expect.stringContaining('looks fine'),
-  );
+  expect(logSpy).toHaveBeenCalledWith(expect.stringContaining('looks fine'));
 });