exit miner thread behavior for late blockfound tenures, add tests to CI

Author: kantai

.github/workflows/bitcoin-tests.yml

@@ -143,6 +143,8 @@ jobs:
           - tests::signer::v0::outgoing_signers_ignore_block_proposals
           - tests::signer::v0::injected_signatures_are_ignored_across_boundaries
           - tests::signer::v0::fast_sortition
+          - tests::signer::v0::single_miner_empty_sortition
+          - tests::signer::v0::multiple_miners_empty_sortition
           - tests::nakamoto_integrations::burn_ops_integration_test
           - tests::nakamoto_integrations::check_block_heights
           - tests::nakamoto_integrations::clarity_burn_state

testnet/stacks-node/src/nakamoto_node/miner.rs

@@ -138,6 +138,15 @@ pub enum MinerReason {
     },
 }
 
+impl MinerReason {
+    pub fn is_late_block(&self) -> bool {
+        match self {
+            Self::BlockFound { ref late } => *late,
+            Self::Extended { .. } => false,
+        }
+    }
+}
+
 impl std::fmt::Display for MinerReason {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         match self {
@@ -434,6 +443,11 @@ impl BlockMinerThread {
                     "Failed to open chainstate DB. Cannot mine! {e:?}"
                 ))
             })?;
+        if self.last_block_mined.is_some() && self.reason.is_late_block() {
+            info!("Miner: finished mining a late tenure");
+            return Err(NakamotoNodeError::StacksTipChanged);
+        }
+
         let new_block = loop {
             // If we're mock mining, we may not have processed the block that the
             // actual tenure winner committed to yet. So, before attempting to