summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEdwin Mons <edwin.mons@isode.com>2017-07-14 08:13:16 (GMT)
committerEdwin Mons <edwin.mons@isode.com>2017-07-14 09:17:40 (GMT)
commit0b4aaa1def9625ca3b4ddf869c988a68483135f8 (patch)
treea1c79e37def8ba3842d982db1734a1e60ad47639 /QA/Checker
parentfdf808cde2f3004aa61cba3bb60cb60a46665e4e (diff)
downloadswift-0b4aaa1def9625ca3b4ddf869c988a68483135f8.zip
swift-0b4aaa1def9625ca3b4ddf869c988a68483135f8.tar.bz2
Get rid of plural(s)
The stock test reporter doesn't use test(s), but test or tests based on the actual number printed. This brings the CppUnitTestResultPrinter more in line. Replaced two fflush(stdout)s with their C++ counterparts on std::cout. Test-information: Unit tests pass on macOS 10.12 Change-Id: Id9fb1f932d0d68cbc8e019f7748c7ecc3623556a
Diffstat (limited to 'QA/Checker')
-rw-r--r--QA/Checker/CppUnitTestResultPrinter.cpp24
1 files changed, 16 insertions, 8 deletions
diff --git a/QA/Checker/CppUnitTestResultPrinter.cpp b/QA/Checker/CppUnitTestResultPrinter.cpp
index 475411b..75eea72 100644
--- a/QA/Checker/CppUnitTestResultPrinter.cpp
+++ b/QA/Checker/CppUnitTestResultPrinter.cpp
@@ -53,6 +53,13 @@ namespace testing {
using namespace internal;
+static std::string plural(const std::string msg1, const std::string msg2, int n) {
+ if (n == 1) {
+ return msg1;
+ }
+ return msg2;
+}
+
static std::string PrintTestPartResultToString(const TestPartResult& testPartResult) {
return (Message() << internal::FormatFileLocation(testPartResult.file_name(), testPartResult.line_number())
<< " " << (testPartResult.type() == TestPartResult::Type::kSuccess ? "Success" : "Failure") << std::endl << testPartResult.message()).GetString();
@@ -70,8 +77,9 @@ void CppUnitTestResultPrinter::OnTestIterationStart(const UnitTest& unitTest, in
ColoredPrintf(COLOR_YELLOW, "Note: Randomizing tests' orders with a seed of %d .\n", unitTest.random_seed());
}
ColoredPrintf(COLOR_GREEN, "[==========] ");
- std::cout << "Running " << unitTest.test_to_run_count() << " test(s) from " << unitTest.test_case_to_run_count() << " test case(s)" << std::endl;
- fflush(stdout);
+ std::cout << "Running " << unitTest.test_to_run_count() << " " << plural("test", "tests", unitTest.test_to_run_count());
+ std::cout << " from " << unitTest.test_case_to_run_count() << " " << plural("test case", "test cases", unitTest.test_case_to_run_count()) << "." << std::endl;
+ std::cout.flush();
}
void CppUnitTestResultPrinter::OnTestPartResult(const TestPartResult& testPartResult) {
@@ -93,29 +101,29 @@ void CppUnitTestResultPrinter::OnTestEnd(const TestInfo& testInfo) {
void CppUnitTestResultPrinter::OnTestIterationEnd(const UnitTest& unitTest, int) {
ColoredPrintf(COLOR_GREEN, "\n[==========] ");
- std::cout << unitTest.test_to_run_count() << " test(s) from " << unitTest.test_case_to_run_count() << " test case(s) ran. ";
+ std::cout << unitTest.test_to_run_count() << " " << plural("test", "tests", unitTest.test_to_run_count()) << " from " << unitTest.test_case_to_run_count() << " " << plural("test case", "test cases", unitTest.test_case_to_run_count()) << " ran. ";
if (GTEST_FLAG(print_time)) {
std::cout << "(" << StreamableToString(unitTest.elapsed_time()).c_str() << " ms total)";
}
ColoredPrintf(COLOR_GREEN, "\n[ PASSED ] ");
- std::cout << unitTest.successful_test_count() << std::endl;
+ std::cout << unitTest.successful_test_count() << " " << plural("test", "tests", unitTest.successful_test_count()) << "." << std::endl;
int num_failures = unitTest.failed_test_count();
if (!unitTest.Passed()) {
ColoredPrintf(COLOR_RED, "[ FAILED ] ");
- std::cout << unitTest.failed_test_count() << " test(s), listed below:" << std::endl;
+ std::cout << unitTest.failed_test_count() << " " << plural("test", "tests", unitTest.failed_test_count()) << ", listed below:" << std::endl;
PrintFailedTests(unitTest);
- std::cout << std::endl << num_failures << " FAILED " << ((num_failures == 1) ? "TEST" : "TESTS") << std::endl;
+ std::cout << std::endl << num_failures << " FAILED " << plural("TEST", "TESTS", num_failures) << std::endl;
}
int num_disabled = unitTest.reportable_disabled_test_count();
if (num_disabled && !GTEST_FLAG(also_run_disabled_tests)) {
if (!num_failures) {
std::cout << std::endl;
}
- std::cout << num_disabled << " test(s) were disabled." << std::endl << std::endl;
+ std::cout << num_disabled << " " << plural("test", "tests", num_disabled) << " were disabled." << std::endl << std::endl;
}
std::cout << testFailures_.rdbuf();
- fflush(stdout);
+ std::cout.flush();
}
// Internal helper for printing the list of failed tests.