Skip to content

Commit

Permalink
Added jasmine-reporters as dependency and added option to use a defau…
Browse files Browse the repository at this point in the history
…lt junitxmlreporter
  • Loading branch information
TheRaven committed Aug 31, 2011
1 parent 1a44b52 commit 67400ed
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 5 deletions.
1 change: 1 addition & 0 deletions README.md
Expand Up @@ -28,5 +28,6 @@ You can supply the following arguments:
indicates passing (green) or failing (red) specs
* <code>--noColor</code>, do not use color in the output
* <code>--verbose</code>, verbose output as the specs are run
* <code>--junitreport</code>, export tests results as junitreport xml format'

Checkout spec/SampleSpecs.js to see how to use it.
13 changes: 12 additions & 1 deletion lib/jasmine-node/cli.js
Expand Up @@ -13,6 +13,13 @@ var showColors = true;
var extentions = "js";
var match = '.'

var junitreport = {
report: false,
savePath : "./reports/",
useDotNotation: true,
consolidate: true
}

var args = process.argv.slice(2);

while(args.length) {
Expand Down Expand Up @@ -46,6 +53,9 @@ while(args.length) {

require.paths.unshift(dir);
break;
case '--junitreport':
junitreport.report = true;
break;
default:
if (arg.match(/^--/)) help();
specFolder = Path.join(process.cwd(), arg);
Expand All @@ -65,7 +75,7 @@ jasmine.executeSpecsInFolder(specFolder, function(runner, log){
} else {
process.exit(1);
}
}, isVerbose, showColors, new RegExp(match + "spec\\.(" + extentions + ")$", 'i'));
}, isVerbose, showColors, new RegExp(match + "spec\\.(" + extentions + ")$", 'i'), junitreport);

function help(){
sys.print([
Expand All @@ -78,6 +88,7 @@ function help(){
, ' -i, --include DIR - add given directory to node include paths'
, ' --verbose - print extra information per each test run'
, ' --coffee - load coffee-script which allows execution .coffee files'
, ' --junitreport - export tests results as junitreport xml format'
, ''
].join("\n"));

Expand Down
10 changes: 7 additions & 3 deletions lib/jasmine-node/index.js
Expand Up @@ -39,7 +39,7 @@ jasmine.loadHelpersInFolder=function(folder, matcher)
}
}

jasmine.executeSpecsInFolder = function(folder, done, isVerbose, showColors, matcher){
jasmine.executeSpecsInFolder = function(folder, done, isVerbose, showColors, matcher, junitreport){
var log = [];
var columnCounter = 0;
var start = 0;
Expand All @@ -62,7 +62,11 @@ jasmine.executeSpecsInFolder = function(folder, done, isVerbose, showColors, mat
}

var jasmineEnv = jasmine.getEnv();
jasmineEnv.reporter = {
if(junitreport.report) {
require("jasmine-reporters");
jasmineEnv.addReporter(new jasmine.JUnitXmlReporter(junitreport.savePath, junitreport.consolidate, junitreport.useDotNotation));
}
jasmineEnv.addReporter ({
log: function(str){
},

Expand Down Expand Up @@ -139,7 +143,7 @@ jasmine.executeSpecsInFolder = function(folder, done, isVerbose, showColors, mat
}
(done||noop)(runner, log);
}
};
});
jasmineEnv.execute();
};

Expand Down
2 changes: 1 addition & 1 deletion package.json
Expand Up @@ -16,7 +16,7 @@
]
, "maintainers" : "Martin Häger <martin.haeger@gmail.com>"
, "licenses" : ["MIT"]
, "dependencies" : { "coffee-script" : ">=1.0.1"}
, "dependencies" : { "coffee-script" : ">=1.0.1", "jasmine-reporters" : "0.1.0"}
, "bin" : "bin/jasmine-node"
, "main" : "lib/jasmine-node"
}

0 comments on commit 67400ed

Please sign in to comment.