fix(axios-extension): log the level set by the response node (#1048)

SAP · May 31, 2023 · aeb4cd8 · aeb4cd8
1 parent 50d1d36
commit aeb4cd8
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/.changeset/new-crabs-invite.md b/.changeset/new-crabs-invite.md
@@ -0,0 +1,5 @@
+---
+'@sap-ux/axios-extension': patch
+---
+
+handle entry severity in logging
diff --git a/packages/axios-extension/src/abap/message.ts b/packages/axios-extension/src/abap/message.ts
@@ -76,7 +76,7 @@ export function prettyPrintMessage({ msg, log, host }: { msg: string; log: Logge
         logFullURL({ host, path: jsonMsg['longtext_url'], log });
         if (jsonMsg.details) {
             jsonMsg.details.forEach((entry) => {
-                log.info(entry.message);
+                logLevel(entry.severity, entry.message, log);
             });
         }
     } catch (error) {

diff --git a/packages/axios-extension/test/abap/message.test.ts b/packages/axios-extension/test/abap/message.test.ts
@@ -14,14 +14,16 @@ describe('message helpers', () => {
                 longtext_url: '/abc/de',
                 details: [
                     { code: '1', message: '~message', severity: 'info' },
-                    { code: '2', message: '~message', severity: 'info' }
+                    { code: '2', message: '~message', severity: 'warning' }
                 ]
             };
 
             const infoMock = (log.info = jest.fn());
+            const warningMock = (log.warn = jest.fn());
             prettyPrintMessage({ msg: JSON.stringify(msg), log, host });
             // log main message, two messages for the full url, and each detail
-            expect(infoMock).toBeCalledTimes(1 + 2 + msg.details.length);
+            expect(infoMock).toBeCalledTimes(4);
+            expect(warningMock).toBeCalledTimes(1);
         });
 
         test('log none JSON message for debugging', () => {