Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Commit c1c565d

Browse files
author
Michal Vlasák
committed
Add more tests
1 parent 544fc1f commit c1c565d

1 file changed

Lines changed: 30 additions & 2 deletions

File tree

tests/serializers.test.js

Lines changed: 30 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ test('Enable custom path', () => {
175175
expect(loggerWrites).toBeCalled();
176176
});
177177

178-
test.only('Some express headers are enabled by default', () => {
178+
test('Some express headers are enabled by default', () => {
179179
const loggerWrites = jest.fn();
180180
const logger = loggerFactory({
181181
streams: [
@@ -211,7 +211,7 @@ test.only('Some express headers are enabled by default', () => {
211211
});
212212
});
213213

214-
test.only('Express fields and headers can be enabled', () => {
214+
test('Express fields and headers can be enabled', () => {
215215
const loggerWrites = jest.fn();
216216
const logger = loggerFactory({
217217
enableFields: ['req.protocol', 'req.headers.host'],
@@ -239,3 +239,31 @@ test.only('Express fields and headers can be enabled', () => {
239239
expect(loggerWrites).toBeCalled();
240240
});
241241
});
242+
243+
test('Default express headers can be disabled', () => {
244+
const loggerWrites = jest.fn();
245+
const logger = loggerFactory({
246+
disableFields: ['req.headers.user-agent'],
247+
streams: [
248+
{
249+
stream: new stream.Writable({
250+
write: (chunk, encoding, next) => {
251+
const json = JSON.parse(chunk);
252+
expect(json.req.headers['user-agent']).toBeUndefined();
253+
loggerWrites();
254+
next();
255+
},
256+
}),
257+
},
258+
],
259+
});
260+
const app = express();
261+
const request = supertest(app);
262+
app.use(logger.express);
263+
return request
264+
.get('/')
265+
.set('x-deviceid', '1234')
266+
.then(() => {
267+
expect(loggerWrites).toBeCalled();
268+
});
269+
});

0 commit comments

Comments
 (0)