test: cover case of a post-compile script with cjs

fixtures/ts-cjs-post-compile/package.json

@@ -0,0 +1,3 @@
+{
+  "type": "commonjs"
+}

fixtures/ts-cjs-post-compile/postCompile.ts

@@ -0,0 +1 @@
+console.log('Doing stuff')

fixtures/ts-cjs-post-compile/src/add.ts

@@ -0,0 +1,4 @@
+
+export function add (x: number, y: number): number {
+  return x + y
+}

fixtures/ts-cjs-post-compile/test/add.test.ts

@@ -0,0 +1,7 @@
+import { test } from 'node:test'
+import { add } from '../src/add.js'
+import { strictEqual } from 'node:assert'
+
+test('add', () => {
+  strictEqual(add(1, 2), 3)
+})

fixtures/ts-cjs-post-compile/test/add2.test.ts

@@ -0,0 +1,7 @@
+import { test } from 'node:test'
+import { add } from '../src/add.js'
+import { strictEqual } from 'node:assert'
+
+test('add2', () => {
+  strictEqual(add(3, 2), 5)
+})

fixtures/ts-cjs-post-compile/tsconfig.json

@@ -0,0 +1,24 @@
+{
+  "$schema": "https://json.schemastore.org/tsconfig",
+  "compilerOptions": {
+    "outDir": "dist",
+    "sourceMap": true,
+    "target": "ES2022",
+    "module": "NodeNext",
+    "moduleResolution": "NodeNext",
+    "esModuleInterop": true,
+    "strict": true,
+    "resolveJsonModule": true,
+    "removeComments": true,
+    "newLine": "lf",
+    "noUnusedLocals": true,
+    "noFallthroughCasesInSwitch": true,
+    "isolatedModules": true,
+    "forceConsistentCasingInFileNames": true,
+    "skipLibCheck": true,
+    "lib": [
+      "ESNext"
+    ],
+    "incremental": true
+  }
+}

lib/run.js

@@ -115,7 +115,7 @@ export default async function runWithTypeScript (config) {
     let outDir = ''
     if (config['post-compile'] && tsconfigPath) {
       const tsconfig = JSON.parse(await readFile(tsconfigPath))
-      outDir = tsconfig.compilerOptions.outDir ?? ''
+      outDir = tsconfig.compilerOptions.outDir
     }
     let start = Date.now()
     tscChild = execa('node', [tscPath, ...typescriptCliArgs], { cwd })

test/cli.test.js

@@ -89,7 +89,7 @@ test('gh reporter', async () => {
   strictEqual(stdout.indexOf('::notice') >= 0, true)
 })
 
-test('post-compile hook should be correctly called', async () => {
+test('Post compile script should be executed when --post-compile  is sent with esm', async () => {
   const cwd = join(import.meta.url, '..', 'fixtures', 'ts-esm-post-compile')
   const { stdout } = await execa('node', [
     borp,
@@ -100,3 +100,14 @@ test('post-compile hook should be correctly called', async () => {
 
   strictEqual(stdout.indexOf('Post compile hook complete') >= 0, true, 'Post compile message should be found in stdout')
 })
+
+test('Post compile script should be executed when --post-compile  is sent with cjs', async () => {
+  const { stdout } = await execa('node', [
+    borp,
+    '--post-compile=postCompile.ts'
+  ], {
+    cwd: join(import.meta.url, '..', 'fixtures', 'ts-cjs-post-compile')
+  })
+
+  strictEqual(stdout.indexOf('Post compile hook complete') >= 0, true, 'Post compile message should be found in stdout')
+})

test/watch.test.js

@@ -110,7 +110,7 @@ test('add', () => {
   await completed
 })
 
-test('watch with post compile hook', async (t) => {
+test('watch with post compile hook should call the hook the right number of times', async (t) => {
   const { strictEqual, completed, ok } = tspl(t, { plan: 2 })
 
   const dir = path.resolve(await mkdtemp('.test-watch-with-post-compile-hook'))