Add protocol support to staker and validator test helps.

Add tests for legacy add_stake.

Author: ii-cruz

blockchain/src/blockchain/history_sync.rs

@@ -208,11 +208,9 @@ impl Blockchain {
                 // Also, this works only if a single macro block is missing between history items.
                 let batch_number = Policy::batch_at(hist_tx.block_number);
                 if batch_number > prev_batch
-                    && block_state
-                        .last()
-                        .map_or(false, |block_state: &BlockState| {
-                            !Policy::is_macro_block_at(block_state.number)
-                        })
+                    && block_state.last().is_some_and(|block_state: &BlockState| {
+                        !Policy::is_macro_block_at(block_state.number)
+                    })
                 {
                     debug!(
                         history_item_block_number = hist_tx.block_number,

primitives/account/src/account/staking_contract/staker.rs

@@ -290,6 +290,7 @@ impl StakingContract {
         tx_logger: &mut TransactionLog,
     ) -> Result<AddStakeReceipt, AccountError> {
         if protocol_version < Policy::ADD_STAKE_PROTOCOL_UPGRADE_VERSION {
+            log::warn!(%protocol_version, "Adding stake using old protocol version");
             return self.legacy_add_stake(store, staker_address, value, tx_logger);
         }
         // Get the staker.