Process all arguments debug.js style (#134)

* Process all arguments debug.js style

* add test for Process all arguments debug.js style

* removed comment line

---------

Co-authored-by: Andrey Kan <andrey.kan@dowjones.com>

debug.js

@@ -1,5 +1,7 @@
 'use strict'
 
+var util = require('util');
+
 module.exports = debug
 
 function debug (namespace) {
@@ -16,7 +18,8 @@ function debug (namespace) {
   function disabled () {}
   disabled.enabled = false
   function enabled () {
-    return log.apply(logger, arguments)
+    let message = util.format.apply(util, arguments) // this is how debug.js formats argeuments
+    return log.apply(logger, [message])
   }
   enabled.enabled = true
 

test/index.js

@@ -2,6 +2,8 @@
 var execSync = require('child_process').execSync
 var tap = require('tap')
 var through = require('through2')
+const pinoDebug = require("../index");
+const debug = require("debug");
 var test = tap.test
 
 const debugModules = [
@@ -342,3 +344,20 @@ test('does not invalidate strict mode', (t) => {
   t.is(require('./fixtures/strict-mode'), true)
   t.end()
 })
+
+test('Process all arguments debug.js style', (t) => {
+  var testOptions={"option1":"value1"};
+  process.env.DEBUG = 'ns1'
+  var pinoDebug = require('../')
+  var stream = through((line, _, cb) => {
+    var obj = JSON.parse(line)
+    var expectedMsg = "test { option1: 'value1' }";
+    t.is(obj.msg, expectedMsg)
+    t.is(obj.ns, 'ns1')
+    t.end()
+  })
+  pinoDebug(require('pino')({level: 'debug'}, stream))
+  var debug = require('debug')
+
+  debug('ns1')('test', testOptions)
+})