Re-enable the failed-message channel preview e2e test

[andremion]

Goal

Re-enable the e2e test test_errorIndicatorShownInPreview_whenMessageFailedToBeSent, ignored since January 2025, and fix the selector bug that made it fail. The SDK behavior is correct: the channel preview shows the failed delivery icon for a message with SyncStatus.FAILED_PERMANENTLY (confirmed by the CI failure screenshot and UI hierarchy dump, which show the icon rendered with the expected resource id). The test failed because it asserted the wrong tag.

Resolves AND-256

Implementation

• The channel preview renders the failed state via MessageReadStatusIcon, tagged Stream_MessageReadStatus_isError. The page object reused the message list's Stream_MessageFailedIcon tag, which never exists in the channel list, so the assertion could only pass when a leftover message-list node was still present during the pane transition. The read, pending, and sent tags are shared between the two surfaces, which is why only the failed variant was broken.
• ChannelListPage.Channel.deliveryStatusIsFailed now points to Stream_MessageReadStatus_isError, and the test asserts it through a new assertFailedMessageDeliveryStatusInPreview helper.
• Removed the @Ignore from the test. No production code changes.

🎨 UI Changes

No UI changes.

Testing

1. In the app: open a channel, force message sends to fail, send a message, and go back to the channel list. The preview shows the message with the failed delivery icon.
2. The fixed test passed 3 of 3 normal local runs, plus 1 run against a mock patched to delay the failure response by 3 seconds, so the assertion also holds when the failure lands after the navigation back.

[github-actions]

PR checklist ✅

All required conditions are satisfied:

• Title length is OK (or ignored by label).
• At least one pr: label exists.
• Sections ### Goal, ### Implementation, and ### Testing are filled, or the PR is bot-authored.
• An issue is linked (Linear ticket or GitHub issue), or the PR is bot-authored.

🎉 Great job! This PR is ready for review.

[andremion]

@CodeRabbit review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Walkthrough

The preview failure test for message delivery status is re-enabled by removing its @Ignore annotation.

Changes

Message delivery test execution

Layer / File(s)	Summary
Enable preview failure test stream-chat-android-compose-sample/src/androidTestE2eDebug/.../MessageDeliveryStatusTests.kt	Removes the skip annotation from test_errorIndicatorShownInPreview_whenMessageFailedToBeSent, allowing it to run in the test suite.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• GetStream/stream-chat-android#6573: Similarly re-enables ignored MessageDeliveryStatusTests cases.

Poem

A bunny found a test asleep,
And nudged it from its hiding heap.
No more ignore, the checks now run,
Delivery errors meet the sun! 🐰

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly summarizes the main change: re-enabling the failed-message channel preview e2e test.
Description check	✅ Passed	The description matches the template well, with Goal, Implementation, UI Changes, and Testing sections filled in.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch andrerego/and-256-chatcompose-failed-message-should-be-displayed-in-channel

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

SDK Size Comparison 📏

SDK	Before	After	Difference	Status
stream-chat-android-client	5.95 MB	5.95 MB	0.00 MB	🟢
stream-chat-android-ui-components	11.21 MB	11.21 MB	0.00 MB	🟢
stream-chat-android-compose	12.69 MB	12.69 MB	0.00 MB	🟢

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[stream-public-bot]

🚀 Available in v7.7.0