Skip to content

add unit tests for build scripts#1641

Closed
tejas0077 wants to merge 2 commits intomicrosoft:mainfrom
tejas0077:maint/build-script-tests
Closed

add unit tests for build scripts#1641
tejas0077 wants to merge 2 commits intomicrosoft:mainfrom
tejas0077:maint/build-script-tests

Conversation

@tejas0077
Copy link
Copy Markdown
Contributor

Adds unit tests for 4 build scripts as requested by @romanlutz in #1569.

Closes #1570

  • test_prepare_package.py — 9 tests covering npm lookup, install/build failures, and frontend copy logic
  • test_validate_docs.py — 14 tests covering TOC parsing, missing file detection, and orphaned file checks
  • test_generate_rss.py — 10 tests covering date extraction and blog markdown parsing
  • test_check_links.py — 13 tests covering URL extraction, fragment stripping, and relative URL resolution

Added 46 new unit tests across 4 test files covering all 4 build scripts.
All 65 tests in tests/unit/build_scripts/ pass locally:

python -m pytest tests/unit/build_scripts/ -v

65 passed

No documentation changes required — this is a pure test addition.
No JupyText changes required — no notebook modifications.

@tejas0077
Copy link
Copy Markdown
Contributor Author

Closing in favour of cleaner PR #1642

@tejas0077 tejas0077 closed this Apr 22, 2026
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.

MAINT add tests for build scripts

1 participant