Skip to content

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

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

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

Conversation

@ChrisRackauckas-Claude

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

Copy link
Copy Markdown
Contributor

Converts the test suite to the SciMLTesting v1.2 run_tests harness (folder-discovery model).

  • test/runtests.jl becomes using SciMLTesting; run_tests().
  • The Core testsets are split into self-contained top-level files (fast_log2_tests.jl, fast_pow_tests.jl, other_ad_engines_tests.jl), and the Enzyme group moves to test/Enzyme/ (enzyme_forward_tests.jl, enzyme_reverse_tests.jl), run in the main test env — matching the pre-conversion dispatch.
  • QA stays in test/qa/ with its sub-env. test/test_groups.toml is unchanged; GROUP=All still runs Core + Enzyme (QA excluded), exactly as the old dispatcher.
  • Adds SciMLTesting + SafeTestsets to the root and test/qa test deps; drops Pkg (the harness owns all Pkg operations).

Validated locally on Julia 1.11: GROUP=Core (1200 + 5 + 4 tests) and GROUP=QA (11 pass + 1 expected @test_broken) both pass.

Ignore until reviewed by @ChrisRackauckas.

🤖 Generated with Claude Code

Replace the hand-written GROUP dispatcher in test/runtests.jl with the
SciMLTesting v1.2 folder-discovery run_tests harness. The Core testsets are
split into self-contained top-level files (fast_log2_tests.jl, fast_pow_tests.jl,
other_ad_engines_tests.jl) and the Enzyme group moves to test/Enzyme/
(enzyme_forward_tests.jl, enzyme_reverse_tests.jl), run in the main test env to
match the pre-conversion behavior. QA stays in test/qa/ with its env. Adds
SciMLTesting + SafeTestsets to the root and qa test deps and drops Pkg (the
harness owns Pkg ops). test_groups.toml is unchanged; GROUP=All still runs
Core + Enzyme (QA excluded), matching the old dispatcher.

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 bfd80f5 into SciML:main Jun 14, 2026
17 of 21 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