Skip to content

Commit cb029a2

Browse files
test(integration): update discoverRoundtrip for the A11 discharge
The integration discoverRoundtrip test asserted the OLD A11-rider behavior (advertisement excludes listChanged). The rider was discharged in this PR; update the assertion to expect {tools: {listChanged: true}} (a designed existing-test modification — the rider's own acceptance criterion, same mandate as the discover.test.ts unit-test inversion).
1 parent e49a844 commit cb029a2

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

test/integration/test/client/discoverRoundtrip.test.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,9 @@ describe('server/discover round-trip against a modern server', () => {
8080
expect(client.getNegotiatedProtocolVersion()).toBe(MODERN);
8181
expect(client.getServerVersion()).toEqual({ name: 'dual-era-server', version: '2.0.0' });
8282
expect(client.getInstructions()).toBe('dual era');
83-
// The advertisement excludes listChanged-class capabilities, visible end to end.
84-
expect(client.getServerCapabilities()).toEqual({ tools: {} });
83+
// The advertisement carries listChanged-class capabilities now that
84+
// the serving entries serve subscriptions/listen, visible end to end.
85+
expect(client.getServerCapabilities()).toEqual({ tools: { listChanged: true } });
8586

8687
expect(bodies.some(b => b.includes('"initialize"'))).toBe(false);
8788
expect(bodies[0]).toContain('server/discover');

0 commit comments

Comments
 (0)