Fix #139

pino.js

@@ -406,8 +406,8 @@ function genLog (z) {
     }
     p = n.length = arguments.length - l
     if (p > 1) {
-      l = countInterp(a, '%j')
-      if (l && typeof a === 'string') {
+      l = typeof a === 'string' ? countInterp(a, '%j') : 0
+      if (l) {
         n.length = l + countInterp(a, '%d') + countInterp(a, '%s') + 1
         o = `${util.format.apply(null, n)}`
       } else {

test/basic.test.js

@@ -292,6 +292,42 @@ test('correctly escape msg strings', function (t) {
   instance.fatal('this contains "')
 })
 
+// https://github.com/pinojs/pino/issues/139
+test('object and format string', function (t) {
+  var instance = pino(sink(function (chunk, enc, cb) {
+    delete chunk.time
+    t.deepEqual(chunk, {
+      pid: pid,
+      hostname: hostname,
+      level: 30,
+      msg: 'foo bar',
+      v: 1
+    })
+    t.end()
+    cb()
+  }))
+
+  instance.info({}, 'foo %s', 'bar')
+})
+
+test('object and format string property', function (t) {
+  var instance = pino(sink(function (chunk, enc, cb) {
+    delete chunk.time
+    t.deepEqual(chunk, {
+      pid: pid,
+      hostname: hostname,
+      level: 30,
+      msg: 'foo bar',
+      answer: 42,
+      v: 1
+    })
+    t.end()
+    cb()
+  }))
+
+  instance.info({ answer: 42 }, 'foo %s', 'bar')
+})
+
 test('correctly strip undefined when returned from toJSON', function (t) {
   t.plan(1)