Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve coverage collection #32994

Closed
tobiasdiez opened this issue Dec 8, 2021 · 2 comments
Closed

Improve coverage collection #32994

tobiasdiez opened this issue Dec 8, 2021 · 2 comments

Comments

@tobiasdiez
Copy link
Contributor

The current coverage collection is relatively basic as one only keeps track whether a function has a (doc)test or not. On the other hand, there are more advanced tools like coverage.py that give more detailed information about which lines and branches are actually executed.
These reports can uploaded e.g. to codecov to get a nice visual representation of the coverage and track changes in coverage (in particular for PRs).

In this ticket, we integrate coverage.py in the sage workflow. In particular,

  1. Add configuration for coverage.py.
  2. Add tox target to run it.
  3. Add github workflow that uploads coverage to codecov. (maybe as a follow-up ticket)

This is part of #28936 and will improve integration of pytest with the coverage collection.

Depends on #33355

CC: @mkoeppe @tscrim @mjungmath @egourgoulhon

Component: doctest coverage

Issue created by migration from https://trac.sagemath.org/ticket/32994

@tobiasdiez tobiasdiez added this to the sage-9.6 milestone Dec 8, 2021
@mkoeppe
Copy link
Contributor

mkoeppe commented Mar 5, 2022

comment:1

dup of #33355

@mkoeppe
Copy link
Contributor

mkoeppe commented Mar 5, 2022

Dependencies: #33355

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants