Merge pull request #5103 from stacks-network/hotfix/mock-signature-message-slot

Reuse BlockResponse slot for MockSignature message type

Author: wileyj

libsigner/src/v0/messages.rs

@@ -185,8 +185,7 @@ impl SignerMessage {
     pub fn msg_id(&self) -> Option<MessageSlotID> {
         match self {
             Self::BlockProposal(_) | Self::BlockPushed(_) => None,
-            Self::BlockResponse(_) => Some(MessageSlotID::BlockResponse),
-            Self::MockSignature(_) => Some(MessageSlotID::MockSignature),
+            Self::BlockResponse(_) | Self::MockSignature(_) => Some(MessageSlotID::BlockResponse), // Mock signature reuses the same slot as block response since its exclusively used in Epoch 2.5
         }
     }
 }

testnet/stacks-node/src/tests/signer/v0.rs

@@ -2212,8 +2212,8 @@ fn mock_sign_epoch_25() {
             std::thread::sleep(Duration::from_millis(100));
             let messages: Vec<SignerMessage> = StackerDB::get_messages(
                 stackerdb
-                    .get_session_mut(&MessageSlotID::MockSignature)
-                    .expect("Failed to get BlockResponse stackerdb session"),
+                    .get_session_mut(&MessageSlotID::BlockResponse)
+                    .expect("Failed to get BlockResponse stackerdb session"), // Epoch 2.5 MockSignatures use the BlockResponse slot
                 &signer_slot_ids,
             )
             .expect("Failed to get message from stackerdb");