Add single suite summary output format

cegonse · Oct 11, 2024 · 6c832ff · 6c832ff
1 parent dc82dca
commit 6c832ff
Show file tree

Hide file tree

Showing 4 changed files with 43 additions and 7 deletions.
diff --git a/src/arg-parser.hpp b/src/arg-parser.hpp
@@ -1,7 +1,8 @@
 #pragma once
 #include "types.hpp"
 #include <stdexcept>
-#include <string>
+#include <iostream>
+#include <cstring>
 
 namespace cest
 {
@@ -28,6 +29,11 @@ namespace cest
           options.generate_test_report = true;
         }
 
+        if (strcmp(argv[i], "-o") == 0 || strcmp(argv[i], "--only-suite-result") == 0)
+        {
+          options.only_test_suite_result = true;
+        }
+
         if (strcmp(argv[i], "-s") == 0 || strcmp(argv[i], "--seed") == 0)
         {
           if (i + 1 < argc)
@@ -39,6 +45,7 @@ namespace cest
             }
             catch (const std::invalid_argument &err)
             {
+              std::cout << "Invalid seed value: " << argv[i + 1] << std::endl;
             }
           }
         }

diff --git a/src/main.hpp b/src/main.hpp
@@ -33,7 +33,11 @@ int main(int argc, const char *argv[])
 
   cest::configureFocusedTestSuite(root_suite);
   cest::runTestSuite(root_suite);
-  cest::printTestSuiteResult(root_suite);
+
+  if (command_line_options.only_test_suite_result)
+    cest::printSuiteSummaryResult(root_suite);
+  else
+    cest::printTestSuiteResult(root_suite);
 
   return 0;
 }
diff --git a/src/output.hpp b/src/output.hpp
@@ -24,25 +24,30 @@ namespace cest
               << std::endl;
     std::cout << "Command line options:" << std::endl;
     std::cout << "    -r/--randomize: Randomize test executions" << std::endl;
+    std::cout << "    -o/--only-suite-result: Only output the test suite result" << std::endl;
     std::cout << "    -s/--seed <seed>: Inject seed for randomization uses (unsigned integer)";
     std::cout << std::endl;
   }
 
-  void printTestCaseResult(cest::TestCase *test_case)
+  void printTestBadge(bool failed, bool skipped = false)
   {
-    if (test_case->failed)
+    if (failed)
     {
       std::cout << ASCII_BACKGROUND_RED << ASCII_BLACK << ASCII_BOLD << " FAIL " << ASCII_RESET;
     }
-    else if (test_case->condition == cest::TestCaseCondition::Skipped)
+    else if (skipped)
     {
       std::cout << ASCII_BACKGROUND_YELLOW << ASCII_BLACK << ASCII_BOLD << " SKIP " << ASCII_RESET;
     }
     else
     {
       std::cout << ASCII_BACKGROUND_GREEN << ASCII_BLACK << ASCII_BOLD << " PASS " << ASCII_RESET;
     }
+  }
 
+  void printTestCaseResult(cest::TestCase *test_case)
+  {
+    printTestBadge(test_case->failed, test_case->condition == cest::TestCaseCondition::Skipped);
     std::cout << ASCII_GRAY << " " << test_case->fn.file << ":" << test_case->fn.line << ASCII_RESET << ASCII_BOLD << " " << test_case->name << ASCII_RESET << std::endl;
 
     if (test_case->failed)
@@ -53,12 +58,31 @@ namespace cest
 
   void printTestSuiteResult(cest::TestSuite *suite)
   {
-    std::cout << ASCII_BOLD << suite->name << ASCII_RESET << std::endl;
-
     for (cest::TestCase *test_case : suite->test_cases)
       printTestCaseResult(test_case);
 
     for (auto &pair : suite->test_suites)
       printTestSuiteResult(pair.second);
   }
+
+  void printSuiteSummaryResult(cest::TestSuite *suite)
+  {
+    bool any_test_failed = false;
+
+    for (size_t i = 0; i < suite->test_cases.size(); ++i)
+    {
+      cest::TestCase *test_case = suite->test_cases[i];
+      if (test_case->failed)
+      {
+        any_test_failed = true;
+        break;
+      }
+    }
+
+    printTestBadge(any_test_failed);
+    std::cout << ASCII_BOLD << " " << suite->name << ASCII_RESET << std::endl;
+
+    for (auto &pair : suite->test_suites)
+      printSuiteSummaryResult(pair.second);
+  }
 }
diff --git a/src/types.hpp b/src/types.hpp
@@ -63,6 +63,7 @@ namespace cest
     unsigned int random_seed;
     bool random_seed_present;
     bool generate_test_report;
+    bool only_test_suite_result;
   };
 
   struct CestGlobals