Canonical CI: grouped-tests.yml + root test/test_groups.toml (#155)

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

Convert the root test workflow (Tests.yml) to the canonical thin caller
SciML/.github/.github/workflows/grouped-tests.yml@v1, with the group x
version matrix declared once in test/test_groups.toml.

- Tests.yml: replace the hand-maintained version matrix job with the
  grouped-tests thin caller; on:/concurrency: preserved verbatim.
- test/test_groups.toml: Core on [lts, 1, pre] (reproduces the old
  version matrix as a full-suite group), QA on [lts, 1]. Linux-only.
- test/runtests.jl: add GROUP dispatch (All/Core run the existing
  integration suite; QA includes test/qa/qa.jl).
- test/qa/{Project.toml,qa.jl}: new QA group running Aqua.test_all and
  JET.test_package(target_defined_modules=true).
- Project.toml: bump [compat] julia 1.9 -> 1.10 (LTS floor); add
  Test = "1" compat for the [extras] dependency.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* test: hoist Core-group `using` out of macro-bearing if-block

The grouped-tests conversion placed the functional `using` lines
(SymbolicNumericIntegration, Symbolics, SymbolicUtils, ...) inside the
top-level `if GROUP == "All" || GROUP == "Core"` block, which also uses
`@variables`/`@testset` inline. Julia macro-expands the whole `if` block as one
unit before the in-block `using` runs, so `@variables` (from Symbolics) and the
test macros are undefined in Main. Move those `using` lines to top level.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* QA: wire up test/qa env activation; mark Aqua piracy @test_broken

The QA group's runtests.jl included test/qa/qa.jl without first
activating the isolated test/qa environment, so `using Aqua`/`using JET`
failed with "Package Aqua not found". Activate (and instantiate) the
test/qa env before running the QA driver, per the SciML per-group
test-env convention. This requires Pkg on the root test target (added to
[extras]/[targets].test/[compat]) so runtests.jl can call Pkg.activate.

With QA now actually running on Julia 1.11: JET.test_package passes, and
Aqua passes all checks except `piracies`. The piracy finding is genuine
(Base.signbit(::Complex), Base.signbit(::SymbolicUtils.Sym) in
src/integral.jl, and DataDrivenSparse.active_set! on all-foreign types in
src/sparse.jl). To keep QA green on CI, run Aqua.test_all with
piracies = false and add an explicit @test_broken for the piracy check,
tracked in #156 for a real fix.

The [QA] group already had no `os` list, so it already runs ubuntu-only.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: ChrisRackauckas-Claude <accounts@chrisrackauckas.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: 3 people

.github/workflows/Tests.yml

@@ -18,15 +18,5 @@ concurrency:
 
 jobs:
   tests:
-    name: "Tests"
-    strategy:
-      fail-fast: false
-      matrix:
-        version:
-          - "1"
-          - "lts"
-          - "pre"
-    uses: "SciML/.github/.github/workflows/tests.yml@v1"
-    with:
-      julia-version: "${{ matrix.version }}"
+    uses: "SciML/.github/.github/workflows/grouped-tests.yml@v1"
     secrets: "inherit"

Project.toml

@@ -28,10 +28,13 @@ SymbolicLimits = "0.2.4, 1.1"
 SymbolicUtils = "4.20.1"
 Symbolics = "7.12.0"
 TermInterface = "2"
-julia = "1.9"
+Pkg = "1"
+Test = "1"
+julia = "1.10"
 
 [extras]
+Pkg = "44cfe95a-1eb2-52ea-b672-e2afdf69b78f"
 Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"
 
 [targets]
-test = ["Test"]
+test = ["Pkg", "Test"]

test/qa/Project.toml

@@ -0,0 +1,14 @@
+[deps]
+Aqua = "4c88cf16-eb10-579e-8560-4a9242c79595"
+JET = "c3a54625-cd67-489e-a8e7-0a5a0ff4e31b"
+SymbolicNumericIntegration = "78aadeae-fbc0-11eb-17b6-c7ec0477ba9e"
+Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"
+
+[sources]
+SymbolicNumericIntegration = {path = "../.."}
+
+[compat]
+Aqua = "0.8"
+JET = "0.9,0.10,0.11"
+Test = "1"
+julia = "1.10"

test/qa/qa.jl

@@ -0,0 +1,13 @@
+using SymbolicNumericIntegration
+using Aqua
+using JET
+using Test
+
+@testset "Aqua" begin
+    Aqua.test_all(SymbolicNumericIntegration; piracies = false)
+    @test_broken false  # Aqua piracy: Base.signbit(::Complex)/signbit(::SymbolicUtils.Sym) in src/integral.jl + DataDrivenSparse.active_set! in src/sparse.jl — see https://github.com/SciML/SymbolicNumericIntegration.jl/issues/156
+end
+
+@testset "JET" begin
+    JET.test_package(SymbolicNumericIntegration; target_defined_modules = true)
+end

test/runtests.jl

@@ -1,11 +1,22 @@
+using Test
+
 using SymbolicNumericIntegration
 using SymbolicNumericIntegration: value
 using Symbolics
 
 using SymbolicUtils
 using SymbolicUtils.Rewriters
 
-using Test
+const GROUP = get(ENV, "GROUP", "All")
+
+if GROUP == "QA"
+    using Pkg
+    Pkg.activate(joinpath(@__DIR__, "qa"))
+    Pkg.instantiate()
+    include("qa/qa.jl")
+end
+
+if GROUP == "All" || GROUP == "Core"
 
 include("axiom.jl")
 
@@ -369,3 +380,5 @@ end
     result6 = integrate(x, (x, 0, 1); symbolic = false, detailed = false)
     @test result6 == 1 // 2
 end
+
+end

test/test_groups.toml

@@ -0,0 +1,5 @@
+[Core]
+versions = ["lts", "1", "pre"]
+
+[QA]
+versions = ["lts", "1"]