From c8c95123afd21b3a489ad007be409105ec2d81f5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Beno=C3=AEt=20Legat?= <benoit.legat@gmail.com>
Date: Fri, 4 Apr 2025 11:30:29 +0200
Subject: [PATCH 1/2] Allow using Test.runtests in other packages

---
 src/Test/Test.jl | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/src/Test/Test.jl b/src/Test/Test.jl
index 06c7116e65..b45c4e6e40 100644
--- a/src/Test/Test.jl
+++ b/src/Test/Test.jl
@@ -173,6 +173,7 @@ version_added(::F) where {F} = v"0.10.5"  # The default for any unlabeled tests.
         warn_unsupported::Bool = false,
         exclude_tests_after::VersionNumber = v"999.0.0",
         verbose::Bool = false,
+        test_module = @__MODULE__,
     )
 
 Run all tests in `MathOptInterface.Test` on `model`.
@@ -201,6 +202,8 @@ Run all tests in `MathOptInterface.Test` on `model`.
    is released with a new test.
  * `verbose` is a `Bool` that controls whether the name of the test is printed
    before executing it. This can be helpful when debugging.
+ * `test_module` is a `Module` where all the functions starting with `test_`
+   are considered as tests.
 
 See also: [`setup_test`](@ref).
 
@@ -227,13 +230,14 @@ function runtests(
     warn_unsupported::Bool = false,
     verbose::Bool = false,
     exclude_tests_after::VersionNumber = v"999.0.0",
+    test_module = @__MODULE__,
 )
-    tests = filter(names(@__MODULE__; all = true)) do name
+    tests = filter(names(test_module; all = true)) do name
         return startswith("$name", "test_")
     end
-    tests = string.(tests)
+    test_names = string.(tests)
     for ex in exclude
-        if ex in tests && any(t -> ex != t && occursin(ex, t), tests)
+        if ex in test_names && any(t -> ex != t && occursin(ex, t), test_names)
             @warn(
                 "The exclude string \"$ex\" is ambiguous because it exactly " *
                 "matches a test, but it also partially matches another. Use " *
@@ -242,11 +246,9 @@ function runtests(
             )
         end
     end
-    for name_sym in names(@__MODULE__; all = true)
+    for name_sym in tests
         name = string(name_sym)
-        if !startswith(name, "test_")
-            continue  # All test functions start with test_
-        elseif !isempty(include) && !any(s -> occursin(s, name), include)
+        if !isempty(include) && !any(s -> occursin(s, name), include)
             continue
         elseif !isempty(exclude) && any(s -> occursin(s, name), exclude)
             continue
@@ -254,7 +256,7 @@ function runtests(
         if verbose
             @info "Running $name"
         end
-        test_function = getfield(@__MODULE__, name_sym)
+        test_function = getfield(test_module, name_sym)
         if version_added(test_function) > exclude_tests_after
             if verbose
                 println("  Skipping test because of `exclude_tests_after`")

From abc51be892535ba6fd79ddd0d138d926eccbb6e0 Mon Sep 17 00:00:00 2001
From: Oscar Dowson <odow@users.noreply.github.com>
Date: Tue, 8 Apr 2025 15:58:08 +1200
Subject: [PATCH 2/2] Update Test.jl

---
 src/Test/Test.jl | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Test/Test.jl b/src/Test/Test.jl
index b45c4e6e40..fdfcacd6e0 100644
--- a/src/Test/Test.jl
+++ b/src/Test/Test.jl
@@ -173,10 +173,10 @@ version_added(::F) where {F} = v"0.10.5"  # The default for any unlabeled tests.
         warn_unsupported::Bool = false,
         exclude_tests_after::VersionNumber = v"999.0.0",
         verbose::Bool = false,
-        test_module = @__MODULE__,
+        test_module = MathOptInterface.Test,
     )
 
-Run all tests in `MathOptInterface.Test` on `model`.
+Run all tests in `test_module`, which defaults to `MathOptInterface.Test`, on `model`.
 
 ## Configuration arguments