add tests

Author: gameroman

knip.jsonc

@@ -0,0 +1,10 @@
+{
+  "$schema": "./node_modules/knip/schema-jsonc.json",
+  "entry": [
+    "src/index.ts",
+    "src/test-utils/*.ts"
+  ],
+  "ignoreDependencies": [
+    "@tanstack/intent"
+  ]
+}

src/commands/__snapshots__/lint.test.ts.snap

@@ -0,0 +1,47 @@
+// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
+
+exports[`lint command > runKnip > detects unused exports and unused files 1`] = `
+[
+  {
+    "code": "unused-file",
+    "column": undefined,
+    "file": "src/unused-file.ts",
+    "level": "warning",
+    "line": undefined,
+    "message": "Unused file",
+    "tool": "knip",
+  },
+  {
+    "code": "unused-export",
+    "column": 20,
+    "file": "src/used.ts",
+    "level": "warning",
+    "line": 3,
+    "message": "Unused export 'unusedExport'",
+    "tool": "knip",
+  },
+]
+`;
+
+exports[`lint command > runOxlint > detects violations in bad code 1`] = `
+[
+  {
+    "code": "eslint(no-unused-vars)",
+    "column": 5,
+    "file": "src/index.ts",
+    "level": "error",
+    "line": 1,
+    "message": "Variable 'x' is declared but never used. Unused variables should start with a '_'.",
+    "tool": "oxlint",
+  },
+  {
+    "code": "eslint(no-var)",
+    "column": 1,
+    "file": "src/index.ts",
+    "level": "error",
+    "line": 1,
+    "message": "Unexpected var, use let or const instead.",
+    "tool": "oxlint",
+  },
+]
+`;

src/commands/lint.test.ts

@@ -0,0 +1,82 @@
+import { describe, it, expect, beforeEach, afterEach } from "vitest";
+import { createFixture } from "../test-utils/index.ts";
+import { runOxlint, runKnip } from "./lint.ts";
+import { fileURLToPath } from "node:url";
+
+describe("lint command", () => {
+	let originalCwd: string;
+	let fixture: Awaited<ReturnType<typeof createFixture>>;
+
+	beforeEach(() => {
+		originalCwd = process.cwd();
+	});
+
+	afterEach(async () => {
+		process.chdir(originalCwd);
+		if (fixture) await fixture.cleanup();
+	});
+
+	describe("runOxlint", () => {
+		it("detects violations in bad code", async () => {
+			fixture = await createFixture({
+				src: {
+					"index.ts": "var x = 1;",
+				},
+			});
+			process.chdir(fileURLToPath(fixture.root));
+
+			const violations = await runOxlint(["./src"]);
+
+			expect(violations).not.toEqual([]);
+			expect(violations).toMatchSnapshot();
+		});
+
+		it("returns empty array for clean code", async () => {
+			fixture = await createFixture({
+				src: {
+					"index.ts": `
+						export const x = 1;
+					`,
+				},
+			});
+			process.chdir(fileURLToPath(fixture.root));
+
+			const violations = await runOxlint(["./src"]);
+
+			expect(violations).toEqual([]);
+		});
+	});
+
+	describe("runKnip", () => {
+		it("detects unused exports and unused files", async () => {
+			fixture = await createFixture({
+				"package.json": {
+					name: "test-pkg",
+					version: "1.0.0",
+					type: "module",
+					exports: "./src/index.ts",
+				},
+				src: {
+					"index.ts": `
+						import { used } from "./used";
+						console.log(used);
+						export const value = 42; 
+					`,
+					"used.ts": `
+						export const used = "used";
+						export const unusedExport = "unusedExport";
+					`,
+					"unused-file.ts": `
+						export const unused = "unused";
+					`,
+				},
+			});
+			process.chdir(fileURLToPath(fixture.root));
+
+			const violations = await runKnip();
+
+			expect(violations).not.toEqual([]);
+			expect(violations).toMatchSnapshot();
+		});
+	});
+});

src/commands/lint.ts

@@ -22,7 +22,7 @@ interface Violation {
 
 // -- Tool Runners --
 
-async function runOxlint(targets: string[], fix?: boolean): Promise<Violation[]> {
+export async function runOxlint(targets: string[], fix?: boolean): Promise<Violation[]> {
 	const args = ["-c", oxlintConfig, "--format=json", ...targets];
 	if (fix) args.push("--fix");
 	const result = await x(local("oxlint"), args, { throwOnError: false });
@@ -74,7 +74,7 @@ interface KnipJsonReport {
 	issues: KnipJsonIssue[];
 }
 
-async function runKnip(): Promise<Violation[]> {
+export async function runKnip(): Promise<Violation[]> {
 	const args = ["--no-progress", "--reporter", "json"];
 	const result = await x(local("knip"), args, { throwOnError: false });
 	if (!result.stdout.trim()) return [];