Abstract away persistence and cache

[dr-orlovsky]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 11.15385% with 231 lines in your changes missing coverage. Please review.

Project coverage is 16.4%. Comparing base (a98b19b) to head (79056c9).

Files with missing lines	Patch %	Lines
persistence/mem/src/cache.rs	0.0%	137 Missing ⚠️
src/wallet.rs	0.0%	82 Missing ⚠️
src/data.rs	70.7%	12 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff            @@
##           master     #92      +/-   ##
=========================================
+ Coverage     4.7%   16.4%   +11.7%     
=========================================
  Files          24       5      -19     
  Lines        2341     671    -1670     
=========================================
  Hits          110     110              
+ Misses       2231     561    -1670     

Flag	Coverage Δ
rust	16.4% <11.2%> (+11.7%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[Crayon-Shin-chan-bitlightlabs]

@dr-orlovsky Thank you for the effort you’ve put into this PR.

I think it would be best to abstract register_psbt as well, as a default method in the trait. That way, we can reuse its implementation without needing to worry about details like WalletTx and other internal constructs.

[dr-orlovsky]

I think it would be best to abstract register_psbt as well, as a default method in the trait. That way, we can reuse its implementation without needing to worry about details like WalletTx and other internal constructs.

Done. Also did further simplification and enhancements