chore: add tests for child logging

Author: carbonrobot

src/index.ts

@@ -1,4 +1,4 @@
-import pino, { BaseLogger, DestinationStream, LoggerOptions, LevelMapping } from 'pino';
+import pino, { BaseLogger, DestinationStream, LoggerOptions, LevelMapping, Bindings } from 'pino';
 import { GlobalContextStorageProvider, ContextStorageProvider, ContextMap } from './context';
 
 export interface ExtendedPinoOptions extends LoggerOptions {

src/tests/index.spec.ts

@@ -47,6 +47,16 @@ tap.test('should log an error message with apiRequestId', (t) => {
   t.end();
 });
 
+tap.test('should add tags with a child logger', (t) => {
+  const [log, output] = createLogger();
+  const childLogger = log.child({ userId: 12 });
+
+  log.withRequest({}, { awsRequestId: '9048989' });
+  childLogger.info('Message with userId');
+  t.matchSnapshot(output.buffer);
+  t.end();
+});
+
 /**
  * Creates a test logger and output buffer for assertions
  * Returns the logger and the buffer

tap-snapshots/src-tests-index.spec.ts-TAP.test.js

@@ -5,6 +5,10 @@
  * Make sure to inspect the output below.  Do not ignore changes!
  */
 'use strict'
+exports[`src/tests/index.spec.ts TAP should add tags with a child logger > must match snapshot 1`] = `
+2016-12-01T06:00:00.000Z	9048989	INFO	Message with userId	{"level":30,"time":1480572000000,"userId":12,"awsRequestId":"9048989","x-correlation-id":"9048989","msg":"Message with userId"}
+`
+
 exports[`src/tests/index.spec.ts TAP should log a simple info message > must match snapshot 1`] = `
 2016-12-01T06:00:00.000Z	undefined	INFO	Simple message	{"level":30,"time":1480572000000,"msg":"Simple message"}
 `