Skip to content

Commit 90ec385

Browse files
logaretmclaude
andcommitted
test: add default-fallback test for partial errorMessages override
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 28c3499 commit 90ec385

1 file changed

Lines changed: 14 additions & 0 deletions

File tree

packages/feedback/test/core/sendFeedback.test.ts

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -293,6 +293,20 @@ describe('sendFeedback', () => {
293293
).rejects.toMatch('custom forbidden text');
294294
});
295295

296+
it('falls back to default messages for codes not in errorMessages', async () => {
297+
mockSdk();
298+
vi.spyOn(getClient()!.getTransport()!, 'send').mockImplementation(() => {
299+
return Promise.resolve({ statusCode: 400 });
300+
});
301+
302+
// Only override ERROR_FORBIDDEN — a 400 should still use the default generic message.
303+
await expect(
304+
sendFeedback({ message: 'mi' }, { errorMessages: { ERROR_FORBIDDEN: 'custom forbidden text' } }),
305+
).rejects.toMatch(
306+
'Unable to send feedback. This could be because of network issues, or because you are using an ad-blocker.',
307+
);
308+
});
309+
296310
it('handles 400 transport error', async () => {
297311
mockSdk();
298312
vi.spyOn(getClient()!.getTransport()!, 'send').mockImplementation(() => {

0 commit comments

Comments
 (0)