Skip to content

test: add sphinx-build integration coverage#124

Merged
Technologicat merged 1 commit intoTechnologicat:masterfrom
BlocksecPHD:test/sphinx-build-integration-114
Apr 15, 2026
Merged

test: add sphinx-build integration coverage#124
Technologicat merged 1 commit intoTechnologicat:masterfrom
BlocksecPHD:test/sphinx-build-integration-114

Conversation

@BlocksecPHD
Copy link
Copy Markdown
Contributor

Summary

  • add an integration test that runs a real sphinx-build HTML build for the .. callgraph:: directive
  • use a tiny fake dot executable so the test exercises Sphinx's Graphviz rendering path without depending on a system Graphviz install in CI
  • assert that the generated HTML references the rendered SVG output and keeps the zoomable callgraph behavior wired up

Closes #114.

Testing

  • ruff check .
  • pytest tests/test_sphinx.py -q
  • pytest tests/test_coverage.py tests/test_sphinx.py tests/test_writers.py -q
  • pytest tests/ -v

Notes

  • the fake dot stub is intentional: it keeps the test focused on the Sphinx integration boundary instead of the external Graphviz binary implementation

@Technologicat Technologicat merged commit e8c8b23 into Technologicat:master Apr 15, 2026
@Technologicat
Copy link
Copy Markdown
Owner

Looks good.

Merged with minor edits (capture stderr from subprocess; run sphinx-build via python -m sphinx; and add Windows support for the stub).

Thanks for the contribution!

— Technologicat

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.

Sphinx extension: add integration test with sphinx-build

2 participants