@sprucelabs/spruce-cli 19.1.5 → 19.1.7

package/node_modules/@typescript-eslint/visitor-keys/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@typescript-eslint/visitor-keys",
-  "version": "5.58.0",
+  "version": "5.59.6",
   "description": "Visitor keys used to help traverse the TypeScript-ESTree AST",
   "keywords": [
     "eslint",
@@ -39,7 +39,7 @@
     "typecheck": "tsc -p tsconfig.json --noEmit"
   },
   "dependencies": {
-    "@typescript-eslint/types": "5.58.0",
+    "@typescript-eslint/types": "5.59.6",
     "eslint-visitor-keys": "^3.3.0"
   },
   "devDependencies": {
@@ -56,5 +56,5 @@
       ]
     }
   },
-  "gitHead": "0c6b63658aa5b01188480150fd777a1fa9b277eb"
+  "gitHead": "5b0e577f2552e8b2c53a3fb22edc9d219589b937"
 }

package/package.json

@@ -17,7 +17,7 @@
     ]
   },
   "license": "MIT",
-  "version": "19.1.5",
+  "version": "19.1.7",
   "bin": {
     "spruce": "./build/index.js"
   },
@@ -71,25 +71,25 @@
     "watch.tsc": "tsc -w --sourceMap"
   },
   "dependencies": {
-    "@babel/cli": "^7.21.0",
+    "@babel/cli": "^7.21.5",
     "@babel/plugin-proposal-decorators": "^7.21.0",
-    "@babel/runtime": "^7.21.0",
+    "@babel/runtime": "^7.21.5",
     "@jest/reporters": "^29.5.0",
-    "@sprucelabs/error": "^5.0.601",
-    "@sprucelabs/globby": "^0.0.7",
-    "@sprucelabs/heartwood-view-controllers": "^102.1.53",
-    "@sprucelabs/jest-json-reporter": "^7.0.25",
-    "@sprucelabs/mercury-client": "^39.0.18",
-    "@sprucelabs/mercury-event-emitter": "^39.0.18",
-    "@sprucelabs/mercury-types": "^44.0.18",
-    "@sprucelabs/schema": "^28.6.9",
-    "@sprucelabs/spruce-core-schemas": "^38.0.18",
-    "@sprucelabs/spruce-event-utils": "^36.0.19",
-    "@sprucelabs/spruce-skill-utils": "^28.1.125",
-    "@sprucelabs/spruce-templates": "^19.1.5",
+    "@sprucelabs/error": "^5.1.2",
+    "@sprucelabs/globby": "^1.0.1",
+    "@sprucelabs/heartwood-view-controllers": "^103.6.12",
+    "@sprucelabs/jest-json-reporter": "^7.0.43",
+    "@sprucelabs/mercury-client": "^40.1.25",
+    "@sprucelabs/mercury-event-emitter": "^40.1.25",
+    "@sprucelabs/mercury-types": "^45.0.16",
+    "@sprucelabs/schema": "^28.6.28",
+    "@sprucelabs/spruce-core-schemas": "^38.1.16",
+    "@sprucelabs/spruce-event-utils": "^37.0.18",
+    "@sprucelabs/spruce-skill-utils": "^29.0.11",
+    "@sprucelabs/spruce-templates": "^19.1.7",
     "@typescript-eslint/eslint-plugin": "^5.27.1",
     "@typescript-eslint/parser": "^5.27.1",
-    "cfonts": "^3.1.1",
+    "cfonts": "^3.2.0",
     "chalk": "4.1.2",
     "chokidar": "^3.5.3",
     "cli-table3": "^0.6.3",
@@ -103,8 +103,8 @@
     "md5": "^2.3.0",
     "open": "^8.4.2",
     "ora": "5.4.1",
-    "semver": "^7.4.0",
-    "string-argv": "^0.3.1",
+    "semver": "^7.5.1",
+    "string-argv": "^0.3.2",
     "terminal-image": "1.2.1",
     "terminal-kit": "sprucelabsai/terminal-kit",
     "tree-kill": "^1.2.2",
@@ -113,47 +113,47 @@
     "uuid": "^9.0.0"
   },
   "devDependencies": {
-    "@sprucelabs/data-stores": "^23.0.12",
-    "@sprucelabs/jest-sheets-reporter": "^3.0.4",
-    "@sprucelabs/mercury-core-events": "^20.0.17",
-    "@sprucelabs/resolve-path-aliases": "^1.1.211",
-    "@sprucelabs/spruce-conversation-plugin": "^57.0.24",
-    "@sprucelabs/spruce-deploy-plugin": "^57.0.24",
-    "@sprucelabs/spruce-store-plugin": "^57.0.24",
-    "@sprucelabs/spruce-test-fixtures": "^57.0.24",
-    "@sprucelabs/test": "^7.7.440",
-    "@sprucelabs/test-utils": "^3.4.38",
+    "@sprucelabs/data-stores": "^24.0.9",
+    "@sprucelabs/jest-sheets-reporter": "^3.0.21",
+    "@sprucelabs/mercury-core-events": "^21.1.14",
+    "@sprucelabs/resolve-path-aliases": "^1.1.219",
+    "@sprucelabs/spruce-conversation-plugin": "^58.0.21",
+    "@sprucelabs/spruce-deploy-plugin": "^58.0.21",
+    "@sprucelabs/spruce-store-plugin": "^58.0.21",
+    "@sprucelabs/spruce-test-fixtures": "^58.0.21",
+    "@sprucelabs/test": "^7.7.447",
+    "@sprucelabs/test-utils": "^3.4.57",
     "@types/blessed": "^0.1.21",
     "@types/eslint": "^8.37.0",
     "@types/fs-extra": "^11.0.1",
     "@types/inquirer": "^8.2.1",
     "@types/lodash": "^4.14.194",
     "@types/md5": "^2.3.2",
-    "@types/node": "^18.15.11",
+    "@types/node": "^20.2.0",
     "@types/promise.allsettled": "^1.0.3",
     "@types/ps-node": "^0.1.1",
-    "@types/semver": "^7.3.13",
+    "@types/semver": "^7.5.0",
     "@types/sha1": "^1.1.3",
     "@types/slug": "^5.0.3",
-    "@types/superagent": "^4.1.16",
+    "@types/superagent": "^4.1.17",
     "@types/terminal-kit": "^2.5.1",
     "@types/uuid": "^9.0.1",
     "chokidar-cli": "^3.0.0",
     "concurrently": "^8.0.1",
     "conventional-changelog-sprucelabs": "^1.1.2",
     "dotenv": "^16.0.3",
-    "eslint": "^8.38.0",
+    "eslint": "^8.40.0",
     "eslint-config-spruce": "^10.12.0",
     "find-process": "^1.4.7",
     "jest": "^29.5.0",
     "jest-circus": "^29.5.0",
     "jest-junit": "^16.0.0",
     "jest-reporters": "^0.0.2",
-    "prettier": "^2.8.7",
+    "prettier": "^2.8.8",
     "ps-node": "^0.1.6",
     "ts-jest": "^29.1.0",
     "ts-node": "^10.9.1",
-    "tsc-watch": "^6.0.0",
+    "tsc-watch": "^6.0.4",
     "tsconfig-paths": "^4.2.0"
   },
   "testSkillCache": {
@@ -590,5 +590,5 @@
       "terminal-kit"
     ]
   },
