chore: Drop pointless utxo_tests.rs

[xdustinface]

It only tests the UTXOCollection defined within the mock module inside the file which is not used anywhere else.

Summary by CodeRabbit

• Tests
  • Removed UTXO-related test suite to streamline test infrastructure.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

Removes the utxo_tests module declaration from the test module registry and deletes the entire utxo_tests.rs file containing UTXO unit tests and mock framework. Eliminates test coverage for UTXO creation, spending, balance calculations, selection, and coinbase handling without changes to functional code.

Changes

Cohort / File(s)	Summary
Test module cleanup key-wallet/src/tests/mod.rs, key-wallet/src/tests/utxo_tests.rs	Removed utxo_tests module declaration; deleted entire test file containing UTXO lifecycle tests, mock Utxo/UTXOCollection structs, and helper functions for test UTXO construction

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Tests removed with a hop and a bound,
UTXO mocks no longer around,
The module's declared, but the code's been tossed,
A cleanup complete—though some coverage is lost! 🗑️

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and specifically describes the main change: removing the utxo_tests.rs file. It uses a clear commit-style format and directly references what is being removed.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cd12793 and 2858e3f.

📒 Files selected for processing (2)

• key-wallet/src/tests/mod.rs
• key-wallet/src/tests/utxo_tests.rs

💤 Files with no reviewable changes (2)

• key-wallet/src/tests/mod.rs
• key-wallet/src/tests/utxo_tests.rs

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (20)

• GitHub Check: Ubuntu ARM / wallet
• GitHub Check: Documentation
• GitHub Check: Pre-commit (windows-latest)
• GitHub Check: Pre-commit (ubuntu-latest)
• GitHub Check: fuzz (dash_deser_net_msg)
• GitHub Check: fuzz (hashes_sha256)
• GitHub Check: fuzz (dash_deserialize_block)
• GitHub Check: fuzz (hashes_cbor)
• GitHub Check: fuzz (dash_outpoint_string)
• GitHub Check: fuzz (dash_deserialize_address)
• GitHub Check: fuzz (hashes_json)
• GitHub Check: fuzz (dash_deserialize_witness)
• GitHub Check: fuzz (dash_script_bytes_to_asm_fmt)
• GitHub Check: fuzz (hashes_sha512_256)
• GitHub Check: Pre-commit (macos-latest)
• GitHub Check: fuzz (hashes_sha512)
• GitHub Check: fuzz (dash_deserialize_amount)
• GitHub Check: fuzz (dash_deserialize_script)
• GitHub Check: fuzz (hashes_sha1)
• GitHub Check: fuzz (hashes_ripemd160)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}