Add test coverage

Author: hectorhdzg

plugins/node/opentelemetry-instrumentation-bunyan/test/bunyan.test.ts

@@ -320,14 +320,40 @@ describe('BunyanInstrumentation', () => {
       assert.strictEqual(rec.attributes.aProperty, 'bar');
     });
 
-    it('log record level configuration (higher than default)', () => {
-      instrumentation.setConfig({ logSeverity: SeverityNumber.WARN });
+    it('log record error level', () => {
+      instrumentation.setConfig({ logSeverity: SeverityNumber.FATAL });
+      // Setting `logSeverity` only has an impact on Loggers created
+      // *after* it is set. So we cannot test with the `log` created in
+      // `beforeEach()` above.
+      log = Logger.createLogger({ name: 'test-logger-name', stream });
+      log.error('error log');
+      log.fatal('fatal log');
+      const logRecords = memExporter.getFinishedLogRecords();
+      // Only one log record match configured severity
+      assert.strictEqual(logRecords.length, 1);
+      assert.strictEqual(logRecords[0].body, 'fatal log');
+    });
 
+    it('log record error level', () => {
+      instrumentation.setConfig({ logSeverity: SeverityNumber.ERROR });
       // Setting `logSeverity` only has an impact on Loggers created
       // *after* it is set. So we cannot test with the `log` created in
       // `beforeEach()` above.
       log = Logger.createLogger({ name: 'test-logger-name', stream });
+      log.warn('warn log');
+      log.error('error log');
+      const logRecords = memExporter.getFinishedLogRecords();
+      // Only one log record match configured severity
+      assert.strictEqual(logRecords.length, 1);
+      assert.strictEqual(logRecords[0].body, 'error log');
+    });
 
+    it('log record warn level', () => {
+      instrumentation.setConfig({ logSeverity: SeverityNumber.WARN });
+      // Setting `logSeverity` only has an impact on Loggers created
+      // *after* it is set. So we cannot test with the `log` created in
+      // `beforeEach()` above.
+      log = Logger.createLogger({ name: 'test-logger-name', stream });
       log.info('info log');
       log.warn('warn log');
       const logRecords = memExporter.getFinishedLogRecords();
@@ -336,12 +362,25 @@ describe('BunyanInstrumentation', () => {
       assert.strictEqual(logRecords[0].body, 'warn log');
     });
 
-    it('log record level configuration (lower than default)', () => {
+    it('log record info level', () => {
+      instrumentation.setConfig({ logSeverity: SeverityNumber.INFO });
+      // Setting `logSeverity` only has an impact on Loggers created
+      // *after* it is set. So we cannot test with the `log` created in
+      // `beforeEach()` above.
+      log = Logger.createLogger({ name: 'test-logger-name', stream });
+      log.debug('debug log');
+      log.info('info log');
+      const logRecords = memExporter.getFinishedLogRecords();
+      // Only one log record match configured severity
+      assert.strictEqual(logRecords.length, 1);
+      assert.strictEqual(logRecords[0].body, 'info log');
+    });
+
+    it('log record debug level', () => {
       instrumentation.setConfig({ logSeverity: SeverityNumber.DEBUG });
       log = Logger.createLogger({ name: 'test-logger-name', stream });
       log.info('info log');
       log.debug('debug log');
-
       // Just the log.info() writes to `stream`.
       sinon.assert.calledOnce(writeSpy);
       // Both log.info() and log.debug() should be written to the OTel Logs SDK.
@@ -350,6 +389,22 @@ describe('BunyanInstrumentation', () => {
       assert.strictEqual(logRecords[0].body, 'info log');
       assert.strictEqual(logRecords[1].body, 'debug log');
     });
+
+    it('log record trace level', () => {
+      instrumentation.setConfig({ logSeverity: SeverityNumber.TRACE });
+      log = Logger.createLogger({ name: 'test-logger-name', stream });
+      log.info('info log');
+      log.debug('debug log');
+      log.debug('trace log');
+      // Just the log.info() writes to `stream`.
+      sinon.assert.calledOnce(writeSpy);
+      // Both log.info() and log.debug() should be written to the OTel Logs SDK.
+      const logRecords = memExporter.getFinishedLogRecords();
+      assert.strictEqual(logRecords.length, 3);
+      assert.strictEqual(logRecords[0].body, 'info log');
+      assert.strictEqual(logRecords[1].body, 'debug log');
+      assert.strictEqual(logRecords[2].body, 'trace log');
+    });
   });
 
   describe('disabled instrumentation', () => {