@sprucelabs/spruce-cli 24.1.0 → 24.1.1

package/src/__tests__/behavioral/DeployingASkill.test.ts

@@ -1,7 +1,7 @@
 import { diskUtil } from '@sprucelabs/spruce-skill-utils'
 import { test, assert } from '@sprucelabs/test-utils'
 import { errorAssert } from '@sprucelabs/test-utils'
-import CommandService from '../../services/CommandService'
+import CommandServiceImpl from '../../services/CommandService'
 import LintService from '../../services/LintService'
 import AbstractCliTest from '../../tests/AbstractCliTest'
 import testUtil from '../../tests/utilities/test.utility'
@@ -15,43 +15,46 @@ export default class DeployingASkillTest extends AbstractCliTest {
 
     protected static async beforeEach() {
         await super.beforeEach()
-        CommandService.fakeCommand('which heroku', {
+        CommandServiceImpl.fakeCommand('which heroku', {
             code: 0,
         })
-        CommandService.fakeCommand('grep api.heroku.com ~/.netrc', {
+        CommandServiceImpl.fakeCommand('grep api.heroku.com ~/.netrc', {
             code: 0,
         })
-        CommandService.fakeCommand('git status', {
+        CommandServiceImpl.fakeCommand('git status', {
             code: 0,
         })
-        CommandService.fakeCommand('git init', {
+        CommandServiceImpl.fakeCommand('git init', {
             code: 0,
             stdout: 'Initialized empty Git repository in',
         })
 
-        CommandService.fakeCommand('git ls-remote heroku', {
+        CommandServiceImpl.fakeCommand('git ls-remote heroku', {
             code: 0,
         })
 
-        CommandService.fakeCommand('which git', {
+        CommandServiceImpl.fakeCommand('which git', {
             code: 0,
         })
 
-        CommandService.fakeCommand('heroku create good-heroku-name', {
+        CommandServiceImpl.fakeCommand('heroku create good-heroku-name', {
             code: 0,
         })
 
-        CommandService.fakeCommand('heroku create bad-heroku-name', {
+        CommandServiceImpl.fakeCommand('heroku create bad-heroku-name', {
             code: 1,
         })
 
-        CommandService.fakeCommand('heroku buildpacks:set heroku/nodejs', {
+        CommandServiceImpl.fakeCommand('heroku buildpacks:set heroku/nodejs', {
             code: 0,
         })
 
-        CommandService.fakeCommand('git push --set-upstream heroku master', {
-            code: 0,
-        })
+        CommandServiceImpl.fakeCommand(
+            'git push --set-upstream heroku master',
+            {
+                code: 0,
+            }
+        )
 
         diskUtil.writeFile(this.resolvePath('Procfile'), 'web: npm run boot')
     }
@@ -142,7 +145,7 @@ export default class DeployingASkillTest extends AbstractCliTest {
 
     @test()
     protected static async errorsIfHerokuClientNotInstalled() {
-        CommandService.fakeCommand('which heroku', {
+        CommandServiceImpl.fakeCommand('which heroku', {
             code: 1,
         })
 
@@ -165,7 +168,7 @@ export default class DeployingASkillTest extends AbstractCliTest {
 
     @test()
     protected static async errorsIfGitNotInstalled() {
-        CommandService.fakeCommand('which git', {
+        CommandServiceImpl.fakeCommand('which git', {
             code: 1,
         })
 
@@ -188,7 +191,7 @@ export default class DeployingASkillTest extends AbstractCliTest {
 
     @test()
     protected static async errorsIfNotInGitRepo() {
-        CommandService.fakeCommand('git status', {
+        CommandServiceImpl.fakeCommand('git status', {
             code: 128,
             stderr: 'fatal: not a git repository (or any of the parent directories): .git',
         })
@@ -217,7 +220,7 @@ export default class DeployingASkillTest extends AbstractCliTest {
 
     @test()
     protected static async canCreateGitRepoIfNeeded() {
-        CommandService.fakeCommand('git status', {
+        CommandServiceImpl.fakeCommand('git status', {
             code: 128,
         })
 
@@ -233,7 +236,7 @@ export default class DeployingASkillTest extends AbstractCliTest {
 
         await this.waitForInput()
 
-        CommandService.fakeCommand('git status', {
+        CommandServiceImpl.fakeCommand('git status', {
             code: 0,
         })
 
@@ -246,7 +249,7 @@ export default class DeployingASkillTest extends AbstractCliTest {
 
     @test()
     protected static async errorsWhenNotLoggedIntoHerkou() {
-        CommandService.fakeCommand('grep api.heroku.com ~/.netrc', {
+        CommandServiceImpl.fakeCommand('grep api.heroku.com ~/.netrc', {
             code: 1,
         })
 
@@ -329,7 +332,7 @@ export default class DeployingASkillTest extends AbstractCliTest {
 
     @test()
     protected static async failsWhenDeclineToCreateRemoteBranch() {
-        CommandService.fakeCommand('git ls-remote heroku', {
+        CommandServiceImpl.fakeCommand('git ls-remote heroku', {
             code: 128,
         })
 
@@ -361,7 +364,7 @@ export default class DeployingASkillTest extends AbstractCliTest {
 
     @test()
     protected static async asksForHerokuAppName() {
-        CommandService.fakeCommand('git ls-remote heroku', {
+        CommandServiceImpl.fakeCommand('git ls-remote heroku', {
             code: 128,
         })
 
@@ -397,7 +400,7 @@ export default class DeployingASkillTest extends AbstractCliTest {
 
     @test()
     protected static async keepsAskingForAppNameUntilAGoodOneIsSelected() {
-        CommandService.fakeCommand('git ls-remote heroku', {
+        CommandServiceImpl.fakeCommand('git ls-remote heroku', {
             code: 128,
         })
         await this.FeatureFixture().installCachedFeatures('deploy')
@@ -439,7 +442,7 @@ export default class DeployingASkillTest extends AbstractCliTest {
 
     @test()
     protected static async failsWithPendingChangesToCommit() {
-        CommandService.fakeCommand('git status', {
+        CommandServiceImpl.fakeCommand('git status', {
             code: 0,
             stdout: 'Changes not staged for commit',
         })

package/src/__tests__/behavioral/EnablingAndDisablingCache.test.ts

@@ -4,7 +4,7 @@ import {
     DISABLE_NPM_CACHE_COMMAND,
     ENABLE_NPM_CACHE_COMMAND,
 } from '../../features/cache/constants'
-import CommandService from '../../services/CommandService'
+import CommandServiceImpl from '../../services/CommandService'
 import AbstractCliTest from '../../tests/AbstractCliTest'
 
 export default class EnablingAndDisablingCacheTest extends AbstractCliTest {
@@ -21,7 +21,7 @@ export default class EnablingAndDisablingCacheTest extends AbstractCliTest {
 
     @test()
     protected static async returnsErrorWhenEnablingIfDockerIsNotInstalled() {
-        CommandService.fakeCommand(/which docker/gis, {
+        CommandServiceImpl.fakeCommand(/which docker/gis, {
             code: 1,
         })
 
@@ -39,7 +39,7 @@ export default class EnablingAndDisablingCacheTest extends AbstractCliTest {
 
     @test()
     protected static async returnsErrorWhenDockerNotEnabled() {
-        CommandService.fakeCommand(/npm config/gis, {
+        CommandServiceImpl.fakeCommand(/npm config/gis, {
             code: 1,
             stderr: 'which',
         })
@@ -52,13 +52,13 @@ export default class EnablingAndDisablingCacheTest extends AbstractCliTest {
 
     @test()
     protected static async canEnableCache() {
-        CommandService.fakeCommand(/which docker/gis, {
+        CommandServiceImpl.fakeCommand(/which docker/gis, {
             code: 0,
         })
-        CommandService.fakeCommand(ENABLE_NPM_CACHE_COMMAND, {
+        CommandServiceImpl.fakeCommand(ENABLE_NPM_CACHE_COMMAND, {
             code: 0,
         })
-        CommandService.fakeCommand(DISABLE_NPM_CACHE_COMMAND, {
+        CommandServiceImpl.fakeCommand(DISABLE_NPM_CACHE_COMMAND, {
             code: 0,
         })
 
@@ -73,7 +73,7 @@ export default class EnablingAndDisablingCacheTest extends AbstractCliTest {
 
     @test()
     protected static async returnsErrorWhenDisablingIfDockerIsNotInstalled() {
-        CommandService.fakeCommand(/which docker/gis, {
+        CommandServiceImpl.fakeCommand(/which docker/gis, {
             code: 1,
         })
 
@@ -91,10 +91,10 @@ export default class EnablingAndDisablingCacheTest extends AbstractCliTest {
 
     @test()
     protected static async returnsErrorWhenDisablingIfDockerIsNotStarted() {
-        CommandService.fakeCommand(/which docker/gis, {
+        CommandServiceImpl.fakeCommand(/which docker/gis, {
             code: 0,
         })
-        CommandService.fakeCommand(/npm config/gis, {
+        CommandServiceImpl.fakeCommand(/npm config/gis, {
             code: 1,
         })
 
@@ -112,7 +112,7 @@ export default class EnablingAndDisablingCacheTest extends AbstractCliTest {
 
     @test()
     protected static async returnsErrorIfCacheIsNotInstalled() {
-        CommandService.fakeCommand(/npm config/gis, {
+        CommandServiceImpl.fakeCommand(/npm config/gis, {
             code: 1,
             stderr: 'tsanoehusnatohu snatoh No such container staoheu saotnhu ',
         })
@@ -125,13 +125,13 @@ export default class EnablingAndDisablingCacheTest extends AbstractCliTest {
 
     @test()
     protected static async canEnableCacheMultipleTimes() {
-        CommandService.fakeCommand(/which docker/gis, {
+        CommandServiceImpl.fakeCommand(/which docker/gis, {
             code: 0,
         })
-        CommandService.fakeCommand(DISABLE_NPM_CACHE_COMMAND, {
+        CommandServiceImpl.fakeCommand(DISABLE_NPM_CACHE_COMMAND, {
             code: 0,
         })
-        CommandService.fakeCommand(ENABLE_NPM_CACHE_COMMAND, {
+        CommandServiceImpl.fakeCommand(ENABLE_NPM_CACHE_COMMAND, {
             code: 0,
         })
 

package/src/__tests__/behavioral/lint/LintService.test.ts

@@ -1,5 +1,5 @@
 import { test, assert, generateId } from '@sprucelabs/test-utils'
-import CommandService from '../../../services/CommandService'
+import CommandServiceImpl from '../../../services/CommandService'
 import LintService from '../../../services/LintService'
 import AbstractCliTest from '../../../tests/AbstractCliTest'
 
@@ -19,7 +19,7 @@ export default class LintServiceTest extends AbstractCliTest {
     protected static async throwsWhenLintReturnsMessage() {
         await this.FeatureFixture().installCachedFeatures('skills')
 
-        CommandService.fakeCommand(/node/, {
+        CommandServiceImpl.fakeCommand(/node/, {
             code: 0,
             stdout: `[{"filePath":"/Users/taylorromero/Development/SpruceLabs/spruce-appointments-skill/src/.spruce/stores/stores.types.ts","messages":[{"fatal":false,"severity":1,"message":"File ignored by default.  Use a negated ignore pattern (like \\"--ignore-pattern '!<relative/path/to/filename>'\\") to override."}],"errorCount":1,"warningCount":0,"fixableErrorCount":0,"fixableWarningCount":0,"usedDeprecatedRules":[]}]\n`,
         })
@@ -31,7 +31,7 @@ export default class LintServiceTest extends AbstractCliTest {
     protected static async worksWhenNoMessagesReturned() {
         await this.FeatureFixture().installCachedFeatures('skills')
 
-        CommandService.fakeCommand(/node/, {
+        CommandServiceImpl.fakeCommand(/node/, {
             code: 0,
             stdout: `[{"filePath":"/Users/taylorromero/Development/SpruceLabs/spruce-appointments-skill/src/.spruce/stores/stores.types.ts","messages":[{"fatal":false,"severity":1,"message":"File ignored by default.  Use a negated ignore pattern (like \\"--ignore-pattern '!<relative/path/to/filename>'\\") to override."}],"errorCount":0,"warningCount":1,"fixableErrorCount":0,"fixableWarningCount":0,"usedDeprecatedRules":[]}]\n`,
         })

package/src/__tests__/behavioral/node/CreatingANodeModule.test.ts

@@ -1,6 +1,6 @@
 import { diskUtil } from '@sprucelabs/spruce-skill-utils'
 import { test, assert } from '@sprucelabs/test-utils'
-import CommandService from '../../../services/CommandService'
+import CommandServiceImpl from '../../../services/CommandService'
 import AbstractCliTest from '../../../tests/AbstractCliTest'
 import testUtil from '../../../tests/utilities/test.utility'
 
@@ -12,7 +12,7 @@ export default class CreatingANodeModuleTest extends AbstractCliTest {
 
     @test()
     protected static async canCreateAtDestination() {
-        CommandService.fakeCommand(new RegExp(/yarn add/gis), {
+        CommandServiceImpl.fakeCommand(new RegExp(/yarn add/gis), {
             code: 0,
         })
         const promise = this.Action('node', 'create', {

package/src/__tests__/behavioral/onboard/StartingOnboarding.test.ts

@@ -1,6 +1,6 @@
 import { test, assert } from '@sprucelabs/test-utils'
 import CliGlobalEmitter, { globalContract } from '../../../GlobalEmitter'
-import CommandService from '../../../services/CommandService'
+import CommandServiceImpl from '../../../services/CommandService'
 import AbstractCliTest from '../../../tests/AbstractCliTest'
 
 class SpyEmitter extends CliGlobalEmitter {
@@ -32,7 +32,7 @@ export default class StartingOnboardingTest extends AbstractCliTest {
 
     @test()
     protected static async onboardingThroughSkillCreateThenShutsOff() {
-        CommandService.fakeCommand(/yarn/, { code: 0 })
+        CommandServiceImpl.fakeCommand(/yarn/, { code: 0 })
 
         const onboardAction = this.Action('onboard', 'onboard')
         const onboardPromise = onboardAction.execute({})

package/src/__tests__/behavioral/skill/CreatingASkill.test.ts

@@ -1,12 +1,12 @@
 import { test, assert } from '@sprucelabs/test-utils'
-import CommandService from '../../../services/CommandService'
+import CommandServiceImpl from '../../../services/CommandService'
 import AbstractCliTest from '../../../tests/AbstractCliTest'
 import testUtil from '../../../tests/utilities/test.utility'
 
 export default class CreatingASkillTest extends AbstractCliTest {
     protected static async beforeEach() {
         await super.beforeEach()
-        CommandService.fakeCommand(new RegExp(/npm.*?install .*?/gis), {
+        CommandServiceImpl.fakeCommand(new RegExp(/npm.*?install .*?/gis), {
             code: 0,
         })
     }

package/src/__tests__/behavioral/skill/RebuildingASkill.test.ts

@@ -1,6 +1,6 @@
 import { test, assert } from '@sprucelabs/test-utils'
 import { errorAssert } from '@sprucelabs/test-utils'
-import CommandService from '../../../services/CommandService'
+import CommandServiceImpl from '../../../services/CommandService'
 import AbstractSkillTest from '../../../tests/AbstractSkillTest'
 
 export default class RebuildingASkillTest extends AbstractSkillTest {
@@ -13,7 +13,7 @@ export default class RebuildingASkillTest extends AbstractSkillTest {
 
     @test()
     protected static async runsExpectedCommand() {
-        CommandService.fakeCommand('yarn rebuild', { code: 0 })
+        CommandServiceImpl.fakeCommand('yarn rebuild', { code: 0 })
 
         const results = await this.Action('skill', 'rebuild').execute({
             shouldPlayGames: false,
@@ -24,7 +24,7 @@ export default class RebuildingASkillTest extends AbstractSkillTest {
 
     @test()
     protected static async handlesError() {
-        CommandService.fakeCommand('yarn rebuild', { code: 1 })
+        CommandServiceImpl.fakeCommand('yarn rebuild', { code: 1 })
 
         const results = await this.Action('skill', 'rebuild').execute({
             shouldPlayGames: false,

package/src/__tests__/behavioral/stores/CreatingADataStore.test.ts

@@ -2,7 +2,7 @@ import { diskUtil } from '@sprucelabs/spruce-skill-utils'
 import { test, assert } from '@sprucelabs/test-utils'
 import { errorAssert } from '@sprucelabs/test-utils'
 import '@sprucelabs/spruce-store-plugin'
-import CommandService from '../../../services/CommandService'
+import CommandServiceImpl from '../../../services/CommandService'
 import AbstractSkillTest from '../../../tests/AbstractSkillTest'
 import testUtil from '../../../tests/utilities/test.utility'
 
@@ -159,11 +159,11 @@ export default class CreatingDataStoresTest extends AbstractSkillTest {
 
     @test()
     protected static async upgradeRestoresDataStoreTypes() {
-        CommandService.fakeCommand('yarn rebuild', {
+        CommandServiceImpl.fakeCommand('yarn rebuild', {
             code: 0,
         })
 
-        CommandService.fakeCommand(/yarn add/gis, {
+        CommandServiceImpl.fakeCommand(/yarn add/gis, {
             code: 0,
         })
 

package/src/__tests__/behavioral/tests/migrationToInstance/StaticToInstanceMigrator.test.ts

@@ -5,6 +5,11 @@ import AbstractSpruceTest, {
     suite,
     generateId,
 } from '@sprucelabs/test-utils'
+import {
+    CommandService,
+    ExecuteCommandOptions,
+} from '../../../../services/CommandService'
+import LintService from '../../../../services/LintService'
 import StaticTestFinderImpl, {
     StaticTestFinder,
 } from '../../../../tests/staticToInstanceMigration/StaticTestFinder'
@@ -23,6 +28,7 @@ export default class StaticToInstanceMigratorTest extends AbstractSpruceTest {
     private readFiles: string[] = []
     private fakedFileContents: Record<string, string> = {}
     private lastWrittenFile?: { destination: string; contents: string }
+    private spyCommandService: SpyCommandService
 
     public constructor() {
         super()
@@ -48,10 +54,16 @@ export default class StaticToInstanceMigratorTest extends AbstractSpruceTest {
         this.testFinder = StaticTestFinderImpl.Finder() as FakeStaticTestFinder
         this.testFileMigrator =
             StaticToInstanceTestFileMigratorImpl.Migrator() as FakeFileMigrator
+        this.spyCommandService = new SpyCommandService()
+        const lintService = new LintService(
+            this.cwd,
+            () => this.spyCommandService
+        )
 
         this.migrator = StaticToInstanceMigratorImpl.Migrator({
             testFinder: this.testFinder,
             testFileMigrator: this.testFileMigrator,
+            lintService,
         })
     }
 
@@ -62,7 +74,7 @@ export default class StaticToInstanceMigratorTest extends AbstractSpruceTest {
             StaticToInstanceMigratorImpl.Migrator()
         )
         errorAssert.assertError(err, 'MISSING_PARAMETERS', {
-            parameters: ['testFinder', 'testFileMigrator'],
+            parameters: ['testFinder', 'testFileMigrator', 'lintService'],
         })
     }
 
@@ -203,6 +215,33 @@ export default class StaticToInstanceMigratorTest extends AbstractSpruceTest {
         assert.isUndefined(this.lastWrittenFile)
     }
 
+    @test()
+    protected async lintsMigratedFiles() {
+        await this.run()
+        assert.isTrue(this.spyCommandService.didExecute)
+        assert.doesInclude(
+            this.spyCommandService.lastCommand,
+            `await cli.lintFiles(['**/*.ts'])`
+        )
+    }
+
+    @test()
+    protected async shouldLintAfterMigrating() {
+        this.setFakedFinderResults(['match'])
+        this.setFakedFileContents({
+            match: generateId(),
+        })
+
+        StaticToInstanceMigratorImpl.diskUtil.writeFile = () => {
+            assert.isFalse(
+                this.spyCommandService.didExecute,
+                'Should not lint yet'
+            )
+        }
+
+        await this.run()
+    }
+
     private setMigrateFileResponse(contents: string) {
         this.testFileMigrator.fakedMigrateResponse = contents
     }
@@ -250,3 +289,22 @@ class FakeFileMigrator implements StaticToInstanceTestFileMigrator {
         )
     }
 }
+
+class SpyCommandService implements CommandService {
+    public didExecute = false
+    public lastCommand?: string
+    public async execute(
+        cmd: string,
+        options?: ExecuteCommandOptions
+    ): Promise<{ stdout: string }> {
+        this.lastCommand = options?.args?.[1] ?? cmd
+        this.didExecute = true
+        return {
+            stdout: generateId(),
+        }
+    }
+    public kill(): void {}
+    public pid(): number | undefined {
+        return 0
+    }
+}

package/src/__tests__/behavioral/upgrading/UpdatingDependencies2.test.ts

@@ -1,7 +1,7 @@
 import { test, assert } from '@sprucelabs/test-utils'
 import { random, uniq } from 'lodash'
 import UpdateDependenciesAction from '../../../features/node/actions/UpdateDependenciesAction'
-import CommandService from '../../../services/CommandService'
+import CommandServiceImpl from '../../../services/CommandService'
 import AbstractCliTest from '../../../tests/AbstractCliTest'
 import { NpmPackage } from '../../../types/cli.types'
 
@@ -27,7 +27,7 @@ export default class UpdateDependencies2Test extends AbstractCliTest {
 
         const passedArgs: string[] = []
 
-        CommandService.fakeCommand(/(npm|yarn).*?(install|add)/, {
+        CommandServiceImpl.fakeCommand(/(npm|yarn).*?(install|add)/, {
             code: 0,
             callback: (_, args) => {
                 passedArgs.push(...args)
@@ -51,7 +51,7 @@ export default class UpdateDependencies2Test extends AbstractCliTest {
 
         let passedArgs: any[] = []
 
-        CommandService.fakeCommand(/(npm|yarn).*?(install|add)/, {
+        CommandServiceImpl.fakeCommand(/(npm|yarn).*?(install|add)/, {
             code: 0,
             callback: (_, args) => {
                 passedArgs.push(args)

package/src/__tests__/behavioral/upgrading/UpgradingANodeModule.test.ts

@@ -1,6 +1,6 @@
 import { diskUtil } from '@sprucelabs/spruce-skill-utils'
 import { test, assert } from '@sprucelabs/test-utils'
-import CommandService from '../../../services/CommandService'
+import CommandServiceImpl from '../../../services/CommandService'
 import AbstractCliTest from '../../../tests/AbstractCliTest'
 import uiAssert from '../../../tests/utilities/uiAssert.utility'
 
@@ -16,9 +16,9 @@ export default class UpgradingANodeModuleTest extends AbstractCliTest {
 
     @test()
     protected static async buildErrorsGetPassedThroughToResults() {
-        CommandService.fakeCommand(/yarn (add|install)/gis, { code: 0 })
-        CommandService.fakeCommand(/yarn clean.build/gis, { code: 1 })
-        CommandService.fakeCommand(/yarn build.dev/gis, { code: 0 })
+        CommandServiceImpl.fakeCommand(/yarn (add|install)/gis, { code: 0 })
+        CommandServiceImpl.fakeCommand(/yarn clean.build/gis, { code: 1 })
+        CommandServiceImpl.fakeCommand(/yarn build.dev/gis, { code: 0 })
 
         await this.emitter.on('feature.did-execute', () => {
             return {
@@ -33,7 +33,7 @@ export default class UpgradingANodeModuleTest extends AbstractCliTest {
 
     @test()
     protected static async upgradingWritesExpectedFiles() {
-        CommandService.fakeCommand(/build/gis, { code: 0 })
+        CommandServiceImpl.fakeCommand(/build/gis, { code: 0 })
 
         const shouldNotBeFound = [
             'src/.spruce/skill.ts',
@@ -58,7 +58,7 @@ export default class UpgradingANodeModuleTest extends AbstractCliTest {
             diskUtil.writeFile(tsConfig, 'beenChanged')
         }
 
-        CommandService.fakeCommand(/yarn/gi, {
+        CommandServiceImpl.fakeCommand(/yarn/gi, {
             code: 0,
         })
 

package/src/__tests__/behavioral/upgrading/UpgradingANonSpruceNodeModule.test.ts

@@ -1,6 +1,6 @@
 import { HASH_SPRUCE_DIR, diskUtil } from '@sprucelabs/spruce-skill-utils'
 import { test, assert, generateId } from '@sprucelabs/test-utils'
-import CommandService from '../../../services/CommandService'
+import CommandServiceImpl from '../../../services/CommandService'
 import PkgService from '../../../services/PkgService'
 import AbstractCliTest from '../../../tests/AbstractCliTest'
 import ScriptUpdaterImpl, {
@@ -61,7 +61,7 @@ export default class UpgradingANonSpruceNodeModuleTest extends AbstractCliTest {
 
         let wasHit = false
 
-        CommandService.fakeCommand(/yarn.*?-W/, {
+        CommandServiceImpl.fakeCommand(/yarn.*?-W/, {
             code: 0,
             callback: () => {
                 wasHit = true
@@ -79,7 +79,7 @@ export default class UpgradingANonSpruceNodeModuleTest extends AbstractCliTest {
 
         const commands: string[] = []
 
-        CommandService.fakeCommand(/yarn.*?add .*/gis, {
+        CommandServiceImpl.fakeCommand(/yarn.*?add .*/gis, {
             code: 0,
             callback: (command, args) => {
                 commands.push([command, ...args].join(' '))
@@ -107,7 +107,7 @@ export default class UpgradingANonSpruceNodeModuleTest extends AbstractCliTest {
     protected static async runsUpgradeInEveryPackageInWorkspace() {
         this.setWorkspaces(['packages/*'])
 
-        CommandService.clearFakedResponses()
+        CommandServiceImpl.clearFakedResponses()
 
         const moduleName = 'test1'
         const path = this.resolvePath('packages', moduleName)

package/src/__tests__/behavioral/upgrading/UpgradingASkill3.test.ts

@@ -3,7 +3,7 @@ import { diskUtil } from '@sprucelabs/spruce-skill-utils'
 import { test, assert } from '@sprucelabs/test-utils'
 import SyncAction from '../../../features/error/actions/SyncAction'
 import UpdateDependenciesAction from '../../../features/node/actions/UpdateDependenciesAction'
-import CommandService from '../../../services/CommandService'
+import CommandServiceImpl from '../../../services/CommandService'
 import AbstractCliTest from '../../../tests/AbstractCliTest'
 import testUtil from '../../../tests/utilities/test.utility'
 export default class UpgradingASkill3Test extends AbstractCliTest {
@@ -17,14 +17,14 @@ export default class UpgradingASkill3Test extends AbstractCliTest {
         }
 
         await super.beforeEach()
-        CommandService.fakeCommand(new RegExp(/yarn rebuild/gis), {
+        CommandServiceImpl.fakeCommand(new RegExp(/yarn rebuild/gis), {
             code: 0,
         })
     }
 
     @test()
     protected static async doesNotAddResolvePathAliasesToDependenciesAfterUpgrade() {
-        CommandService.clearFakedResponses()
+        CommandServiceImpl.clearFakedResponses()
         await this.FeatureFixture().installCachedFeatures('views')
 
         await this.Action('node', 'upgrade').execute({})
@@ -36,7 +36,7 @@ export default class UpgradingASkill3Test extends AbstractCliTest {
 
     @test()
     protected static async featuresNotEnabledDontInstall() {
-        CommandService.clearFakedResponses()
+        CommandServiceImpl.clearFakedResponses()
         await this.FeatureFixture().installCachedFeatures('schemas')
 
         const results = await this.Action('node', 'upgrade').execute({})
@@ -93,7 +93,7 @@ export default class UpgradingASkill3Test extends AbstractCliTest {
             return {}
         }
 
-        CommandService.fakeCommand('yarn clean.build', {
+        CommandServiceImpl.fakeCommand('yarn clean.build', {
             code: 0,
             callback: () => {
                 wasCleanBuildCalled = true
@@ -102,7 +102,7 @@ export default class UpgradingASkill3Test extends AbstractCliTest {
 
         let wasBuildDevCalled = false
 
-        CommandService.fakeCommand('yarn build.dev', {
+        CommandServiceImpl.fakeCommand('yarn build.dev', {
             code: 0,
             callback: () => {
                 wasBuildDevCalled = true
@@ -217,17 +217,17 @@ export default class UpgradingASkill3Test extends AbstractCliTest {
     private static disableCleanBuildAndYarnAdd() {
         this.disableCleanAndBuild()
 
-        CommandService.fakeCommand(/yarn.*?add/gis, {
+        CommandServiceImpl.fakeCommand(/yarn.*?add/gis, {
             code: 0,
         })
     }
 
     private static disableCleanAndBuild() {
-        CommandService.fakeCommand('yarn clean.build', {
+        CommandServiceImpl.fakeCommand('yarn clean.build', {
             code: 0,
         })
 
-        CommandService.fakeCommand('yarn build.dev', {
+        CommandServiceImpl.fakeCommand('yarn build.dev', {
             code: 0,
         })
     }