gabriel / musehub public
fix tests dev
AI Agent gabriel · 130 days ago · May 5, 2026 · Diff

fix(tests): replace bare uuid4() with content-addressed IDs

Replace 6 Category-D bare UUID usages and 20 malformed long_id(uuid.uuid4().hex) calls with semantically correct content-addressed IDs:

- test_snapshot_symbol_indexer.py: 20× long_id(uuid.uuid4().hex) → blob_id(uuid.uuid4().bytes) - test_migrations.py: repo_id → compute_repo_id(), identity_id → compute_identity_id() - test_musehub_coord.py: repo fixture repo_id → compute_repo_id() - test_musehub_search.py: commit_id → blob_id(uuid.uuid4().bytes) - test_agent_registration.py: mock_key.identity_id → compute_identity_id() - test_musehub_ui_team.py: user_id → compute_identity_id()

sha256:8733ffe52549293004d5ed21c1c868ba04efb3a68b5f156bb8fee14570a55554 sha
sha256:3e7cf268c7084f397f98e0e6050d1119e4f2f5ec8a3480af2aa064bc7f24e36b parent
+6 ~24 symbols
sha256:44c1a6d71d0ce7b796135e4f437a3018e328066cd5b884a6ce48746192430e2d snapshot
+6
symbols added
~24
symbols modified
0
dead code introduced
Semantic Changes 30 symbols
~ tests/test_agent_registration.py .py 1 symbol added, 2 symbols modified
+ compute_identity_id import import compute_identity_id L20–20
~ TestRegisterAgentIdentityService
~ _make_mock_session
~ tests/test_migrations.py .py 3 symbols added, 3 symbols modified
+ compute_identity_id import import compute_identity_id L41–41
+ compute_repo_id import import compute_repo_id L41–41
+ datetime import import datetime L35–35
~ TestMigrationE2E
~ test_identity_row_persists_at_head
~ test_repo_data_persists_at_head
~ tests/test_musehub_coord.py .py 1 symbol added, 2 symbols modified
+ compute_repo_id import import compute_repo_id L47–47
~ public_repo
~ repo
~ tests/test_musehub_search.py .py 1 symbol added, 1 symbol modified
+ blob_id import import blob_id L41–41
~ _make_search_commit
~ tests/test_musehub_ui_team.py .py 1 symbol modified
~ test_collaborators_settings_page_has_remove_button_htmx
~ tests/test_snapshot_symbol_indexer.py .py 15 symbols modified
~ TestBackfillContentIdsFromSnapshots
~ _seed_snapshot_and_commit
~ test_corrupt_manifest_blob_is_skipped_gracefully
~ test_dry_run_returns_count_without_writing
~ test_fills_content_id_from_manifest
~ test_idempotent
~ test_repo_id_none_fills_all_repos
~ test_skips_entries_already_with_content_id
~ test_skips_entry_when_path_absent_from_manifest
~ test_skips_symbol_level_addresses
~ TestBackfillGenesisOps
~ _seed_bad_birth
~ test_only_corrects_oldest_entry_not_later_modifies
~ TestBackfillRawOpsFromCommits
~ _seed_commit_with_meta
← Older Oldest on dev
All commits
Newer → Latest on dev

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:8733ffe52549293004d5ed21c1c868ba04efb3a68b5f156bb8fee14570a55554 --body "your comment"