feat: child dependencies will now have their deps mocked too

Author: fent

README.md

@@ -21,11 +21,16 @@ module.exports = (url) => {
 
 **test.js**
 ```js
+const request = require('request')
+
 const mockedRequest = (url, options, callback) => {
-  // Mock a request here.
+  // Log all requests.
+  console.log('Request made:', url);
+  request(url, options, callback)
 };
 
 const foo = muk('./foo', {
+  // Will overwrite all requires of "request" with our own version.
   request: mockedRequest
 });
 ```

lib/index.js

@@ -16,13 +16,14 @@ module.exports = (filename, deps) => {
 
   // Load children.
   const children = {};
-  Object.keys(deps).forEach((key) => {
+  for (let key in deps) {
     const childpath = Module._resolveFilename(key, m);
     const child = children[childpath] = new Module(childpath, m);
+    require.cache[childpath] = child;
     child.paths = Module._nodeModulePaths(path.dirname(childpath));
     child.loaded = true;
     child.exports = deps[key];
-  });
+  }
 
   m.require = (path) => {
     const childpath = Module._resolveFilename(path, m);
@@ -37,8 +38,12 @@ module.exports = (filename, deps) => {
   // Load module.
   m.load(filename);
 
-  // Delete module from cache so it can be required normally.
+  // Clear so mocked modules can be required normally.
   delete require.cache[filename];
+  for (let key in deps) {
+    let childpath = Module._resolveFilename(key, m);
+    delete require.cache[childpath];
+  }
 
   return m.exports;
 };

test/custom/child.js

@@ -0,0 +1 @@
+module.exports = require('fs');

test/custom/parent.js

@@ -0,0 +1,9 @@
+const fs = require('fs');
+const child = require('./child');
+
+
+module.exports = (callback) => {
+  process.nextTick(() => {
+    callback(fs, child);
+  });
+};

test/require-test.js

@@ -56,3 +56,13 @@ describe('Mock required relative file', () => {
 describe('Mock required relative file in a different dir', () => {
   require('./dir')('./custom', './bar');
 });
+
+describe.only('Mock modules required by children too', () => {
+  it('Mocks dependency in both parent and children', () => {
+    let mymock = {};
+    muk('./custom/parent.js', { 'fs': mymock })((fs, child) => {
+      assert.equal(fs, mymock);
+      assert.equal(child, mymock);
+    });
+  });
+});