fix: return slotsealed when dump omits pubkey

[reez]

Description

Dumping the active (sealed) slot without a CVC causes the firmware to omit pubkey.

Our iOS app calls dump during scans; the generated Rust bindings assumed pubkey was always present and did unwrap(), which panicked and crashed the iOS app.

This change replaces the unwrap() with an ok_or(DumpError::SlotSealed) so callers receive a proper DumpError instead of aborting the process.

Also introduced #![allow(deprecated)] inside the emulator test module to silence the bdk_wallet::SignOptions deprecation because it was failing on CI because of that

Notes to the reviewers

Crash on iOS app found by @RockRockX2 reez/SatsBuddy#37

Changelog notice

Checklists

All Submissions:

• I've signed all my commits
• I followed the contribution guidelines
• I ran cargo fmt and cargo clippy before committing

New Features:

• I've added tests for the new feature
• I've added docs for the new feature

Bugfixes:

• This pull request breaks the existing API
• I've added tests to reproduce the issue which are now passing
• I'm linking the issue being fixed by this PR

[notmandatory]

ACK 1d3561d

Thanks for the fix! Should I cut a new release or can you keep testing from the master branch incase there's something else to fix?

[reez]

yeah I can keep testing from the master branch 👍