fix: run build before smoke test

Author: Yukaii

.github/workflows/test.yml

@@ -54,5 +54,8 @@ jobs:
       - name: Install dependencies
         run: pnpm install --frozen-lockfile
 
+      - name: Build project
+        run: pnpm run build
+
       - name: Run smoke tests
-        run: pnpm run test:smoke
+        run: pnpm exec mocha --forbid-only "test/smoke/**/*.test.ts"

package.json

@@ -74,7 +74,7 @@
     "postpack": "shx rm -f oclif.manifest.json",
     "posttest": "pnpm run lint",
     "prepack": "pnpm run build && oclif manifest && oclif readme",
-    "test": "mocha --forbid-only \"test/**/*.test.ts\"",
+    "test": "mocha --forbid-only \"test/**/*.test.ts\" --exclude \"test/smoke/**/*\"",
     "test:unit": "mocha --forbid-only \"test/**/*.test.ts\" --exclude \"test/integration/**/*\" --exclude \"test/smoke/**/*\"",
     "test:smoke": "pnpm run build && mocha --forbid-only \"test/smoke/**/*.test.ts\"",
     "version": "oclif readme && git add README.md"

test/smoke/cli.test.ts

@@ -9,6 +9,7 @@ const execAsync = promisify(exec)
 /**
  * Verify that build artifacts exist
  * This is a prerequisite for smoke tests
+ * Throws an error if build artifacts are missing
  */
 function ensureBuildExists(): void {
   // Use process.cwd() which works in both CommonJS and ES modules
@@ -47,6 +48,7 @@ describe('Smoke Tests: Built CLI Binary', () => {
 
   before(() => {
     // Ensure build exists before running tests
+    // This will throw an error if build is missing, causing tests to fail explicitly
     ensureBuildExists()
   })