Skip to content

Run python doctest against scripts - #6402

Open
janniklasrose wants to merge 4 commits into
mainfrom
janniklasrose/tools-py-doctest
Open

Run python doctest against scripts#6402
janniklasrose wants to merge 4 commits into
mainfrom
janniklasrose/tools-py-doctest

Conversation

@janniklasrose

@janniklasrose janniklasrose commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Changes

Run doctest against python scripts.

Remove import-time side-effect from gh_report.py.

Why

Agents already write doctests, now they run on checks (local & CI).

Allows (follow-up PRs) to add more test coverage to the scripts we use.

Tests

CI green. Existing doctest failed when modifying behaviour of a function locally.

@github-actions

Copy link
Copy Markdown
Contributor

Waiting for approval

Based on git history, these people are best suited to review:

  • @pietern -- recent work in ./
  • @denik -- recent work in ./, tools/

Eligible reviewers: @andrewnester, @anton-107, @lennartkats-db, @rclarey, @renaudhartert-db, @rugpanov, @shreyas-goenka, @simonfaltum

Suggestions based on git history. See OWNERS for ownership rules.

@eng-dev-ecosystem-bot

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: 7d8f853

Run: 33088148453

Env 🔄​flaky 💚​RECOVERED 🙈​SKIP ✅​pass 🙈​skip Time
💚​ aws linux 1 4 274 1191 4:11
💚​ aws windows 1 4 276 1189 4:35
💚​ azure linux 1 4 273 1191 4:02
🔄​ azure windows 1 4 275 1189 6:51
💚​ gcp linux 1 4 274 1191 4:24
💚​ gcp windows 1 4 276 1189 4:39
Test Name aws linux aws windows azure linux azure windows gcp linux gcp windows
🔄​ TestAccept 💚​R 💚​R 💚​R 🔄​f 💚​R 💚​R
🙈​ TestAccept/bundle/invariant/no_drift 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/bundle/resources/vector_search_endpoints/drift/recreated_same_name 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/bundle/resources/vector_search_indexes/recreate/embedding_dimension 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/ssh/connection 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
Top 3 slowest tests (at least 2 minutes):
duration env testname
4:35 gcp windows TestAccept
4:32 aws windows TestAccept
3:42 azure windows TestAccept

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