Skip to content

Move the doctests into their own test group - #340

Merged
michakraus merged 1 commit into
mainfrom
tests/doctests-group
Oct 9, 2026
Merged

michakraus merged 1 commit into
mainfrom
tests/doctests-group

Conversation

@michakraus

Copy link
Copy Markdown
Member

What

test/quality/doctests.jl moves from the group slow into its own group doctests. An empty ARGS runs core and slow, so Pkg.test() and every job of the CI test matrix no longer run the doctests. Pkg.test(test_args = ["doctests"]) runs them, and the CI Doctests job stays their runner.

Why

The test convention now has a sixth group, doctests. The doctests belong on one pinned job, as the comment in CI.yml says; the matrix ran them on every Julia version and every system as well.

Checks

  • The layout check of the test convention: clean.
  • The group doctests passes on Julia 1.13.1: the test runner reported a pass in 28 s, with the Test Summary "selected | Pass 1 Total 1 16.4s".
  • The changed test/runtests.jl is formatted, and NFC.
  • CHANGELOG.md: one added line under [Unreleased].

Open pull requests that also change test/runtests.jl: #338 Add LatentVectorFieldLoss: the reduced vector field against the pushed-forward full one, #337 Add ReducedHamiltonianLoss: fit a decoder to states and reduced-Hamiltonian gradients, #336 Add StandardAutoencoder, an autoencoder of Dense layers, #329 Add PODArch for proper orthogonal decomposition, #321 Check the Poincaré integral invariant of a decoded loop, #310 Train through GeometricOptimizers' TrainingOptimizer (G4), one method per leaf, #207 Parametric Generalized Hamiltonian Neural Networks. The later one to merge rebases.

🤖 Generated with Claude Code

Pkg.test() no longer runs test/quality/doctests.jl. The file is the
group doctests, which an empty ARGS does not run; the CI Doctests job
stays the runner of the doctests.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@codecov

codecov Bot commented Oct 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.64%. Comparing base (d7c8960) to head (22d67e9).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #340   +/-   ##
=======================================
  Coverage   86.64%   86.64%           
=======================================
  Files          76       76           
  Lines        2786     2786           
=======================================
  Hits         2414     2414           
  Misses        372      372           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@michakraus
michakraus merged commit 7c7778e into main Oct 9, 2026
15 of 16 checks passed
@michakraus
michakraus deleted the tests/doctests-group branch October 9, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant