@form8ion/javascript 15.8.3 → 16.0.0-beta.1

package/src/package/lifter.js

@@ -1,6 +1,5 @@
 import {promises as fs} from 'node:fs';
 import deepmerge from 'deepmerge';
-import {info} from '@travi/cli-messages';
 import {writePackageJson} from '@form8ion/javascript-core';
 
 import sortPackageProperties from './property-sorter.js';
@@ -17,8 +16,8 @@ export default async function liftPackageJson({
   packageManager,
   vcs,
   pathWithinParent
-}) {
-  info('Updating `package.json`', {level: 'secondary'});
+}, {logger}) {
+  logger.info('Updating `package.json`', {level: 'secondary'});
 
   const existingPackageJsonContents = JSON.parse(await fs.readFile(`${projectRoot}/package.json`, 'utf-8'));
   const {scripts: liftedScripts, dependencies: scriptDependencies} = liftScripts({
@@ -43,5 +42,5 @@ export default async function liftPackageJson({
     devDependencies,
     projectRoot,
     packageManager
-  });
+  }, {logger});
 }

package/src/package/lifter.test.js

@@ -34,6 +34,7 @@ describe('package.json lifter', () => {
   const config = any.simpleObject();
   const tags = any.listOf(any.word);
   const scriptDependencies = any.simpleObject();
+  const logger = {info: () => undefined};
 
   beforeEach(() => {
     when(defineVcsHostDetails).calledWith(vcs, pathWithinParent).thenReturn(vcsDetails);
@@ -53,7 +54,10 @@ describe('package.json lifter', () => {
       .calledWith({...existingPackageContents, ...vcsDetails, scripts: liftedScripts})
       .thenReturn(config);
 
-    await liftPackage({dependencies, devDependencies, projectRoot, packageManager, vcs, pathWithinParent, scripts});
+    await liftPackage(
+      {dependencies, devDependencies, projectRoot, packageManager, vcs, pathWithinParent, scripts},
+      {logger}
+    );
 
     expect(writePackageJson).toHaveBeenCalledWith({projectRoot, config});
     expect(processDependencies).toHaveBeenCalledWith({
@@ -61,7 +65,7 @@ describe('package.json lifter', () => {
       devDependencies,
       projectRoot,
       packageManager
-    });
+    }, {logger});
   });
 
   it('should update keywords if tags are provided', async () => {
@@ -73,7 +77,7 @@ describe('package.json lifter', () => {
       .calledWith({...existingPackageContents, ...vcsDetails, scripts: liftedScripts, keywords: tags})
       .thenReturn(config);
 
-    await liftPackage({dependencies, projectRoot, packageManager, vcs, pathWithinParent, scripts, tags});
+    await liftPackage({dependencies, projectRoot, packageManager, vcs, pathWithinParent, scripts, tags}, {logger});
 
     expect(writePackageJson).toHaveBeenCalledWith({projectRoot, config});
   });
@@ -93,7 +97,7 @@ describe('package.json lifter', () => {
       })
       .thenReturn(config);
 
-    await liftPackage({dependencies, projectRoot, packageManager, vcs, pathWithinParent, scripts, tags});
+    await liftPackage({dependencies, projectRoot, packageManager, vcs, pathWithinParent, scripts, tags}, {logger});
 
     expect(writePackageJson).toHaveBeenCalledWith({projectRoot, config});
   });

package/src/package/scaffolder.js

@@ -1,4 +1,3 @@
-import {info} from '@travi/cli-messages';
 import {writePackageJson} from '@form8ion/javascript-core';
 
 import buildPackageName from './package-name.js';
@@ -12,8 +11,8 @@ export default async function scaffoldPackage({
   license,
   author,
   description
-}) {
-  info('Configuring package.json');
+}, {logger}) {
+  logger.info('Configuring package.json');
 
   const packageName = buildPackageName(projectName, scope);
 

package/src/package/scaffolder.test.js

@@ -13,6 +13,8 @@ vi.mock('./package-name.js');
 vi.mock('./details.js');
 
 describe('package scaffolder', () => {
+  const logger = {info: () => undefined};
+
   it('should create the package file', async () => {
     const projectName = any.string();
     const packageName = any.string();
@@ -40,7 +42,7 @@ describe('package scaffolder', () => {
       license,
       author,
       description
-    })).toEqual({packageName});
+    }, {logger})).toEqual({packageName});
     expect(writePackageJson).toHaveBeenCalledWith({projectRoot, config: packageDetails});
   });
 });

