Fix CI on kedro-sphinx-theme

astrojuanlu commented 3 months ago

Tests run, but there are no actual tests https://github.com/kedro-org/kedro-sphinx-theme/blob/d77aaa1/tests/test_import.py
There are no branch protections in this repo just yet

astrojuanlu commented 5 days ago

After a bit of exploration by @Huongg it wasn't clear what to test from a Sphinx theme. Some high-profile themes don't have any tests at all...

@Huongg can we add a simple smoke test that verifies that the contents of the binary build artifact?

In other words: uv build produces 2 files, one sdist in .tar.gz format and one binary wheel in .whl format (they get uploaded to PyPI with uv publish). The smoke test should check that the contents of the .whl include

An /assets/ folder with >=1 CSS and JS files,
A /theme/kedro-sphinx-theme/ folder with >=1 HTML files, and
A /theme/kedro-sphinx-theme/theme.conf file

?

(Note that .whl files are Zip files, you can do unzip path.whl or open them with the zipfile module)

astrojuanlu commented 4 days ago

Fixed by #15

kedro-org / kedro-sphinx-theme

Fix CI on kedro-sphinx-theme #10