test: build and test script test

Author: syi0808

tests/fixtures/build-script/package.json

@@ -0,0 +1,7 @@
+{
+  "name": "build-script-fixtures",
+  "version": "0.0.1",
+  "scripts": {
+    "build": "build"
+  }
+}

tests/fixtures/test-script/package.json

@@ -0,0 +1,7 @@
+{
+  "name": "test-script-fixtures",
+  "version": "0.0.1",
+  "scripts": {
+    "test": "test"
+  }
+}

tests/options.test.ts

@@ -0,0 +1,9 @@
+import { test } from "vitest";
+import { runPubmCli } from "./utils/cli";
+import path from "node:path";
+
+test('build script options', async () => {
+  const { controller } = runPubmCli('0.0.2', '--contents', path.resolve(import.meta.dirname, './fixtures/build-script'), '--no-pre-check', '--no-tests', '--build-script', 'not-exist-build-scripts');
+
+  await controller.waitForStderr("✖ Checking if test and build scripts exist");
+});

tests/utils/cli.ts

@@ -2,6 +2,7 @@ import type { Readable, Writable } from 'node:stream';
 import { stripVTControlCharacters } from 'node:util';
 import { isCI } from 'std-env';
 import { type Options, exec } from 'tinyexec';
+import path from 'node:path';
 
 // Based on https://github.com/vitest-dev/vitest/blob/main/test/test-utils/index.ts
 
@@ -116,55 +117,44 @@ function isWritable(stream: any): stream is Writable {
 	return stream && typeof stream?.write === 'function';
 }
 
-export async function runPubmCli(
-	command: string,
-	_options?: Partial<Options>,
+export function runPubmCli(
+	_options?: Partial<Options> | string,
 	...args: string[]
-): Promise<{
+): {
 	controller: CliController;
 	exitCode: number | undefined;
 	stdout: string;
 	stderr: string;
 	waitForClose: () => Promise<unknown>;
-}> {
+} {
 	let options = _options;
 
 	if (typeof _options === 'string') {
 		args.unshift(_options);
 		options = undefined;
 	}
 
-	const subprocess = exec(command, args, options as Options).process;
+	const subprocess = exec(path.resolve(import.meta.dirname, '../../bin/cli.js'), args, options as Options).process!;
 	const controller = new CliController({
 		// biome-ignore lint/style/noNonNullAssertion: <explanation>
-		stdin: subprocess!.stdin!,
+		stdin: subprocess.stdin!,
 		// biome-ignore lint/style/noNonNullAssertion: <explanation>
-		stdout: subprocess!.stdout!,
+		stdout: subprocess.stdout!,
 		// biome-ignore lint/style/noNonNullAssertion: <explanation>
-		stderr: subprocess!.stderr!,
+		stderr: subprocess.stderr!,
 	});
 
-	let setDone: (value?: unknown) => void;
-
 	const isDone = new Promise((resolve) => {
-		setDone = resolve;
+		subprocess?.on('exit', resolve);
 	});
 
-	subprocess?.on('exit', () => setDone());
-
-	function output() {
-		return {
-			controller,
-			exitCode: subprocess?.exitCode ?? undefined,
-			stdout: controller.stdout || '',
-			stderr: controller.stderr || '',
-			waitForClose: () => isDone,
-		};
-	}
-
-	await isDone;
-
-	return output();
+	return {
+		controller,
+		exitCode: subprocess?.exitCode ?? undefined,
+		stdout: controller.stdout || '',
+		stderr: controller.stderr || '',
+		waitForClose: () => isDone,
+	};
 }
 
 export const DOWN = '\x1B\x5B\x42';