Skip to content

Commit

Permalink
Adds tests for loading reporters w/ relative/absolute paths (#2773)
Browse files Browse the repository at this point in the history
* Adds tests for loading reporters w/ relative/absolute paths

* Fixes the test based on CR

* Replaces '+' with path.join()
  • Loading branch information
Sulabh Bista authored and dasilvacontin committed Apr 24, 2017
1 parent 73929ad commit 0a93024
Show file tree
Hide file tree
Showing 2 changed files with 53 additions and 0 deletions.
26 changes: 26 additions & 0 deletions test/integration/fixtures/simple-reporter.js
@@ -0,0 +1,26 @@
var baseReporter = require('../../../lib/reporters/base');
module.exports = simplereporter;

function simplereporter (runner) {
baseReporter.call(this, runner);

runner.on('suite', function (suite) {
console.log('on(\'suite\') called');
});

runner.on('fail', function (test, err) {
console.log('on(\'fail\') called');
});

runner.on('pass', function (test) {
console.log('on(\'pass\') called');
});

runner.on('test end', function (test, err) {
console.log('on(\'test end\') called');
});

runner.on('end', function () {
console.log('on(\'end\') called');
});
}
27 changes: 27 additions & 0 deletions test/integration/reporters.spec.js
Expand Up @@ -60,4 +60,31 @@ describe('reporters', function () {
});
});
});

describe('loader', function () {
it('loads a reporter from a path relative to the current working directory', function (done) {
var reporterAtARelativePath = 'test/integration/fixtures/simple-reporter.js';

var args = ['--reporter=' + reporterAtARelativePath];

run('passing.fixture.js', args, function (err, result) {
assert(!err);
assert.equal(result.code, 0);
done();
});
});

it('loads a reporter from an absolute path', function (done) {
// Generates an absolute path string
var reporterAtAnAbsolutePath = path.join(process.cwd(), 'test/integration/fixtures/simple-reporter.js');

var args = ['--reporter=' + reporterAtAnAbsolutePath];

run('passing.fixture.js', args, function (err, result) {
assert(!err);
assert.equal(result.code, 0);
done();
});
});
});
});

0 comments on commit 0a93024

Please sign in to comment.