ci: add no-valgrind tag to more tests

All tests taking longer than 1s in a recent CI run.
james-d-mitchell · Sep 17, 2024 · 03d86f6 · 03d86f6
1 parent 424412d
commit 03d86f6
Show file tree

Hide file tree

Showing 24 changed files with 125 additions and 115 deletions.
diff --git a/tests/test-cong.cpp b/tests/test-cong.cpp
@@ -995,7 +995,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("Congruence",
                           "035",
                           "stellar_monoid S4",
-                          "[quick][cong]") {
+                          "[quick][cong][no-valgrind]") {
     auto                    rg = ReportGuard(false);
     Presentation<word_type> p  = zero_rook_monoid(4);
 
@@ -1172,7 +1172,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("Congruence",
                           "042",
                           "const_contains",
-                          "[quick][cong]") {
+                          "[quick][cong][no-valgrind]") {
     auto                    rg = ReportGuard(false);
     Presentation<word_type> p;
     p.alphabet(2);

diff --git a/tests/test-cutting.cpp b/tests/test-cutting.cpp
@@ -67,7 +67,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("Cutting",
                           "002",
                           "step_hen Stephen test case 004",
-                          "[cutting][quick]") {
+                          "[cutting][quick][no-valgrind]") {
     ToWord                         to_word("xyzXYZ");
     InversePresentation<word_type> p;
     p.alphabet(to_word("xyzXYZ"));
@@ -87,7 +87,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("Cutting",
                           "003",
                           "symmetric_inverse_semigroup",
-                          "[cutting][quick]") {
+                          "[cutting][quick][no-valgrind]") {
     auto p = to_inverse_presentation(fpsemigroup::symmetric_inverse_monoid(4));
     REQUIRE(p.alphabet() == 01234567_w);
     REQUIRE(p.inverses() == 45670123_w);
@@ -128,7 +128,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("Cutting",
                           "005",
                           "dual_symmetric_inverse_monoid",
-                          "[cutting][quick]") {
+                          "[cutting][quick][no-valgrind]") {
     auto p = to_inverse_presentation(
         fpsemigroup::dual_symmetric_inverse_monoid(4));
     REQUIRE(p.contains_empty_word());

diff --git a/tests/test-fpsemi-examples-1.cpp b/tests/test-fpsemi-examples-1.cpp
@@ -642,7 +642,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("fpsemi-examples",
                           "034",
                           "uniform_block_bijection_monoid(5)",
-                          "[fpsemi-examples][quick]") {
+                          "[fpsemi-examples][quick][no-valgrind]") {
     auto   rg = ReportGuard(REPORT);
     size_t n  = 5;
     auto   p  = uniform_block_bijection_monoid(n, author::FitzGerald);
@@ -683,7 +683,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("fpsemi-examples",
                           "038",
                           "orientation_preserving_reversing_monoid(5)",
-                          "[fpsemi-examples][quick]") {
+                          "[fpsemi-examples][quick][no-valgrind]") {
     auto        rg = ReportGuard(REPORT);
     size_t      n  = 5;
     ToddCoxeter tc(congruence_kind::twosided,
@@ -756,7 +756,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("fpsemi-examples",
                           "044",
                           "symmetric_group(6) Carmichael",
-                          "[fpsemi-examples][quick]") {
+                          "[fpsemi-examples][quick][no-valgrind]") {
     auto        rg = ReportGuard(REPORT);
     size_t      n  = 6;
     ToddCoxeter tc(congruence_kind::twosided,
@@ -767,7 +767,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("fpsemi-examples",
                           "045",
                           "symmetric_group(6) Moore index 0",
-                          "[fpsemi-examples][quick]") {
+                          "[fpsemi-examples][quick][no-valgrind]") {
     auto   rg = ReportGuard(REPORT);
     size_t n  = 6;
 
@@ -779,7 +779,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("fpsemi-examples",
                           "046",
                           "symmetric_group(7) Moore index 1",
-                          "[fpsemi-examples][quick]") {
+                          "[fpsemi-examples][quick][no-valgrind]") {
     auto        rg = ReportGuard(REPORT);
     size_t      n  = 7;
     ToddCoxeter tc(congruence_kind::twosided,
@@ -839,7 +839,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("fpsemi-examples",
                           "049",
                           "full_transformation_monoid(5) Iwahori",
-                          "[fpsemi-examples][quick]") {
+                          "[fpsemi-examples][quick][no-valgrind]") {
     auto   rg = ReportGuard(REPORT);
     size_t n  = 5;
 
@@ -851,7 +851,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("fpsemi-examples",
                           "050",
                           "partial_transformation_monoid(5) Sutov",
-                          "[fpsemi-examples][quick]") {
+                          "[fpsemi-examples][quick][no-valgrind]") {
     auto        rg = ReportGuard(REPORT);
     size_t      n  = 5;
     ToddCoxeter tc(congruence_kind::twosided,
@@ -938,7 +938,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("fpsemi-examples",
                           "058",
                           "cyclic_inverse_monoid(8) Fernandes index 1",
-                          "[fpsemi-examples][quick]") {
+                          "[fpsemi-examples][quick][no-valgrind]") {
     auto        rg = ReportGuard(REPORT);
     size_t      n  = 8;
     ToddCoxeter tc(congruence_kind::twosided,
@@ -1051,7 +1051,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("fpsemi-examples",
                           "072",
                           "partial_brauer_monoid(4)",
-                          "[fpsemi-examples][quick]") {
+                          "[fpsemi-examples][quick][no-valgrind]") {
     auto   rg = ReportGuard(REPORT);
     size_t n  = 4;
 

diff --git a/tests/test-froidure-pin-bmat8.cpp b/tests/test-froidure-pin-bmat8.cpp
@@ -39,7 +39,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin",
                           "015",
                           "(BMat8) regular boolean mat monoid 4",
-                          "[quick][froidure-pin][bmat8]") {
+                          "[quick][froidure-pin][bmat8][no-valgrind]") {
     auto               rg = ReportGuard(false);
     std::vector<BMat8> gens
         = {BMat8({{0, 1, 0, 0}, {1, 0, 0, 0}, {0, 0, 1, 0}, {0, 0, 0, 1}}),
@@ -253,7 +253,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin",
                           "018",
                           "(BMat8) exception prefix",
-                          "[quick][froidure-pin][bmat8]") {
+                          "[quick][froidure-pin][bmat8][no-valgrind]") {
     std::vector<BMat8> gens
         = {BMat8({{0, 1, 0, 0}, {1, 0, 0, 0}, {0, 0, 1, 0}, {0, 0, 0, 1}}),
            BMat8({{0, 1, 0, 0}, {0, 0, 1, 0}, {0, 0, 0, 1}, {1, 0, 0, 0}}),
@@ -275,12 +275,13 @@ namespace libsemigroups {
            BMat8({{0, 1, 0, 0}, {0, 0, 1, 0}, {0, 0, 0, 1}, {1, 0, 0, 0}}),
            BMat8({{1, 1, 0, 0}, {1, 0, 1, 0}, {0, 1, 1, 1}, {0, 1, 1, 1}})};
     FroidurePin<BMat8> S(gens);
+    // TODO(0) there are no tests in here!
   }
 
   LIBSEMIGROUPS_TEST_CASE("FroidurePin",
                           "020",
                           "(BMat8) exception first_letter",
-                          "[quick][froidure-pin][bmat8]") {
+                          "[quick][froidure-pin][bmat8][no-valgrind]") {
     std::vector<BMat8> gens
         = {BMat8({{0, 1, 0, 0}, {1, 0, 0, 0}, {0, 0, 1, 0}, {0, 0, 0, 1}}),
            BMat8({{0, 1, 0, 0}, {0, 0, 1, 0}, {0, 0, 0, 1}, {1, 0, 0, 0}}),
@@ -302,12 +303,13 @@ namespace libsemigroups {
            BMat8({{0, 1, 0, 0}, {0, 0, 1, 0}, {0, 0, 0, 1}, {1, 0, 0, 0}}),
            BMat8({{1, 1, 0, 0}, {1, 0, 1, 0}, {0, 1, 1, 1}, {0, 1, 1, 1}})};
     FroidurePin<BMat8> S(gens);
+    // TODO(0) there are no tests in here!
   }
 
   LIBSEMIGROUPS_TEST_CASE("FroidurePin",
                           "022",
                           "(BMat8) exception current_length",
-                          "[quick][froidure-pin][bmat8]") {
+                          "[quick][froidure-pin][bmat8][no-valgrind]") {
     std::vector<BMat8> gens
         = {BMat8({{0, 1, 0, 0}, {1, 0, 0, 0}, {0, 0, 1, 0}, {0, 0, 0, 1}}),
            BMat8({{0, 1, 0, 0}, {0, 0, 1, 0}, {0, 0, 0, 1}, {1, 0, 0, 0}}),
@@ -323,7 +325,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin",
                           "023",
                           "(BMat8) exception product_by_reduction",
-                          "[quick][froidure-pin][bmat8]") {
+                          "[quick][froidure-pin][bmat8][no-valgrind]") {
     std::vector<BMat8> gens
         = {BMat8({{0, 1, 0, 0}, {1, 0, 0, 0}, {0, 0, 1, 0}, {0, 0, 0, 1}}),
            BMat8({{1, 1, 0, 0}, {1, 0, 1, 0}, {0, 1, 1, 1}, {0, 1, 1, 1}})};
@@ -345,7 +347,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin",
                           "024",
                           "(BMat8) exception fast_product",
-                          "[quick][froidure-pin][bmat8]") {
+                          "[quick][froidure-pin][bmat8][024]") {
     std::vector<BMat8> gens
         = {BMat8({{0, 1, 0, 0}, {1, 0, 0, 0}, {0, 0, 1, 0}, {0, 0, 0, 1}}),
            BMat8({{1, 1, 0, 0}, {1, 0, 1, 0}, {0, 1, 1, 1}, {0, 1, 1, 1}})};

diff --git a/tests/test-froidure-pin-intmat.cpp b/tests/test-froidure-pin-intmat.cpp
@@ -246,7 +246,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin<IntMat>",
                           "038",
                           "exception: prefix, suffix, first_letter",
-                          "[quick][froidure-pin][element]") {
+                          "[quick][froidure-pin][element][no-valgrind]") {
     test010<IntMat<2>>();
     test010<IntMat<>>();
   }

diff --git a/tests/test-froidure-pin-transf.cpp b/tests/test-froidure-pin-transf.cpp
@@ -998,7 +998,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin<Transf<>>",
                           "098",
                           "sorted_position, sorted_at",
-                          "[quick][froidure-pin][transf]") {
+                          "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -1556,7 +1556,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin<Transf<>>",
                           "109",
                           "copy_closure [partly enumerated]",
-                          "[quick][froidure-pin][transf]") {
+                          "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -1665,7 +1665,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin<Transf<>>",
                           "112",
                           "copy_closure [fully enumerated]",
-                          "[quick][froidure-pin][transf]") {
+                          "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -1772,7 +1772,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin<Transf<>>",
                           "115",
                           "relations",
-                          "[quick][froidure-pin][transf]") {
+                          "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -1811,7 +1811,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin<Transf<>>",
                           "116",
                           "relations [copy_closure, duplicate gens]",
-                          "[quick][froidure-pin][transf]") {
+                          "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -1846,7 +1846,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin<Transf<>>",
                           "117",
                           "relations [copy_add_generators, duplicate gens]",
-                          "[quick][froidure-pin][transf]") {
+                          "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -1881,7 +1881,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin<Transf<>>",
                           "118",
                           "relations [from copy, not enumerated]",
-                          "[quick][froidure-pin][transf]") {
+                          "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -1904,7 +1904,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin<Transf<>>",
                           "119",
                           "relations [from copy, partly enumerated]",
-                          "[quick][froidure-pin][transf]") {
+                          "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -1931,7 +1931,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin<Transf<>>",
                           "120",
                           "relations [from copy, fully enumerated]",
-                          "[quick][froidure-pin][transf]") {
+                          "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -1959,7 +1959,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin<Transf<>>",
                           "121",
                           "relations [from copy_closure, not enumerated]",
-                          "[quick][froidure-pin][transf]") {
+                          "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -1988,7 +1988,7 @@ namespace libsemigroups {
       "FroidurePin<Transf<>>",
       "122",
       "relations [from copy_add_generators, not enumerated]",
-      "[quick][froidure-pin][transf]") {
+      "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -2013,7 +2013,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin<Transf<>>",
                           "123",
                           "relations [from copy_closure, partly enumerated]",
-                          "[quick][froidure-pin][transf]") {
+                          "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -2037,7 +2037,7 @@ namespace libsemigroups {
       "FroidurePin<Transf<>>",
       "124",
       "relations [from copy_add_generators, partly enumerated]",
-      "[quick][froidure-pin][transf]") {
+      "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -2064,7 +2064,7 @@ namespace libsemigroups {
   LIBSEMIGROUPS_TEST_CASE("FroidurePin<Transf<>>",
                           "125",
                           "relations [from copy_closure, fully enumerated]",
-                          "[quick][froidure-pin][transf]") {
+                          "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));
@@ -2090,7 +2090,7 @@ namespace libsemigroups {
       "FroidurePin<Transf<>>",
       "126",
       "relations [from copy_add_generators, fully enumerated]",
-      "[quick][froidure-pin][transf]") {
+      "[quick][froidure-pin][transf][no-valgrind]") {
     auto                  rg = ReportGuard(REPORT);
     FroidurePin<Transf<>> S;
     S.add_generator(Transf<>({0, 1, 2, 3, 4, 5}));

diff --git a/tests/test-gabow.cpp b/tests/test-gabow.cpp
@@ -237,7 +237,10 @@ namespace libsemigroups {
   //   }
   // }
 
-  LIBSEMIGROUPS_TEST_CASE("Gabow", "009", "large cycle", "[quick][gabow]") {
+  LIBSEMIGROUPS_TEST_CASE("Gabow",
+                          "009",
+                          "large cycle",
+                          "[quick][gabow][no-valgrind]") {
     WordGraph<size_t> wg;
     wg.add_to_out_degree(1);
     word_graph::add_cycle(wg, 100000);