Skip to content

Comments

Add infra for RPC testing #746

Merged
Rigidity merged 7 commits intomainfrom
rpc-tests
Feb 19, 2026
Merged

Add infra for RPC testing #746
Rigidity merged 7 commits intomainfrom
rpc-tests

Conversation

@Rigidity
Copy link
Collaborator

@Rigidity Rigidity commented Feb 17, 2026

Note

Medium Risk
Moderate risk because it changes core wallet initialization/logging behavior (global tracing setup, test-mode toggles) and extends sync manager commands to accept injected peers, which could affect runtime behavior if misused.

Overview
Adds an in-process RPC test harness for sage-rpc by exposing make_router(...), introducing a PeerSimulator-backed tests.rs, and adding the needed dev-dependencies.

Refactors RPC TLS setup by extracting WalletCertVerifier and load_rustls_config into dedicated modules, and updates Sage::new to accept a test flag that disables file logging, avoids re-initializing the global tracing subscriber, and configures faster sync timeouts for tests.

Extends sage-wallet sync control with a new SyncCommand::AddPeer to inject a pre-connected peer/receiver (used by tests), updates CLI/Tauri call sites for the new Sage::new(..., test) signature, and includes minor CI (print Prettier version) plus small NFT UI formatting tweaks.

Written by Cursor Bugbot for commit c0cc2ae. This will update automatically on new commits. Configure here.

@Rigidity
Copy link
Collaborator Author

Thanks to #745 for the general design idea

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@Rigidity Rigidity merged commit 6fd4105 into main Feb 19, 2026
8 checks passed
@Rigidity Rigidity deleted the rpc-tests branch February 19, 2026 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant