协作头脑风暴
brainstorming
在实现前澄清需求、探索设计选择,并形成清晰的开发方向。
test-cli-application【开发与工程】Integration tests for Node.js CLI app via built-in node:test. exec helper, output assertions, fs state verify, cleanup hooks, JSON parsing, err cases, state restoration after destructive. Use → add tests to existing CLI, test new cmd, verify adapter behavior across frameworks, setup CI.
独立 GitHub 快照 · 自动分类 · 以原作者文档为准
Integration tests via built-in node:test + execSync.
cli/index.js)import { describe, it, before, after } from 'node:test';
import assert from 'node:assert/strict';
import { execSync } from 'child_process';
import { existsSync, rmSync } from 'fs';
import { resolve } from 'path';
const CLI = 'node cli/index.js';
const ROOT = process.cwd();
function run(args) {
return execSync(`${CLI} ${args}`, {
cwd: ROOT,
encoding: 'utf8',
timeout: 10000,
});
}
Design decisions:
node:test built-in — no test runner depexecSync runs CLI as subprocess → tests actual binary, not internalsencoding: 'utf8' → strings for regexROOT for reproducibilityGot: Test file imports from node:test, working run() helper.
If err: node:test not avail → Node < 18. Upgrade | polyfill.
Verify CLI starts, parses args, expected output shapes:
describe('meta', () => {
it('shows version', () => {
const out = run('--version');
assert.match(out, /\d+\.\d+\.\d+/);
});
it('shows help with all commands', () => {
const out = run('--help');
assert.match(out, /install/);
assert.match(out, /list/);
assert.match(out, /detect/);
});
});
describe('registry', () => {
it('list shows expected counts', () => {
const out = run('list --domains');
assert.match(out, /\d+ domains/);
});
it('search finds known items', () => {
const out = run('search "docker"');
assert.match(out, /result\(s\) for "docker"/);
});
it('search returns 0 for nonsense', () => {
const out = run('search "xyzzy-nonexistent"');
assert.match(out, /0 result/);
});
});
Smoke patterns:
--version + --help always workGot: Smoke tests confirm CLI functional, data loaded.
If err: Registry counts change often → use \d+ not hardcoded #s.
Create → verify → delete sequences w/ cleanup:
describe('install', () => {
const testPath = resolve(ROOT, '.agents/skills/commit-changes');
after(() => {
// Always clean up, even if tests fail
try { rmSync(testPath); } catch {}
try { rmSync(resolve(ROOT, '.agents/skills'), { recursive: true }); } catch {}
try { rmSync(resolve(ROOT, '.agents'), { recursive: true }); } catch {}
});
it('dry-run does not create files', () => {
const out = run('install commit-changes --dry-run');
assert.match(out, /DRY RUN/);
assert.ok(!existsSync(testPath));
});
it('installs creates the target', () => {
run('install commit-changes');
assert.ok(existsSync(testPath));
});
it('skips already installed', () => {
const out = run('install commit-changes');
assert.match(out, /skipped/);
});
it('uninstall removes the target', () => {
run('uninstall commit-changes');
assert.ok(!existsSync(testPath));
});
});
Cleanup rules:
after() not afterEach() — lifecycle tests build on each othertry/catch — must not fail suiteGot: Tests run in sequence within describe, cleanup runs even on fail.
If err: Tests run parallel (non-default in node:test) → force sequential w/ { concurrency: 1 }.
Test each adapter's target path w/o changes:
describe('adapter: cursor (dry-run)', () => {
it('targets .cursor/skills/ path', () => {
const out = run('install commit-changes --framework cursor --dry-run');
assert.match(out, /\.cursor\/skills/i);
});
});
describe('adapter: copilot (dry-run)', () => {
it('targets .github/ path', () => {
const out = run('install commit-changes --framework copilot --dry-run');
assert.match(out, /\.github/i);
});
});
Pattern scales to any adapters. Each test:
--framework bypasses auto-detection--dry-run → no files createdGot: 1 describe per adapter, each w/ ≥ path assertion.
If err: Adapter doesn't exist in proj → fails w/ "Unknown framework". Correct — adapter tests should only exist for implemented.
describe('errors', () => {
it('rejects unknown items', () => {
assert.throws(
() => run('install nonexistent-skill-xyz'),
/No matching items|Unknown/,
);
});
it('rejects unknown framework', () => {
assert.throws(
() => run('install commit-changes --framework nonexistent'),
/Unknown framework/,
);
});
it('handles missing state gracefully', () => {
assert.throws(
() => run('scatter nonexistent-team'),
/not burning|Unknown/,
);
});
});
Err testing patterns:
assert.throws catches non-zero exits from execSyncGot: All err paths → non-zero exits + helpful msgs.
If err: execSync throws on non-zero. Err's stderr | stdout has msg. Check error.stdout if regex doesn't match.
describe('json output', () => {
it('campfire --json outputs valid JSON', () => {
const out = run('campfire --json');
const data = JSON.parse(out);
assert.ok(typeof data.totalTeams === 'number');
assert.ok(Array.isArray(data.fires));
});
it('gather --dry-run --json outputs structured data', () => {
const out = run('gather tending --dry-run --json');
// JSON may follow a DRY RUN header — extract from first '{'
const jsonStart = out.indexOf('{');
assert.ok(jsonStart >= 0, 'Should contain JSON');
const data = JSON.parse(out.slice(jsonStart));
assert.equal(data.team, 'tending');
});
});
JSON testing gotchas:
{Got: JSON output parseable + contains expected keys.
If err: JSON.parse fails → cmd may mix human text + JSON. Fix cmd → pure JSON in --json mode | extract substring.
describe('stateful commands', () => {
const stateDir = resolve(ROOT, '.agent-almanac');
after(() => {
// Remove state file created by tests
try { rmSync(stateDir, { recursive: true }); } catch {}
});
// Tests that create/modify state...
});
// Restore symlinks that destructive tests may remove
describe('destructive tests', () => {
after(() => {
// Restore symlinks that scatter/uninstall removed
const skills = ['heal', 'meditate', 'remote-viewing'];
for (const skill of skills) {
const link = resolve(ROOT, `.claude/skills/${skill}`);
if (!existsSync(link)) {
try {
execSync(`ln -s ../../skills/${skill} ${link}`, { cwd: ROOT });
} catch {}
}
}
});
});
State restoration rules:
.agent-almanac/state.json) → cleaned after testsscatter/uninstall → restoredagent-almanac.yml) created by init → removedafter() runs reverse declaration order → declare restore hooks lastGot: Suite leaves proj in same state found.
If err: CI reports leftover files → add cleanup to after(). Use git status after to detect leaked state.
node --test cli/test/cli.test.js--version, --help, registry loading\d+ regex | read dynamic vs 329 skills.node:test runs suites in declaration order default, but within suite may not. Lifecycle suites (create → verify → delete) within single describe for guarantee.after() still runs. Throw in before() → subsequent + after() may not. Keep before() minimal.execSync. Pipe echo y | | ensure --yes always passed..claude/skills/ | .agents/skills/ modify working tree. CI may fail on "dirty WD" checks. Always clean.scaffold-cli-command — build cmds these tests verifybuild-cli-plugin — build adapters tested in Step 4design-cli-output — output patterns tests assert against