@erickmerchant/conditional-watch - npm Package Compare versions

Comparing version 2.1.5 to 2.1.6

package.json

 {
   "name": "@erickmerchant/conditional-watch",
-  "version": "2.1.5",
+  "version": "2.1.6",
   "description": "Watch using recursive-watch or not",
@@ -24,3 +24,3 @@ "main": "index.js",
   "devDependencies": {
-    "mockery": "^2.1.0",
+    "proxyquire": "^2.1.0",
     "standard": "^11.0.1",
@@ -27,0 +27,0 @@ "tape": "^4.9.1"

test.js

 const test = require('tape')
-const mockery = require('mockery')
-const mockerySettings = {
-  useCleanCache: true,
-  warnOnReplace: false,
-  warnOnUnregistered: false
-}
+const proxyquire = require('proxyquire').noPreserveCache()
 
@@ -22,41 +17,32 @@ test('false - returned value', function (t) {
 test('true - watching', function (t) {
-  mockery.enable(mockerySettings)
-
   t.plan(4)
 
-  mockery.registerMock('debounce-fn', function (fn, options) {
-    t.deepEqual(options, {wait: 150})
+  proxyquire('./index', {
+    'debounce-fn': function (fn, options) {
+      t.deepEqual(options, {wait: 150})
 
-    return fn
-  })
+      return fn
+    },
+    'recursive-watch': function (directory, fn) {
+      t.equal(directory, './foo')
 
-  mockery.registerMock('recursive-watch', function (directory, fn) {
-    t.equal(directory, './foo')
-
-    fn('a-file')
-  })
-
-  require('./index')(true, './foo', function (files) {
+      fn('a-file')
+    }
+  })(true, './foo', function (files) {
     t.ok(1)
   })
-
-  mockery.disable()
-
-  mockery.deregisterAll()
 })
 
 test('true - debounced', function (t) {
-  mockery.enable(mockerySettings)
-
   t.plan(2)
 
-  mockery.registerMock('recursive-watch', function (directory, fn) {
-    t.equal(directory, './foo')
+  proxyquire('./index', {
+    'recursive-watch': function (directory, fn) {
+      t.equal(directory, './foo')
 
-    fn('a-file')
+      fn('a-file')
 
-    fn('b-file')
-  })
-
-  require('./index')(true, './foo', function (files) {
+      fn('b-file')
+    }
+  })(true, './foo', function (files) {
     if (files) {
@@ -66,11 +52,5 @@ t.deepEqual(files, ['a-file', 'b-file'])
   })
-
-  mockery.disable()
-
-  mockery.deregisterAll()
 })
 
 test('true - multiple', function (t) {
-  mockery.enable(mockerySettings)
-
   t.plan(2)
@@ -80,11 +60,10 @@
 
-  mockery.registerMock('debounce-fn', function (fn, options) {
-    return fn
-  })
-
-  mockery.registerMock('recursive-watch', function (directory, fn) {
-    directories.push(directory)
-  })
-
-  require('./index')(true, ['./foo', './bar'], function (files) {
+  proxyquire('./index', {
+    'debounce-fn': function (fn, options) {
+      return fn
+    },
+    'recursive-watch': function (directory, fn) {
+      directories.push(directory)
+    }
+  })(true, ['./foo', './bar'], function (files) {
     t.ok(1)
@@ -94,6 +73,2 @@
   })
-
-  mockery.disable()
-
-  mockery.deregisterAll()
 })

No alert changes

Worsened metrics

Total package byte prevSize

3719

decreased by-9.05%

Lines of code

75

decreased by-12.79%

No dependency changes