refactor: fix tests

Author: Michael Hladky

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

@@ -9,7 +9,7 @@ describe('runAutorunExecutor', () => {
     Object.entries(process.env).filter(([k]) => k.startsWith('CP_')),
   );
   const executeProcessSpy = vi.spyOn(executeProcessModule, 'executeProcess');
-  let loggerSpy: Awaited<typeof import('@code-pushup/utils')>['logger'];
+  let logger: import('@code-pushup/utils').Logger;
 
   beforeAll(() => {
     Object.entries(process.env)
@@ -24,8 +24,8 @@ describe('runAutorunExecutor', () => {
   });
 
   beforeEach(async () => {
-    const { logger } = await import('@code-pushup/utils');
-    loggerSpy = logger;
+    const utils = await import('@code-pushup/utils');
+    logger = utils.logger;
     vi.unstubAllEnvs();
     executeProcessSpy.mockResolvedValue({
       bin: 'npx ...',
@@ -125,7 +125,7 @@ describe('runAutorunExecutor', () => {
     });
 
     expect(output.command).not.toContain('--verbose');
-    expect(loggerSpy.warn).toHaveBeenCalledTimes(0);
+    expect(logger.warn).toHaveBeenCalledTimes(0);
   });
 
   it('should log env var in dryRun information if verbose is set', async () => {
@@ -140,18 +140,18 @@ describe('runAutorunExecutor', () => {
     expect(executeProcessSpy).toHaveBeenCalledTimes(0);
 
     expect(output.command).not.toContain('--verbose');
-    expect(loggerSpy.warn).toHaveBeenCalledTimes(1);
-    expect(loggerSpy.warn).toHaveBeenCalledWith(
+    expect(logger.warn).toHaveBeenCalledTimes(1);
+    expect(logger.warn).toHaveBeenCalledWith(
       expect.stringContaining('CP_VERBOSE="true"'),
     );
   });
 
   it('should log command if dryRun is set', async () => {
     await runAutorunExecutor({ dryRun: true }, executorContext('utils'));
 
-    expect(loggerSpy.command).toHaveBeenCalledTimes(0);
-    expect(loggerSpy.warn).toHaveBeenCalledTimes(1);
-    expect(loggerSpy.warn).toHaveBeenCalledWith(
+    expect(logger.command).toHaveBeenCalledTimes(0);
+    expect(logger.warn).toHaveBeenCalledTimes(1);
+    expect(logger.warn).toHaveBeenCalledWith(
       expect.stringContaining('DryRun execution of'),
     );
   });