Fix Ledger Tests to be Feature-Gated #32
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix Ledger Tests to be Feature-Gated
This PR fixes the CI failure in PR #31 by properly feature-gating the Ledger wallet tests. The issue was that the CI workflow runs tests with
--no-default-features
, but the Ledger implementation requires theledger
feature to be enabled.Changes
#[cfg(feature = "ledger")]
to the Ledger tests moduleThis ensures that the tests are only compiled and run when the
ledger
feature is enabled, preventing compilation errors when running with--no-default-features
.Link to Devin run: https://app.devin.ai/sessions/bc2a0d70f17f44518478567e1d0f8730
Requested by: Jimmy