From 7b3b5b98586449afd0b6996ba9fd7aec8308bbc6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Juan=20Diego=20Garc=C3=ADa?= Date: Mon, 30 Mar 2026 18:02:56 -0500 Subject: [PATCH] fix: remove share public from integration tests (#938) --- server/tests/integration/test_multitrack_pipeline.py | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/server/tests/integration/test_multitrack_pipeline.py b/server/tests/integration/test_multitrack_pipeline.py index 34e799b3..1a8a9257 100644 --- a/server/tests/integration/test_multitrack_pipeline.py +++ b/server/tests/integration/test_multitrack_pipeline.py @@ -162,15 +162,9 @@ async def test_multitrack_pipeline_end_to_end( ), f"Expected at least 2 speakers for multitrack, got {len(participants)}" # 7. Verify email transcript notification - # The send_email pipeline task should have: - # a) Set the transcript to public share_mode - # b) Sent an email to TEST_EMAIL via Mailpit - transcript_resp = await api_client.get(f"/transcripts/{transcript_id}") - transcript_resp.raise_for_status() - transcript_data = transcript_resp.json() - assert ( - transcript_data.get("share_mode") == "public" - ), "Transcript should be set to public when email recipients exist" + # The send_email pipeline task should have sent an email to TEST_EMAIL via Mailpit. + # Note: share_mode is only set to "public" when meeting has email_recipients; + # room-level emails do NOT change share_mode. # Poll Mailpit for the delivered email (send_email task runs async after finalize) messages = await poll_mailpit_messages(mailpit_client, TEST_EMAIL, max_wait=30)