e2e: Ignore the flaky thread quote jump test with the scrolling setup - #6592
Conversation
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.
PR checklist ✅All required conditions are satisfied:
🎉 Great job! This PR is ready for review. |
|
@CodeRabbit review |
✅ Action performedReview finished.
|
WalkthroughThe quoted-reply end-to-end test now imports JUnit’s ChangesQuoted-reply test execution
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
SDK Size Comparison 📏
|
|
|
🚀 Available in v7.7.0 |



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
@Ignorewith the AND-1332 reference on the single flaky test. The failures are in the test setup, before the jump behavior under test: awaitToAppeartimeout finding the target message after a fixed number of scroll-ups (CI), and aStaleObjectExceptionclicking 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.
spotlessCheckand the e2e androidTest compilation pass. The remainingQuotedReplyTestscontinue to run on CI.Summary by CodeRabbit