Skip to content

fix: avoid field spillover in parsing DS API responses#1191

Merged
keynslug merged 1 commit intomain-3.xfrom
fix/ds/parse-spillover
Apr 24, 2026
Merged

fix: avoid field spillover in parsing DS API responses#1191
keynslug merged 1 commit intomain-3.xfrom
fix/ds/parse-spillover

Conversation

@keynslug
Copy link
Copy Markdown
Contributor

@keynslug keynslug commented Apr 24, 2026

Summary

This PR fixes a bug in GetDSReplicationStatus that was causing transitions field context to spill over to next items in DBs array.

Port of #1190's commit.

This commit fixes a bug in `GetDSReplicationStatus` that was
causing `transitions` field context to spill over to next items
in `DBs` array.
@keynslug keynslug marked this pull request as ready for review April 24, 2026 15:25
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.86%. Comparing base (18e0cda) to head (e1b644c).
⚠️ Report is 5 commits behind head on main-3.x.

Additional details and impacted files
@@             Coverage Diff              @@
##           main-3.x    #1191      +/-   ##
============================================
+ Coverage     77.33%   77.86%   +0.53%     
============================================
  Files            48       48              
  Lines          3948     3917      -31     
============================================
- Hits           3053     3050       -3     
+ Misses          742      722      -20     
+ Partials        153      145       -8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@keynslug keynslug merged commit aeac026 into main-3.x Apr 24, 2026
16 checks passed
@keynslug keynslug deleted the fix/ds/parse-spillover branch April 24, 2026 21:47
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