diff --git a/.changeset/new-crabs-invite.md b/.changeset/new-crabs-invite.md new file mode 100644 index 0000000000..58761dc2ba --- /dev/null +++ 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 index 9b45e2a385..e2fee697a3 100644 --- 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 index b476945110..eff82c4af9 100644 --- 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', () => {