llm-security/tests/scanners/ci-integration.test.mjs
Kjell Tore Guttormsen c4471617a4
test(ci): give policy.json ci.failOn / ci.compact a real test
The "--fail-on via policy.json" block wrote a policy.json into a tmp root it
never scanned and passed --fail-on on the CLI, so ci.failOn / ci.compact had
no test at all. The rewrite scans a copy of grade-a-project as the process's
own working tree (the only place policy.json is honored since v8.1.0):
WARNING with 0 critical exits 1 by default, and exits 0 only if
`ci.failOn: 'critical'` is read from the policy. Measured: each branch of
main() mutated away turns exactly its own test red (fail 1), real code 4/4.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2026-09-22 21:00:31 +02:00

169 lines
7.3 KiB
JavaScript

// ci-integration.test.mjs — Tests for --fail-on and --compact CI flags
import { describe, it, beforeEach, afterEach, after } from 'node:test';
import { spawn } from 'node:child_process';
import { strict as assert } from 'node:assert';
import { resolve, dirname, join } from 'node:path';
import { fileURLToPath } from 'node:url';
import { mkdirSync, writeFileSync, rmSync, readFileSync, existsSync, cpSync } from 'node:fs';
import { tmpdir } from 'node:os';
import { materializeTree } from '../helpers/payload-trees.mjs';
import { mkOwnTreeDir } from '../helpers/own-tree.mjs';
const __dirname = dirname(fileURLToPath(import.meta.url));
const ORCHESTRATOR = resolve(__dirname, '../../scanners/scan-orchestrator.mjs');
const CLI = resolve(__dirname, '../../bin/llm-security.mjs');
const poisonedTree = materializeTree('memory-scan/poisoned-project');
after(poisonedTree.cleanup);
const POISONED = poisonedTree.dir;
const CLEAN = resolve(__dirname, '../fixtures/posture-scan/grade-a-project');
function run(args, timeout = 120000, cwd = undefined) {
return new Promise((resolve) => {
const chunks = [];
const errChunks = [];
const child = spawn('node', [ORCHESTRATOR, ...args], {
cwd,
timeout,
stdio: ['ignore', 'pipe', 'pipe'],
});
child.stdout.on('data', (c) => chunks.push(c));
child.stderr.on('data', (c) => errChunks.push(c));
child.on('close', (code) => {
resolve({
code: code ?? 1,
stdout: Buffer.concat(chunks).toString(),
stderr: Buffer.concat(errChunks).toString(),
});
});
});
}
describe('--fail-on flag', () => {
it('exit 0 when --fail-on critical and no critical findings', async () => {
const { code } = await run([CLEAN, '--fail-on', 'critical']);
assert.equal(code, 0, 'should exit 0 — no critical findings in clean fixture');
});
it('exit 1 when --fail-on critical and critical findings exist', async () => {
const { code } = await run([POISONED, '--fail-on', 'critical']);
assert.equal(code, 1, 'should exit 1 — poisoned fixture has critical findings');
});
it('exit 1 when --fail-on high and high findings exist', async () => {
const { code } = await run([POISONED, '--fail-on', 'high']);
assert.equal(code, 1, 'should exit 1 — poisoned fixture has high findings');
});
it('exit 1 when --fail-on medium and medium findings exist', async () => {
const { code } = await run([CLEAN, '--fail-on', 'medium']);
// grade-a-project produces medium from taint/toxic-flow
const { code: code2 } = await run([POISONED, '--fail-on', 'medium']);
assert.equal(code2, 1, 'should exit 1 — poisoned fixture has medium+ findings');
});
it('preserves existing exit codes without --fail-on', async () => {
const { code } = await run([POISONED]);
// Poisoned project produces BLOCK verdict → exit 2
assert.equal(code, 2, 'should exit 2 (BLOCK verdict) without --fail-on');
});
it('rejects invalid --fail-on value', async () => {
const { code, stderr } = await run(['.', '--fail-on', 'invalid']);
assert.equal(code, 1, 'should exit 1 for invalid severity');
assert.ok(stderr.includes('--fail-on must be one of'), 'should print validation error');
});
});
describe('--compact flag', () => {
it('outputs one-liner format to stdout (not JSON)', async () => {
const { stdout } = await run([POISONED, '--compact']);
assert.ok(!stdout.includes('"scanners"'), 'should not contain JSON envelope key');
assert.ok(stdout.includes('[CRITICAL]') || stdout.includes('[HIGH]'), 'should contain severity prefix');
assert.ok(stdout.includes('---'), 'should contain summary separator');
assert.ok(stdout.includes('Verdict:'), 'should contain verdict summary line');
});
it('writes full JSON to --output-file, compact aggregate to stdout', async () => {
const tmpFile = resolve(tmpdir(), `llm-security-ci-test-${Date.now()}.json`);
try {
const { stdout } = await run([POISONED, '--compact', '--output-file', tmpFile]);
assert.ok(existsSync(tmpFile), 'output file should exist');
const content = JSON.parse(readFileSync(tmpFile, 'utf8'));
assert.ok(content.scanners, 'file should contain full JSON with scanners key');
const stdoutParsed = JSON.parse(stdout);
assert.ok(stdoutParsed.aggregate, 'stdout should contain compact aggregate JSON');
} finally {
if (existsSync(tmpFile)) rmSync(tmpFile);
}
});
it('with --output-file writes one-liner findings to stderr', async () => {
const tmpFile = resolve(tmpdir(), `llm-security-ci-test-${Date.now()}.json`);
try {
const { stderr } = await run([POISONED, '--compact', '--output-file', tmpFile]);
assert.ok(
stderr.includes('[CRITICAL]') || stderr.includes('[HIGH]'),
'stderr should contain one-liner findings in compact+output-file mode'
);
} finally {
if (existsSync(tmpFile)) rmSync(tmpFile);
}
});
});
describe('--fail-on + --compact combined', () => {
it('exit 0 with compact output when below threshold', async () => {
const { code, stdout } = await run([CLEAN, '--fail-on', 'critical', '--compact']);
assert.equal(code, 0, 'should exit 0 — no critical findings');
assert.ok(stdout.includes('Verdict:'), 'should still show compact summary');
});
});
// v8.1.1: the policy file is honored only for the caller's own working tree
// (scanners/lib/own-working-tree.mjs), so these run the orchestrator with the
// fixture as its cwd and '.' as its target. grade-a-project is WARNING with 0
// critical (default exit 1), so `ci.failOn: 'critical'` read from the policy
// is the only way to get exit 0 — the test cannot pass without reading it.
describe('--fail-on / --compact via policy.json (own working tree)', () => {
let ownDir;
function writePolicy(ci) {
mkdirSync(join(ownDir, '.llm-security'), { recursive: true });
writeFileSync(join(ownDir, '.llm-security', 'policy.json'), JSON.stringify({ ci }));
}
beforeEach(() => {
ownDir = mkOwnTreeDir('ci-policy-own-');
cpSync(CLEAN, ownDir, { recursive: true });
});
afterEach(() => {
rmSync(ownDir, { recursive: true, force: true });
});
it('known-positive: without policy the fixture exits 1 (WARNING, no critical)', async () => {
const { code, stdout } = await run(['.'], 120000, ownDir);
const env = JSON.parse(stdout);
assert.equal(env.aggregate.counts.critical, 0);
assert.equal(code, 1, 'WARNING verdict without --fail-on should exit 1');
});
it('reads failOn from policy.json ci section', async () => {
writePolicy({ failOn: 'critical' });
const { code } = await run(['.'], 120000, ownDir);
assert.equal(code, 0, 'policy ci.failOn: critical with no critical findings should exit 0');
});
it('CLI --fail-on overrides policy.json', async () => {
writePolicy({ failOn: 'critical' });
const { code } = await run(['.', '--fail-on', 'medium'], 120000, ownDir);
assert.equal(code, 1, 'CLI --fail-on medium should override policy failOn: critical');
});
it('reads compact from policy.json ci section', async () => {
writePolicy({ compact: true });
const { stdout } = await run(['.'], 120000, ownDir);
assert.ok(stdout.includes('Verdict:'), 'policy ci.compact: true should print the compact summary');
assert.throws(() => JSON.parse(stdout), 'compact output should not be the JSON envelope');
});
});