chore: set up console mock setup for all packages, replace console.log with info

Tlacenka · Tlacenka · commit 49d4f94534b9 · 2023-11-29T18:14:20.000+01:00
diff --git a/packages/models/vite.config.integration.ts b/packages/models/vite.config.integration.ts
@@ -13,5 +13,9 @@ export default defineConfig({
     environment: 'node',
     include: ['src/**/*.integration.test.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
     globalSetup: ['global-setup.ts'],
+    setupFiles: [
+      '../../testing-utils/src/lib/setup/console.mock.ts',
+      '../../testing-utils/src/lib/setup/reset.mocks.ts',
+    ],
   },
 });
diff --git a/packages/models/vite.config.unit.ts b/packages/models/vite.config.unit.ts
@@ -13,5 +13,9 @@ export default defineConfig({
     environment: 'node',
     include: ['src/**/*.unit.test.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
     globalSetup: ['global-setup.ts'],
+    setupFiles: [
+      '../../testing-utils/src/lib/setup/console.mock.ts',
+      '../../testing-utils/src/lib/setup/reset.mocks.ts',
+    ],
   },
 });
diff --git a/packages/nx-plugin/src/generators/configuration/generator.ts b/packages/nx-plugin/src/generators/configuration/generator.ts
@@ -18,7 +18,7 @@ export async function addToProjectGenerator(
   const { root } = projectConfiguration;
 
   if (tree.exists(path.join(root, 'code-pushup.config.ts'))) {
-    console.log('Code PushUp already configured for this project');
+    console.info('Code PushUp already configured for this project');
     return;
   }
 
diff --git a/packages/nx-plugin/vite.config.integration.ts b/packages/nx-plugin/vite.config.integration.ts
@@ -12,5 +12,9 @@ export default defineConfig({
     },
     environment: 'node',
     include: ['src/**/*.integration.test.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
+    setupFiles: [
+      '../../testing-utils/src/lib/setup/console.mock.ts',
+      '../../testing-utils/src/lib/setup/reset.mocks.ts',
+    ],
   },
 });
diff --git a/packages/nx-plugin/vite.config.unit.ts b/packages/nx-plugin/vite.config.unit.ts
@@ -12,5 +12,9 @@ export default defineConfig({
     },
     environment: 'node',
     include: ['src/**/*.unit.test.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
+    setupFiles: [
+      '../../testing-utils/src/lib/setup/console.mock.ts',
+      '../../testing-utils/src/lib/setup/reset.mocks.ts',
+    ],
   },
 });
diff --git a/packages/plugin-eslint/vite.config.integration.ts b/packages/plugin-eslint/vite.config.integration.ts
@@ -13,5 +13,9 @@ export default defineConfig({
     environment: 'node',
     include: ['src/**/*.integration.test.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
     globalSetup: ['global-setup.ts'],
+    setupFiles: [
+      '../../testing-utils/src/lib/setup/console.mock.ts',
+      '../../testing-utils/src/lib/setup/reset.mocks.ts',
+    ],
   },
 });
diff --git a/packages/plugin-eslint/vite.config.unit.ts b/packages/plugin-eslint/vite.config.unit.ts
@@ -13,5 +13,9 @@ export default defineConfig({
     environment: 'node',
     include: ['src/**/*.unit.test.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
     globalSetup: ['global-setup.ts'],
+    setupFiles: [
+      '../../testing-utils/src/lib/setup/console.mock.ts',
+      '../../testing-utils/src/lib/setup/reset.mocks.ts',
+    ],
   },
 });
diff --git a/packages/plugin-lighthouse/vite.config.integration.ts b/packages/plugin-lighthouse/vite.config.integration.ts
@@ -13,5 +13,9 @@ export default defineConfig({
     environment: 'node',
     include: ['src/**/*.integration.test.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
     globalSetup: ['global-setup.ts'],
+    setupFiles: [
+      '../../testing-utils/src/lib/setup/console.mock.ts',
+      '../../testing-utils/src/lib/setup/reset.mocks.ts',
+    ],
   },
 });
diff --git a/packages/plugin-lighthouse/vite.config.unit.ts b/packages/plugin-lighthouse/vite.config.unit.ts
@@ -13,5 +13,9 @@ export default defineConfig({
     environment: 'node',
     include: ['src/**/*.unit.test.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
     globalSetup: ['global-setup.ts'],
+    setupFiles: [
+      '../../testing-utils/src/lib/setup/console.mock.ts',
+      '../../testing-utils/src/lib/setup/reset.mocks.ts',
+    ],
   },
 });
diff --git a/testing-utils/vite.config.unit.ts b/testing-utils/vite.config.unit.ts
@@ -12,5 +12,9 @@ export default defineConfig({
     },
     environment: 'node',
     include: ['src/**/*.unit.test.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
+    setupFiles: [
+      'src/lib/setup/console.mock.ts',
+      'src/lib/setup/reset.mocks.ts',
+    ],
   },
 });
diff --git a/tools/scripts/start-local-registry.ts b/tools/scripts/start-local-registry.ts
@@ -66,7 +66,7 @@ function startLocalRegistry({
         const port = parseInt(
           data.toString().match(/localhost:(?<port>\d+)/)?.groups?.port,
         );
-        console.log('Local registry started on port ' + port);
+        console.info('Local registry started on port ' + port);
 
         const registry = `http://localhost:${port}`;
         process.env.npm_config_registry = registry;
@@ -80,7 +80,7 @@ function startLocalRegistry({
         process.env.YARN_NPM_REGISTRY_SERVER = registry;
         process.env.YARN_UNSAFE_HTTP_WHITELIST = 'localhost';
 
-        console.log('Set npm and yarn config registry to ' + registry);
+        console.info('Set npm and yarn config registry to ' + registry);
 
         resolve(() => {
           childProcess.kill();
@@ -94,11 +94,11 @@ function startLocalRegistry({
       process.stderr.write(data);
     });
     childProcess.on('error', err => {
-      console.log('local registry error', err);
+      console.error('local registry error', err);
       reject(err);
     });
     childProcess.on('exit', code => {
-      console.log('local registry exit', code);
+      console.info('local registry exit', code);
       if (code !== 0) {
         reject(code);
       } else {

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ export async function addToProjectGenerator(`
`18`	`18`	`const { root } = projectConfiguration;`
`19`	`19`
`20`	`20`	`if (tree.exists(path.join(root, 'code-pushup.config.ts'))) {`
`21`		`- console.log('Code PushUp already configured for this project');`
	`21`	`+ console.info('Code PushUp already configured for this project');`
`22`	`22`	`return;`
`23`	`23`	`}`
`24`	`24`