homelab/stack
1
0
Fork 0
Code Issues 262 Pull Requests 2 Actions Packages Projects 1 Releases Wiki Activity

notebooks: add tests for new layout — config, snippets, compose validation #218

New Issue
Closed
opened 2026-03-25 01:20:03 +00:00 by kert · 1 comment
kert commented 2026-03-25 01:20:03 +00:00
Owner
Copy Link

Tasks

  • Test: infra/marimo/marimo.toml is valid TOML with required sections
  • Test: all 10 snippet files exist in infra/marimo/snippets/
  • Test: home-page-patched.js exists in infra/marimo/
  • Test: notebooks/ contains ONLY .py files (no config, no JS, no CSS)
  • Test: docker compose config --quiet passes
  • Test: no stale references to old paths in compose.yml, Dockerfile, stack.toml
  • Full test suite passes with zero regressions
## Tasks - [ ] Test: `infra/marimo/marimo.toml` is valid TOML with required sections - [ ] Test: all 10 snippet files exist in `infra/marimo/snippets/` - [ ] Test: `home-page-patched.js` exists in `infra/marimo/` - [ ] Test: `notebooks/` contains ONLY `.py` files (no config, no JS, no CSS) - [ ] Test: `docker compose config --quiet` passes - [ ] Test: no stale references to old paths in compose.yml, Dockerfile, stack.toml - [ ] Full test suite passes with zero regressions
kert added this to the P27: Notebook Separation of Concerns — config, snippets, assets out of notebooks/ milestone 2026-03-25 01:20:03 +00:00
kert commented 2026-03-25 01:35:34 +00:00
Author
Owner
Copy Link

Complete.

18 tests in tests/test_notebook_layout.py:

TestNotebooksClean (6): no config files, only .py, no .marimo-config, no .marimo.toml, no snippets, no home-page JS
TestInfraMarimo (8): dir exists, marimo.toml valid TOML with required sections, snippet path updated, 10 snippets present by name, home-page JS exists, no loch.css duplicate
TestNoStaleReferences (4): compose no old config mount, no old homepage mount, uses infra/marimo, stack.toml path_filter includes infra/marimo

✅ 18/18 passed. Full suite: 12,055 passed, zero regressions.

Complete. 18 tests in `tests/test_notebook_layout.py`: **TestNotebooksClean (6):** no config files, only .py, no .marimo-config, no .marimo.toml, no snippets, no home-page JS **TestInfraMarimo (8):** dir exists, marimo.toml valid TOML with required sections, snippet path updated, 10 snippets present by name, home-page JS exists, no loch.css duplicate **TestNoStaleReferences (4):** compose no old config mount, no old homepage mount, uses infra/marimo, stack.toml path_filter includes infra/marimo ✅ 18/18 passed. Full suite: **12,055 passed**, zero regressions.
kert closed this issue 2026-03-25 01:35:34 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
api
REST API (FastAPI)
 auth
Authentication and authorization
 bib
Bibliography, Zotero, citations
 ci
CI/CD pipelines and Woodpecker plugins
 cli
CLI commands and UX
 cloud
Cloud provider abstraction (AWS, GCP, Azure, self-hosted)
 config
Configuration and wiring
 data
Data loading, CCLF, BCDA, seeds
 databricks
Databricks jobs and deployment
 docs
Documentation and generated references
 image:api
image:docs
image:mc
image:notebooks
image:zotero
infra
lakehouse
Nessie, Polaris, Trino, Iceberg
 llm
llm module — local RAG over comments + Zotero corpus
 notebooks
marimo notebook errors (auto-filed by nb-watcher/nb-integration)
 packaging
Python packaging, optional deps, build system
 pfs
pipeline
Pipeline execution and validation
 pipeline:ci
pipeline:deploy
pipeline:harden
pipeline:infra-ci
pipeline:rebuild-all
pipeline:release
qpp
quality
Testing, coverage, error handling
 step:build
step:deploy
step:hadolint
step:lint
step:publish
step:scan
step:test
step:validate
styles
Charts, figures, visual theming
 tree
Project directory structure and file organization
No Label
Milestone
No items
No Milestone P27: Notebook Separation of Concerns — config, snippets, assets out of notebooks/
Projects
Clear projects
No project
Assignees
Clear assignees
kert renovate-bot
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: homelab/stack#218
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?