Skip to content

Commit 8702218

Browse files
committed
fix: change test for 500 error responses
1 parent 767cad8 commit 8702218

3 files changed

Lines changed: 13 additions & 7 deletions

File tree

server/controllers/user.controller/__tests__/authManagement/updateSettings.test.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -523,7 +523,7 @@ describe('user.controller > auth management > updateSettings (email, username, p
523523

524524
describe('and when there is any other error', () => {
525525
beforeEach(async () => {
526-
User.findById = jest.fn().mockRejectedValue('db error');
526+
User.findById = jest.fn().mockRejectedValue('Internal server error');
527527
requestBody = minimumValidRequest;
528528
request.setBody(requestBody);
529529
await updateSettings(
@@ -534,7 +534,9 @@ describe('user.controller > auth management > updateSettings (email, username, p
534534
});
535535
it('returns a 500 error', () => {
536536
expect(response.status).toHaveBeenCalledWith(500);
537-
expect(response.json).toHaveBeenCalledWith({ error: 'db error' });
537+
expect(response.json).toHaveBeenCalledWith({
538+
error: 'Internal server error'
539+
});
538540
});
539541
});
540542
});

server/controllers/user.controller/__tests__/helpers.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ describe('user.controller > helpers', () => {
8787
);
8888
expect(response.status).toHaveBeenCalledWith(500);
8989
expect(response.json).toHaveBeenCalledWith({
90-
error: 'async error'
90+
error: 'Internal server error'
9191
});
9292
});
9393
});

server/controllers/user.controller/__tests__/userPreferences.test.ts

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ describe('user.controller > user preferences', () => {
8484
it('returns 500 if saving preferences fails', async () => {
8585
mockUser = createMockUser({
8686
preferences: { ...mockUserPreferences, theme: AppThemeOptions.LIGHT },
87-
save: jest.fn().mockRejectedValue(new Error('DB error'))
87+
save: jest.fn().mockRejectedValue(new Error('Internal server error'))
8888
});
8989

9090
User.findById = jest
@@ -101,7 +101,9 @@ describe('user.controller > user preferences', () => {
101101
);
102102

103103
expect(response.status).toHaveBeenCalledWith(500);
104-
expect(response.json).toHaveBeenCalledWith({ error: expect.any(Error) });
104+
expect(response.json).toHaveBeenCalledWith({
105+
error: 'Internal server error'
106+
});
105107
});
106108
});
107109

@@ -155,7 +157,7 @@ describe('user.controller > user preferences', () => {
155157
it('returns 500 if saving cookieConsent fails', async () => {
156158
mockUser = createMockUser({
157159
cookieConsent: CookieConsentOptions.ALL,
158-
save: jest.fn().mockRejectedValue(new Error('DB error'))
160+
save: jest.fn().mockRejectedValue(new Error('Internal server error'))
159161
});
160162

161163
User.findById = jest
@@ -172,7 +174,9 @@ describe('user.controller > user preferences', () => {
172174
);
173175

174176
expect(response.status).toHaveBeenCalledWith(500);
175-
expect(response.json).toHaveBeenCalledWith({ error: expect.any(Error) });
177+
expect(response.json).toHaveBeenCalledWith({
178+
error: 'Internal server error'
179+
});
176180
});
177181
});
178182
});

0 commit comments

Comments
 (0)