Skip to content

test(monero-rpc-pool): add database and status coverage#1000

Draft
copp1723 wants to merge 7 commits intoeigenwallet:masterfrom
copp1723:codex/801-rpc-pool-tests
Draft

test(monero-rpc-pool): add database and status coverage#1000
copp1723 wants to merge 7 commits intoeigenwallet:masterfrom
copp1723:codex/801-rpc-pool-tests

Conversation

@copp1723
Copy link
Copy Markdown

@copp1723 copp1723 commented May 11, 2026

Refs #801

Summary

  • Add unit coverage for node address formatting, node record delegation, and health success-rate math.
  • Add database tests for migrations, default node seeding, health-check aggregation, reliable node stats, and unknown-node handling.
  • Add pool/app tests for current status reporting, status broadcasts, bandwidth sample gating, and initial app status publication.
  • Add proxy tests for cloneable request/response buffering, JSON-RPC method/error extraction, clearnet whitelist behavior, and handler error JSON.
  • Add tempfile as a dev dependency for isolated SQLite database directories.

Why

monero-rpc-pool had no library tests. These cover the core database, status, app startup, and proxy helper behavior without requiring live Monero nodes, Tor, or external services.

Verification

  • RUSTUP_TOOLCHAIN=stable cargo fmt --check --manifest-path monero-rpc-pool/Cargo.toml
  • git diff --check
  • CARGO_HOME=/private/tmp/cargo-home RUSTUP_TOOLCHAIN=stable cargo test --manifest-path monero-rpc-pool/Cargo.toml --lib (15 passed)

@Einliterflasche
Copy link
Copy Markdown

Per our AI policy we require all communication to be exclusively human written. Until this requirement is met, we can't spare the effort to review this.

@Einliterflasche Einliterflasche marked this pull request as draft May 11, 2026 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants