Skip to content

Pytestless doctests 2 - #196

Draft
pp-mo wants to merge 20 commits into
SciTools:mainfrom
pp-mo:pytestless_doctests_2
Draft

Pytestless doctests 2#196
pp-mo wants to merge 20 commits into
SciTools:mainfrom
pp-mo:pytestless_doctests_2

Conversation

@pp-mo

@pp-mo pp-mo commented Feb 12, 2026

Copy link
Copy Markdown
Member

Replaces #177
Rebased to resolve conflicts,
and, since we moved the repo into SciTools, also replace the branch with one in my fork (pp-mo/ncdata)

##TODO
Issues inherited from #177

(and some more added since)

  • output reports currently say like e.g.
    1 item had failures:
    17 of 30 in lenient_metadata.rst
    ***Test Failed*** 17 failures and 9 skipped tests.
    • this should be clearer + especially include count of successes, so : "ok/bad/skip add up to total"
  • ability to detect + skip internal sections that APPEAR like doctests, e.g. in ".. codeblock::"
    (dropped: non-trival and would require deeper engagement with the doctests api, beyond 'testmod/testfile)

@pp-mo pp-mo mentioned this pull request Feb 12, 2026
@trexfeathers
trexfeathers marked this pull request as draft March 18, 2026 10:35
@pp-mo pp-mo mentioned this pull request Mar 26, 2026
18 tasks
@pp-mo
pp-mo force-pushed the pytestless_doctests_2 branch from ba1577c to 84f674f Compare August 10, 2026 10:12
@pp-mo
pp-mo force-pushed the pytestless_doctests_2 branch from 795ba92 to 76b3439 Compare August 20, 2026 16:39
@pp-mo

pp-mo commented Aug 20, 2026

Copy link
Copy Markdown
Member Author

status update 2026-08-20

current version has simplifications to the runner
tests okay-->fail for python 3.10,3.11 --> 3.12/3.13
= iris 3.14 --> iris3.16 (needing set_auto_chartostring support)

Now ticks all the above to-do boxes.
I think this code is now "good-to-go" as far as the doctest-runner and its tests are concerned,
but this branch is now a mess : a bunch of unrelated commits mixed in.

Will get the tests+CI fixed, and then re-propose this code
- possibly also using scitools templating ?

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant