Skip to content

Commit

Permalink
Merge pull request #202 from pelias/calculate-pass-rate-using-improve…
Browse files Browse the repository at this point in the history
…ments

Use improvements when calculating pass rate
  • Loading branch information
orangejulius authored Jun 30, 2020
2 parents 3f40b7f + 3dcc636 commit ab94468
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
4 changes: 3 additions & 1 deletion output_generators/autocomplete.js
Original file line number Diff line number Diff line change
Expand Up @@ -91,15 +91,17 @@ function prettyPrintSuiteResults( suiteResults, config, testSuites ){

console.log( '\nAggregate test results'.blue );
console.log( 'Pass: ' + suiteResults.stats.pass.toString().green );
console.log( 'Improvements: ' + suiteResults.stats.improvement.toString().green );
console.log( 'Fail: ' + suiteResults.stats.fail.toString().yellow );
console.log( 'Placeholders: ' + suiteResults.stats.placeholder.toString().cyan );

var numRegressions = suiteResults.stats.regression;
var regressionsColor = ( numRegressions > 0 ) ? 'red' : 'yellow';
var total = suiteResults.stats.pass + suiteResults.stats.fail + suiteResults.stats.regression;
var total = suiteResults.stats.pass + suiteResults.stats.fail + suiteResults.stats.regression + suiteResults.stats.improvement;
var pass = total - numRegressions;

console.log( 'Regressions: ' + numRegressions.toString()[ regressionsColor ] );
console.log( 'Total tests: ' + total );
console.log( 'Took %sms', suiteResults.stats.timeTaken );
console.log( 'Test success rate %s%%', percentageForDisplay(total, pass));
console.log( '' );
Expand Down
3 changes: 2 additions & 1 deletion output_generators/terminal.js
Original file line number Diff line number Diff line change
Expand Up @@ -128,10 +128,11 @@ function prettyPrintSuiteResults( suiteResults, config, testSuites ){
console.log( 'Placeholders: ' + suiteResults.stats.placeholder.toString().cyan );

var numRegressions = suiteResults.stats.regression;
var total = suiteResults.stats.pass + suiteResults.stats.fail + suiteResults.stats.regression;
var total = suiteResults.stats.pass + suiteResults.stats.fail + suiteResults.stats.regression + suiteResults.stats.improvement;
var pass = total - numRegressions;

console.log( 'Regressions: ' + numRegressions.toString().red);
console.log( 'Total tests: ' + total );
console.log( 'Took %sms', suiteResults.stats.timeTaken );
console.log( 'Test success rate %s%%', percentageForDisplay(total,pass));

Expand Down

0 comments on commit ab94468

Please sign in to comment.