Skip to content

fix: missing pending messages feature flag check (WPB-28301) - #5216

Open
sbakhtiarov wants to merge 1 commit into
developfrom
fix/pending-messages-crash
Open

fix: missing pending messages feature flag check (WPB-28301)#5216
sbakhtiarov wants to merge 1 commit into
developfrom
fix/pending-messages-crash

Conversation

@sbakhtiarov

@sbakhtiarov sbakhtiarov commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

https://wearezeta.atlassian.net/browse/WPB-28301

https://wearezeta.atlassian.net/browse/WPB-28301

What's new in this PR?

Checking pending messages feature flag to prevent resending messages when feature is disabled.

@codecov

codecov Bot commented Aug 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.45%. Comparing base (18b1028) to head (a64949e).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #5216   +/-   ##
========================================
  Coverage    52.45%   52.45%           
========================================
  Files          739      739           
  Lines        25702    25704    +2     
  Branches      4215     4216    +1     
========================================
+ Hits         13481    13483    +2     
  Misses       10933    10933           
  Partials      1288     1288           
Files with missing lines Coverage Δ
.../kotlin/com/wire/android/GlobalObserversManager.kt 92.30% <100.00%> (+0.20%) ⬆️

Continue to review full report in Codecov by Harness.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions

Copy link
Copy Markdown
Contributor

Test Results

2 621 tests  +1   2 621 ✅ +1   1m 23s ⏱️ -2s
  333 suites ±0       0 💤 ±0 
  333 files   ±0       0 ❌ ±0 

Results for commit a64949e. ± Comparison against base commit 18b1028.

@github-actions

Copy link
Copy Markdown
Contributor

APKs built during tests are available here. Scroll down to Artifacts!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant