refactor: reuse utils logic

Author: John Doe

packages/nx-plugin/src/executors/cli/executor.int.test.ts

@@ -1,17 +1,34 @@
-import { afterEach, expect, vi } from 'vitest';
+import { afterEach, beforeEach, expect, vi } from 'vitest';
 import { executorContext } from '@code-pushup/test-nx-utils';
-import * as executeProcessModule from '../../internal/execute-process.js';
 import runAutorunExecutor from './executor.js';
 import * as utils from './utils.js';
 
+const { executeProcessSpy } = vi.hoisted(() => ({
+  executeProcessSpy: vi.fn().mockResolvedValue({
+    code: 0,
+    stdout: '',
+    stderr: '',
+    date: new Date().toISOString(),
+    duration: 100,
+  }),
+}));
+
+vi.mock('@code-pushup/utils', async () => {
+  const utils = await vi.importActual('@code-pushup/utils');
+  return {
+    ...utils,
+    executeProcess: executeProcessSpy,
+  };
+});
+
 describe('runAutorunExecutor', () => {
   const parseAutorunExecutorOptionsSpy = vi.spyOn(
     utils,
     'parseAutorunExecutorOptions',
   );
-  const executeProcessSpy = vi.spyOn(executeProcessModule, 'executeProcess');
 
   beforeEach(() => {
+    executeProcessSpy.mockClear();
     executeProcessSpy.mockResolvedValue({
       bin: 'npx ...',
       code: 0,

packages/nx-plugin/src/executors/cli/executor.ts

@@ -1,6 +1,4 @@
 import { type ExecutorContext, logger } from '@nx/devkit';
-import { isVerbose } from '@code-pushup/utils';
-import { executeProcess } from '../../internal/execute-process.js';
 import {
   createCliCommandObject,
   createCliCommandString,
@@ -42,10 +40,10 @@ export default async function runAutorunExecutor(
     logger.warn(`DryRun execution of: ${commandString}`);
   } else {
     try {
+      const { executeProcess } = await import('@code-pushup/utils');
       await executeProcess({
         ...createCliCommandObject({ command, args: cliArgumentObject, bin }),
         ...(context.cwd ? { cwd: context.cwd } : {}),
-        ...(isVerbose() || verbose ? { verbose: true } : {}),
       });
     } catch (error) {
       logger.error(error);

packages/nx-plugin/src/executors/cli/executor.unit.test.ts

@@ -2,16 +2,32 @@ import { logger } from '@nx/devkit';
 import { afterAll, afterEach, beforeEach, expect, vi } from 'vitest';
 import { executorContext } from '@code-pushup/test-nx-utils';
 import { MEMFS_VOLUME } from '@code-pushup/test-utils';
-import * as executeProcessModule from '../../internal/execute-process.js';
 import runAutorunExecutor from './executor.js';
 
+const { executeProcessSpy } = vi.hoisted(() => ({
+  executeProcessSpy: vi.fn().mockResolvedValue({
+    code: 0,
+    stdout: '',
+    stderr: '',
+    date: new Date().toISOString(),
+    duration: 100,
+  }),
+}));
+
+vi.mock('@code-pushup/utils', async () => {
+  const utils = await vi.importActual('@code-pushup/utils');
+  return {
+    ...utils,
+    executeProcess: executeProcessSpy,
+  };
+});
+
 describe('runAutorunExecutor', () => {
   const processEnvCP = Object.fromEntries(
     Object.entries(process.env).filter(([k]) => k.startsWith('CP_')),
   );
   const loggerInfoSpy = vi.spyOn(logger, 'info');
   const loggerWarnSpy = vi.spyOn(logger, 'warn');
-  const executeProcessSpy = vi.spyOn(executeProcessModule, 'executeProcess');
 
   beforeAll(() => {
     Object.entries(process.env)
@@ -25,6 +41,7 @@ describe('runAutorunExecutor', () => {
 
   beforeEach(() => {
     vi.unstubAllEnvs();
+    executeProcessSpy.mockClear();
     executeProcessSpy.mockResolvedValue({
       bin: 'npx ...',
       code: 0,