fix(test): discover project tests before running

Author: chrisbbreuer

storage/framework/core/actions/src/test/feature.ts

@@ -1,16 +1,3 @@
-import process from 'node:process'
-import { log } from '@stacksjs/cli'
-import { projectPath } from '@stacksjs/path'
+import { runTestSuites } from './runner'
 
-const proc = Bun.spawn(['sh', '-c', 'bun test ./tests/feature/**'], {
-  cwd: projectPath(),
-  stdio: ['inherit', 'inherit', 'inherit'], // Inherit stdio to see the output in the console
-  // env: process.env, // Pass the environment variables
-})
-
-const exitCode = await proc.exited
-
-if (exitCode !== 0) {
-  log.error('Tests failed')
-  process.exit(exitCode ?? 1)
-}
+await runTestSuites(['feature'])

storage/framework/core/actions/src/test/index.ts

@@ -1,6 +1,4 @@
-import process from 'node:process'
-import { log } from '@stacksjs/cli'
-import { projectPath } from '@stacksjs/path'
+import { runTestSuites } from './runner'
 
 // Test runner shells out to `bun test` rather than running tests in-process so
 // each test file gets a fresh module graph (no leakage between suites).
@@ -11,19 +9,6 @@ import { projectPath } from '@stacksjs/path'
 //    had loaded once but the test runner spun a fresh subprocess per file
 //    and re-paid the cost.
 //
-//  - The glob list includes `./tests/integration/**` so that suite ships
-//    alongside `feature/` + `unit/` instead of being silently ignored.
-const proc = Bun.spawn(
-  ['sh', '-c', 'bun test --timeout 30000 ./tests/feature/** ./tests/integration/** ./tests/unit/**'],
-  {
-    cwd: projectPath(),
-    stdio: ['inherit', 'inherit', 'inherit'], // Inherit stdio to see the output in the console
-  },
-)
-
-const exitCode = await proc.exited
-
-if (exitCode !== 0) {
-  log.error('Tests failed')
-  process.exit(exitCode ?? 1)
-}
+// The project-wide filter intentionally includes root-level tests as well as
+// feature, integration, and unit suites.
+await runTestSuites([''], 30000)

storage/framework/core/actions/src/test/runner.ts

@@ -0,0 +1,40 @@
+import process from 'node:process'
+import { existsSync } from 'node:fs'
+import { join } from 'node:path'
+import { log } from '@stacksjs/cli'
+import { projectPath } from '@stacksjs/path'
+
+const testFiles = new Bun.Glob('**/*.{test,spec}.{js,jsx,ts,tsx,mjs,mts,cjs,cts}')
+
+export async function runTestSuites(suites: string[], timeout?: number): Promise<void> {
+  const project = projectPath()
+  const filters = suites.flatMap((suite) => {
+    const suiteDirectory = join(project, 'tests', suite)
+    if (!existsSync(suiteDirectory))
+      return []
+
+    return [...testFiles.scanSync({ cwd: suiteDirectory, onlyFiles: true })]
+      .map(file => suite ? `./tests/${suite}/${file}` : `./tests/${file}`)
+  })
+
+  if (filters.length === 0) {
+    log.info('No matching tests found.')
+    return
+  }
+
+  const args = ['bun', 'test']
+  if (timeout)
+    args.push('--timeout', String(timeout))
+  args.push(...filters)
+
+  const proc = Bun.spawn(args, {
+    cwd: project,
+    stdio: ['inherit', 'inherit', 'inherit'],
+  })
+  const exitCode = await proc.exited
+
+  if (exitCode !== 0) {
+    log.error('Tests failed')
+    process.exit(exitCode ?? 1)
+  }
+}

storage/framework/core/actions/src/test/unit.ts

@@ -1,16 +1,3 @@
-import process from 'node:process'
-import { log } from '@stacksjs/cli'
-import { projectPath } from '@stacksjs/path'
+import { runTestSuites } from './runner'
 
-const proc = Bun.spawn(['sh', '-c', 'bun test ./tests/unit/**'], {
-  cwd: projectPath(),
-  stdio: ['inherit', 'inherit', 'inherit'], // Inherit stdio to see the output in the console
-  // env: process.env, // Pass the environment variables
-})
-
-const exitCode = await proc.exited
-
-if (exitCode !== 0) {
-  log.error('Tests failed')
-  process.exit(exitCode ?? 1)
-}
+await runTestSuites(['unit'])