Conversation
JordiManyer
commented
Mar 12, 2026
- Removed TestApp and sysimage compilation completely.
- Split tests into different runners.
- Now supporting newest stable julia release.
Removed TestApp and sysimage compilation completely. Split tests into different runners. Now supporting newest stable julia release.
There was a problem hiding this comment.
Pull request overview
This PR overhauls the CI pipeline by removing the TestApp module and sysimage compilation, splitting tests into categorized runners controlled by a TESTCASE environment variable, and updating Julia version pins to use 'lts' and '1' (latest stable).
Changes:
- Removed
TestAppmodule, itsProject.toml, sysimage compilation scripts, and allTestApp.prefixed references in MPI tests. - Split sequential and MPI tests into named categories (geometry, fespaces, physics, transient/adaptivity, misc) selectable via
TESTCASEenv var, with a CI matrix that runs each as a separate job. - Updated CI to use
julia-actions/julia-runtest@v1, added concurrency control, and modernized Julia version specifiers.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/ci.yml | Consolidated into single matrix job with TESTCASE, removed sysimage steps, updated Julia versions |
| test/runtests.jl | Routes to sequential or MPI tests based on TESTCASE |
| test/sequential/runtests.jl | Groups sequential tests into categories gated by TESTCASE |
| test/mpi/runtests.jl | Removed sysimage support, added OpenMPI --oversubscribe flag |
| test/mpi/runtests_np4.jl | Replaced TestApp with direct includes of test modules |
| test/mpi/runtests_np4_body.jl | Replaced TestApp. prefixes, added TESTCASE gating for MPI test categories |
| test/TestApp/* (5 files) | Deleted TestApp module, compilation scripts, and related config |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #197 +/- ##
===========================================
+ Coverage 0.00% 85.11% +85.11%
===========================================
Files 15 16 +1
Lines 4228 4254 +26
===========================================
+ Hits 0 3621 +3621
+ Misses 4228 633 -3595
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|