@unito/integration-sdk 0.1.6 → 0.1.8

package/test/resources/logger.test.ts

@@ -1,10 +1,10 @@
 import assert from 'node:assert/strict';
 import { describe, it } from 'node:test';
-import Logger from '../../src/resources/logger.js';
+import { default as Logger, Metadata } from '../../src/resources/logger.js';
 
 describe('Logger', () => {
   it('metadata', () => {
-    let metadata: Record<string, unknown> = { correlation_id: 'test' };
+    let metadata: Metadata = { correlation_id: 'test' };
     const logger = new Logger(metadata);
 
     // Changing initial object should not affect the logger
@@ -42,7 +42,7 @@ describe('Logger', () => {
     logger.log('test');
     assert.strictEqual(logSpy.mock.calls.length, 1);
     assert.deepEqual(logSpy.mock.calls[0]?.arguments, [
-      JSON.stringify({ message: 'test', correlation_id: '123456789' }),
+      JSON.stringify({ correlation_id: '123456789', message: 'test' }),
     ]);
 
     const errorSpy = testContext.mock.method(global.console, 'error', () => {});
@@ -50,7 +50,7 @@ describe('Logger', () => {
     logger.error('test');
     assert.strictEqual(errorSpy.mock.calls.length, 1);
     assert.deepEqual(errorSpy.mock.calls[0]?.arguments, [
-      JSON.stringify({ message: 'test', correlation_id: '123456789' }),
+      JSON.stringify({ correlation_id: '123456789', message: 'test' }),
     ]);
 
     const warnSpy = testContext.mock.method(global.console, 'warn', () => {});
@@ -58,7 +58,7 @@ describe('Logger', () => {
     logger.warn('test');
     assert.strictEqual(warnSpy.mock.calls.length, 1);
     assert.deepEqual(warnSpy.mock.calls[0]?.arguments, [
-      JSON.stringify({ message: 'test', correlation_id: '123456789' }),
+      JSON.stringify({ correlation_id: '123456789', message: 'test' }),
     ]);
 
     const infoSpy = testContext.mock.method(global.console, 'info', () => {});
@@ -66,7 +66,7 @@ describe('Logger', () => {
     logger.info('test');
     assert.strictEqual(infoSpy.mock.calls.length, 1);
     assert.deepEqual(infoSpy.mock.calls[0]?.arguments, [
-      JSON.stringify({ message: 'test', correlation_id: '123456789' }),
+      JSON.stringify({ correlation_id: '123456789', message: 'test' }),
     ]);
 
     const debugSpy = testContext.mock.method(global.console, 'debug', () => {});
@@ -74,7 +74,60 @@ describe('Logger', () => {
     logger.debug('test');
     assert.strictEqual(debugSpy.mock.calls.length, 1);
     assert.deepEqual(debugSpy.mock.calls[0]?.arguments, [
-      JSON.stringify({ message: 'test', correlation_id: '123456789' }),
+      JSON.stringify({ correlation_id: '123456789', message: 'test' }),
+    ]);
+  });
+
+  it('merges message payload with metadata', testContext => {
+    const metadata = { correlation_id: '123456789', http: { method: 'GET' } };
+    const logger = new Logger(metadata);
+
+    const logSpy = testContext.mock.method(global.console, 'log', () => {});
+    assert.strictEqual(logSpy.mock.calls.length, 0);
+    logger.log('test', { error: { code: '200', message: 'Page Not Found' } });
+    assert.strictEqual(logSpy.mock.calls.length, 1);
+    assert.deepEqual(logSpy.mock.calls[0]?.arguments, [
+      JSON.stringify({
+        correlation_id: '123456789',
+        http: { method: 'GET' },
+        error: { code: '200', message: 'Page Not Found' },
+        message: 'test',
+      }),
+    ]);
+  });
+
+  it('overwrites conflicting metadata keys (1st level) with message payload', testContext => {
+    const metadata = { correlation_id: '123456789', http: { method: 'GET' } };
+    const logger = new Logger(metadata);
+
+    const logSpy = testContext.mock.method(global.console, 'log', () => {});
+    assert.strictEqual(logSpy.mock.calls.length, 0);
+    logger.log('test', { http: { status_code: 200 } });
+    assert.strictEqual(logSpy.mock.calls.length, 1);
+    assert.deepEqual(logSpy.mock.calls[0]?.arguments, [
+      JSON.stringify({
+        correlation_id: '123456789',
+        http: { status_code: 200 },
+        message: 'test',
+      }),
+    ]);
+  });
+
+  it('snakify keys of Message and Metadata', testContext => {
+    const metadata = { correlationId: '123456789', http: { method: 'GET', statusCode: 200 } };
+    const logger = new Logger(metadata);
+
+    const logSpy = testContext.mock.method(global.console, 'log', () => {});
+    assert.strictEqual(logSpy.mock.calls.length, 0);
+    logger.log('test', { errorContext: { errorCode: 200, errorMessage: 'Page Not Found' } });
+    assert.strictEqual(logSpy.mock.calls.length, 1);
+    assert.deepEqual(logSpy.mock.calls[0]?.arguments, [
+      JSON.stringify({
+        correlation_id: '123456789',
+        http: { method: 'GET', status_code: 200 },
+        error_context: { error_code: 200, error_message: 'Page Not Found' },
+        message: 'test',
+      }),
     ]);
   });
 });

package/test/resources/provider.test.ts

@@ -48,7 +48,7 @@ describe('Provider', () => {
         },
       },
     ]);
-    assert.deepEqual(actualResponse, { status: 200, headers: response.headers, data: { data: 'value' } });
+    assert.deepEqual(actualResponse, { status: 200, headers: response.headers, body: { data: 'value' } });
   });
 
   it('post with url encoded body', async context => {
@@ -86,7 +86,7 @@ describe('Provider', () => {
         },
       },
     ]);
-    assert.deepEqual(actualResponse, { status: 201, headers: response.headers, data: { data: 'value' } });
+    assert.deepEqual(actualResponse, { status: 201, headers: response.headers, body: { data: 'value' } });
   });
 
   it('put with json body', async context => {
@@ -125,7 +125,7 @@ describe('Provider', () => {
         },
       },
     ]);
-    assert.deepEqual(actualResponse, { status: 201, headers: response.headers, data: { data: 'value' } });
+    assert.deepEqual(actualResponse, { status: 201, headers: response.headers, body: { data: 'value' } });
   });
 
   it('patch with query params', async context => {
@@ -164,7 +164,7 @@ describe('Provider', () => {
         },
       },
     ]);
-    assert.deepEqual(actualResponse, { status: 201, headers: response.headers, data: { data: 'value' } });
+    assert.deepEqual(actualResponse, { status: 201, headers: response.headers, body: { data: 'value' } });
   });
 
   it('delete', async context => {
@@ -196,7 +196,7 @@ describe('Provider', () => {
         },
       },
     ]);
-    assert.deepEqual(actualResponse, { status: 204, headers: response.headers, data: undefined });
+    assert.deepEqual(actualResponse, { status: 204, headers: response.headers, body: undefined });
   });
 
   it('uses rate limiter if provided', async context => {
@@ -247,7 +247,7 @@ describe('Provider', () => {
         },
       },
     ]);
-    assert.deepEqual(actualResponse, { status: 204, headers: response.headers, data: undefined });
+    assert.deepEqual(actualResponse, { status: 204, headers: response.headers, body: undefined });
   });
 
   it('throws on invalid json response', async context => {