Skip to content

test: run the Python quick start in CI - #224

Merged
gouyt13 merged 2 commits into
VectorDB-NTU:mainfrom
OptiSolvD:test/215-run-quick-start-in-ci
Nov 8, 2026
Merged

gouyt13 merged 2 commits into
VectorDB-NTU:mainfrom
OptiSolvD:test/215-run-quick-start-in-ci

Conversation

@OptiSolvD

@OptiSolvD OptiSolvD commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Adds tests/python/test_quick_start.py, which reads docs/docs/quick_start.md,
extracts the ```python blocks in order and runs them in one shared namespace
against the installed package. The test reads the Markdown, so there is no second
copy of the example. It runs from a temporary working directory, and the save/load
block already uses TemporaryDirectory, so no files are left behind.

Closes #215

No API, saved-index format or dependency changes.

Testing

  • python -m pytest tests/python/test_quick_start.py -v: 1 passed
  • Mutation check: renaming nprobe to n_probe in block 1 of quick_start.md
    makes the test fail with a TypeError that names the block (reverted afterwards)
  • ruff format --check tests/python and ruff check tests/python: pass
  • git diff --check clean; git status shows only the new test file
  • bash scripts/check-python.sh: passed successfully.
    • 76 files already formatted.
    • All checks passed.

@OptiSolvD
OptiSolvD requested a review from gouyt13 as a code owner November 7, 2026 14:38
@github-actions github-actions Bot added the release:maintenance Release category managed from the PR title label Nov 7, 2026
Docs-only changes selected only the documentation build, so an edit to
docs/docs/quick_start.md that broke its Python blocks passed CI. Select
the Linux source-install job for that page and run test_quick_start.py
in its smoke tests.
@gouyt13
gouyt13 merged commit d6c7e25 into VectorDB-NTU:main Nov 8, 2026
71 of 76 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:maintenance Release category managed from the PR title

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Run the Python quick start in CI

2 participants