gabriel / musehub public
test task/mwp-6-phase3 #3 / 4
AI Agent gabriel · 20 days ago · Jun 29, 2026 · Diff

test: re-enable MWP-6 phase 4 wire regression suite (MWP6_40–MWP6_46)

Remove all phase-4 skip markers and rewrite 7 test files to match the current wire protocol API:

- test_mpack_rate_limiting_phase4.py (MWP6_46): removed 2 function skips - test_mpack_size_gates.py (MWP6_45): removed 1 skip; rewrote oversized-wire and at-limit tests to use build_wire_mpack + MemoryBackend directly - test_push_ff_check.py (MWP6_42): removed module skip; rewrote _upload_and_unpack to use MUSE wire format + blob_id key - test_mpack_delta_e2e.py (MWP6_43): removed function skip; rewrote push helper to use MUSE wire format + direct backend storage - test_mpack_content_scanning_phase3.py (MWP6_44): rewrote _push_mpack to use HTTP unpack-mpack; rewrote 4 blocklist tests to block before push; deleted unimplemented content.scan test; fixed DMCA tests to use blob_id - test_mpack_fetch_phase3.py (MWP6_40): full rewrite — PF-1/2/3 now use HTTP push + index job + force_build=True to match actual wire_fetch_mpack API (mpack_url, not mpack_urls; MPackNotReadyError on cold cache) - test_mpack_gc_phase4.py (MWP6_41): fixed _make_mpack to use build_wire_mpack with blobs key; fixed PG-3 to read blobs key; rewrote PG-6 to use correct result shape and force_build=True

All 46 tests pass.

