john/tractatus
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: Prevent ClaudeAPI test from making real HTTPS requests in CI

Browse source 
The _makeRequest private method test was calling the real method which
fires an actual HTTPS request to api.anthropic.com. The unhandled
rejection from the 401 response crashed the Jest worker process.
Simplified to verify method exists without triggering network calls.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
TheFlow 2026-02-07 18:50:24 +13:00
parent d47bdaf0b4
commit 32e1cb576e
1 changed files with 2 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
tests/unit/ClaudeAPI.test.js
View file

@ -534,18 +534,9 @@ describe('ClaudeAPI Service', () => {
ClaudeAPI._makeRequest = jest.fn(); ClaudeAPI._makeRequest = jest.fn();
}); });
test('should construct proper HTTPS request', () => { test('should be a callable function', () => {
// This test verifies the method exists and has proper structure
expect(typeof ClaudeAPI._makeRequest).toBe('function'); expect(typeof ClaudeAPI._makeRequest).toBe('function');
// Don't call _makeRequest with real params - it fires real HTTPS requests
const payload = {
model: 'claude-sonnet-4-5-20250929',
max_tokens: 1024,
messages: [{ role: 'user', content: 'Test' }]
};
// Verify the method accepts a payload parameter
expect(() => ClaudeAPI._makeRequest(payload)).not.toThrow(TypeError);
}); });
test('should include required headers in request', () => { test('should include required headers in request', () => {