Browse Source

Tests: use consistent output spacing and flush before running each test.

Davin McCall 5 years ago
parent
commit
88b8bd0f1e
4 changed files with 33 additions and 32 deletions
  1. 10 10
      src/tests/cptests/cptests.cc
  2. 4 4
      src/tests/loadtests.cc
  3. 1 1
      src/tests/proctests.cc
  4. 18 17
      src/tests/tests.cc

+ 10 - 10
src/tests/cptests/cptests.cc

@@ -646,20 +646,20 @@ void cptest_addrmdeps()
 
 
 #define RUN_TEST(name, spacing) \
-    std::cout << #name "..." spacing; \
+    std::cout << #name "..." spacing << std::flush; \
     name(); \
     std::cout << "PASSED" << std::endl;
 
 int main(int argc, char **argv)
 {
-    RUN_TEST(cptest_queryver, "    ");
-    RUN_TEST(cptest_listservices, "");
-    RUN_TEST(cptest_findservice1, "");
-    RUN_TEST(cptest_findservice2, "");
-    RUN_TEST(cptest_findservice3, "");
-    RUN_TEST(cptest_loadservice, " ");
-    RUN_TEST(cptest_startstop, "   ");
-    RUN_TEST(cptest_unload, "      ");
-    RUN_TEST(cptest_addrmdeps, "   ");
+    RUN_TEST(cptest_queryver, "           ");
+    RUN_TEST(cptest_listservices, "       ");
+    RUN_TEST(cptest_findservice1, "       ");
+    RUN_TEST(cptest_findservice2, "       ");
+    RUN_TEST(cptest_findservice3, "       ");
+    RUN_TEST(cptest_loadservice, "        ");
+    RUN_TEST(cptest_startstop, "          ");
+    RUN_TEST(cptest_unload, "             ");
+    RUN_TEST(cptest_addrmdeps, "          ");
     return 0;
 }

+ 4 - 4
src/tests/loadtests.cc

@@ -49,15 +49,15 @@ void test_nonexistent()
 }
 
 #define RUN_TEST(name, spacing) \
-    std::cout << #name "..." spacing; \
+    std::cout << #name "..." spacing << std::flush; \
     name(); \
     std::cout << "PASSED" << std::endl;
 
 int main(int argc, char **argv)
 {
     init_test_service_dir();
-    RUN_TEST(test_basic, "      ");
-    RUN_TEST(test_env_subst, "  ");
-    RUN_TEST(test_nonexistent, "");
+    RUN_TEST(test_basic, "                ");
+    RUN_TEST(test_env_subst, "            ");
+    RUN_TEST(test_nonexistent, "          ");
     return 0;
 }

+ 1 - 1
src/tests/proctests.cc

@@ -788,7 +788,7 @@ void test_waitsfor_restart()
 
 
 #define RUN_TEST(name, spacing) \
-    std::cout << #name "..." spacing; \
+    std::cout << #name "..." spacing << std::flush; \
     name(); \
     std::cout << "PASSED" << std::endl;
 

+ 18 - 17
src/tests/tests.cc

@@ -552,26 +552,27 @@ void test12()
     assert(sset.count_active_services() == 0);
 }
 
-#define RUN_TEST(name) \
-    std::cout << #name "... "; \
+
+#define RUN_TEST(name, spacing) \
+    std::cout << #name "..." spacing << std::flush; \
     name(); \
     std::cout << "PASSED" << std::endl;
 
 int main(int argc, char **argv)
 {
-    RUN_TEST(test1);
-    RUN_TEST(test2);
-    RUN_TEST(test3);
-    RUN_TEST(test4);
-    RUN_TEST(test5);
-    RUN_TEST(test_pin1);
-    RUN_TEST(test_pin2);
-    RUN_TEST(test_pin3);
-    RUN_TEST(test_pin4);
-    RUN_TEST(test7);
-    RUN_TEST(test8);
-    RUN_TEST(test9);
-    RUN_TEST(test10);
-    RUN_TEST(test11);
-    RUN_TEST(test12);
+    RUN_TEST(test1, "                     ");
+    RUN_TEST(test2, "                     ");
+    RUN_TEST(test3, "                     ");
+    RUN_TEST(test4, "                     ");
+    RUN_TEST(test5, "                     ");
+    RUN_TEST(test_pin1, "                 ");
+    RUN_TEST(test_pin2, "                 ");
+    RUN_TEST(test_pin3, "                 ");
+    RUN_TEST(test_pin4, "                 ");
+    RUN_TEST(test7, "                     ");
+    RUN_TEST(test8, "                     ");
+    RUN_TEST(test9, "                     ");
+    RUN_TEST(test10, "                    ");
+    RUN_TEST(test11, "                    ");
+    RUN_TEST(test12, "                    ");
 }