sha256:815aa52697f56e4edc6f51cafc0b0c92a2048796c6d44cad027af8fcd153173b sha
sha256:effb54731c40ed34c82360dd10252c9f2b7bf6e9eff2efa9d57a6a0ea7817261 parent
+37 ~23 −23 symbols
sha256:890fb553dd79a15ca0d5f7933847da4643749a096a985359ad738bbc82cb436e snapshot
+37
symbols added
~23
symbols modified
−23
symbols removed
0
dead code introduced
Semantic Changes 83 symbols
~ tests/test_mpack_content_scanning_phase3.py .py 2 symbols added, 2 symbols removed, 10 symbols modified
− hashlib import import hashlib L15–15
− test_content_scan_jobs_enqueued_after_indexing function async_function test_content_scan_jobs_enqueued_after_indexing L504–536
+ _push_mpack_expect_blocked function async_function _push_mpack_expect_blocked L204–232
+ build_wire_mpack import import build_wire_mpack L33–33
~ _push_mpack
~ build_mpack
~ test_blocked_check_fires_before_minio_puts
~ test_blocked_object_quarantines_mpack
~ test_clean_mpack_bypasses_blocklist_unimpeded
~ test_dmca_takedown_adds_hashes_to_blocklist
~ test_dmca_takedown_idempotent
~ test_dmca_takedown_marks_repos_dmca_hold
~ test_dmca_takedown_quarantines_existing_minio_objects
~ test_multiple_blocked_objects_all_reported
~ tests/test_mpack_delta_e2e.py .py 1 symbol added, 1 symbol removed, 2 symbols modified
− httpx import import httpx L19–19
+ build_wire_mpack import import build_wire_mpack L30–30
~ build_mpack
~ test_mpack_push_delta_e2e
~ tests/test_mpack_fetch_phase3.py .py 32 symbols added, 13 symbols removed
− _make_commit_chain function function _make_commit_chain L101–140
− _make_mpack function function _make_mpack L35–52
− _push_and_index function async_function _push_and_index L61–98
− _store_mpack function async_function _store_mpack L55–58
− compute_identity_id import import compute_identity_id L27–27
− create_repo import import create_repo L28–28
− db import import db L26–26
− hashlib import import hashlib L16–16
− select import import select L22–22
− pytestmark variable variable pytestmark L21–21
− test_pf1_fetch_returns_mpack_urls function async_function test_pf1_fetch_returns_mpack_urls L148–175
− test_pf2_mpack_urls_count_equals_distinct_mpacks function async_function test_pf2_mpack_urls_count_equals_distinct_mpacks L183–221
− test_pf3_mpack_fetch_includes_commit_and_snapshot_metadata function async_function test_pf3_mpack_fetch_includes_commit_and_snapshot_metadata L229–259
+ _AUTH_CTX variable variable _AUTH_CTX L37–42
+ _BLOB_SIZE variable variable _BLOB_SIZE L45–45
+ _N_FILES variable variable _N_FILES L44–44
+ _http_push function async_function _http_push L144–177
+ _make_local_repo function function _make_local_repo L82–92
+ _populate function function _populate L95–141
+ _run_index_job function async_function _run_index_job L180–211
+ client function async_function client L51–65
+ ASGITransport import import ASGITransport L20–20
+ AsyncClient import import AsyncClient L20–20
+ CommitRecord import import CommitRecord L31–31
+ MSignContext import import MSignContext L23–23
+ SnapshotRecord import import SnapshotRecord L33–33
+ app import import app L25–25
+ build_mpack import import build_mpack L28–28
+ build_wire_mpack import import build_wire_mpack L28–28
+ compute_commit_id import import compute_commit_id L30–30
+ compute_snapshot_id import import compute_snapshot_id L30–30
+ get_db import import get_db L24–24
+ muse_dir import import muse_dir L29–29
+ optional_signed_request import import optional_signed_request L23–23
+ pathlib import import pathlib L15–15
+ pytest_asyncio import import pytest_asyncio L19–19
+ require_signed_request import import require_signed_request L23–23
+ write_branch_ref import import write_branch_ref L32–32
+ write_commit import import write_commit L31–31
+ write_object import import write_object L27–27
+ write_snapshot import import write_snapshot L33–33
+ repo function async_function repo L69–77
+ test_pf1_fetch_returns_mpack_url function async_function test_pf1_fetch_returns_mpack_url L217–246
+ test_pf2_delta_fetch_returns_only_new_commits function async_function test_pf2_delta_fetch_returns_only_new_commits L252–284
+ test_pf3_cold_cache_raises_mpack_not_ready function async_function test_pf3_cold_cache_raises_mpack_not_ready L290–315
~ tests/test_mpack_gc_phase4.py .py 1 symbol added, 2 symbols removed, 3 symbols modified
− hashlib import import hashlib L26–26
− pytestmark variable variable pytestmark L32–32
+ build_wire_mpack import import build_wire_mpack L32–32
~ _make_mpack
~ test_pg3_consolidated_mpack_contains_all_objects
~ test_pg6_fetch_works_after_gc_and_index_converges
~ tests/test_mpack_rate_limiting_phase4.py .py 2 symbols modified
~ test_daily_limit_allows_push_under_quota
~ test_daily_limit_is_per_user_not_global
~ tests/test_mpack_size_gates.py .py 3 symbols removed, 3 symbols modified
− AsyncMock import import AsyncMock L20–20
− hashlib import import hashlib L19–19
− patch import import patch L20–20
~ test_presign_accepts_mpack_at_limit
~ test_unpack_mpack_accepts_counts_at_limit
~ test_unpack_mpack_rejects_oversized_wire_bytes
~ tests/test_push_ff_check.py .py 1 symbol added, 2 symbols removed, 3 symbols modified
− httpx import import httpx L21–21
− pytestmark variable variable pytestmark L43–43
+ build_wire_mpack import import build_wire_mpack L31–31
~ _add_commit
~ _upload_and_unpack
~ build_mpack
Older test: MWP-6 Phase 3 — revive 5 skipped mpack index/validation te… sha256:effb54731c40ed34c82360dd10252c9f2b7bf6e9eff2efa9d57a6a0ea7817261 All commits Newer test: re-enable MWP-6 phase 5 subprocess E2E suite (MWP6_50–MWP6_52) sha256:3cab1a247ab9842d8c6843ff4f19db0152b95dab75f2f381a483c859e393ee83

0 comments

No comments yet. Be the first to start the discussion.

To add a comment, use the Muse CLI: muse hub commit comment sha256:815aa52697f56e4edc6f51cafc0b0c92a2048796c6d44cad027af8fcd153173b --body "your comment"