quick-window-91705
03/16/2026, 6:28 PMchilly-france-44439
03/16/2026, 6:38 PMnutritious-balloon-65098
03/16/2026, 6:39 PMnutritious-balloon-65098
03/16/2026, 6:43 PM00daaa41d (PR #12773 — "Load workspace hooks for V1 conversations") was merged to main with a broken test class TestLoadHooksFromWorkspace in tests/unit/app_server/test_live_status_app_conversation_service.py. All 9 tests in that class failed with:
TypeError: LiveStatusAppConversationService.__init__() missing 2 required positional arguments: 'pending_message_service' and 'max_num_conversations_per_sandbox'
Fix: Added the two missing constructor parameters to `TestLoadHooksFromWorkspace.setup_method()`:
- pending_message_service=self.mock_pending_message_service (with a new Mock() instance)
- max_num_conversations_per_sandbox=20
This matches the exact pattern already used by the other two test classes in the same file.
Verification: All 9 previously failing tests pass locally.
Status:
- ✅ Changes already pushed to branch fix/test-missing-init-params
- ✅ PR created: https://github.com/OpenHands/OpenHands/pull/13424
- ✅ Only 1 file changed, 3 lines added — minimal and focused fix
- ✅ No extraneous changeschilly-france-44439
03/16/2026, 6:47 PMquick-window-91705
03/16/2026, 6:49 PM