Skip to content

e2e: Ignore the flaky thread quote jump test with the scrolling setup - #6592

Merged
andremion merged 1 commit into
developfrom
andrerego/and-1332-stabilize-the-thread-quote-jump-e2e-test-with-the-scrolling
Jul 28, 2026
Merged

e2e: Ignore the flaky thread quote jump test with the scrolling setup#6592
andremion merged 1 commit into
developfrom
andrerego/and-1332-stabilize-the-thread-quote-jump-e2e-test-with-the-scrolling

Conversation

@andremion

@andremion andremion commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Goal

test_quotedReplyNotInList_whenUserAddsQuotedReply_InThread (AllureId 5892) was re-enabled by #6589 and turned out unstable: it failed the e2e batch on two unrelated PRs (#6588, #6587) after the merge, and it reproduces locally at roughly 2 of 5 runs. This ignores that one test again so open PRs are unblocked, until it is stabilized.

Resolves nothing by itself; the stabilization work is tracked in AND-1332.

Implementation

Adds @Ignore with the AND-1332 reference on the single flaky test. The failures are in the test setup, before the jump behavior under test: a waitToAppear timeout finding the target message after a fixed number of scroll-ups (CI), and a StaleObjectException clicking the channel (local). The three participant variants of the same scenario have deterministic setups, do not fail, and stay enabled, so the jump fix keeps e2e coverage.

Testing

Test-only change. spotlessCheck and the e2e androidTest compilation pass. The remaining QuotedReplyTests continue to run on CI.

Summary by CodeRabbit

  • Tests
    • Temporarily disabled a quoted-reply end-to-end test while a known issue is being addressed.

test_quotedReplyNotInList_whenUserAddsQuotedReply_InThread fails
intermittently in its setup (a fixed number of scroll-ups to reach the
quoted target, and stale-node clicks), and since it was re-enabled it
blocks unrelated PRs. The three participant variants of the same
scenario have deterministic setups and stay enabled.
@github-actions

github-actions Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

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 andremion added the pr:test Test-only changes label Jul 28, 2026
@andremion

Copy link
Copy Markdown
Contributor Author

@CodeRabbit review

@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown
✅ 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

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

The quoted-reply end-to-end test now imports JUnit’s Ignore annotation and is excluded from execution with a reference to issue AND-1332.

Changes

Quoted-reply test execution

Layer / File(s) Summary
Ignore the quoted-reply thread scenario
stream-chat-android-compose-sample/.../QuotedReplyTests.kt
Adds JUnit’s Ignore import and annotates test_quotedReplyNotInList_whenUserAddsQuotedReply_InThread with the issue URL.

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

Possibly related PRs

Suggested reviewers: velikovpetar, testableapple, aleksandar-apostolov

Poem

A bunny found a test in flight,
And tucked it safely out of sight.
“AND-1332,” the carrot said,
“This check can rest inside its bed.”
🐇 Tests will hop when bugs are fed.

🚥 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 and concisely describes the main change: ignoring a flaky e2e test in the thread quote flow.
Description check ✅ Passed The description covers Goal, Implementation, and Testing with enough detail, and the missing UI/GIF sections are reasonable for a test-only change.
✨ 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-1332-stabilize-the-thread-quote-jump-e2e-test-with-the-scrolling

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

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

@andremion
andremion marked this pull request as ready for review July 28, 2026 10:01
@andremion
andremion requested a review from a team as a code owner July 28, 2026 10:01
@andremion
andremion enabled auto-merge (squash) July 28, 2026 10:01
@github-actions

Copy link
Copy Markdown
Contributor

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.22 MB 11.22 MB 0.00 MB 🟢
stream-chat-android-compose 12.69 MB 12.69 MB 0.00 MB 🟢

@sonarqubecloud

Copy link
Copy Markdown

@andremion
andremion disabled auto-merge July 28, 2026 10:23
@andremion
andremion merged commit 8ab0391 into develop Jul 28, 2026
20 of 21 checks passed
@andremion
andremion deleted the andrerego/and-1332-stabilize-the-thread-quote-jump-e2e-test-with-the-scrolling branch July 28, 2026 10:28
@stream-public-bot stream-public-bot added the released Included in a release label Jul 30, 2026
@stream-public-bot

Copy link
Copy Markdown
Contributor

🚀 Available in v7.7.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:test Test-only changes released Included in a release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants