Skip to content

[blob-store·tests] pytest coverage for the publish logic (#2) - #3

Closed
chondl wants to merge 1 commit into
cph-bucket-first-servingfrom
bucket-first-serving-tests
Closed

[blob-store·tests] pytest coverage for the publish logic (#2)#3
chondl wants to merge 1 commit into
cph-bucket-first-servingfrom
bucket-first-serving-tests

Conversation

@chondl

@chondl chondl commented Jul 10, 2026

Copy link
Copy Markdown
Owner

Track 2 · companion tests for #2 (bucket-first-serving). Base: bucket-first-serving; merge after #2. Test infrastructure only (backend/tests/, pytest.ini, requirements-dev.txt) — no production code changes. Optional.

Note: this branch was cut from an earlier commit of #2 and does not include #2's final commit (match-page-from-event-blob). It still merges cleanly onto #2 and only adds test files; rebasing onto #2's tip is a harmless cleanup.


Follow-up to #2. Dependency-light pytest suite for the pure manifest/content-addressing module (src/google/publish.py): hash determinism, versioned/historical key formats, manifest round-trip, copy-on-write skip of unchanged blobs, carry-forward of unrendered manifest entries, and hist-epoch handling.

Kept separate from #2 because the project has no existing test infrastructure; this branch adds backend/tests/, backend/pytest.ini, and backend/requirements-dev.txt only.

cd backend
pip install -r requirements-dev.txt
python -m pytest tests/ -q   # 9 passed

@chondl
chondl force-pushed the bucket-first-serving branch from e7f8a83 to 032db39 Compare July 10, 2026 07:05
@chondl
chondl force-pushed the bucket-first-serving-tests branch 2 times, most recently from bf10bfb to 3aaa6d4 Compare July 10, 2026 18:30
@chondl chondl changed the title Add pytest coverage for the publish logic [blob-store·tests] pytest coverage for the publish logic (#2) Jul 10, 2026
@chondl
chondl force-pushed the bucket-first-serving-tests branch from 3aaa6d4 to 6fa5b86 Compare July 10, 2026 21:04
Dependency-light tests for the pure manifest/content-addressing module:
hash determinism, key formats, manifest round-trip, copy-on-write skip,
carry-forward of unrendered entries, and hist-epoch handling. Kept off
the main branch since the project has no test infrastructure;
requirements-dev.txt and pytest.ini are scoped to backend/.

    pip install -r requirements-dev.txt
    python -m pytest tests/ -q
@chondl
chondl force-pushed the bucket-first-serving-tests branch from 6fa5b86 to 8fa8791 Compare July 10, 2026 21:47
@chondl chondl closed this Jul 19, 2026
@chondl
chondl deleted the bucket-first-serving-tests branch July 19, 2026 18:54
@chondl

chondl commented Jul 19, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #19 — reopened as #19 [09] on cph-bucket-first-serving-tests.

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.

1 participant