Tweaks for transaction-level fuzzing (#2744)

Tweak register blockhash for tests + add optional feature set parameter into bank for fuzzing

ledger/src/blockstore_processor.rs

@@ -865,6 +865,7 @@ pub(crate) fn process_blockstore_for_bank_0(
         None,
         exit,
         None,
+        None,
     );
     let bank0_slot = bank0.slot();
     let bank_forks = BankForks::new_rw_arc(bank0);

program-test/src/lib.rs

@@ -867,6 +867,7 @@ impl ProgramTest {
             None,
             Arc::default(),
             None,
+            None,
         );
 
         // Add commonly-used SPL programs as a convenience to the user

runtime/src/bank.rs

@@ -987,6 +987,7 @@ impl Bank {
         #[allow(unused)] collector_id_for_tests: Option<Pubkey>,
         exit: Arc<AtomicBool>,
         #[allow(unused)] genesis_hash: Option<Hash>,
+        #[allow(unused)] feature_set: Option<FeatureSet>,
     ) -> Self {
         let accounts_db = AccountsDb::new_with_config(
             paths,
@@ -1005,6 +1006,11 @@ impl Bank {
         bank.transaction_debug_keys = debug_keys;
         bank.cluster_type = Some(genesis_config.cluster_type);
 
+        #[cfg(feature = "dev-context-only-utils")]
+        {
+            bank.feature_set = Arc::new(feature_set.unwrap_or_default());
+        }
+
         #[cfg(not(feature = "dev-context-only-utils"))]
         bank.process_genesis_config(genesis_config);
         #[cfg(feature = "dev-context-only-utils")]
@@ -3157,7 +3163,6 @@ impl Bank {
             w_blockhash_queue
                 .register_hash(blockhash, self.fee_rate_governor.lamports_per_signature);
         }
-        self.update_recent_blockhashes_locked(&w_blockhash_queue);
     }
 
     /// Tell the bank which Entry IDs exist on the ledger. This function assumes subsequent calls
@@ -6631,6 +6636,7 @@ impl Bank {
             Some(Pubkey::new_unique()),
             Arc::default(),
             None,
+            None,
         )
     }
 
@@ -6655,6 +6661,7 @@ impl Bank {
             Some(Pubkey::new_unique()),
             Arc::default(),
             None,
+            None,
         )
     }
 

runtime/src/bank/partitioned_epoch_rewards/mod.rs

@@ -380,6 +380,7 @@ mod tests {
             Some(Pubkey::new_unique()),
             Arc::default(),
             None,
+            None,
         );
 
         // Fill bank_forks with banks with votes landing in the next slot
@@ -488,6 +489,7 @@ mod tests {
             Some(Pubkey::new_unique()),
             Arc::default(),
             None,
+            None,
         );
 
         let stake_account_stores_per_block =

runtime/src/bank/tests.rs

@@ -9010,6 +9010,7 @@ fn test_epoch_schedule_from_genesis_config() {
         None,
         Arc::default(),
         None,
+        None,
     ));
 
     assert_eq!(bank.epoch_schedule(), &genesis_config.epoch_schedule);
@@ -9041,6 +9042,7 @@ where
         None,
         Arc::default(),
         None,
+        None,
     ));
     let vote_and_stake_accounts =
         load_vote_and_stake_accounts(&bank).vote_with_stake_delegations_map;
@@ -12645,6 +12647,7 @@ fn test_rehash_with_skipped_rewrites() {
         Some(Pubkey::new_unique()),
         Arc::new(AtomicBool::new(false)),
         None,
+        None,
     ));
     // This test is only meaningful while the bank hash contains rewrites.
     // Once this feature is enabled, it may be possible to remove this test entirely.
@@ -12707,6 +12710,7 @@ fn test_rebuild_skipped_rewrites() {
         Some(Pubkey::new_unique()),
         Arc::new(AtomicBool::new(false)),
         None,
+        None,
     ));
     // This test is only meaningful while the bank hash contains rewrites.
     // Once this feature is enabled, it may be possible to remove this test entirely.
@@ -12818,6 +12822,7 @@ fn test_get_accounts_for_bank_hash_details(skip_rewrites: bool) {
         Some(Pubkey::new_unique()),
         Arc::new(AtomicBool::new(false)),
         None,
+        None,
     ));
     // This test is only meaningful while the bank hash contains rewrites.
     // Once this feature is enabled, it may be possible to remove this test entirely.