Add tests for loading reporters w/ relative/absolute paths

[dasilvacontin]

I'd like to have tests for the new feature added in #2659.

Gonna do this today after I review the rest of the un-released commits. Let me know if you want to tackle it for me.

[sul4bh]

I can help with writing a test to cover #2659 if you give me some pointers on what to cover in the test.

[dasilvacontin]

@sul4bh Thanks!

It should cover:

• using a relative path as the argument for the reporter option
• using an absolute path as the argument for the reporter option

So maybe write a very short and useless reporter, and use this helper to execute the mocha binary, and check its output and that there haven't been any error:

mocha/test/integration/helpers.js

Lines 44 to 83 in 7a05a6c

	/**
	* Invokes the mocha binary on the code of the body of the function.
	* Accepts an array of additional command line args to pass. The callback is
	* invoked with a summary of the run, in addition to its output. The summary
	* includes the number of passing, pending, and failing tests, as well as the
	* exit code. Useful for testing different reporters.
	*
	* Example response:
	* {
	* pending: 0,
	* passing: 0,
	* failing: 1,
	* code: 1,
	* output: '...'
	* }
	*
	* @param {function} fixture
	* @param {array} args
	* @param {function} fn
	*/
	runMochaFunction: function (fixture, args, fn) {
	var path = resolveFixturePath(fixture.name + '.js' || 'tempfile.js');
	args = args || [];
	var fixtureContent = 'var fn = ' + fixture.toString() + '; fn()';
	fs.writeFileSync(path, fixtureContent, 'utf8');
	function cleanup () {
	fs.unlink(path);
	fn.apply(this, arguments);
	}
	invokeMocha(args.concat(['-C', path]), function (err, res) {
	if (err) {
	return cleanup(err);
	}
	cleanup(null, getSummary(res));
	});
	},

.

Ping me on Gitter if anything! :)

[sul4bh]

Awesome. I am on it!

[dasilvacontin]

@sul4bh ❤️

[dasilvacontin]

Closed via #2773.