make it so that default level is set when optionsInner is used

chrisspiegl · Mar 22, 2022 · 75fd853 · 75fd853
1 parent fbb3074
commit 75fd853
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/index.js b/index.js
@@ -104,7 +104,8 @@ function isString(value) {
 function pnotice(namespace, options = {}) {
   return (text, level = 'INFO', optionsInner = {}) => {
     if (!isString(level)) {
-      optionsInner = level
+      optionsInner = { ...level }
+      level = undefined
     }
 
     if (optionsInner.level) {

diff --git a/test.js b/test.js
@@ -96,14 +96,25 @@ if (config.chat.id) {
     t.is(response?.status, 'ok')
   })
 
+  test('handle no level but optionsInner (with level) as second argument', async (t) => {
+    const pn = pnotice('pntest', {
+      disabled: false,
+      debug: true,
+      chat: config.chat,
+      env: 'test',
+    })
+    const response = await pn('handle no level but optionsInner (with level) as second argument', { level: 'TESTING', silent: true })
+    t.is(response?.status, 'ok')
+  })
+
   test('handle no level but optionsInner as second argument', async (t) => {
     const pn = pnotice('pntest', {
       disabled: false,
       debug: true,
       chat: config.chat,
       env: 'test',
     })
-    const response = await pn('handle no level but optionsInner as second argument', { level: 'TESTING', silent: true })
+    const response = await pn('handle no level but optionsInner as second argument', { silent: true })
     t.is(response?.status, 'ok')
   })