tiennm99/ccs
1
0
Fork 0
mirror of https://github.com/tiennm99/ccs.git synced 2026-09-02 20:24:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(proxy): update tests for tool_result+text and disconnect handler changes

Browse Source
This commit is contained in:
Grandis SYF
2026-04-20 13:06:00 -04:00
committed by Tam Nhu Tran
parent a0f91761ed
commit b2a4579098
2 changed files with 6 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/unit/proxy/messages-route.test.ts
+4 -6
View File
@@ -31,17 +31,15 @@ describe('attachDisconnectAbortHandlers', () => {
); );
expect(req.listenerCount('aborted')).toBe(1); expect(req.listenerCount('aborted')).toBe(1);
expect(req.listenerCount('close')).toBe(1); expect(req.listenerCount('close')).toBe(0);
expect(req.socket.listenerCount('close')).toBe(1); expect(req.socket.listenerCount('close')).toBe(1);
expect(res.listenerCount('close')).toBe(1); expect(res.listenerCount('close')).toBe(0);
expect(res.socket.listenerCount('close')).toBe(1); expect(res.socket.listenerCount('close')).toBe(1);
cleanup(); cleanup();
expect(req.listenerCount('aborted')).toBe(0); expect(req.listenerCount('aborted')).toBe(0);
expect(req.listenerCount('close')).toBe(0);
expect(req.socket.listenerCount('close')).toBe(0); expect(req.socket.listenerCount('close')).toBe(0);
expect(res.listenerCount('close')).toBe(0);
expect(res.socket.listenerCount('close')).toBe(0); expect(res.socket.listenerCount('close')).toBe(0);
}); });
@@ -60,9 +58,9 @@ describe('attachDisconnectAbortHandlers', () => {
} }
); );
req.emit('close'); req.emit('aborted');
req.socket.emit('close'); req.socket.emit('close');
res.emit('close'); res.socket.emit('close');
expect(controller.signal.aborted).toBe(true); expect(controller.signal.aborted).toBe(true);
expect(disconnectCount).toBe(1); expect(disconnectCount).toBe(1);
tests/unit/proxy/transformers/request-transformer-regressions.test.ts
+2 -2
View File
@@ -120,7 +120,7 @@ describe('ProxyRequestTransformer regressions', () => {
}, },
], ],
}) })
).toThrow('tool_result blocks must come before other user content'); ).not.toThrow();
expect(() => expect(() =>
new ProxyRequestTransformer().transform({ new ProxyRequestTransformer().transform({
@@ -135,7 +135,7 @@ describe('ProxyRequestTransformer regressions', () => {
}, },
], ],
}) })
).toThrow('must start with tool_result blocks for pending tool_use ids'); ).toThrow('must include tool_result blocks for pending tool_use ids');
}); });
it('rejects tool_result content that cannot be represented as OpenAI tool text', () => { it('rejects tool_result content that cannot be represented as OpenAI tool text', () => {