Option to also clear cache for children modules

[jasonkhanlar]

Add option to also delete cache for all child modules of the specified module using { children: true } passed as 2nd argument

[sindresorhus]

Would it maybe be smart to run the children through clear(), making it recursive, instead of deleting them directly. That way the children too would be run through the code on line 26-35 that deletes the children from their parents, preventing memory leaks.

[jasonkhanlar]

Oh, I was just working on a recursive implementation before I notice your comment.

[jasonkhanlar]

Mm, yes, it probably would have been easier to use clear() and comparing root directory to make sure files are nested inside.

[sindresorhus]

I'm ok with adding this option, but it needs to be documented and tested.

[jasonkhanlar]

I'm ok with adding this option, but it needs to be documented and tested.

I'm not sure how to write a test in test.js for clearModule('modulename', { children: true });
but I'll try working from:

test('clearModule()', t => {
        const id = './fixture';
        t.is(require(id)(), 1);
        t.is(require(id)(), 2);
        m(id);
        t.is(require(id)(), 1);
});

I'm working on a script to add to test.js. It's not pretty, but I'm currently playing around with this, trying to pass some tests, and gather some debug data:

test('clearModule("fixture", {children: true})', t => {
	const id = './fixture';
	t.is(require(id)(), 1);
	t.is(require(id)(), 2);
	m(id);
	t.is(require(id)(), 1);

	const scriptsCachedBeforeRequire = Object.keys(require.cache).length;
	const path = require('path');
	const id2 = 'resolve-from';
	const resolveFrom = require(id2);
	const id3 = 'caller-path';
	const callerPath = require(id3);
	const id2Path = resolveFrom(path.dirname(callerPath()), id2);
	const id2Dir = path.dirname(id2Path);
	const id3Path = resolveFrom(path.dirname(callerPath()), id3);
	const id3Dir = path.dirname(id3Path);
	console.log(id2Dir);
	console.log(id3Dir);
	const scriptsCachedAfterRequire = Object.keys(require.cache).length;
	console.log(scriptsCachedBeforeRequire);
	console.log(scriptsCachedAfterRequire);
	t.is(require(id)(), 13); // Should fail intentionally
});

[sindresorhus]

Closing this in favor of #9.