Skip to content

Try general python test workflow#3398

Open
jeromekelleher wants to merge 1 commit intotskit-dev:mainfrom
jeromekelleher:try-general-python
Open

Try general python test workflow#3398
jeromekelleher wants to merge 1 commit intotskit-dev:mainfrom
jeromekelleher:try-general-python

Conversation

@jeromekelleher
Copy link
Member

@jeromekelleher jeromekelleher commented Feb 24, 2026

This refactors the main tests computing coverage to use the general workflow. Turns out a lot of the complexity isn't needed any more with the uv setup (yay for uv!).

The odds and ends that don't fit in elsewhere are included as bespoke python tests.

@codecov
Copy link

codecov bot commented Feb 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.10%. Comparing base (e470639) to head (6285e13).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3398      +/-   ##
==========================================
+ Coverage   93.73%   94.10%   +0.37%     
==========================================
  Files          48       48              
  Lines       51206    51206              
  Branches     5716     5716              
==========================================
+ Hits        47997    48187     +190     
+ Misses       2782     2593     -189     
+ Partials      427      426       -1     
Flag Coverage Δ
C 91.33% <ø> (ø)
c-python 76.94% <ø> (ø)
python-tests 96.56% <ø> (ø)
python-tests-no-jit 33.42% <ø> (?)

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

Components Coverage Δ
Python API 98.88% <ø> (ø)
Python C interface 91.20% <ø> (ø)
C library 93.56% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

And create local bespoke tests for corner cases
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.

1 participant