Canonical CI: grouped-tests.yml + root test/test_groups.toml

[ChrisRackauckas-Claude]

Summary

Converts the root test workflow (.github/workflows/Test.yml, name: "Tests") from a hand-maintained version × group matrix into the canonical grouped-tests.yml@v1 thin caller, with the matrix declared once in a root test/test_groups.toml.

What changed

• .github/workflows/Test.yml — the matrix test job (formerly tests.yml@v1 with a 3-version × 5-group matrix) becomes the thin caller:

  jobs:
    tests:
      uses: "SciML/.github/.github/workflows/grouped-tests.yml@v1"
      with:
        coverage: false
      secrets: "inherit"

  Filename, name: "Tests", on: triggers, and concurrency: are preserved verbatim. coverage: false is kept (the old job disabled coverage). group-env-name is omitted because runtests.jl reads the canonical GROUP var; check-bounds, coverage-directories, apt-packages are all left at their defaults (the old job used the defaults). Linux-only: no os field (default ubuntu-latest).
• test/test_groups.toml (new) — [Core] on ["lts","1","pre"], [QA] on ["lts","1"].
• Canonical Core/QA refactor — this repo had no Aqua/JET; its only QA-style check was the ExplicitImports import-hygiene test running inline in the Misc group. That check is moved into an isolated QA group:
  • test/qa/Project.toml — [deps] = Catalyst (via [sources] path = "../..") + ExplicitImports + Test; [compat] julia = "1.10", ExplicitImports = "1.15".
  • test/qa/qa.jl — the moved ExplicitImports checks.
  • runtests.jl — when GROUP == "QA", activate test/qa, Pkg.develop the repo root, instantiate, and include("qa/qa.jl"). The functional suite (formerly Modeling/Simulation/Hybrid/Misc/Spatial) now runs under Core (and the local default All); the fine-grained group names still work for local runs.
• Project.toml — ExplicitImports dropped from [extras], [targets].test, and [compat] (it now lives only in the QA env). julia compat floor is already "1.10" (LTS), unchanged.

Extension tests remain owned by the separate TestExtensions.yml workflow (untouched), as before; the root matrix does not run the Extensions group. All other workflow files are untouched.

Matrix match

compute_affected_sublibraries.jl . --root-matrix (from SciML/.github@v1) emits exactly the intended canonical matrix:

group	versions	runner
Core	lts, 1, pre	ubuntu-latest
QA	lts, 1	ubuntu-latest

= 5 cells, all ubuntu-latest. This is a deliberate consolidation of the old 15-cell matrix (5 functional groups × 3 versions): the 5 functional groups collapse into a single Core group (running the union of all functional tests) on the same 3 versions, and the inline ExplicitImports check becomes a dedicated QA group on [lts, 1]. Same test coverage, canonical Core/QA shape. Emitted matrix verified to match the authored test_groups.toml exactly (5/5).

QA validation

Catalyst is a heavy package (ModelingToolkitBase / Symbolics / SymbolicUtils precompile, > ~20 min cold), so the QA group was not run locally — CI validates it. The QA structure was statically verified: test_groups.toml, test/qa/Project.toml, and Project.toml all parse as TOML; Test.yml parses as YAML; runtests.jl and qa/qa.jl parse as Julia; the [sources] path ../.. resolves to the Catalyst Project.toml. No tests were skipped, broken, or silenced.

Ignore until reviewed by @ChrisRackauckas.

🤖 Generated with Claude Code

[isaacsas]

Why is this eliminating the current group structure? That was intentional as otherwise CI times become gigantic.

[ChrisRackauckas]

Nothing is eliminated.

[ChrisRackauckas]

The matrix is exactly the same.

[isaacsas]

The PR says it explicitly now runs all tests in one group except for QA though, and most of the prior groups are not present in the group CI file.

[ChrisRackauckas]

Okay that's an error. They aren't supposed to be in the CI file, but they aren't reflected in the test_groups. I'll get that fixed.

[ChrisRackauckas]

This sis good now.