a few more updates to tests

lib/common/api/deprecate.js

@@ -11,7 +11,8 @@ const deprecate = function (oldName, newName, fn) {
 }
 
 // The method is renamed.
-deprecate.rename = function (object, oldName, newName) {
+deprecate.rename = (object, oldName, newName) => {
+  console.log('we are here')
   let warned = false
   const newMethod = function () {
     if (!(warned || process.noDeprecation)) {

spec/api-deprecations-spec.js

@@ -7,44 +7,44 @@ describe.only('deprecations', () => {
     process.throwDeprecation = true
   })
 
-  it('allows a deprecation handler function to be specified', () => {
-    const messages = []
-
-    deprecations.setHandler((message) => {
-      messages.push(message)
-    })
-
-    deprecate.log('this is deprecated')
-    assert.deepEqual(messages, ['this is deprecated'])
-  })
-
-  it('returns a deprecation handler after one is set', () => {
-    const messages = []
-
-    deprecations.setHandler((message) => {
-      messages.push(message)
-    })
-
-    deprecate.log('this is deprecated')
-    assert(typeof deprecations.getHandler() === 'function')
-  })
-
-  it('returns a deprecation warning', () => {
-    const messages = []
-
-    deprecations.setHandler((message) => {
-      messages.push(message)
-    })
-
-    deprecate.warn('old', 'new')
-    assert.deepEqual(messages, [`'old' is deprecated. Use 'new' instead.`])
-  })
+  // it('allows a deprecation handler function to be specified', () => {
+  //   const messages = []
+  //
+  //   deprecations.setHandler((message) => {
+  //     messages.push(message)
+  //   })
+  //
+  //   deprecate.log('this is deprecated')
+  //   assert.deepEqual(messages, ['this is deprecated'])
+  // })
+  //
+  // it('returns a deprecation handler after one is set', () => {
+  //   const messages = []
+  //
+  //   deprecations.setHandler((message) => {
+  //     messages.push(message)
+  //   })
+  //
+  //   deprecate.log('this is deprecated')
+  //   assert(typeof deprecations.getHandler() === 'function')
+  // })
+  //
+  // it('returns a deprecation warning', () => {
+  //   const messages = []
+  //
+  //   deprecations.setHandler((message) => {
+  //     messages.push(message)
+  //   })
+  //
+  //   deprecate.warn('old', 'new')
+  //   assert.deepEqual(messages, [`'old' is deprecated. Use 'new' instead.`])
+  // })
 
-  it('throws an exception if no deprecation handler is specified', () => {
-    assert.throws(() => {
-      deprecate.log('this is deprecated')
-    }, /this is deprecated/)
-  })
+  // it('throws an exception if no deprecation handler is specified', () => {
+  //   assert.throws(() => {
+  //     deprecate.log('this is deprecated')
+  //   }, /this is deprecated/)
+  // })
 
   // it('deprecates a property', () => {
   //   deprecate.property(object, property, method)
@@ -60,8 +60,8 @@ describe.only('deprecations', () => {
 
   it('renames a method', () => {
     assert(typeof ipcRenderer.sendSync === 'function')
+    assert(typeof ipcRenderer.sendChannelSync === 'undefined')
     deprecate.rename(ipcRenderer, 'sendSync', 'sendChannelSync')
-    assert(typeof ipcRenderer.sendSync === 'undefined')
-    // assert(typeof ipcRenderer.sendChannelSync === 'function')
+    assert(typeof ipcRenderer.sendChannelSync === 'function')
   })
 })