gabriel / muse public
fix task/fix-warnings-and-tests #1 / 2
gabriel · 167 days ago · Apr 14, 2026 · Diff

fix: update tests to match current API shapes

- test_push_have_filter: filter_objects mock returns FilterObjectsResult TypedDict (missing=[], bases={}) — code accesses result['missing'] not result[0], so bare [] raised TypeError - test_integrity_I7_history_walk: use _make_diverging_branches(root,110,110) for TestFindMergeBaseAtCap — tip_b was an ancestor of tip_a in the same linear chain so BFS found the base before hitting the cap; truly diverging branches with 110 commits each need 220+ visits to find base - test_pack_missing_snapshot_integrity: build_pack_from_walk now raises ValueError('Push aborted: N snapshot(s) missing') instead of silently excluding commits; update all affected tests to expect the raise so they are authoritative for the strict, no-dangling-refs behaviour - test_cmd_hub_hardening: sync proposal view→read, issue edit→update, issue get→read, hub_url→hub param renames from CLI refactor

sha256:45145c8cf01709237a7c07ce3e469ea5286b86076b3baec2bfb02a3ab3054b41 sha
sha256:ca9d0ced9904fcfe452d65cfcaa48b05a37b7e0f055989c910a7cea0fa39145c parent
+4 ~69 −3 symbols
sha256:efbd8182a63b024ce4bb59ba307296338908410924ef8d8f2cc01ba9e33811a3 snapshot
+4
symbols added
~69
symbols modified
−3
symbols removed
0
dead code introduced
Semantic Changes 76 symbols
~ tests/test_cmd_hub_hardening.py .py 58 symbols modified
~ TestIssueE2E
~ test_create_then_edit_flow
~ test_edit_agent_json_pipeline
~ test_validation_error_does_not_leak_network
~ TestIssueEditHardening
~ test_body_only_patch
~ test_both_title_and_body_in_patch
~ test_hub_override_used
~ test_json_passthrough
~ test_json_short_flag
~ test_no_fields_error_message
~ test_no_fields_exits_nonzero_no_network
~ test_no_hub_exits_nonzero
~ test_no_identity_exits_nonzero
~ test_outside_repo_exits_nonzero
~ test_patch_endpoint_includes_number
~ test_text_mode_no_json_on_stdout
~ test_text_mode_success_message
~ test_title_only_patch
~ test_uses_patch_method
~ TestIssueEditSecurity
~ test_all_validation_before_network
~ test_empty_title_exits_nonzero_no_network
~ test_empty_title_shows_error_message
~ test_negative_number_exits_nonzero_no_network
~ test_repo_flag_routes_correctly
~ test_title_at_max_length_accepted
~ test_title_too_long_exits_nonzero_no_network
~ test_title_too_long_shows_char_count
~ test_whitespace_only_title_exits_nonzero_no_network
~ test_zero_number_exits_nonzero_no_network
~ test_zero_number_shows_helpful_message
~ TestIssueEditStress
~ test_body_only_no_title_validation
~ test_large_number_accepted
~ test_positive_number_one_accepted
~ TestIssueGetHardening
~ test_ansi_in_author_sanitized
~ test_ansi_in_title_sanitized
~ test_invalid_number_message_mentions_positive
~ test_json_output_contains_number_and_title
~ test_json_passthrough_does_not_emit_stderr_summary
~ test_json_short_flag
~ test_negative_number_exits_nonzero_no_network
~ test_open_state_shows_correct_icon
~ test_text_output_goes_to_stderr_not_stdout
~ test_text_shows_number_title_author
~ test_zero_number_exits_nonzero_no_network
~ TestIssueSubparserRegistration
~ test_edit_help_contains_exit_codes
~ test_edit_j_alias_accepted
~ TestNewSubcommandsE2E
~ test_full_crud_sequence
~ test_get_agent_pipeline
~ TestNewSubcommandsRegistration
~ test_all_five_subcommands_present
~ test_get_help_shows_quickstart
~ test_get_in_issue_help
~ tests/test_integrity_I7_history_walk.py .py 3 symbols modified
~ TestFindMergeBaseAtCap
~ test_a_side_cap_raises_muse_cli_error
~ test_error_message_mentions_config_key
~ tests/test_pack_missing_snapshot_integrity.py .py 3 symbols added, 3 symbols removed, 5 symbols modified
− test_pack_excludes_commit_when_snapshot_missing method method test_pack_excludes_commit_when_snapshot_missing L179–193
− test_pack_excludes_only_broken_commit_leaves_others method method test_pack_excludes_only_broken_commit_leaves_others L207–224
− test_pack_bundle_is_always_internally_consistent method method test_pack_bundle_is_always_internally_consistent L282–304
+ test_pack_raises_when_any_snapshot_missing_in_chain method method test_pack_raises_when_any_snapshot_missing_in_chain L211–224
+ test_pack_raises_when_snapshot_missing method method test_pack_raises_when_snapshot_missing L187–197
+ test_pack_aborts_on_chain_with_gaps method method test_pack_aborts_on_chain_with_gaps L282–298
~ TestBuildPackExcludesCommitsWithMissingSnapshot
~ test_no_warning_when_all_snapshots_present
~ test_pack_bundle_snapshot_list_and_commit_list_are_consistent
~ test_warning_emitted_when_snapshot_missing
~ TestMissingSnapshotRegressionInvariant
~ tests/test_push_have_filter.py .py 1 symbol added, 3 symbols modified
+ FilterObjectsResult import import FilterObjectsResult L60–60
~ _push_mock
~ PresignResponse
~ TransportError
Older fix: suppress tree-sitter int-arg DeprecationWarning for scss grammar sha256:ca9d0ced9904fcfe452d65cfcaa48b05a37b7e0f055989c910a7cea0fa39145c All commits
Newer → Latest on task/fix-warnings-and-tests

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:45145c8cf01709237a7c07ce3e469ea5286b86076b3baec2bfb02a3ab3054b41 --body "your comment"