package/src/project-type/application/scaffolder.js

@@ -1,10 +1,9 @@
 import {mergeIntoExistingPackageJson} from '@form8ion/javascript-core';
-import {info} from '@travi/cli-messages';
 
 const defaultBuildDirectory = 'public';
 
-export default async function scaffoldApplication({projectRoot}) {
-  info('Scaffolding Application Details');
+export default async function scaffoldApplication({projectRoot}, {logger}) {
+  logger.info('Scaffolding Application Details');
 
   await mergeIntoExistingPackageJson({projectRoot, config: {private: true}});
 

package/src/project-type/application/scaffolder.test.js

@@ -1,6 +1,6 @@
 import {mergeIntoExistingPackageJson} from '@form8ion/javascript-core';
 
-import {afterEach, describe, expect, it, vi} from 'vitest';
+import {describe, expect, it, vi} from 'vitest';
 import any from '@travi/any';
 
 import scaffoldApplication from './scaffolder.js';
@@ -8,15 +8,13 @@ import scaffoldApplication from './scaffolder.js';
 vi.mock('@form8ion/javascript-core');
 
 describe('application project-type scaffolder', () => {
-  afterEach(() => {
-    vi.clearAllMocks();
-  });
+  const logger = {info: () => undefined};
 
   it('should scaffold the details specific to an application project-type', async () => {
     const projectRoot = any.string();
     const buildDirectory = 'public';
 
-    expect(await scaffoldApplication({projectRoot})).toEqual({
+    expect(await scaffoldApplication({projectRoot}, {logger})).toEqual({
       scripts: {
         clean: `rimraf ./${buildDirectory}`,
         start: `node ./${buildDirectory}/index.js`,

package/src/project-type/monorepo/scaffolder.js

@@ -1,8 +1,7 @@
 import {mergeIntoExistingPackageJson} from '@form8ion/javascript-core';
-import {info} from '@travi/cli-messages';
 
-export default async function scaffoldMonorepo({projectRoot}) {
-  info('Scaffolding Monorepo Details');
+export default async function scaffoldMonorepo({projectRoot}, {logger}) {
+  logger.info('Scaffolding Monorepo Details');
 
   await mergeIntoExistingPackageJson({projectRoot, config: {private: true}});
 

package/src/project-type/monorepo/scaffolder.test.js

@@ -1,6 +1,6 @@
 import {mergeIntoExistingPackageJson} from '@form8ion/javascript-core';
 
-import {describe, vi, it, expect, afterEach} from 'vitest';
+import {describe, expect, it, vi} from 'vitest';
 import any from '@travi/any';
 
 import scaffoldMonorepo from './scaffolder.js';
@@ -9,13 +9,10 @@ vi.mock('@form8ion/javascript-core');
 
 describe('monorepo project-type scaffolder', () => {
   const projectRoot = any.string();
-
-  afterEach(() => {
-    vi.clearAllMocks();
-  });
+  const logger = {info: () => undefined};
 
   it('should scaffold the monorepo specific project details', async () => {
-    expect(await scaffoldMonorepo({projectRoot})).toEqual({
+    expect(await scaffoldMonorepo({projectRoot}, {logger})).toEqual({
       nextSteps: [{
         summary: 'Add packages to your new monorepo',
         description: 'Leverage [@form8ion/add-package-to-monorepo](https://npm.im/@form8ion/add-package-to-monorepo)'

package/src/project-type/package/scaffolder.js

@@ -1,5 +1,4 @@
 import deepmerge from 'deepmerge';
-import {info} from '@travi/cli-messages';
 import {dialects, mergeIntoExistingPackageJson} from '@form8ion/javascript-core';
 
 import determinePackageAccessLevelFromProjectVisibility from '../publishable/access-level.js';
@@ -19,8 +18,8 @@ export default async function scaffoldPackageProjectType({
   dialect,
   provideExample,
   publishRegistry
-}) {
-  info('Scaffolding Package Details');
+}, {logger}) {
+  logger.info('Scaffolding Package Details');
 
   const packageAccessLevel = determinePackageAccessLevelFromProjectVisibility({projectVisibility: visibility});
   const [detailsForBuild, publishableResults] = await Promise.all([

package/src/project-type/package/scaffolder.test.js

@@ -35,6 +35,7 @@ describe('package project-type scaffolder', () => {
   const documentation = any.simpleObject();
   const decisions = any.simpleObject();
   const buildDetailsResults = any.simpleObject();
+  const logger = {info: () => undefined};
 
   beforeEach(() => {
     when(documentationScaffolder.default)
@@ -68,7 +69,7 @@ describe('package project-type scaffolder', () => {
       decisions,
       dialect,
       provideExample
-    })).toEqual({
+    }, {logger})).toEqual({
       ...publishableResults,
       ...buildDetailsResults,
       documentation,
@@ -113,7 +114,7 @@ describe('package project-type scaffolder', () => {
       packageBundlers,
       decisions,
       provideExample
-    })).toEqual({
+    }, {logger})).toEqual({
       ...publishableResults,
       ...buildDetailsResults,
       documentation,
@@ -153,7 +154,7 @@ describe('package project-type scaffolder', () => {
       decisions,
       dialect,
       provideExample
-    })).toEqual({
+    }, {logger})).toEqual({
       ...publishableResults,
       ...buildDetailsResults,
       documentation,
@@ -199,7 +200,7 @@ describe('package project-type scaffolder', () => {
       packageBundlers,
       dialect,
       provideExample
-    })).toEqual({
+    }, {logger})).toEqual({
       ...publishableResults,
       ...buildDetailsResults,
       documentation,
@@ -239,7 +240,7 @@ describe('package project-type scaffolder', () => {
       dialect,
       provideExample,
       packageBundlers
-    });
+    }, {logger});
 
     expect(mergeIntoExistingPackageJson).toHaveBeenCalledWith({
       projectRoot,

package/src/project-type/publishable/access-level.js

@@ -1,3 +1,3 @@
 export default function mapProjectVisibility({projectVisibility}) {
-  return 'Public' === projectVisibility ? 'public' : 'restricted';
+  return 'OSS' === projectVisibility ? 'public' : 'restricted';
 }

package/src/project-type/publishable/access-level.test.js

@@ -1,13 +1,15 @@
 import {describe, expect, it} from 'vitest';
+import any from '@travi/any';
 
 import determinePackageAccessLevelFromProjectVisibility from './access-level.js';
 
 describe('package access level', () => {
-  it('should return `public` when project visibility is `Public`', () => {
-    expect(determinePackageAccessLevelFromProjectVisibility({projectVisibility: 'Public'})).toEqual('public');
+  it('should return `public` when project visibility is `OSS`', () => {
+    expect(determinePackageAccessLevelFromProjectVisibility({projectVisibility: 'OSS'})).toEqual('public');
   });
 
   it('should return `restricted` when the project visibility is `Private`', () => {
-    expect(determinePackageAccessLevelFromProjectVisibility({projectVisibility: 'Private'})).toEqual('restricted');
+    expect(determinePackageAccessLevelFromProjectVisibility({projectVisibility: any.fromList(['ISS', 'CS'])}))
+      .toEqual('restricted');
   });
 });

package/src/project-type/scaffolder.js

@@ -18,7 +18,7 @@ export default async function scaffoldProjectType({
   dialect,
   provideExample,
   publishRegistry
-}) {
+}, {logger}) {
   switch (projectType) {
     case projectTypes.PACKAGE:
       return scaffoldPackageType({
@@ -33,9 +33,9 @@ export default async function scaffoldProjectType({
         dialect,
         provideExample,
         publishRegistry
-      });
+      }, {logger});
     case projectTypes.APPLICATION:
-      return scaffoldApplicationType({projectRoot});
+      return scaffoldApplicationType({projectRoot}, {logger});
     case projectTypes.CLI:
       return scaffoldCliType({
         visibility,
@@ -46,7 +46,7 @@ export default async function scaffoldProjectType({
         packageBundlers
       });
     case projectTypes.MONOREPO:
-      return scaffoldMonorepoType({projectRoot});
+      return scaffoldMonorepoType({projectRoot}, {logger});
     case 'Other':
       return {};
     default:

package/src/project-type/scaffolder.test.js

@@ -1,6 +1,6 @@
 import {projectTypes} from '@form8ion/javascript-core';
 
-import {afterEach, describe, expect, it, vi} from 'vitest';
+import {describe, expect, it, vi} from 'vitest';
 import any from '@travi/any';
 import {when} from 'vitest-when';
 
@@ -27,10 +27,7 @@ describe('project-type scaffolder', () => {
   const dialect = any.word();
   const provideExample = any.boolean();
   const packageBundlers = any.simpleObject();
-
-  afterEach(() => {
-    vi.clearAllMocks();
-  });
+  const logger = {info: () => undefined};
 
   it('should apply the package-type scaffolder when the project-type is `Package`', async () => {
     const scope = any.word();
@@ -46,7 +43,7 @@ describe('project-type scaffolder', () => {
       dialect,
       provideExample,
       publishRegistry
-    }).thenResolve(results);
+    }, {logger}).thenResolve(results);
 
     expect(await projectTypeScaffolder({
       projectType: projectTypes.PACKAGE,
@@ -61,11 +58,11 @@ describe('project-type scaffolder', () => {
       dialect,
       provideExample,
       publishRegistry
-    })).toEqual(results);
+    }, {logger})).toEqual(results);
   });
 
   it('should apply the application-type scaffolder when the project-type is `Application`', async () => {
-    when(scaffoldApplicationType).calledWith({projectRoot}).thenResolve(results);
+    when(scaffoldApplicationType).calledWith({projectRoot}, {logger}).thenResolve(results);
 
     expect(await projectTypeScaffolder({
       projectType: projectTypes.APPLICATION,
@@ -75,7 +72,7 @@ describe('project-type scaffolder', () => {
       packageManager,
       decisions,
       visibility
-    })).toEqual(results);
+    }, {logger})).toEqual(results);
   });
 
   it('should apply the cli-type scaffolder when the project-type is `CLI`', async () => {
@@ -91,24 +88,26 @@ describe('project-type scaffolder', () => {
       publishRegistry,
       decisions,
       packageBundlers
-    })).toEqual(results);
+    }, {logger})).toEqual(results);
   });
 
   it('should apply the monorepo-type scaffolder when the project-type is `Monorepo`', async () => {
-    when(scaffoldMonorepoType).calledWith({projectRoot}).thenResolve(results);
+    when(scaffoldMonorepoType).calledWith({projectRoot}, {logger}).thenResolve(results);
 
-    expect(await projectTypeScaffolder({projectRoot, projectType: projectTypes.MONOREPO, packageManager, decisions}))
-      .toEqual(results);
+    expect(await projectTypeScaffolder(
+      {projectRoot, projectType: projectTypes.MONOREPO, packageManager, decisions},
+      {logger}
+    )).toEqual(results);
   });
 
   it('should not throw an error when the project-type is `Other`', async () => {
-    expect(await projectTypeScaffolder({projectType: 'Other'})).toEqual({});
+    expect(await projectTypeScaffolder({projectType: 'Other'}, {logger})).toEqual({});
   });
 
   it('should throw an error for an unknown project-type', async () => {
     const projectType = any.word();
 
-    await expect(() => projectTypeScaffolder({projectType}))
-      .rejects.toThrowError(`The project-type of ${projectType} is invalid`);
+    await expect(() => projectTypeScaffolder({projectType}, {logger}))
+      .rejects.toThrow(`The project-type of ${projectType} is invalid`);
   });
 });

package/src/prompts/questions.js

@@ -1,7 +1,6 @@
 import {packageManagers, projectTypes} from '@form8ion/javascript-core';
 import {prompt as promptWithInquirer} from '@form8ion/overridable-prompts';
 import {questionNames as commonQuestionNames, questions as commonQuestions} from '@travi/language-scaffolder-prompts';
-import {warn} from '@travi/cli-messages';
 
 import {execa} from 'execa';
 import npmConfFactory from '../../thirdparty-wrappers/npm-conf.js';
@@ -43,7 +42,8 @@ export async function prompt(
   vcs,
   decisions,
   configs,
-  pathWithinParent
+  pathWithinParent,
+  {logger}
 ) {
   const npmConf = npmConfFactory();
 
@@ -52,7 +52,7 @@ export async function prompt(
     maybeLoggedInNpmUsername = (await execa('npm', ['whoami'])).stdout;
   } catch (failedExecutionResult) {
     if (!decisions[questionNames.SCOPE]) {
-      warn('No logged in user found with `npm whoami`. Login with `npm login` '
+      logger.warn('No logged in user found with `npm whoami`. Login with `npm login` '
         + 'to use your npm account name as the package scope default.');
     }
   }

package/src/prompts/questions.test.js

@@ -59,6 +59,7 @@ describe('prompts', () => {
     [questionNames.DIALECT]: dialect,
     [questionNames.PROVIDE_EXAMPLE]: provideExample
   };
+  const logger = {info: () => undefined, warn: () => undefined};
 
   beforeEach(() => {
     when(commonPrompts.questions)
@@ -166,7 +167,7 @@ describe('prompts', () => {
       ], decisions)
       .thenResolve({...answers, [questionNames.CONFIGURE_LINTING]: any.word()});
 
-    expect(await prompt(ciServices, hosts, visibility, vcs, decisions, configs)).toEqual({
+    expect(await prompt(ciServices, hosts, visibility, vcs, decisions, configs, undefined, {logger})).toEqual({
       tests,
       projectType,
       ci,
@@ -188,7 +189,7 @@ describe('prompts', () => {
     when(execa).calledWith('npm', ['whoami']).thenResolve({stdout: npmUser});
     prompts.prompt.mockResolvedValue({...answers, [questionNames.CONFIGURE_LINTING]: false});
 
-    expect(await prompt(ciServices, {}, visibility, vcs, decisions)).toEqual({
+    expect(await prompt(ciServices, {}, visibility, vcs, decisions, undefined, undefined, {logger})).toEqual({
       tests,
       projectType,
       ci,
@@ -211,7 +212,7 @@ describe('prompts', () => {
       .thenReturn(commonQuestions);
     prompts.prompt.mockResolvedValue(answers);
 
-    await prompt(ciServices, {}, 'Private', vcs, null, null, pathWithinParent);
+    await prompt(ciServices, {}, 'Private', vcs, null, null, pathWithinParent, {logger});
 
     const [questions] = prompts.prompt.mock.lastCall;
     expect(questions.filter(question => questionNames.NODE_VERSION_CATEGORY === question.name).length).toEqual(0);
@@ -225,7 +226,7 @@ describe('prompts', () => {
       .thenReturn(commonQuestions);
     prompts.prompt.mockResolvedValue(answers);
 
-    await prompt(ciServices, {}, 'Private', vcs, null, null, pathWithinParent);
+    await prompt(ciServices, {}, 'Private', vcs, null, null, pathWithinParent, {logger});
 
     const [questions] = prompts.prompt.mock.lastCall;
     expect(questions.filter(question => questionNames.SHOULD_BE_SCOPED === question.name).length).toEqual(0);
@@ -239,7 +240,7 @@ describe('prompts', () => {
       .thenReturn(commonQuestions);
     prompts.prompt.mockResolvedValue(answers);
 
-    await prompt(ciServices, {}, 'Public', vcs, {}, null, pathWithinParent);
+    await prompt(ciServices, {}, 'Public', vcs, {}, null, pathWithinParent, {logger});
 
     const [questions] = prompts.prompt.mock.lastCall;
     expect(questions.filter(question => questionNames.SHOULD_BE_SCOPED === question.name).length).toEqual(1);

package/src/scaffolder.js

@@ -1,5 +1,4 @@
 import deepmerge from 'deepmerge';
-import {info} from '@travi/cli-messages';
 import {projectTypes, scaffoldChoice} from '@form8ion/javascript-core';
 import {scaffold as scaffoldCommitConvention} from '@form8ion/commit-convention';
 
@@ -18,8 +17,8 @@ import buildDocumentationCommand from './documentation/generation-command.js';
 import {scaffold as scaffoldVerification} from './verification/index.js';
 import {scaffold as scaffoldCodeStyle} from './code-style/index.js';
 
-export default async function scaffoldJavascript(options) {
-  info('Initializing JavaScript project');
+export default async function scaffoldJavascript(options, {logger}) {
+  logger.info('Initializing JavaScript project');
 
   const {
     projectRoot,
@@ -55,9 +54,9 @@ export default async function scaffoldJavascript(options) {
     provideExample,
     packageManager,
     dialect
-  } = await prompt(ciServices, hosts, visibility, vcs, decisions, configs, pathWithinParent);
+  } = await prompt(ciServices, hosts, visibility, vcs, decisions, configs, pathWithinParent, {logger});
 
-  info('Writing project files', {level: 'secondary'});
+  logger.info('Writing project files', {level: 'secondary'});
 
   const {packageName} = await scaffoldPackage({
     projectRoot,
@@ -67,7 +66,7 @@ export default async function scaffoldJavascript(options) {
     license,
     author,
     description
-  });
+  }, {logger});
   const projectTypeResults = await scaffoldProjectType({
     projectType,
     projectRoot,
@@ -86,7 +85,7 @@ export default async function scaffoldJavascript(options) {
     dialect,
     provideExample,
     publishRegistry: configs.registries.publish
-  });
+  }, {logger});
   const verificationResults = await scaffoldVerification({
     projectRoot,
     dialect,
@@ -100,7 +99,7 @@ export default async function scaffoldJavascript(options) {
     pathWithinParent
   });
   const [nodeVersion, npmResults, dialectResults, codeStyleResults] = await Promise.all([
-    scaffoldNodeVersion({projectRoot, nodeVersionCategory}),
+    scaffoldNodeVersion({projectRoot, nodeVersionCategory}, {logger}),
     scaffoldNpmConfig({projectType, projectRoot}),
     scaffoldDialect({
       dialect,

package/src/scaffolder.test.js

@@ -62,6 +62,7 @@ describe('javascript project scaffolder', () => {
   const mergedVcsIgnore = any.simpleObject();
   const mergedNextSteps = any.listOf(any.simpleObject);
   const mergedResults = {...any.simpleObject(), vcsIgnore: mergedVcsIgnore, nextSteps: mergedNextSteps};
+  const logger = {info: () => undefined};
 
   beforeEach(() => {
     const projectName = any.word();
@@ -121,7 +122,7 @@ describe('javascript project scaffolder', () => {
         }
       });
     when(prompt)
-      .calledWith(ciServices, hosts, visibility, vcs, decisions, configs, pathWithinParent)
+      .calledWith(ciServices, hosts, visibility, vcs, decisions, configs, pathWithinParent, {logger})
       .thenResolve({
         packageManager,
         dialect,
@@ -138,7 +139,7 @@ describe('javascript project scaffolder', () => {
     when(scaffoldDocumentation).calledWith({packageManager, projectTypeResults}).thenReturn(documentation);
     when(buildDocumentationCommand).calledWith(packageManager).thenReturn(documentationCommand);
     when(scaffoldPackage)
-      .calledWith({projectRoot, projectName, scope, dialect, license, author, description})
+      .calledWith({projectRoot, projectName, scope, dialect, license, author, description}, {logger})
       .thenResolve(packageResults);
     when(scaffoldProjectType)
       .calledWith({
@@ -159,7 +160,7 @@ describe('javascript project scaffolder', () => {
         scope,
         tests,
         publishRegistry
-      })
+      }, {logger})
       .thenResolve(projectTypeResults);
     when(scaffoldVerification)
       .calledWith({
@@ -175,7 +176,7 @@ describe('javascript project scaffolder', () => {
         integrationTestFrameworks
       })
       .thenResolve(verificationResults);
-    when(scaffoldNodeVersion).calledWith({projectRoot, nodeVersionCategory}).thenResolve(nodeVersionResults);
+    when(scaffoldNodeVersion).calledWith({projectRoot, nodeVersionCategory}, {logger}).thenResolve(nodeVersionResults);
     when(scaffoldDialect)
       .calledWith({dialect, configs, projectRoot, projectType, testFilenamePattern})
       .thenResolve(dialectResults);
@@ -235,7 +236,7 @@ describe('javascript project scaffolder', () => {
   });
 
   it('should scaffold the javascript details', async () => {
-    const results = await scaffold(options);
+    const results = await scaffold(options, {logger});
 
     expect(scaffoldPackageManager).toHaveBeenCalledWith({projectRoot, packageManager});
     expect(results).toEqual({

package/src/tester.js

@@ -1,9 +1,7 @@
-import {info} from '@travi/cli-messages';
-
 import {test as nvmIsUsed} from './node-version/index.js';
 import {test as jsPackageManagerIsUsed} from './package-managers/index.js';
 
-export default async function projectUsesJavaScript({projectRoot}) {
+export default async function projectUsesJavaScript({projectRoot}, {logger}) {
   const [nvmFound, jsPackageManagerFound] = await Promise.all([
     nvmIsUsed({projectRoot}),
     jsPackageManagerIsUsed({projectRoot})
@@ -11,7 +9,7 @@ export default async function projectUsesJavaScript({projectRoot}) {
 
   const jsProjectFound = nvmFound || jsPackageManagerFound;
 
-  if (jsProjectFound) info('JavaScript Project Detected');
+  if (jsProjectFound) logger.info('JavaScript Project Detected');
 
   return jsProjectFound;
 }

package/src/tester.test.js

@@ -1,4 +1,4 @@
-import {vi, it, describe, afterEach, expect} from 'vitest';
+import {describe, expect, it, vi} from 'vitest';
 import any from '@travi/any';
 import {when} from 'vitest-when';
 
@@ -11,27 +11,24 @@ vi.mock('./package-managers/index.js');
 
 describe('javascript predicate', () => {
   const projectRoot = any.string();
-
-  afterEach(() => {
-    vi.clearAllMocks();
-  });
+  const logger = {info: () => undefined};
 
   it('should return `true` if nvm is detected', async () => {
     when(nvmIsUsed).calledWith({projectRoot}).thenResolve(true);
 
-    expect(await testApplicability({projectRoot})).toBe(true);
+    expect(await testApplicability({projectRoot}, {logger})).toBe(true);
   });
 
   it('should return `true` if a javascript package manager is detected', async () => {
     when(jsPackageManagerIsUsed).calledWith({projectRoot}).thenResolve(true);
 
-    expect(await testApplicability({projectRoot})).toBe(true);
+    expect(await testApplicability({projectRoot}, {logger})).toBe(true);
   });
 
   it('should return `false` if neither nvm nor a js package manager is found', async () => {
     when(nvmIsUsed).calledWith({projectRoot}).thenResolve(false);
     when(jsPackageManagerIsUsed).calledWith({projectRoot}).thenResolve(false);
 
-    expect(await testApplicability({projectRoot})).toBe(false);
+    expect(await testApplicability({projectRoot}, {logger})).toBe(false);
   });
 });