Merge pull request #8 from ta2edchimp/ava-bug

Use proxyquire to mock fs object ...

Author: Kent C. Dodds

package.json

@@ -37,9 +37,9 @@
     "eslint": "2.4.0",
     "eslint-config-kentcdodds": "6.0.0",
     "ghooks": "1.0.3",
-    "mock-fs": "3.8.0",
     "npm-run-all": "1.5.3",
     "nyc": "6.1.1",
+    "proxyquire": "1.7.4",
     "semantic-release": "4.3.5",
     "validate-commit-msg": "2.4.0"
   },

test/bin.js

@@ -0,0 +1,9 @@
+import test from 'ava'
+
+test.todo('test bin later')
+
+try {
+  require('../bin') // requiring now for coverage until this is tested
+} catch (error) {
+  // ignore the inevitable error
+}

test.js renamed to test/index.js

@@ -1,25 +1,10 @@
-import mock from 'mock-fs'
+import proxyquire from 'proxyquire'
 import test from 'ava'
-import findNewRules from './index'
 
-try {
-  require('./bin') // requiring now for coverage until this is tested
-} catch (error) {
-  // ignore the inevitable error
-}
-
-test.before(() => {
-  mock({
-    './node_modules/eslint/lib/rules': {
-      'foo-rule.js': '',
-      'bar-rule.js': '',
-      'baz-thing.js': '',
-    },
-  })
-})
-
-test.after(() => {
-  mock.restore()
+const findNewRules = proxyquire('../index', {
+  fs: {
+    readdirSync: () => ['foo-rule.js', 'bar-rule.js', 'baz-thing.js'],
+  },
 })
 
 test('returns the difference between what it finds in eslint/lib/rules and the rules array it is passed', t => {
@@ -32,4 +17,3 @@ test('returns an empty array if there is no difference', t => {
   t.true(Array.isArray(missingRules))
   t.same(missingRules.length, 0)
 })
-