test: remove obsolete console mocks

Author: matejchalk

packages/models/src/lib/implementation/schemas.unit.test.ts

@@ -38,6 +38,10 @@ describe('weightSchema', () => {
 });
 
 describe('docsUrlSchema', () => {
+  beforeAll(() => {
+    vi.spyOn(console, 'warn').mockImplementation(() => {});
+  });
+
   it('should accept a valid URL', () => {
     expect(() =>
       docsUrlSchema.parse(

packages/plugin-typescript/src/lib/utils.unit.test.ts

@@ -115,10 +115,10 @@ describe('getCategoryRefsFromGroups', () => {
 });
 
 describe('logSkippedAudits', () => {
-  it('should not warn when all audits are included', () => {
+  it('should not print anything when all audits are included', () => {
     logSkippedAudits(AUDITS);
 
-    expect(console.warn).not.toHaveBeenCalled();
+    expect(logger.info).not.toHaveBeenCalled();
   });
 
   it('should warn about skipped audits', () => {

testing/test-setup-config/src/lib/vitest-config-factory.unit.test.ts

@@ -29,7 +29,6 @@ describe('createVitestConfig', () => {
           ],
           globalSetup: ['../../global-setup.ts'],
           setupFiles: expect.arrayContaining([
-            '../../testing/test-setup/src/lib/console.mock.ts',
             '../../testing/test-setup/src/lib/reset.mocks.ts',
             '../../testing/test-setup/src/lib/fs.mock.ts',
           ]),
@@ -47,9 +46,6 @@ describe('createVitestConfig', () => {
       const config = createVitestConfig('test-package', 'unit');
 
       const setupFiles = config.test!.setupFiles;
-      expect(setupFiles).toContain(
-        '../../testing/test-setup/src/lib/console.mock.ts',
-      );
       expect(setupFiles).toContain(
         '../../testing/test-setup/src/lib/reset.mocks.ts',
       );
@@ -121,9 +117,6 @@ describe('createVitestConfig', () => {
       const config = createVitestConfig('test-package', 'int');
 
       const setupFiles = config.test!.setupFiles;
-      expect(setupFiles).toContain(
-        '../../testing/test-setup/src/lib/console.mock.ts',
-      );
       expect(setupFiles).toContain(
         '../../testing/test-setup/src/lib/logger.mock.ts',
       );
@@ -172,9 +165,9 @@ describe('createVitestConfig', () => {
       expect(setupFiles).toContain(
         '../../testing/test-setup/src/lib/reset.mocks.ts',
       );
-      // Should NOT include console, fs, git, etc.
+      // Should NOT include fs, git, etc.
       expect(setupFiles).not.toContain(
-        '../../testing/test-setup/src/lib/console.mock.ts',
+        '../../testing/test-setup/src/lib/git.mock.ts',
       );
       expect(setupFiles).not.toContain(
         '../../testing/test-setup/src/lib/fs.mock.ts',

testing/test-setup-config/src/lib/vitest-setup-files.ts

@@ -7,9 +7,9 @@ import type { TestKind } from './vitest-config-factory.js';
  * which is why they use `../../` to navigate to the workspace root first.
  */
 const CUSTOM_MATCHERS = [
-  '../../testing/test-setup/src/lib/extend/markdown-table.matcher.ts',
   '../../testing/test-setup/src/lib/extend/jest-extended.matcher.ts',
   '../../testing/test-setup/src/lib/extend/path.matcher.ts',
+  '../../testing/test-setup/src/lib/extend/markdown-table.matcher.ts',
 ] as const;
 
 /**
@@ -19,12 +19,11 @@ const CUSTOM_MATCHERS = [
  * which is why they use `../../` to navigate to the workspace root first.
  */
 const UNIT_TEST_SETUP_FILES = [
-  '../../testing/test-setup/src/lib/fs.mock.ts',
-  '../../testing/test-setup/src/lib/console.mock.ts',
   '../../testing/test-setup/src/lib/reset.mocks.ts',
+  '../../testing/test-setup/src/lib/fs.mock.ts',
+  '../../testing/test-setup/src/lib/logger.mock.ts',
   '../../testing/test-setup/src/lib/git.mock.ts',
   '../../testing/test-setup/src/lib/portal-client.mock.ts',
-  '../../testing/test-setup/src/lib/logger.mock.ts',
   ...CUSTOM_MATCHERS,
 ] as const;
 
@@ -36,10 +35,9 @@ const UNIT_TEST_SETUP_FILES = [
 
  */
 const INT_TEST_SETUP_FILES = [
-  '../../testing/test-setup/src/lib/console.mock.ts',
   '../../testing/test-setup/src/lib/reset.mocks.ts',
-  '../../testing/test-setup/src/lib/chrome-path.mock.ts',
   '../../testing/test-setup/src/lib/logger.mock.ts',
+  '../../testing/test-setup/src/lib/chrome-path.mock.ts',
   ...CUSTOM_MATCHERS,
 ] as const;