Skip to content

Commit c0c8628

Browse files
authored
Merge pull request #435 from ChrisRackauckas-Claude/fix-skip-wrapper-tests-in-all
Skip wrapper tests in GROUP=all to fix NonlinearSolve CI
2 parents 62255e0 + 9ecd2e8 commit c0c8628

2 files changed

Lines changed: 23 additions & 9 deletions

File tree

test/runtests.jl

Lines changed: 19 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,22 @@ const RETESTITEMS_NWORKER_THREADS = parse(
2121

2222
@info "Running tests for group: $(GROUP) with $(RETESTITEMS_NWORKERS) workers"
2323

24-
ReTestItems.runtests(
25-
BoundaryValueDiffEq;
26-
tags = (GROUP == "all" ? nothing : [Symbol(GROUP)]),
27-
nworkers = RETESTITEMS_NWORKERS, nworker_threads = RETESTITEMS_NWORKER_THREADS
28-
)
24+
if GROUP == "wrappers"
25+
# Wrapper tests must be explicitly requested via GROUP=wrappers
26+
ReTestItems.runtests(
27+
joinpath(@__DIR__, "wrappers");
28+
nworkers = RETESTITEMS_NWORKERS, nworker_threads = RETESTITEMS_NWORKER_THREADS
29+
)
30+
else
31+
# For "all" and other groups, exclude wrappers (they have external deps like ODEInterface)
32+
for dir in readdir(@__DIR__)
33+
dirpath = joinpath(@__DIR__, dir)
34+
isdir(dirpath) || continue
35+
dir in ("wrappers", "qa") && continue
36+
ReTestItems.runtests(
37+
dirpath;
38+
tags = (GROUP == "all" ? nothing : [Symbol(GROUP)]),
39+
nworkers = RETESTITEMS_NWORKERS, nworker_threads = RETESTITEMS_NWORKER_THREADS
40+
)
41+
end
42+
end

test/wrappers/odeinterface_tests.jl

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ export ex7_f!, ex7_2pbc1!, ex7_2pbc2!, u0, p, tspan
2929

3030
end
3131

32-
@testitem "BVPM2" setup = [ODEInterfaceWrapperTestSetup] skip = true begin
32+
@testitem "BVPM2" setup = [ODEInterfaceWrapperTestSetup] tags = [:wrappers] skip = true begin
3333
using ODEInterface, RecursiveArrayTools, LinearAlgebra
3434

3535
tpprob = TwoPointBVProblem(
@@ -46,7 +46,7 @@ end
4646
end
4747

4848
# Just test that it runs. BVPSOL only works with linearly separable BCs.
49-
@testitem "BVPSOL" setup = [ODEInterfaceWrapperTestSetup] begin
49+
@testitem "BVPSOL" setup = [ODEInterfaceWrapperTestSetup] tags = [:wrappers] begin
5050
using ODEInterface, OrdinaryDiffEqTsit5, RecursiveArrayTools, NonlinearSolveFirstOrder
5151

5252
tpprob = TwoPointBVProblem(
@@ -100,7 +100,7 @@ end
100100
@test sol_bvpsol isa SciMLBase.ODESolution
101101
end
102102

103-
@testitem "COLNEW" setup = [ODEInterfaceWrapperTestSetup] begin
103+
@testitem "COLNEW" setup = [ODEInterfaceWrapperTestSetup] tags = [:wrappers] begin
104104
using ODEInterface, RecursiveArrayTools
105105

106106
function f!(du, u, p, t)
@@ -122,7 +122,7 @@ end
122122
@test SciMLBase.successful_retcode(sol_colnew)
123123
end
124124

125-
@testitem "COLNEW for multi-points BVP" setup = [ODEInterfaceWrapperTestSetup] begin
125+
@testitem "COLNEW for multi-points BVP" setup = [ODEInterfaceWrapperTestSetup] tags = [:wrappers] begin
126126
using ODEInterface, RecursiveArrayTools
127127

128128
function f!(du, u, p, t)

0 commit comments

Comments
 (0)