Add tests for --reporter and --preprocessor

fixtures/cli-preprocessor/index.js

@@ -0,0 +1,4 @@
+export default function (options) {
+  console.log('hi from js preprocessor');
+  return options;
+}

fixtures/cli-preprocessor/index.ts

@@ -0,0 +1,4 @@
+export default function (options) {
+  console.log('hi from ts preprocessor');
+  return options;
+}

fixtures/cli-preprocessor/package.json

@@ -0,0 +1,3 @@
+{
+  "name": "@fixtures/cli-preprocessor"
+}

fixtures/cli-reporter/index.js

@@ -0,0 +1,3 @@
+export default function () {
+  console.log('hi from js reporter');
+}

fixtures/cli-reporter/index.ts

@@ -0,0 +1,3 @@
+export default function () {
+  console.log('hi from ts reporter');
+}

fixtures/cli-reporter/package.json

@@ -0,0 +1,3 @@
+{
+  "name": "@fixtures/cli-reporter"
+}

fixtures/cli/package.json

@@ -0,0 +1,3 @@
+{
+  "name": "@fixtures/cli"
+}

tests/cli-preprocessor.test.ts

@@ -0,0 +1,27 @@
+import assert from 'node:assert/strict';
+import { execSync } from 'node:child_process';
+import test from 'node:test';
+import { resolve } from '../src/util/path.js';
+
+const cwd = resolve('fixtures/cli-preprocessor');
+
+const exec = (command: string) => {
+  const output = execSync(command.replace(/^knip/, 'node ../../dist/cli.js'), { cwd });
+  return output.toString().trim();
+};
+
+test('knip --preprocessor ./index.js', () => {
+  assert.equal(exec('knip --preprocessor ./index.js'), 'hi from js preprocessor');
+});
+
+test('knip --preprocessor ./index.ts', () => {
+  assert.equal(exec('knip --preprocessor ./index.ts'), 'hi from ts preprocessor');
+});
+
+test('knip --preprocessor knip-preprocessor', () => {
+  assert.equal(exec('knip --preprocessor knip-preprocessor'), 'hi from pkg preprocessor');
+});
+
+test('knip --preprocessor @org/preprocessor', () => {
+  assert.equal(exec('knip --preprocessor @org/preprocessor'), 'hi from scoped preprocessor');
+});

tests/cli-reporter.test.ts

@@ -0,0 +1,27 @@
+import assert from 'node:assert/strict';
+import { execSync } from 'node:child_process';
+import test from 'node:test';
+import { resolve } from '../src/util/path.js';
+
+const cwd = resolve('fixtures/cli-reporter');
+
+const exec = (command: string) => {
+  const output = execSync(command.replace(/^knip/, 'node ../../dist/cli.js'), { cwd });
+  return output.toString().trim();
+};
+
+test('knip --reporter ./index.js', () => {
+  assert.equal(exec('knip --reporter ./index.js'), 'hi from js reporter');
+});
+
+test('knip --reporter ./index.ts', () => {
+  assert.equal(exec('knip --reporter ./index.ts'), 'hi from ts reporter');
+});
+
+test('knip --reporter knip-reporter', () => {
+  assert.equal(exec('knip --reporter knip-reporter'), 'hi from pkg reporter');
+});
+
+test('knip --reporter @org/reporter', () => {
+  assert.equal(exec('knip --reporter @org/reporter'), 'hi from scoped reporter');
+});

tests/cli.test.ts

@@ -2,10 +2,13 @@ import assert from 'node:assert/strict';
 import { execSync } from 'node:child_process';
 import test from 'node:test';
 import { helpText } from '../src/util/cli-arguments.js';
+import { resolve } from '../src/util/path.js';
 import { version } from '../src/version.js';
 
+const cwd = resolve('fixtures/cli');
+
 const exec = (command: string) => {
-  const output = execSync(command.replace(/^knip/, 'node --no-warnings --loader tsx src/cli.ts'));
+  const output = execSync(command.replace(/^knip/, 'node ../../dist/cli.js'), { cwd });
   return output.toString().trim();
 };
 
@@ -16,7 +19,3 @@ test('knip --version', () => {
 test('knip --help', () => {
   assert.equal(exec('knip --help'), helpText);
 });
-
-test('knip', { skip: true }, () => {
-  assert.equal(exec('knip'), '✂️  Excellent, Knip found no issues.');
-});