Skip to content

Conversation

@pinebit
Copy link
Collaborator

@pinebit pinebit commented Jan 29, 2026

This fixes a bug in restoreKey routine that affects "remove operators" edit command.

category: bug
ticket: none

Copy link
Collaborator

@KaloyanTanev KaloyanTanev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be possible to have a test case to catch this scenario?

@pinebit
Copy link
Collaborator Author

pinebit commented Jan 29, 2026

Would it be possible to have a test case to catch this scenario?

I was thinking about this, but let's wait for the full report and I suspect we may need to rework a lot more, including more of the tests...

@pinebit pinebit force-pushed the pinebit/restore-key-fix branch from 715e7ec to d9d0b24 Compare January 29, 2026 13:23
@sonarqubecloud
Copy link

@codecov
Copy link

codecov bot commented Jan 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.57%. Comparing base (33fe701) to head (d9d0b24).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4261      +/-   ##
==========================================
- Coverage   56.61%   56.57%   -0.04%     
==========================================
  Files         237      237              
  Lines       31329    31332       +3     
==========================================
- Hits        17736    17727       -9     
- Misses      11320    11334      +14     
+ Partials     2273     2271       -2     

☔ 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.

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.

3 participants