Skip to content

Commit 36c33eb

Browse files
committed
more test added and passed
1 parent 17d52e4 commit 36c33eb

File tree

4 files changed

+44
-20
lines changed

4 files changed

+44
-20
lines changed

__tests__/getStatusMessage.test.ts

Lines changed: 31 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,44 @@
1-
21
import { StatusCodes } from '../src/statusCodes';
32
import { getStatusMessage } from './../src/getStatusMessage';
43

54
describe('getStatusMessage', () => {
5+
// check if it returns a short message
66
it('should return a short message by default', () => {
77
expect(getStatusMessage(StatusCodes.OK)).toBe('OK');
8+
expect(getStatusMessage(StatusCodes.CREATED)).toBe('Created');
9+
expect(getStatusMessage(StatusCodes.NOT_FOUND)).toBe('Not Found');
10+
expect(getStatusMessage(StatusCodes.INSUFFICIENT_STORAGE)).toBe(
11+
'Insufficient Storage'
12+
);
13+
expect(getStatusMessage(StatusCodes.MOVED_PERMANENTLY)).toBe('Moved Permanently');
14+
expect(getStatusMessage(StatusCodes.PERMANENT_REDIRECT)).toBe('Permanent Redirect');
815
});
916

1017

18+
// check if it returns a long message
1119
it('should return a long message if long = true', () => {
12-
expect(getStatusMessage(StatusCodes.OK, true)).toMatch(/successful/i)
13-
})
20+
expect(getStatusMessage(StatusCodes.OK, true)).toMatch(/successful/i);
21+
expect(getStatusMessage(StatusCodes.CREATED, true)).toMatch(
22+
/successfully/i
23+
);
24+
expect(getStatusMessage(StatusCodes.NOT_FOUND, true)).toMatch(
25+
/cannot find/i
26+
);
27+
expect(getStatusMessage(StatusCodes.INSUFFICIENT_STORAGE, true)).toMatch(
28+
/unable to store/i
29+
);
30+
expect(getStatusMessage(StatusCodes.MOVED_PERMANENTLY, true)).toMatch(
31+
/permanently moved /i
32+
);
33+
expect(getStatusMessage(StatusCodes.PERMANENT_REDIRECT, true)).toMatch(
34+
/permanent redirect/i
35+
);
36+
});
37+
1438

39+
// check if it returns a "Unknown Status" for invalid code
1540
it('should return a "Unknown Status" for invalid code', () => {
1641
expect(getStatusMessage(999 as any)).toBe('Unknown Status');
17-
})
18-
19-
})
20-
42+
expect(getStatusMessage(600 as any, true)).toBe('Unknown Status');
43+
});
44+
});

package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,8 @@
3636
"build": "tsup",
3737
"start": "node dist/index.js",
3838
"dev": "ts-node src/index.ts",
39-
"test": "jest"
39+
"test": "jest",
40+
"check": "npm run build && npm run test"
4041
},
4142
"keywords": [
4243
"http",

src/getStatusMessage.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
// import shortMessages from './shortStatusMessages';
2-
32
import {LongStatusMessages} from './longStatusMessages';
43
import type { StatusCode } from './statusCodes';
54
import {StatusMessages} from './statusMessage';

src/longStatusMessages.ts

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -62,17 +62,17 @@ export const LongStatusMessages = {
6262
451: 'Access to the resource is denied for legal reasons.',
6363

6464
// --- 5xx: Server Errors ---
65-
500: 'Internal Server Error',
66-
501: 'Not Implemented',
67-
502: 'Bad Gateway',
68-
503: 'Service Unavailable',
69-
504: 'Gateway Timeout',
70-
505: 'HTTP Version Not Supported',
71-
506: 'Variant Also Negotiates',
72-
507: 'Insufficient Storage',
73-
508: 'Loop Detected',
74-
510: 'Not Extended',
75-
511: 'Network Authentication Required',
65+
500: 'The server encountered an unexpected internal error.',
66+
501: 'The server does not support the requested feature.',
67+
502: 'Received an invalid response from the upstream server.',
68+
503: 'The server is currently unavailable or overloaded.',
69+
504: 'The gateway timed out while waiting for a response.',
70+
505: 'The HTTP version used is not supported by the server.',
71+
506: 'Content negotiation failed due to a server issue.',
72+
507: 'The server is unable to store the representation needed.',
73+
508: 'The server detected an infinite loop while processing.',
74+
510: 'Further extensions are required to complete the request.',
75+
511: 'Network authentication is required to access this resource.',
7676

7777
// Unofficial
7878
419: 'Page expired due to missing or invalid CSRF token.',

0 commit comments

Comments
 (0)