paperlint 3.1.3 → 3.2.0

package/scripts/rules-are-content-only.test.mjs

@@ -1,61 +0,0 @@
-/**
- * rules-are-content-only's gate over a consumer's `eslint-rules/`: temp trees with planted rules.
- */
-import assert from "node:assert/strict";
-import { fileURLToPath } from "node:url";
-import { spawnSync } from "node:child_process";
-import { test } from "vitest";
-import { useTempDir, writeTree } from "../test/support.mjs";
-import { main as contentMain } from "./rules-are-content-only.mjs";
-
-const capture = () => {
-  const out = [];
-  const io = { log: (s) => out.push(s), err: (s) => out.push(`E ${s}`) };
-  return { out, io };
-};
-
-const root = useTempDir("content-only-");
-
-test("rules-are-content-only: no rules is a failure, a rule asking git is a finding, clean rules pass", () => {
-  writeTree(root, {
-    "empty/.keep": "",
-    "dirty/eslint-rules/asks-git.mjs":
-      'import { execFileSync } from "node:child_process";\nexecFileSync("git", ["log"]);\n',
-    "dirty/eslint-rules/clean.mjs": "export default {};\n",
-    "clean/eslint-rules/clean.mjs": "export default {};\n",
-  });
-  const runs = ["empty", "dirty", "clean"].map((d) => {
-    const { out, io } = capture();
-    return [
-      contentMain({ cwd: `${root}/${d}`, ...io }),
-      out.map((l) => l.replace(root, "<root>").split(". ")[0]),
-    ];
-  });
-  assert.deepEqual(runs, [
-    [1, ["E <root>/empty/eslint-rules: no rule sources"]],
-    [
-      1,
-      [
-        "E asks-git.mjs: imports `git`",
-        "rules-are-content-only: 2 rule sources under <root>/dirty, 1 findings",
-      ],
-    ],
-    [
-      0,
-      ["rules-are-content-only: 1 rule sources under <root>/clean, 0 findings"],
-    ],
-  ]);
-});
-
-test("run as a program over this repository: every rule source is content-only, exit 0", () => {
-  const r = spawnSync(
-    process.execPath,
-    [fileURLToPath(new URL("./rules-are-content-only.mjs", import.meta.url))],
-    { cwd: fileURLToPath(new URL("..", import.meta.url)), encoding: "utf8" },
-  );
-  assert.equal(r.status, 0, r.stderr);
-  assert.match(
-    r.stdout,
-    /^rules-are-content-only: \d+ rule sources under .*, 0 findings\n$/,
-  );
-});

package/scripts/test.mjs

@@ -1,44 +0,0 @@
-/**
- * `npm test`: the unit tests (vitest), then the harnesses (`vigiles test`). BOTH always run.
- *
- * Until #52 this was `vitest run && vigiles test --min=1`. Under `npm run coverage` a single
- * failing vitest test then skipped every harness, and what the run printed last was c8's summary:
- * lines, branches and functions each some six points below the floor. The one failing test sat
- * above that, looking like a consequence rather than the cause. Running both suites keeps the
- * coverage report whole, so a failing test is reported as itself and the coverage figures still
- * mean what they say.
- *
- * Arguments are the harness runner's (`npm test -- --no-skip`), as they were before.
- */
-import { spawnSync } from "node:child_process";
-import { isMain } from "../skills/paper-pipeline/scripts/consumer.mjs";
-
-/** The two suites, in order: what a failure is named as, and the command. */
-export const SUITES = [
-  { name: "vitest", command: ["vitest", "run"] },
-  { name: "harnesses", command: ["vigiles", "test", "--min=1"] },
-];
-
-/**
- * Run every suite, whatever the one before it returned, and name the ones that failed.
- * @returns 0 when every suite passed; otherwise the first failing suite's exit code (1 for a signal).
- */
-export function runSuites(
-  args,
-  { spawn = spawnSync, err = console.error, suites = SUITES } = {},
-) {
-  const failed = suites.flatMap(({ name, command }) => {
-    const [file, ...rest] = command;
-    const r = spawn(file, [...rest, ...(name === "harnesses" ? args : [])], {
-      stdio: "inherit",
-    });
-    return r.status === 0 ? [] : [{ name, code: r.status ?? 1 }];
-  });
-  for (const f of failed)
-    err(
-      `✗ ${f.name} failed (exit ${String(f.code)}) — its own report is above`,
-    );
-  return failed.length === 0 ? 0 : failed[0].code;
-}
-
-if (isMain(import.meta.url)) process.exit(runSuites(process.argv.slice(2)));

package/scripts/test.test.mjs

@@ -1,52 +0,0 @@
-/**
- * `npm test`'s runner: both suites run whatever the first returned, each failure is named, and
- * the harness arguments reach the harness runner only.
- */
-import assert from "node:assert/strict";
-import { test } from "vitest";
-import { runSuites, SUITES } from "./test.mjs";
-
-const fake = (status) => {
-  const calls = [];
-  return {
-    calls,
-    spawn: (file, args) => {
-      calls.push([file, ...args]);
-      return { status: status[file] };
-    },
-  };
-};
-
-test("a failing vitest does not skip the harnesses, and is named as the failure", () => {
-  const { calls, spawn } = fake({ vitest: 1, vigiles: 0 });
-  const said = [];
-  assert.equal(
-    runSuites(["--no-skip"], { spawn, err: (s) => said.push(s) }),
-    1,
-  );
-  assert.deepEqual(calls, [
-    ["vitest", "run"],
-    ["vigiles", "test", "--min=1", "--no-skip"],
-  ]);
-  assert.deepEqual(said, [
-    "✗ vitest failed (exit 1) — its own report is above",
-  ]);
-});
-
-test("both passing is 0 and silent; a harness killed by a signal is exit 1, named", () => {
-  const ok = fake({ vitest: 0, vigiles: 0 });
-  assert.equal(runSuites([], { spawn: ok.spawn, err: assert.fail }), 0);
-  const killed = fake({ vitest: 0, vigiles: null });
-  const said = [];
-  assert.equal(
-    runSuites([], { spawn: killed.spawn, err: (s) => said.push(s) }),
-    1,
-  );
-  assert.deepEqual(said, [
-    "✗ harnesses failed (exit 1) — its own report is above",
-  ]);
-  assert.deepEqual(
-    SUITES.map((s) => s.name),
-    ["vitest", "harnesses"],
-  );
-});