triton6564685/gbrain
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: add explicit test timeouts for CLI subprocess E2E tests

Browse Source 
CLI subprocess tests (Setup Journey, Doctor Command, Parallel Import)
spawn `bun run src/cli.ts` which takes several seconds to JIT compile +
connect. The Bun test framework default 5000ms per-test timeout is too
tight for CI. Added 30-60s timeouts matching each subprocess's own
timeout to prevent false failures.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Garry Tan
2026-04-10 12:37:59 -10:00
parent 3fc6f8b943
commit ad287a84af
1 changed files with 11 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
test/e2e/mechanical.test.ts
View File

@@ -524,7 +524,7 @@ describeE2E('E2E: Setup Journey', () => {
const stdout = new TextDecoder().decode(result.stdout); const stdout = new TextDecoder().decode(result.stdout);
expect(result.exitCode).toBe(0); expect(result.exitCode).toBe(0);
expect(stdout).toContain('Brain ready'); expect(stdout).toContain('Brain ready');
}); }, 30_000);
test('gbrain import imports fixtures via CLI', () => { test('gbrain import imports fixtures via CLI', () => {
const result = Bun.spawnSync({ const result = Bun.spawnSync({
@@ -536,7 +536,7 @@ describeE2E('E2E: Setup Journey', () => {
const stdout = new TextDecoder().decode(result.stdout); const stdout = new TextDecoder().decode(result.stdout);
expect(result.exitCode).toBe(0); expect(result.exitCode).toBe(0);
expect(stdout).toContain('imported'); expect(stdout).toContain('imported');
}); }, 60_000);
test('gbrain search returns results via CLI', () => { test('gbrain search returns results via CLI', () => {
const result = Bun.spawnSync({ const result = Bun.spawnSync({
@@ -548,7 +548,7 @@ describeE2E('E2E: Setup Journey', () => {
const stdout = new TextDecoder().decode(result.stdout); const stdout = new TextDecoder().decode(result.stdout);
expect(result.exitCode).toBe(0); expect(result.exitCode).toBe(0);
expect(stdout.length).toBeGreaterThan(0); expect(stdout.length).toBeGreaterThan(0);
}); }, 30_000);
test('gbrain stats shows page count via CLI', () => { test('gbrain stats shows page count via CLI', () => {
const result = Bun.spawnSync({ const result = Bun.spawnSync({
@@ -558,7 +558,7 @@ describeE2E('E2E: Setup Journey', () => {
timeout: 15_000, timeout: 15_000,
}); });
expect(result.exitCode).toBe(0); expect(result.exitCode).toBe(0);
}); }, 30_000);
test('gbrain health runs via CLI', () => { test('gbrain health runs via CLI', () => {
const result = Bun.spawnSync({ const result = Bun.spawnSync({
@@ -568,7 +568,7 @@ describeE2E('E2E: Setup Journey', () => {
timeout: 15_000, timeout: 15_000,
}); });
expect(result.exitCode).toBe(0); expect(result.exitCode).toBe(0);
}); }, 30_000);
}); });
// ───────────────────────────────────────────────────────────────── // ─────────────────────────────────────────────────────────────────
@@ -787,7 +787,7 @@ describeE2E('E2E: Doctor Command', () => {
timeout: 15_000, timeout: 15_000,
}); });
expect(result.exitCode).toBe(0); expect(result.exitCode).toBe(0);
}); }, 60_000);
test('gbrain doctor --json produces valid JSON', () => { test('gbrain doctor --json produces valid JSON', () => {
const result = Bun.spawnSync({ const result = Bun.spawnSync({
@@ -806,7 +806,7 @@ describeE2E('E2E: Doctor Command', () => {
expect(typeof check.name).toBe('string'); expect(typeof check.name).toBe('string');
expect(typeof check.message).toBe('string'); expect(typeof check.message).toBe('string');
} }
}); }, 30_000);
}); });
// ───────────────────────────────────────────────────────────────── // ─────────────────────────────────────────────────────────────────
@@ -851,7 +851,7 @@ describeE2E('E2E: Parallel Import', () => {
expect(seqPageCount).toBeGreaterThan(0); expect(seqPageCount).toBeGreaterThan(0);
expect(seqChunkCount).toBeGreaterThan(0); expect(seqChunkCount).toBeGreaterThan(0);
}); }, 60_000);
test('parallel import with --workers 2 matches sequential page count', async () => { test('parallel import with --workers 2 matches sequential page count', async () => {
await setupDB(); await setupDB();
@@ -866,7 +866,7 @@ describeE2E('E2E: Parallel Import', () => {
const stats = await callOp('get_stats') as any; const stats = await callOp('get_stats') as any;
expect(stats.page_count).toBe(seqPageCount); expect(stats.page_count).toBe(seqPageCount);
}); }, 60_000);
test('parallel import has same chunk count (no duplicates)', async () => { test('parallel import has same chunk count (no duplicates)', async () => {
const stats = await callOp('get_stats') as any; const stats = await callOp('get_stats') as any;
@@ -912,7 +912,7 @@ describeE2E('E2E: Parallel Import', () => {
const stats = await callOp('get_stats') as any; const stats = await callOp('get_stats') as any;
expect(stats.page_count).toBe(seqPageCount); expect(stats.page_count).toBe(seqPageCount);
expect(stats.chunk_count).toBe(seqChunkCount); expect(stats.chunk_count).toBe(seqChunkCount);
}); }, 60_000);
test('re-import with workers is idempotent', async () => { test('re-import with workers is idempotent', async () => {
// Import again on top of existing data // Import again on top of existing data
@@ -927,7 +927,7 @@ describeE2E('E2E: Parallel Import', () => {
const stats = await callOp('get_stats') as any; const stats = await callOp('get_stats') as any;
expect(stats.page_count).toBe(seqPageCount); expect(stats.page_count).toBe(seqPageCount);
expect(stats.chunk_count).toBe(seqChunkCount); expect(stats.chunk_count).toBe(seqChunkCount);
}); }, 60_000);
}); });
// ───────────────────────────────────────────────────────────────── // ─────────────────────────────────────────────────────────────────