Route sublibrary groups from the root test harness - #4998
Route sublibrary groups from the root test harness#4998ChrisRackauckas-Claude wants to merge 1 commit into
Conversation
|
The current CI failures are outside this PR's one-line QA-routing change:
The routing change itself passes Runic and the completed non-Catalyst downstream/downgrade lanes. I am keeping typo cleanup separate so this PR remains focused. |
|
Baseline CI audit for the unrelated Spell Check and docs
Neither failure is caused by this PR's focused diff. |
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
fe63358 to
626690c
Compare
|
Rebased onto current master and pushed the updated draft branch. Audit remains scoped to test-group routing: the root |
Ignore this PR until it has been reviewed by @ChrisRackauckas.
What changed and why
Pass the repository's
libdirectory to SciMLTesting's rootrun_testscall. This enables the existing monorepo router to resolve prefixed groups such asModelingToolkitBase_Extensionsand run the requested sublibrary tests.This is a one-line test-harness correction. It changes no package source, public API, dependency, or version.
Root cause
The harness conversion in d647d61 from #4610 adopted SciMLTesting but omitted
lib_dirfrom the root call.Its parent accepts the prefixed command but silently performs zero tests:
Current master exposes the omission as a hard routing error instead. The existing report is #4997.
Failing before
On clean current master
38d91d81dcd77319a70abb93882ba6da13510600:GROUP=ModelingToolkitBase_Extensions julia +1.12 --project=. -e 'using Pkg; Pkg.test()'exits before any owning test runs:
Passing after
The identical root command now dispatches to the actual sublibrary suite and exits 0:
The root QA group also passes:
Whole-repository Runic, spelling over the diff, and
git diff --checkpass.Not verified
Other prefixed sublibrary groups were not each run locally; this change supplies the generic
lib_dirconsumed by the shared router. Docs were not run because only the test harness changed. Julia LTS/prerelease were not repeated locally.