Skip to content

get doctests running as part of test actions#713

Merged
JessicaS11 merged 14 commits intodevelopmentfrom
doctests
Feb 13, 2026
Merged

get doctests running as part of test actions#713
JessicaS11 merged 14 commits intodevelopmentfrom
doctests

Conversation

@JessicaS11
Copy link
Copy Markdown
Member

@JessicaS11 JessicaS11 commented Oct 10, 2025

While working on #712, I noticed that the codecov report showed some lines as untested that should have been tested by the docstring (and also that some of those same doctests should be failing!). I discovered that despite --doctest-modules being set in pytest.ini, it wasn't running the docstring tests. This PR modifies the pytest.ini file to help it find all the docstring tests and run them (local testing) and the unit_test.yml action to include them in CI testing. It also starts fixing some of the failing docstring tests.

This lack of docstring tests running explains a not-yet-documented mismatch in what product_summary_info provides in v2 after #685 versus the short summary it previously showed.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 10, 2025

Binder 👈 Launch a binder notebook on this branch for commit 1d22652

I will automatically update this comment whenever this PR is modified

Binder 👈 Launch a binder notebook on this branch for commit 6f83d77

Binder 👈 Launch a binder notebook on this branch for commit 8b73c35

Binder 👈 Launch a binder notebook on this branch for commit 9724ab9

Binder 👈 Launch a binder notebook on this branch for commit 6a29319

Binder 👈 Launch a binder notebook on this branch for commit 1be7168

Binder 👈 Launch a binder notebook on this branch for commit df46b73

Binder 👈 Launch a binder notebook on this branch for commit a884bd4

Binder 👈 Launch a binder notebook on this branch for commit d738452

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.65%. Comparing base (647929a) to head (6f83d77).

Additional details and impacted files
@@             Coverage Diff              @@
##           development     #713   +/-   ##
============================================
  Coverage        65.65%   65.65%           
============================================
  Files               37       37           
  Lines             3051     3051           
  Branches           396      396           
============================================
  Hits              2003     2003           
  Misses             971      971           
  Partials            77       77           

☔ View full report in Codecov by Sentry.
📢 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.

@JessicaS11 JessicaS11 marked this pull request as ready for review February 13, 2026 15:47
@JessicaS11
Copy link
Copy Markdown
Member Author

The unit tests will continue to fail due to #723. Going to merge so we can get that fix through without the other version update distractions.

@JessicaS11 JessicaS11 merged commit a48e938 into development Feb 13, 2026
5 of 9 checks passed
@JessicaS11 JessicaS11 deleted the doctests branch February 13, 2026 16:15
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