Skip to content

Fix flaky tests in CI + failing tests + issue on CI passing with failed tests#2311

Open
coder-with-a-bushido wants to merge 3 commits into
mainfrom
karthi/flaky-test-attempt-2
Open

Fix flaky tests in CI + failing tests + issue on CI passing with failed tests#2311
coder-with-a-bushido wants to merge 3 commits into
mainfrom
karthi/flaky-test-attempt-2

Conversation

@coder-with-a-bushido
Copy link
Copy Markdown
Contributor

@coder-with-a-bushido coder-with-a-bushido commented Apr 1, 2026

Another attempt since it cannot be reproduced locally

Also fixes CI green even when tests fail like in https://github.com/famedly/matrix-dart-sdk/actions/runs/23843600999/job/69505225066?pr=2307

image image

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.00%. Comparing base (19381cf) to head (5329175).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2311      +/-   ##
==========================================
+ Coverage   57.19%   58.00%   +0.81%     
==========================================
  Files         160      160              
  Lines       19795    19795              
==========================================
+ Hits        11321    11482     +161     
+ Misses       8474     8313     -161     
Files with missing lines Coverage Δ
lib/src/utils/device_keys_list.dart 93.20% <100.00%> (-0.40%) ⬇️

... and 10 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 57b883c...5329175. Read the comment docs.

@td-famedly td-famedly marked this pull request as ready for review April 2, 2026 09:03
@td-famedly td-famedly enabled auto-merge April 2, 2026 09:03
Comment thread test/device_keys_list_test.dart
@td-famedly td-famedly disabled auto-merge April 2, 2026 09:04
@td-famedly td-famedly enabled auto-merge April 2, 2026 09:04
@coder-with-a-bushido coder-with-a-bushido force-pushed the karthi/flaky-test-attempt-2 branch from 5329175 to 0426051 Compare April 2, 2026 09:20
@coder-with-a-bushido coder-with-a-bushido changed the title fix: tests getting stuck randomly in CI fix: tests getting stuck randomly in CI + CI passes even with failed tests Apr 2, 2026
@coder-with-a-bushido coder-with-a-bushido force-pushed the karthi/flaky-test-attempt-2 branch from 92192c3 to ffebaf9 Compare April 16, 2026 17:12
@coder-with-a-bushido coder-with-a-bushido changed the title fix: tests getting stuck randomly in CI + CI passes even with failed tests Fix flaky tests in CI + failing tests + issue on CI passing with failed tests Apr 16, 2026
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.

2 participants