Skip to content

test: wait for the channel mute to reach member reads - #286

Merged
mogita merged 1 commit into
mainfrom
test/channel-mute-wait
Nov 14, 2026
Merged

mogita merged 1 commit into
mainfrom
test/channel-mute-wait

Conversation

@mogita

@mogita mogita commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

Problem

ChannelTest > Can mute a channel fails on most CI runs. It mutes a channel and reads the member back at once, and the read can come from an API node that still holds the user from before the mute.

Solution

The two member reads poll with the existing waitFor helper until notifications_muted is true. The same change went into stream-chat-net for the same reason.

How to verify

  1. The CI run on this PR passes, including Can mute a channel.

Note

Low Risk
Test-only change; no production or API behavior is modified.

Overview
Stabilizes Can mute a channel by polling member reads after a mute instead of asserting immediately. Both checks—plain Channel.queryMembers() and the same query with filterCondition("notifications_muted", true)—now run inside the existing waitFor helper until the test user’s member reports notifications_muted as true, with a comment noting short-lived API node caching for offline users.

Reviewed by Cursor Bugbot for commit 3f41046. Bugbot is set up for automated code reviews on this repo. Configure here.

@mogita
mogita merged commit 7d83f6b into main Nov 14, 2026
6 of 7 checks passed
@mogita
mogita deleted the test/channel-mute-wait branch November 14, 2026 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant