Added unit tests for watch-only and refactored the tests in floresta-chain and wire as suggested.

Updated cache_transaction in floresta-watch-only/src/lib.rs for a potential bug. In this function as we cache a new address in the db, the components of the CacheAddress should be empty.

If not, then the same components: balance and utxos related to that script_hash gets pushed again in save_non_mempool_tx, resulting in the balance associated with that sript_hash beccoming twice as much.

@Davidson-Souza, please approval the workflows to run, so I can see if they are failing.

@lla-dane done!

@Davidson-Souza, please approve once more, clippy won't bug this time, probably.

Finally, all the checks passed, @Davidson-Souza please review it, and suggest if I have to change something.

@lla-dane you can run

$ cargo +nightly clippy --all-targets

the same command that actions run in the virtual machine, you can check this at the actions details.

on your local project, with this you have the same feedback than actions without the need to push it to verify.
Nice work that youre doing!

@lla-dane you can run

$ cargo +nightly clippy --all-targets

the same command that actions run in the virtual machine, you can check this at the actions details.

on your local project, with this you have the same feedback than actions without the need to push it to verify. Nice work that youre doing!

yeah that's the thing!, ran that command on my end many times, didn't gave any errors, even ran all the CI checks on my fork too, for some reason they were passing on my fork, but failing on the PR by compilation errors, it was really weird :|

@lla-dane could you please squash some commits? like 15d6c13 can be squashed on 1a5750d and everything after 5abb90c could be squashed on this commit

All commits are squashed into one. I guess some commits included in some previous PRs which got included when I synced my fork, also got squashed in this,

It is showing diff for stuff that changed on master, if you rebase that will go away.

@Davidson-Souza is this PR ready for merge ?

I've left a few minor comments, shouldn't take more than a few minutes to fix. Then I think it's ready to merge.

I've left a few minor comments, shouldn't take more than a few minutes to fix. Then I think it's ready to merge.

Its not showing any unresolved comments here, those that were there before are fixed already I think. Are there any new ones?

Its not showing any unresolved comments here, those that were there before are fixed already I think. Are there any new ones?

Sorry, I forgot to submit

@Davidson-Souza done!