Use MOI.Test.runtests (#20)

* Use MOI.Test.runtests

* Stop checking out MOI

* Revert "Stop checking out MOI"

This reverts commit 1f7c5f1.

Author: blegat

Project.toml

@@ -14,6 +14,6 @@ Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"
 FillArrays = "1.13.0"
 KrylovKit = "0.9.5"
 LinearAlgebra = "1.10"
-MathOptInterface = "1.34.0"
+MathOptInterface = "1.39.0"
 Test = "1.10"
 julia = "1.10"

src/Test/Test.jl

@@ -196,67 +196,4 @@ function MOI.Test.setup_test(
     return
 end
 
-function runtests(
-    model::MOI.ModelLike,
-    config::MOI.Test.Config;
-    include::Vector = String[],
-    exclude::Vector = String[],
-    warn_unsupported::Bool = false,
-    verbose::Bool = false,
-    exclude_tests_after::VersionNumber = v"999.0.0",
-)
-    tests = filter(names(@__MODULE__; all = true)) do name
-        return startswith("$name", "test_")
-    end
-    tests = string.(tests)
-    for ex in exclude
-        if ex in tests && any(t -> ex != t && occursin(ex, t), tests)
-            @warn(
-                "The exclude string \"$ex\" is ambiguous because it exactly " *
-                "matches a test, but it also partially matches another. Use " *
-                "`r\"^$ex\$\"` to exclude the exactly matching test, or " *
-                "`r\"$ex.*\"` to exclude all partially matching tests.",
-            )
-        end
-    end
-    for name_sym in names(@__MODULE__; all = true)
-        name = string(name_sym)
-        if !startswith(name, "test_")
-            continue  # All test functions start with test_
-        elseif !isempty(include) && !any(s -> occursin(s, name), include)
-            continue
-        elseif !isempty(exclude) && any(s -> occursin(s, name), exclude)
-            continue
-        end
-        if verbose
-            @info "Running $name"
-        end
-        test_function = getfield(@__MODULE__, name_sym)
-        if MOI.Test.version_added(test_function) > exclude_tests_after
-            if verbose
-                println("  Skipping test because of `exclude_tests_after`")
-            end
-            continue
-        end
-        @testset "$(name)" begin
-            c = copy(config)
-            tear_down = MOI.Test.setup_test(test_function, model, c)
-            # Make sure to empty the model before every test.
-            MOI.empty!(model)
-            try
-                test_function(model, c)
-            catch err
-                if verbose
-                    println("  Test errored with $(typeof(err))")
-                end
-                MOI.Test._error_handler(err, name, warn_unsupported)
-            end
-            if tear_down !== nothing
-                tear_down()
-            end
-        end
-    end
-    return
-end
-
 end

test/Test/runtests.jl

@@ -23,13 +23,14 @@ end
 function test_runtests()
     # Some tests are excluded because UniversalFallback accepts absolutely
     # everything.
-    LRO.Test.runtests(
+    MOI.Test.runtests(
         MOI.Utilities.MockOptimizer(
             MOI.Utilities.UniversalFallback(MOI.Utilities.Model{Float64}()),
         ),
         MOI.Test.Config(),
         warn_unsupported = true,
         verbose = true,
+        test_module = LRO.Test,
     )
     return
 end