Simplify run_tests in each test file (#202)

odow · web-flow · commit a824296901b3 · 2026-06-30T16:59:45.000+12:00
diff --git a/test/algorithms/test_Chalmet.jl b/test/algorithms/test_Chalmet.jl
@@ -14,12 +14,9 @@ import MultiObjectiveAlgorithms: MOI
 include(joinpath(dirname(@__DIR__), "mock_optimizer.jl"))
 
 function run_tests()
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                getfield(@__MODULE__, name)()
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        getfield(@__MODULE__, name)()
     end
     return
 end
diff --git a/test/algorithms/test_Dichotomy.jl b/test/algorithms/test_Dichotomy.jl
@@ -15,12 +15,9 @@ import MultiObjectiveAlgorithms: MOI
 include(joinpath(dirname(@__DIR__), "mock_optimizer.jl"))
 
 function run_tests()
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                getfield(@__MODULE__, name)()
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        getfield(@__MODULE__, name)()
     end
     return
 end
diff --git a/test/algorithms/test_DominguezRios.jl b/test/algorithms/test_DominguezRios.jl
@@ -18,12 +18,9 @@ function run_tests()
     if Sys.WORD_SIZE == 32
         return  # Skip on 32-bit because HiGHS fails
     end
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                getfield(@__MODULE__, name)()
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        getfield(@__MODULE__, name)()
     end
     return
 end
diff --git a/test/algorithms/test_EpsilonConstraint.jl b/test/algorithms/test_EpsilonConstraint.jl
@@ -16,12 +16,9 @@ include(joinpath(dirname(@__DIR__), "mock_optimizer.jl"))
 include(joinpath(dirname(@__DIR__), "vOptLib.jl"))
 
 function run_tests()
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                getfield(@__MODULE__, name)()
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        getfield(@__MODULE__, name)()
     end
     return
 end
diff --git a/test/algorithms/test_Hierarchical.jl b/test/algorithms/test_Hierarchical.jl
@@ -12,12 +12,9 @@ import MultiObjectiveAlgorithms as MOA
 import MultiObjectiveAlgorithms: MOI
 
 function run_tests()
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                getfield(@__MODULE__, name)()
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        getfield(@__MODULE__, name)()
     end
     return
 end
diff --git a/test/algorithms/test_KirlikSayin.jl b/test/algorithms/test_KirlikSayin.jl
@@ -16,12 +16,9 @@ include(joinpath(dirname(@__DIR__), "problems.jl"))
 include(joinpath(dirname(@__DIR__), "vOptLib.jl"))
 
 function run_tests()
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                getfield(@__MODULE__, name)()
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        getfield(@__MODULE__, name)()
     end
     return
 end
diff --git a/test/algorithms/test_Lexicographic.jl b/test/algorithms/test_Lexicographic.jl
@@ -12,12 +12,9 @@ import MultiObjectiveAlgorithms as MOA
 import MultiObjectiveAlgorithms: MOI
 
 function run_tests()
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                getfield(@__MODULE__, name)()
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        getfield(@__MODULE__, name)()
     end
     return
 end
diff --git a/test/algorithms/test_RandomWeighting.jl b/test/algorithms/test_RandomWeighting.jl
@@ -14,12 +14,9 @@ import MultiObjectiveAlgorithms: MOI
 include(joinpath(dirname(@__DIR__), "mock_optimizer.jl"))
 
 function run_tests()
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                getfield(@__MODULE__, name)()
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        getfield(@__MODULE__, name)()
     end
     return
 end
diff --git a/test/algorithms/test_Sandwiching.jl b/test/algorithms/test_Sandwiching.jl
@@ -15,12 +15,9 @@ import Polyhedra
 include(joinpath(dirname(@__DIR__), "mock_optimizer.jl"))
 
 function run_tests()
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                getfield(@__MODULE__, name)()
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        getfield(@__MODULE__, name)()
     end
     return
 end
diff --git a/test/algorithms/test_TambyVanderpooten.jl b/test/algorithms/test_TambyVanderpooten.jl
@@ -16,12 +16,9 @@ include(joinpath(dirname(@__DIR__), "problems.jl"))
 include(joinpath(dirname(@__DIR__), "vOptLib.jl"))
 
 function run_tests()
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                getfield(@__MODULE__, name)()
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        getfield(@__MODULE__, name)()
     end
     return
 end
diff --git a/test/problems.jl b/test/problems.jl
@@ -6,17 +6,15 @@
 module Problems
 
 using Test
+
 import MathOptInterface as MOI
 import MultiObjectiveAlgorithms as MOA
 
 function run_tests(model::MOI.ModelLike)
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                MOI.empty!(model)
-                getfield(@__MODULE__, name)(model)
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        MOI.empty!(model)
+        getfield(@__MODULE__, name)(model)
     end
     return
 end
diff --git a/test/test_model.jl b/test/test_model.jl
@@ -12,12 +12,9 @@ import MultiObjectiveAlgorithms as MOA
 import MultiObjectiveAlgorithms: MOI
 
 function run_tests()
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                getfield(@__MODULE__, name)()
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        getfield(@__MODULE__, name)()
     end
     return
 end
diff --git a/test/test_utilities.jl b/test/test_utilities.jl
@@ -11,12 +11,9 @@ import MultiObjectiveAlgorithms as MOA
 import MultiObjectiveAlgorithms: MOI
 
 function run_tests()
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                getfield(@__MODULE__, name)()
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        getfield(@__MODULE__, name)()
     end
     return
 end
diff --git a/test/vOptLib.jl b/test/vOptLib.jl
@@ -6,18 +6,16 @@
 module vOptLib
 
 using Test
+
 import JSON
 import MathOptInterface as MOI
 import MultiObjectiveAlgorithms as MOA
 
 function run_tests(model::MOI.ModelLike; kwargs...)
-    for name in names(@__MODULE__; all = true)
-        if startswith("$name", "test_")
-            @testset "$name" begin
-                MOI.empty!(model)
-                getfield(@__MODULE__, name)(model; kwargs...)
-            end
-        end
+    is_test(name) = startswith("$name", "test_")
+    @testset "$name" for name in filter(is_test, names(@__MODULE__; all = true))
+        MOI.empty!(model)
+        getfield(@__MODULE__, name)(model; kwargs...)
     end
     return
 end
