Skip to content

test: cover recheckDeviceStatus catch branch#730

Merged
TaprootFreak merged 1 commit into
stagingfrom
fix/coverage-recheckdevicestatus-catch
Jun 10, 2026
Merged

test: cover recheckDeviceStatus catch branch#730
TaprootFreak merged 1 commit into
stagingfrom
fix/coverage-recheckdevicestatus-catch

Conversation

@TaprootFreak

Copy link
Copy Markdown
Contributor

Summary

  • Adds one missing test for the catch block in recheckDeviceStatus() (lines 242–245 in connect_bitbox_cubit.dart)
  • The scenario: device was uninitialized on first pair → BitboxNotInitialized; user sets up wallet; recheckDeviceStatus is called but createBitboxWallet throws → cubit must fall back to BitboxNotConnected
  • Fixes Coverage Floor Gate failing at 99.9% (4924/4928 lines) in PR Promote: staging -> develop #728

Test plan

  • flutter test test/screens/hardware_connect_bitbox/bloc/connect_bitbox_cubit_test.dart passes
  • Coverage Floor Gate reaches 100%

@TaprootFreak TaprootFreak marked this pull request as ready for review June 10, 2026 08:03
@TaprootFreak TaprootFreak merged commit 8c08820 into staging Jun 10, 2026
11 checks passed
@TaprootFreak TaprootFreak deleted the fix/coverage-recheckdevicestatus-catch branch June 10, 2026 08: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.

1 participant