From cc6a76514baae93ff5032ec48164277299fce2a3 Mon Sep 17 00:00:00 2001 From: hailin Date: Sat, 29 Nov 2025 00:52:53 -0800 Subject: [PATCH] fix: Report completion for all 3 participants in TestCompleteKeygenFlow The test was only reporting completion for one participant, but the session requires ALL participants to report completion before transitioning to "completed" status. This follows the domain logic in ShouldCompleteSession() which checks session.AllCompleted(). Changes: - Added reportCompletion calls for all 3 parties (party_user_device, party_server, party_recovery) - Updated test comment to clarify all participants must report completion --- backend/mpc-system/tests/e2e/keygen_flow_test.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/backend/mpc-system/tests/e2e/keygen_flow_test.go b/backend/mpc-system/tests/e2e/keygen_flow_test.go index 9d17486b..4a6939bb 100644 --- a/backend/mpc-system/tests/e2e/keygen_flow_test.go +++ b/backend/mpc-system/tests/e2e/keygen_flow_test.go @@ -166,9 +166,11 @@ func (s *KeygenFlowTestSuite) TestCompleteKeygenFlow() { statusResp = s.getSessionStatus(sessionID) assert.Equal(s.T(), "in_progress", statusResp.Status) - // Step 9: Report completion (simulate keygen completion) + // Step 9: Report completion for all participants (simulate keygen completion) publicKey := []byte("test-group-public-key-from-keygen") s.reportCompletion(sessionID, "party_user_device", publicKey) + s.reportCompletion(sessionID, "party_server", publicKey) + s.reportCompletion(sessionID, "party_recovery", publicKey) // Step 10: Verify session is completed statusResp = s.getSessionStatus(sessionID)