fix: make sure errors get logged to stderr (#34)

Author: bartveneman

src/cli/cli.ts

@@ -25,8 +25,8 @@ async function cli(cli_args: string[]) {
 		return tap(report, params)
 	}
 	if (params.reporter === 'json') {
-		// oxlint-disable-next-line no-null
-		return console.log(JSON.stringify(report))
+		let log = JSON.stringify(report)
+		return report.report.ok ? console.log(log) : console.error(log)
 	}
 	return pretty(report, params)
 }

src/cli/reporters/pretty.ts

@@ -109,7 +109,9 @@ export function print_lines({ report, context }: Report, params: CliArguments, {
 }
 
 export function print(report: Report, params: CliArguments): void {
+	let logger = report.report.ok ? console.log : console.error
+
 	for (let line of print_lines(report, params, { styleText, print_width: process.stdout.columns })) {
-		console.log(line)
+		logger(line)
 	}
 }

src/cli/reporters/tap.ts

@@ -14,7 +14,7 @@ function ok(n: number, description?: string) {
 }
 
 function not_ok(n: number, description?: string) {
-	console.log(`not ok ${n} ${description ? `- ${description}` : ''}`)
+	console.error(`not ok ${n} ${description ? `- ${description}` : ''}`)
 }
 
 function meta(data: Record<string | number, string | number>) {