-  "gitHead": "ee04cb9291c2130ae9a07b9ee559592b6f9f344b"
+  "gitHead": "6d676a2ac0b1fb23cd625ea6fbe97c3af1b55900"
 }

package/src/__tests__/behavioral/tests/CreatingATest.test.ts

@@ -59,33 +59,12 @@ export default class CreatingBehavioralTestsTest extends AbstractTestTest {
 			testType
 		)
 
-		uiAssert.assertSelectRenderChoice(this.ui, '.', testType)
-		uiAssert.assertSelectRenderChoice(this.ui, `dummy1`, `${testType}/dummy1`)
-
-		await this.ui.sendInput('.')
-		await this.waitAndSelectSubClass()
-
-		await promise
-	}
-
-	@test()
-	protected static async listsManyDirsIfExistInsideTestDir() {
-		await this.installTests()
-
-		const dirs = ['dir1', 'dir2', 'dir3']
-
-		for (const dir of dirs) {
-			this.createTestSubDir('behavioral', dir)
-		}
-
-		const { promise } = await this.installAndStartTestActionAndWaitForInput()
-
-		for (const dir of dirs) {
-			uiAssert.assertSelectRenderChoice(this.ui, `${dir}`, `behavioral/${dir}`)
-		}
-
-		await this.ui.sendInput('.')
+		uiAssert.assertRendersDirectorySelect(
+			this.ui,
+			this.resolvePath('src', '__tests__', testType)
+		)
 
+		await this.ui.sendInput('')
 		await this.waitAndSelectSubClass()
 
 		await promise
@@ -95,13 +74,13 @@ export default class CreatingBehavioralTestsTest extends AbstractTestTest {
 	@test('can select sub dir 2', 'test-2')
 	protected static async selectingAnOptionRendersToSubDir(dirName: string) {
 		await this.installTests()
-		this.createTestSubDir('behavioral', dirName)
+		const dir = this.createTestSubDir('behavioral', dirName)
 
 		const { promise } = await this.installAndStartTestActionAndWaitForInput(
 			'behavioral'
 		)
 
-		await this.ui.sendInput(`${dirName}`)
+		await this.ui.sendInput({ path: dir })
 
 		await this.waitAndSelectSubClass()
 
@@ -118,25 +97,6 @@ export default class CreatingBehavioralTestsTest extends AbstractTestTest {
 		assert.isEqual(expectedPath, results.files?.[0]?.path)
 	}
 
-	@test()
-	protected static async doesNotListFiles() {
-		await this.installTests()
-		this.createTestSubDir('behavioral', 'subdir')
-
-		const file = this.resolveTestDir('behavioral', 'test.ts')
-		diskUtil.writeFile(file, 'what the!?')
-
-		await this.installAndStartTestActionAndWaitForInput()
-
-		uiAssert.assertSelectDidNotRenderChoice(
-			this.ui,
-			'test',
-			`behavioral/test/test.ts`
-		)
-
-		this.ui.reset()
-	}
-
 	@test()
 	protected static async allTestsComeFakedToStart() {
 		const testFile = await this.createTestAndGetFile()
@@ -147,6 +107,7 @@ export default class CreatingBehavioralTestsTest extends AbstractTestTest {
 	private static createTestSubDir(...testDirs: string[]) {
 		const newDir = this.resolveTestDir(...testDirs)
 		diskUtil.createDir(newDir)
+		return newDir
 	}
 
 	private static resolveTestDir(...testDirs: string[]) {

package/src/__tests__/behavioral/tests/RunningTests.test.ts

@@ -3,6 +3,13 @@ import { errorAssert } from '@sprucelabs/test-utils'
 import AbstractTestTest from '../../../tests/AbstractTestTest'
 
 export default class RunningTestsTest extends AbstractTestTest {
+	private static hasCreatedTest: any
+
+	protected static async beforeEach(): Promise<void> {
+		await super.beforeEach()
+		this.hasCreatedTest = false
+	}
+
 	@test()
 	protected static async hasTestAction() {
 		await this.Cli()
@@ -12,35 +19,24 @@ export default class RunningTestsTest extends AbstractTestTest {
 	@test()
 	protected static async runningTestsActuallyRunsTests() {
 		await this.installTests()
-		const creationPromise = this.Action('test', 'create').execute({
-			type: 'behavioral',
+
+		const creationResults = await this.createTest({
 			nameReadable: 'Can book appointment',
 			nameCamel: 'canBookAppointment',
 			namePascal: 'CanBookAppointment',
 		})
 
-		await this.waitForInput()
-		await this.ui.sendInput('')
-
-		const creationResults = await creationPromise
-
 		const file = creationResults.files?.[0]
 		assert.isTruthy(file)
 
 		this.fixBadTest(file.path)
 
-		const promise = this.Action('test', 'create').execute({
-			type: 'behavioral',
+		await this.createTest({
 			nameReadable: 'Can cancel appointment',
 			nameCamel: 'canCancelAppointment',
 			namePascal: 'CanCancelAppointment',
 		})
 
-		await this.waitForInput()
-		await this.ui.sendInput('')
-
-		await promise
-
 		await this.Service('build').build()
 
 		const results = await this.Action('test', 'test').execute({
@@ -66,4 +62,28 @@ export default class RunningTestsTest extends AbstractTestTest {
 			totalTodo: 0,
 		})
 	}
+
+	private static async createTest(options: {
+		nameReadable: string
+		nameCamel: string
+		namePascal: string
+	}) {
+		const creationPromise = this.Action('test', 'create').execute({
+			type: 'behavioral',
+			...options,
+		})
+
+		if (this.hasCreatedTest) {
+			await this.waitForInput()
+			await this.ui.sendInput({ path: '.' })
+		}
+
+		this.hasCreatedTest = true
+
+		await this.waitForInput()
+		await this.ui.sendInput('')
+
+		const creationResults = await creationPromise
+		return creationResults
+	}
 }

package/src/__tests__/behavioral/tests/SelectingAnAbstractTestClass.test.ts

@@ -42,14 +42,14 @@ export default class SelectingAnAbstractTestClassTest extends AbstractTestTest {
 	@test()
 	protected static async canSelectAbstractClassWhileSelectingSubDir() {
 		const testDir = this.resolvePath('src', '__tests__', 'behavioral', 'taco')
+
 		diskUtil.createDir(testDir)
 
 		await this.installAndCopyTestFiles()
 
 		const { promise } = await this.invokeCreateActionAndWaitForInput()
 
-		await this.ui.sendInput('taco')
-
+		await this.ui.sendInput({ path: 'taco' })
 		await this.waitForInput()
 
 		this.selectOptionBasedOnLabel('AbstractBananaTestDifferentThanFileName')

package/src/__tests__/implementation/TestRunner.test.ts

@@ -8,9 +8,11 @@ import { CliInterface } from '../../types/cli.types'
 
 export default class TestRunnerTest extends AbstractTestTest {
 	protected static testRunner: TestRunner
+	private static hasCreatedTest: boolean
 
 	protected static async beforeEach() {
 		await super.beforeEach()
+		this.hasCreatedTest = false
 		this.testRunner = new TestRunner({
 			cwd: this.cwd,
 			commandService: this.Service('command'),
@@ -143,6 +145,13 @@ export default class TestRunnerTest extends AbstractTestTest {
 		})
 
 		await this.waitForInput()
+		if (this.hasCreatedTest) {
+			await this.ui.sendInput('')
+			await this.waitForInput()
+		}
+
+		this.hasCreatedTest = true
+
 		await this.ui.sendInput('')
 
 		const results = await promise

package/src/features/test/actions/CreateAction.ts

@@ -38,10 +38,7 @@ export default class CreateAction extends AbstractAction<OptionsSchema> {
 		const candidates = await testFeature.buildParentClassCandidates()
 
 		if (diskUtil.doesDirExist(resolvedDestination)) {
-			resolvedDestination = await this.promptForSubDir(
-				resolvedDestination,
-				type
-			)
+			resolvedDestination = await this.promptForSubDir(resolvedDestination)
 		}
 
 		if (candidates.length > 0) {
@@ -70,34 +67,18 @@ export default class CreateAction extends AbstractAction<OptionsSchema> {
 			hints: ["run `spruce test` in your skill when you're ready!"],
 		}
 	}
-	private async promptForSubDir(resolvedDestination: string, type: string) {
-		const subdirs = diskUtil
-			.readDir(resolvedDestination)
-			.filter((d) =>
-				diskUtil.isDir(diskUtil.resolvePath(resolvedDestination, d))
-			)
-
-		if (subdirs.length > 0) {
-			const match = await this.ui.prompt({
-				type: 'select',
-				label: 'Where should I write this test?',
-				isRequired: true,
-				options: {
-					choices: [
-						{
-							value: '.',
-							label: `${type}`,
-						},
-						...subdirs.map((dir) => ({
-							value: `${dir}`,
-							label: `${type}/${dir}`,
-						})),
-					],
-				},
-			})
-
-			resolvedDestination = diskUtil.resolvePath(resolvedDestination, match)
-		}
+	private async promptForSubDir(resolvedDestination: string) {
+		const match = await this.ui.prompt({
+			type: 'directory',
+			label: 'Where should I write this test?',
+			isRequired: true,
+			defaultValue: {
+				path: diskUtil.resolvePath(resolvedDestination),
+			},
+		})
+
+		resolvedDestination = diskUtil.resolvePath(resolvedDestination, match.path)
+
 		return resolvedDestination
 	}
 

package/src/interfaces/SpyInterface.ts

@@ -91,11 +91,17 @@ export default class SpyInterface implements GraphicsInterface {
 		return this.invocations[this.invocations.length - 1]
 	}
 
-	public async sendInput(input: string | string[]): Promise<void> {
+	public async sendInput(input: Input): Promise<void> {
 		this.trackInvocation('sendInput', input)
 
 		this.optionallyRenderLine(
-			`Sending input: "${input.length > 0 ? input : 'ENTER'}"`
+			`Sending input: "${
+				(input as FilePromptInput).path
+					? (input as FilePromptInput).path
+					: (input as string).length > 0
+					? input
+					: 'ENTER'
+			}"`
 		)
 
 		if (this.waitForEnterResolver) {
@@ -114,7 +120,7 @@ export default class SpyInterface implements GraphicsInterface {
 
 			resolver(
 				input === '\n' ||
-					input.length === 0 ||
+					(input as string).length === 0 ||
 					(typeof input === 'string' && input.toLowerCase() === 'y')
 			)
 		} else {
@@ -360,3 +366,9 @@ export default class SpyInterface implements GraphicsInterface {
 		this.trackInvocation('clearBelowCursor')
 	}
 }
+
+type FilePromptInput = {
+	path: string
+}
+
+type Input = string | string[] | FilePromptInput

package/src/tests/utilities/uiAssert.utility.ts

@@ -36,6 +36,15 @@ const uiAssert = {
 		})
 	},
 
+	assertRendersDirectorySelect(ui: SpyInterface, defaultValue: string) {
+		const last = ui.getLastInvocation()
+
+		assert.doesInclude(last.options, {
+			type: 'directory',
+			defaultValue: { path: defaultValue },
+		})
+	},
+
 	async assertRendersConfirmWriteFile(ui: SpyInterface) {
 		await ui.waitForInput()