Skip to content

Commit

Permalink
test: add JuliaSimCompiler only to run its integration tests
Browse files Browse the repository at this point in the history
Remove `JuliaSimCompiler` as a test dependency as that would require
having access to the package at all times to run the tests, making it
synonymous with a direct dependency (and less of an extension) in the
context of running tests.
Instead, explicitly `Pkg.add` `JuliaSimCompiler` instead in the
relevant test matrices.
  • Loading branch information
thazhemadam committed Sep 20, 2023
1 parent b64af40 commit d88bf45
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Project.toml
Original file line number Diff line number Diff line change
Expand Up @@ -48,11 +48,11 @@ TermInterface = "0.2, 0.3"
julia = "1.6"

[extras]
JuliaSimCompiler = "8391cb6b-4921-5777-4e45-fd9aab8cb88d"
NonlinearSolve = "8913a72c-1f9b-4ce2-8d82-65094dcecaec"
Pkg = "44cfe95a-1eb2-52ea-b672-e2afdf69b78f"
SafeTestsets = "1bc83da4-3b8d-516f-aca4-4fe02f6d838f"
StableRNGs = "860ef19b-820b-49d6-a774-d7a799459cd3"
Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"

[targets]
test = ["Test", "NonlinearSolve", "SafeTestsets", "StableRNGs", "JuliaSimCompiler"]
test = ["Test", "NonlinearSolve", "SafeTestsets", "StableRNGs", "Pkg"]
3 changes: 3 additions & 0 deletions test/runtests.jl
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
using SafeTestsets
using Pkg

const GROUP = get(ENV, "GROUP", "All")
const is_APPVEYOR = Sys.iswindows() && haskey(ENV, "APPVEYOR")
Expand Down Expand Up @@ -129,11 +130,13 @@ const is_TRAVIS = haskey(ENV, "TRAVIS")
end
############### JSC #####################
if GROUP == "All" || GROUP == "MOL_Interface2_JSC"
Pkg.add("JuliaSimCompiler")
@time @safetestset "MOLFiniteDifference Interface" begin
include("pde_systems/MOLtest2_JSC.jl")
end
end
if GROUP == "All" || GROUP == "MOL_Interface1_JSC"
Pkg.add("JuliaSimCompiler")
@time @safetestset "MOLFiniteDifference Interface" begin
include("pde_systems/MOLtest1_JSC.jl")
end
Expand Down

0 comments on commit d88bf45

Please sign in to comment.