add tests

lib/common/api/deprecate.js

@@ -36,12 +36,10 @@ const deprecate = {
   },
 
   function: (fn, newName) => {
-    // if newName is left blank, a removal warning will be displayed
     const warn = warnOnce(fn.name, newName)
-
     return function () {
-      if (!process.noDeprecation) warn()
-      return fn.apply(this, arguments)
+      warn()
+      fn.apply(this, arguments)
     }
   },
 

spec/api-deprecations-spec.js

@@ -82,9 +82,35 @@ describe('deprecations', () => {
     expect(msg).to.include(prop)
   })
 
+  it('warns exactly once when a function is deprecated with no replacement', () => {
+    let msg
+    deprecations.setHandler(m => { msg = m })
+
+    function oldFn () { return 'hello' }
+    deprecate.function(oldFn)
+    oldFn()
+
+    expect(msg).to.be.a('string')
+    expect(msg).to.include(oldFn)
+  })
+
+  it('warns exactly once when a function is deprecated with a replacement', () => {
+    let msg
+    deprecations.setHandler(m => { msg = m })
+
+    function oldFn () { return 'hello' }
+    function newFn () { return 'goodbye' }
+    deprecate.function(oldFn)
+    oldFn()
+
+    expect(msg).to.be.a('string')
+    expect(msg).to.include(oldFn)
+    expect(msg).to.include(newFn)
+  })
+
   it('warns only once per item', () => {
     const messages = []
-    deprecations.setHandler(message => { messages.push(message) })
+    deprecations.setHandler(message => messages.push(message))
 
     const key = 'foo'
     const val = 'bar'
@@ -125,15 +151,15 @@ describe('deprecations', () => {
 
     const enableCallbackWarnings = () => {
       warnings = []
-      deprecations.setHandler(warning => { warnings.push(warning) })
+      deprecations.setHandler(warning => warnings.push(warning))
       process.enablePromiseAPIs = true
     }
 
     beforeEach(() => {
       deprecations.setHandler(null)
       process.throwDeprecation = true
 
-      promiseFunc = param => new Promise((resolve, reject) => { resolve(param) })
+      promiseFunc = param => new Promise((resolve, reject) => resolve(param))
     })
 
     it('acts as a pass-through for promise-based invocations', async () => {