refactor(mock-consensus): 🔐 remove Keyring

previously, we added a `Keyring` newtype. this was a low-effort, thin
newtype wrapper around a `BTreeMap<VerificationKey, SigningKey>`.

this commit removes it.

NB: this is based on #4001.

crates/test/mock-consensus/src/builder.rs

@@ -6,15 +6,17 @@
 mod init_chain;
 
 use {
-    crate::{keyring::Keyring, TestNode},
+    crate::TestNode,
     bytes::Bytes,
+    ed25519_consensus::{SigningKey, VerificationKey},
+    std::collections::BTreeMap,
 };
 
 /// A buider, used to prepare and instantiate a new [`TestNode`].
 #[derive(Default)]
 pub struct Builder {
     pub app_state: Option<Bytes>,
-    pub keyring: Keyring,
+    pub keyring: BTreeMap<VerificationKey, SigningKey>,
 }
 
 impl TestNode<()> {
@@ -57,9 +59,15 @@ impl Builder {
             );
         }
 
-        Self {
-            keyring: Keyring::new_with_size(1),
-            ..self
-        }
+        // Generate a consensus key.
+        let sk = ed25519_consensus::SigningKey::new(rand_core::OsRng);
+        let vk = sk.verification_key();
+        tracing::trace!(verification_key = ?vk, "generated consensus key");
+
+        // Place it into the keyring.
+        let mut keyring = BTreeMap::new();
+        keyring.insert(vk, sk);
+
+        Self { keyring, ..self }
     }
 }

crates/test/mock-consensus/src/lib.rs

@@ -4,9 +4,13 @@
 //
 //  see penumbra-zone/penumbra#3588.
 
+use {
+    ed25519_consensus::{SigningKey, VerificationKey},
+    std::collections::BTreeMap,
+};
+
 pub mod block;
 pub mod builder;
-pub mod keyring;
 
 mod abci;
 
@@ -23,7 +27,7 @@ pub struct TestNode<C> {
     consensus: C,
     last_app_hash: Vec<u8>,
     height: tendermint::block::Height,
-    keyring: self::keyring::Keyring,
+    keyring: BTreeMap<VerificationKey, SigningKey>,
 }
 
 impl<C> TestNode<C> {