Skip to content

Commit

Permalink
Fix #125 We now use the language failed requests rather than test fai…
Browse files Browse the repository at this point in the history
…lures since that's what this is really showing.
  • Loading branch information
Ardesco committed Feb 26, 2016
1 parent 3615669 commit 792fd2f
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

##Next Version (Release Date TBC) Release Notes

* Issue #125 - Console output is now clearer when talking about failures.
* Merge pull request #111 (Thanks [Gordon](https://github.com/gordon00) - Add support for opening a test file in jmeter:gui
* **BREAKING CHANGE:** Merge pull request #161 (Thanks [Irek P](https://github.com/automatictester)) - Add support for multiple custom properties files

Expand Down
7 changes: 4 additions & 3 deletions src/main/java/com/lazerycode/jmeter/JMeterMojo.java
Original file line number Diff line number Diff line change
Expand Up @@ -77,12 +77,13 @@ void parseTestResults(List<String> results) throws MojoExecutionException, MojoF
}
}
getLog().info(" ");
getLog().info("Test Results:");
getLog().info("Performance Test Results:");
getLog().info(" ");
getLog().info("Tests Run: " + results.size() + ", Failures: " + totalFailureCount);
getLog().info("Total tests: " + results.size());
getLog().info("Failed requests: " + totalFailureCount);
getLog().info(" ");
if (failed) {
throw new MojoFailureException("There were " + totalFailureCount + " test failures. See the JMeter logs at '" + logsDir.getAbsolutePath() + "' for details.");
throw new MojoFailureException("There were " + totalFailureCount + " request failures. See the JMeter logs at '" + logsDir.getAbsolutePath() + "' for more details.");
}
}
}

0 comments on commit 792fd2f

Please sign in to comment.