Skip to content

Use SciMLTesting v1.2 (folder-based run_tests)#115

Merged
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:sciml-testing-rollout
Jun 14, 2026
Merged

Use SciMLTesting v1.2 (folder-based run_tests)#115
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:sciml-testing-rollout

Conversation

@ChrisRackauckas-Claude

@ChrisRackauckas-Claude ChrisRackauckas-Claude commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Converts EllipsisNotation.jl to the SciMLTesting v1.2 folder-discovery model.

test/runtests.jl is now just:

using SciMLTesting
run_tests()

Layout:

  • Core (main test env): top-level test/*.jlbasic.jl, more_generic.jl, cartesian.jl, alloc_tests.jl.
  • QA (sub-env test/qa/): qa.jl and explicit_imports.jl moved into test/qa/ to join the existing jet.jl, so the Core top-level glob does not erroneously pick them up. The test/qa/Project.toml sub-env (Aqua/ExplicitImports/JET) activation is now owned by the harness.

test/test_groups.toml is unchanged. Deps: added SciMLTesting (compat 1) to the root [extras]/[targets].test/[compat] and to test/qa/Project.toml (plus SafeTestsets there); dropped Pkg from the root test deps (no test/*.jl uses it — the v1.2 harness owns all Pkg ops).

Verified end-to-end on Julia 1.11 with SciMLTesting v1.2.0: GROUP=Core (alloc 4/4, basic 27/27, cartesian 6/6, more_generic 5/5) and GROUP=QA (explicit_imports 2/2, jet 14/14, qa 10/10) both pass. The per-GROUP test set matches the previous dispatch (CI runs Core and QA as separate lanes via the centralized grouped-tests workflow).

Ignore until reviewed by @ChrisRackauckas.

Replace the hand-written GROUP dispatch in test/runtests.jl with the
folder-discovery `run_tests()` from SciMLTesting v1.2. Core stays at the
top level (basic, more_generic, cartesian, alloc_tests); QA moves into
test/qa/ (qa.jl + explicit_imports.jl joined the existing jet.jl) so the
top-level Core glob does not pick them up. The qa sub-env activation is
now owned by the harness.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ChrisRackauckas-Claude ChrisRackauckas-Claude changed the title Use SciMLTesting v1.0.0 (run_tests harness) Use SciMLTesting v1.2 (folder-based run_tests) Jun 14, 2026
@ChrisRackauckas ChrisRackauckas marked this pull request as ready for review June 14, 2026 23:54
@ChrisRackauckas ChrisRackauckas merged commit 5e925d0 into SciML:master Jun 14, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants