65 lines
2.1 KiB
Markdown
65 lines
2.1 KiB
Markdown
|
# Task ID: 67
|
||
|
|
|
||
|
|
**Title:** Add integration tests for gated session flow
|
||
|
|
|
||
|
|
**Status:** done
|
||
|
|
|
||
|
|
**Dependencies:** 65 ✓
|
||
|
|
|
||
|
|
**Priority:** high
|
||
|
|
|
||
|
|
**Description:** Create end-to-end integration tests for the complete gated session flow including connect, begin_session, tool calls, and read_prompts.
|
||
|
|
|
||
|
|
**Details:**
|
||
|
|
|
||
|
|
1. Add tests in `/src/mcplocal/tests/integration/gated-flow.test.ts`:
|
||
|
|
```typescript
|
||
|
|
describe('Gated Session Flow Integration', () => {
|
||
|
|
let app: FastifyInstance;
|
||
|
|
let mcpClient: McpClient;
|
||
|
|
|
||
|
|
beforeAll(async () => {
|
||
|
|
app = await createTestApp();
|
||
|
|
// Seed test project with gated=true and test prompts
|
||
|
|
});
|
||
|
|
|
||
|
|
describe('end-to-end gated flow', () => {
|
||
|
|
it('connect → begin_session with tags → tools available → correct prompts', async () => {
|
||
|
|
// 1. Connect to MCP endpoint
|
||
|
|
const session = await mcpClient.connect(app, 'test-project');
|
||
|
|
|
||
|
|
// 2. Verify only begin_session available
|
||
|
|
const toolsBefore = await session.listTools();
|
||
|
|
expect(toolsBefore.map(t => t.name)).toEqual(['begin_session']);
|
||
|
|
|
||
|
|
// 3. Call begin_session
|
||
|
|
const briefing = await session.callTool('begin_session', {
|
||
|
|
tags: ['test', 'integration']
|
||
|
|
});
|
||
|
|
expect(briefing).toContain('matched prompt content');
|
||
|
|
|
||
|
|
// 4. Verify all tools now available
|
||
|
|
const toolsAfter = await session.listTools();
|
||
|
|
expect(toolsAfter.map(t => t.name)).toContain('read_prompts');
|
||
|
|
});
|
||
|
|
});
|
||
|
|
|
||
|
|
describe('end-to-end intercept flow', () => {
|
||
|
|
it('connect → skip begin_session → call tool → keywords extracted → briefing injected', async () => { ... });
|
||
|
|
});
|
||
|
|
|
||
|
|
describe('end-to-end read_prompts', () => {
|
||
|
|
it('after ungating → request more context → additional prompts → no duplicates', async () => { ... });
|
||
|
|
});
|
||
|
|
});
|
||
|
|
```
|
||
|
|
|
||
|
|
**Test Strategy:**
|
||
|
|
|
||
|
|
This task IS the test implementation. Verify:
|
||
|
|
- Happy path tests pass
|
||
|
|
- Intercept path tests pass
|
||
|
|
- read_prompts deduplication works
|
||
|
|
- Tests use realistic data
|
||
|
|
- Tests clean up properly
|