Skip to content

CI: Pin pytest<9.2 - #3501

Merged
dweindl merged 1 commit into
AMICI-dev:mainfrom
dweindl:pin_pytest_xdist_loadgroup
Oct 29, 2026
Merged

dweindl merged 1 commit into
AMICI-dev:mainfrom
dweindl:pin_pytest_xdist_loadgroup

Conversation

@dweindl

@dweindl dweindl commented Oct 29, 2026

Copy link
Copy Markdown
Member

Stopgap for the SBML workflow, which has failed on main since pytest 9.2.0 was released (e.g., this run).

With pytest 9.2, the xdist_group marks are ignored with --dist=loadgroup of pytest-xdist<=3.8.0 (pytest-dev/pytest-xdist#1386, fixed but not yet released). Then, the tests of one SBML semantic test suite case run on different workers, which compile the same model into the same directory concurrently, failing with, e.g., "No such file or directory" for generated sources. The pin can be removed once a fixed pytest-xdist is released.

🤖 Generated with Claude Code

With pytest 9.2, the `xdist_group` marks are ignored with
`--dist=loadgroup` of pytest-xdist<=3.8.0
(pytest-dev/pytest-xdist#1386). Then, the
tests of one SBML semantic test suite case run on different workers,
which compile the same model into the same directory concurrently, so
that the SBML workflow fails.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedpypi/​pytest@​9.2.0 ⏵ 9.1.187100100100100

View full report

@dweindl
dweindl marked this pull request as ready for review October 29, 2026 07:09
@dweindl
dweindl requested a review from a team as a code owner October 29, 2026 07:09
@dweindl
dweindl enabled auto-merge October 29, 2026 07:09
@codecov

codecov Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.26%. Comparing base (153fe89) to head (f574ee4).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3501   +/-   ##
=======================================
  Coverage   80.26%   80.26%           
=======================================
  Files         331      331           
  Lines       23420    23420           
  Branches     1631     1631           
=======================================
  Hits        18797    18797           
  Misses       4617     4617           
  Partials        6        6           
Flag Coverage Δ
cpp 74.85% <ø> (ø)
cpp_python 34.65% <ø> (ø)
petab 46.64% <ø> (ø)
petab_sciml 16.61% <ø> (ø)
petab_sciml_benchmarks 15.20% <ø> (ø)
python 72.58% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@dweindl
dweindl added this pull request to the merge queue Oct 29, 2026
Merged via the queue into AMICI-dev:main with commit 71c6704 Oct 29, 2026
40 checks passed
@dweindl
dweindl deleted the pin_pytest_xdist_loadgroup branch October 29, 2026 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants