diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml index db681182..ab203e68 100644 --- a/.github/workflows/main.yml +++ b/.github/workflows/main.yml @@ -2,7 +2,7 @@ name: Main on: push: - branches: [main] + branches: [main,mollusk-tests-5-withdraw] pull_request: env: diff --git a/program/tests/deactivate.rs b/program/tests/deactivate.rs new file mode 100644 index 00000000..980edaab --- /dev/null +++ b/program/tests/deactivate.rs @@ -0,0 +1,115 @@ +#![allow(clippy::arithmetic_side_effects)] + +mod helpers; + +use { + helpers::{ + context::StakeTestContext, + instruction_builders::{DeactivateConfig, DelegateConfig}, + lifecycle::StakeLifecycle, + utils::parse_stake_account, + }, + mollusk_svm::result::Check, + solana_program_error::ProgramError, + solana_stake_interface::{error::StakeError, state::StakeStateV2}, + solana_stake_program::id, + test_case::test_case, +}; + +#[test_case(false; "activating")] +#[test_case(true; "active")] +fn test_deactivate(activate: bool) { + let mut ctx = StakeTestContext::with_delegation(); + let min_delegation = ctx.minimum_delegation.unwrap(); + + let (stake, mut stake_account) = ctx + .stake_account(StakeLifecycle::Initialized) + .staked_amount(min_delegation) + .build(); + + // Deactivating an undelegated account fails + ctx.process_with(DeactivateConfig { + stake: (&stake, &stake_account), + override_signer: None, + }) + .checks(&[Check::err(ProgramError::InvalidAccountData)]) + .test_missing_signers(false) + .execute(); + + // Delegate + let result = ctx + .process_with(DelegateConfig { + stake: (&stake, &stake_account), + vote: ( + ctx.vote_account.as_ref().unwrap(), + ctx.vote_account_data.as_ref().unwrap(), + ), + }) + .execute(); + stake_account = result.resulting_accounts[0].1.clone().into(); + + if activate { + // Advance epoch to activate + let current_slot = ctx.mollusk.sysvars.clock.slot; + let slots_per_epoch = ctx.mollusk.sysvars.epoch_schedule.slots_per_epoch; + ctx.mollusk.warp_to_slot(current_slot + slots_per_epoch); + } + + // Deactivate with withdrawer fails + ctx.process_with(DeactivateConfig { + stake: (&stake, &stake_account), + override_signer: Some(&ctx.withdrawer), + }) + .checks(&[Check::err(ProgramError::MissingRequiredSignature)]) + .test_missing_signers(false) + .execute(); + + // Deactivate succeeds + let result = ctx + .process_with(DeactivateConfig { + stake: (&stake, &stake_account), + override_signer: None, + }) + .checks(&[ + Check::success(), + Check::all_rent_exempt(), + Check::account(&stake) + .lamports(ctx.rent_exempt_reserve + min_delegation) + .owner(&id()) + .space(StakeStateV2::size_of()) + .build(), + ]) + .test_missing_signers(true) + .execute(); + stake_account = result.resulting_accounts[0].1.clone().into(); + + let clock = ctx.mollusk.sysvars.clock.clone(); + let (_, stake_data, _) = parse_stake_account(&stake_account); + assert_eq!( + stake_data.unwrap().delegation.deactivation_epoch, + clock.epoch + ); + + // Deactivate again fails + ctx.process_with(DeactivateConfig { + stake: (&stake, &stake_account), + override_signer: None, + }) + .checks(&[Check::err(StakeError::AlreadyDeactivated.into())]) + .test_missing_signers(false) + .execute(); + + // Advance epoch + let current_slot = ctx.mollusk.sysvars.clock.slot; + let slots_per_epoch = ctx.mollusk.sysvars.epoch_schedule.slots_per_epoch; + ctx.mollusk.warp_to_slot(current_slot + slots_per_epoch); + + // Deactivate again still fails + ctx.process_with(DeactivateConfig { + stake: (&stake, &stake_account), + override_signer: None, + }) + .checks(&[Check::err(StakeError::AlreadyDeactivated.into())]) + .test_missing_signers(false) + .execute(); +} diff --git a/program/tests/delegate.rs b/program/tests/delegate.rs new file mode 100644 index 00000000..eb65b5dd --- /dev/null +++ b/program/tests/delegate.rs @@ -0,0 +1,214 @@ +#![allow(clippy::arithmetic_side_effects)] + +mod helpers; + +use { + helpers::{ + context::StakeTestContext, + instruction_builders::{DeactivateConfig, DelegateConfig}, + lifecycle::StakeLifecycle, + stake_tracker::MolluskStakeExt, + utils::{create_vote_account, increment_vote_account_credits, parse_stake_account}, + }, + mollusk_svm::result::Check, + solana_account::{AccountSharedData, WritableAccount}, + solana_program_error::ProgramError, + solana_pubkey::Pubkey, + solana_stake_interface::{ + error::StakeError, + state::{Delegation, Stake, StakeStateV2}, + }, + solana_stake_program::id, +}; + +#[test] +fn test_delegate() { + let mut ctx = StakeTestContext::with_delegation(); + let vote_account = *ctx.vote_account.as_ref().unwrap(); + let mut vote_account_data = ctx.vote_account_data.as_ref().unwrap().clone(); + let min_delegation = ctx.minimum_delegation.unwrap(); + + let vote_state_credits = 100u64; + increment_vote_account_credits(&mut vote_account_data, 0, vote_state_credits); + + let (stake, mut stake_account) = ctx + .stake_account(StakeLifecycle::Initialized) + .staked_amount(min_delegation) + .build(); + + // Delegate stake + let result = ctx + .process_with(DelegateConfig { + stake: (&stake, &stake_account), + vote: (&vote_account, &vote_account_data), + }) + .checks(&[ + Check::success(), + Check::all_rent_exempt(), + Check::account(&stake) + .lamports(ctx.rent_exempt_reserve + min_delegation) + .owner(&id()) + .space(StakeStateV2::size_of()) + .build(), + ]) + .test_missing_signers(true) + .execute(); + stake_account = result.resulting_accounts[0].1.clone().into(); + + // Verify that delegate() looks right + let clock = ctx.mollusk.sysvars.clock.clone(); + let (_, stake_data, _) = parse_stake_account(&stake_account); + assert_eq!( + stake_data.unwrap(), + Stake { + delegation: Delegation { + voter_pubkey: vote_account, + stake: min_delegation, + activation_epoch: clock.epoch, + deactivation_epoch: u64::MAX, + ..Delegation::default() + }, + credits_observed: vote_state_credits, + } + ); + + // Advance epoch to activate the stake + let activation_epoch = ctx.mollusk.sysvars.clock.epoch; + ctx.tracker.as_mut().unwrap().track_delegation( + &stake, + min_delegation, + activation_epoch, + &vote_account, + ); + + let slots_per_epoch = ctx.mollusk.sysvars.epoch_schedule.slots_per_epoch; + let current_slot = ctx.mollusk.sysvars.clock.slot; + ctx.mollusk.warp_to_slot_with_stake_tracking( + ctx.tracker.as_ref().unwrap(), + current_slot + slots_per_epoch, + Some(0), + ); + + // Verify that delegate fails as stake is active and not deactivating + ctx.process_with(DelegateConfig { + stake: (&stake, &stake_account), + vote: (&vote_account, ctx.vote_account_data.as_ref().unwrap()), + }) + .checks(&[Check::err(StakeError::TooSoonToRedelegate.into())]) + .test_missing_signers(false) + .execute(); + + // Deactivate + let result = ctx + .process_with(DeactivateConfig { + stake: (&stake, &stake_account), + override_signer: None, + }) + .execute(); + stake_account = result.resulting_accounts[0].1.clone().into(); + + // Create second vote account + let (vote_account2, vote_account2_data) = ctx.create_second_vote_account(); + + // Verify that delegate to a different vote account fails during deactivation + ctx.process_with(DelegateConfig { + stake: (&stake, &stake_account), + vote: (&vote_account2, &vote_account2_data), + }) + .checks(&[Check::err(StakeError::TooSoonToRedelegate.into())]) + .test_missing_signers(false) + .execute(); + + // Verify that delegate succeeds to same vote account when stake is deactivating + let result = ctx + .process_with(DelegateConfig { + stake: (&stake, &stake_account), + vote: (&vote_account, ctx.vote_account_data.as_ref().unwrap()), + }) + .execute(); + stake_account = result.resulting_accounts[0].1.clone().into(); + + // Verify that deactivation has been cleared + let (_, stake_data, _) = parse_stake_account(&stake_account); + assert_eq!(stake_data.unwrap().delegation.deactivation_epoch, u64::MAX); + + // Verify that delegate to a different vote account fails if stake is still active + ctx.process_with(DelegateConfig { + stake: (&stake, &stake_account), + vote: (&vote_account2, &vote_account2_data), + }) + .checks(&[Check::err(StakeError::TooSoonToRedelegate.into())]) + .test_missing_signers(false) + .execute(); + + // Advance epoch again using tracker + let current_slot = ctx.mollusk.sysvars.clock.slot; + let slots_per_epoch = ctx.mollusk.sysvars.epoch_schedule.slots_per_epoch; + ctx.mollusk.warp_to_slot_with_stake_tracking( + ctx.tracker.as_ref().unwrap(), + current_slot + slots_per_epoch, + Some(0), + ); + + // Delegate still fails after stake is fully activated; redelegate is not supported + let (vote_account2, vote_account2_data) = ctx.create_second_vote_account(); + + ctx.process_with(DelegateConfig { + stake: (&stake, &stake_account), + vote: (&vote_account2, &vote_account2_data), + }) + .checks(&[Check::err(StakeError::TooSoonToRedelegate.into())]) + .test_missing_signers(false) + .execute(); +} + +#[test] +fn test_delegate_fake_vote_account() { + let mut ctx = StakeTestContext::with_delegation(); + + // Create fake vote account (not owned by vote program) + let fake_vote_account = Pubkey::new_unique(); + let mut fake_vote_data = create_vote_account(); + fake_vote_data.set_owner(Pubkey::new_unique()); // Wrong owner + + let min_delegation = ctx.minimum_delegation.unwrap(); + let (stake, stake_account) = ctx + .stake_account(StakeLifecycle::Initialized) + .staked_amount(min_delegation) + .build(); + + // Try to delegate to fake vote account + ctx.process_with(DelegateConfig { + stake: (&stake, &stake_account), + vote: (&fake_vote_account, &fake_vote_data), + }) + .checks(&[Check::err(ProgramError::IncorrectProgramId)]) + .test_missing_signers(false) + .execute(); +} + +#[test] +fn test_delegate_non_stake_account() { + let ctx = StakeTestContext::with_delegation(); + + // Create a rewards pool account (program-owned but not a stake account) + let rewards_pool = Pubkey::new_unique(); + let rewards_pool_data = AccountSharedData::new_data_with_space( + ctx.rent_exempt_reserve, + &StakeStateV2::RewardsPool, + StakeStateV2::size_of(), + &id(), + ) + .unwrap(); + + ctx.process_with(DelegateConfig { + stake: (&rewards_pool, &rewards_pool_data), + vote: ( + ctx.vote_account.as_ref().unwrap(), + ctx.vote_account_data.as_ref().unwrap(), + ), + }) + .checks(&[Check::err(ProgramError::InvalidAccountData)]) + .test_missing_signers(false) + .execute(); +} diff --git a/program/tests/helpers/context.rs b/program/tests/helpers/context.rs index 03713b4d..4bfdf5fd 100644 --- a/program/tests/helpers/context.rs +++ b/program/tests/helpers/context.rs @@ -2,62 +2,183 @@ use { super::{ instruction_builders::{InstructionConfig, InstructionExecution}, lifecycle::StakeLifecycle, - utils::{add_sysvars, STAKE_RENT_EXEMPTION}, + stake_tracker::StakeTracker, + utils::{add_sysvars, create_vote_account, STAKE_RENT_EXEMPTION}, }, mollusk_svm::{result::Check, Mollusk}, solana_account::AccountSharedData, solana_instruction::Instruction, solana_pubkey::Pubkey, + solana_stake_interface::state::Lockup, solana_stake_program::id, }; /// Builder for creating stake accounts with customizable parameters -pub struct StakeAccountBuilder { +/// Follows the builder pattern for flexibility and readability +pub struct StakeAccountBuilder<'a> { + ctx: &'a mut StakeTestContext, lifecycle: StakeLifecycle, + staked_amount: u64, + stake_authority: Option, + withdraw_authority: Option, + lockup: Option, + vote_account: Option, + stake_pubkey: Option, } -impl StakeAccountBuilder { +impl StakeAccountBuilder<'_> { + /// Set the staked amount (lamports delegated to validator) + pub fn staked_amount(mut self, amount: u64) -> Self { + self.staked_amount = amount; + self + } + + /// Set a custom stake authority (defaults to ctx.staker) + pub fn stake_authority(mut self, authority: &Pubkey) -> Self { + self.stake_authority = Some(*authority); + self + } + + /// Set a custom withdraw authority (defaults to ctx.withdrawer) + pub fn withdraw_authority(mut self, authority: &Pubkey) -> Self { + self.withdraw_authority = Some(*authority); + self + } + + /// Set a custom lockup (defaults to Lockup::default()) + pub fn lockup(mut self, lockup: &Lockup) -> Self { + self.lockup = Some(*lockup); + self + } + + /// Set a custom vote account (defaults to ctx.vote_account) + pub fn vote_account(mut self, vote_account: &Pubkey) -> Self { + self.vote_account = Some(*vote_account); + self + } + + /// Set a specific stake account pubkey (defaults to Pubkey::new_unique()) + pub fn stake_pubkey(mut self, pubkey: &Pubkey) -> Self { + self.stake_pubkey = Some(*pubkey); + self + } + + /// Build the stake account and return (pubkey, account_data) pub fn build(self) -> (Pubkey, AccountSharedData) { - let stake_pubkey = Pubkey::new_unique(); - let account = self.lifecycle.create_uninitialized_account(); + let stake_pubkey = self.stake_pubkey.unwrap_or_else(Pubkey::new_unique); + let account = self.lifecycle.create_stake_account_fully_specified( + &mut self.ctx.mollusk, + self.ctx + .tracker + .as_mut() + .expect("tracker required for stake account builder"), + &stake_pubkey, + self.vote_account.as_ref().unwrap_or( + self.ctx + .vote_account + .as_ref() + .expect("vote_account required for this lifecycle"), + ), + self.staked_amount, + self.stake_authority.as_ref().unwrap_or(&self.ctx.staker), + self.withdraw_authority + .as_ref() + .unwrap_or(&self.ctx.withdrawer), + self.lockup.as_ref().unwrap_or(&Lockup::default()), + ); (stake_pubkey, account) } } -/// Consolidated test context for stake account tests pub struct StakeTestContext { pub mollusk: Mollusk, pub rent_exempt_reserve: u64, pub staker: Pubkey, pub withdrawer: Pubkey, + pub minimum_delegation: Option, + pub vote_account: Option, + pub vote_account_data: Option, + pub tracker: Option, } impl StakeTestContext { - /// Create a new test context with all standard setup - pub fn new() -> Self { + pub fn minimal() -> Self { let mollusk = Mollusk::new(&id(), "solana_stake_program"); + Self { + mollusk, + rent_exempt_reserve: STAKE_RENT_EXEMPTION, + staker: Pubkey::new_unique(), + withdrawer: Pubkey::new_unique(), + minimum_delegation: None, + vote_account: None, + vote_account_data: None, + tracker: None, + } + } + pub fn with_delegation() -> Self { + let mollusk = Mollusk::new(&id(), "solana_stake_program"); + let minimum_delegation = solana_stake_program::get_minimum_delegation(); + let tracker: StakeTracker = StakeLifecycle::create_tracker_for_test(minimum_delegation); Self { mollusk, rent_exempt_reserve: STAKE_RENT_EXEMPTION, staker: Pubkey::new_unique(), withdrawer: Pubkey::new_unique(), + minimum_delegation: Some(minimum_delegation), + vote_account: Some(Pubkey::new_unique()), + vote_account_data: Some(create_vote_account()), + tracker: Some(tracker), } } + pub fn new() -> Self { + Self::with_delegation() + } + /// Create a stake account builder for the specified lifecycle stage + /// This is the primary method for creating stake accounts in tests. /// /// Example: /// ``` /// let (stake, account) = ctx - /// .stake_account(StakeLifecycle::Uninitialized) + /// .stake_account(StakeLifecycle::Active) + /// .staked_amount(1_000_000) /// .build(); /// ``` pub fn stake_account(&mut self, lifecycle: StakeLifecycle) -> StakeAccountBuilder { - StakeAccountBuilder { lifecycle } + StakeAccountBuilder { + ctx: self, + lifecycle, + staked_amount: 0, + stake_authority: None, + withdraw_authority: None, + lockup: None, + vote_account: None, + stake_pubkey: None, + } + } + + /// Create a lockup that expires in the future + pub fn create_future_lockup(&self, epochs_ahead: u64) -> Lockup { + Lockup { + unix_timestamp: 0, + epoch: self.mollusk.sysvars.clock.epoch + epochs_ahead, + custodian: Pubkey::new_unique(), + } + } + + /// Create a lockup that's currently in force (far future) + pub fn create_in_force_lockup(&self) -> Lockup { + self.create_future_lockup(1_000_000) + } + + /// Create a second vote account (for testing different vote accounts) + pub fn create_second_vote_account(&self) -> (Pubkey, AccountSharedData) { + (Pubkey::new_unique(), create_vote_account()) } - /// Process an instruction + /// Process an instruction with a config-based approach pub fn process_with<'b, C: InstructionConfig>( &self, config: C, @@ -69,7 +190,7 @@ impl StakeTestContext { ) } - /// Process an instruction with optional missing signer testing + /// Internal helper to process an instruction with optional missing signer testing pub(crate) fn process_instruction_maybe_test_signers( &self, instruction: &Instruction, diff --git a/program/tests/helpers/instruction_builders.rs b/program/tests/helpers/instruction_builders.rs index 515a72a0..3249e61e 100644 --- a/program/tests/helpers/instruction_builders.rs +++ b/program/tests/helpers/instruction_builders.rs @@ -103,3 +103,81 @@ impl InstructionConfig for InitializeCheckedConfig<'_> { vec![(*self.stake.0, self.stake.1.clone())] } } + +pub struct DeactivateConfig<'a> { + pub stake: (&'a Pubkey, &'a AccountSharedData), + /// Override signer for testing wrong signer scenarios (defaults to ctx.staker) + pub override_signer: Option<&'a Pubkey>, +} + +impl InstructionConfig for DeactivateConfig<'_> { + fn build_instruction(&self, ctx: &StakeTestContext) -> Instruction { + let signer = self.override_signer.unwrap_or(&ctx.staker); + ixn::deactivate_stake(self.stake.0, signer) + } + + fn build_accounts(&self) -> Vec<(Pubkey, AccountSharedData)> { + vec![(*self.stake.0, self.stake.1.clone())] + } +} + +pub struct DelegateConfig<'a> { + pub stake: (&'a Pubkey, &'a AccountSharedData), + pub vote: (&'a Pubkey, &'a AccountSharedData), +} + +impl InstructionConfig for DelegateConfig<'_> { + fn build_instruction(&self, ctx: &StakeTestContext) -> Instruction { + ixn::delegate_stake(self.stake.0, &ctx.staker, self.vote.0) + } + + fn build_accounts(&self) -> Vec<(Pubkey, AccountSharedData)> { + vec![ + (*self.stake.0, self.stake.1.clone()), + (*self.vote.0, self.vote.1.clone()), + ] + } +} + +pub struct SplitConfig<'a> { + pub source: (&'a Pubkey, &'a AccountSharedData), + pub destination: (&'a Pubkey, &'a AccountSharedData), + pub amount: u64, + pub signer: &'a Pubkey, +} + +impl InstructionConfig for SplitConfig<'_> { + fn build_instruction(&self, _ctx: &StakeTestContext) -> Instruction { + let instructions = ixn::split(self.source.0, self.signer, self.amount, self.destination.0); + instructions[2].clone() // The actual split instruction + } + + fn build_accounts(&self) -> Vec<(Pubkey, AccountSharedData)> { + vec![ + (*self.source.0, self.source.1.clone()), + (*self.destination.0, self.destination.1.clone()), + ] + } +} + +pub struct WithdrawConfig<'a> { + pub stake: (&'a Pubkey, &'a AccountSharedData), + pub recipient: (&'a Pubkey, &'a AccountSharedData), + pub amount: u64, + /// Override signer for testing wrong signer scenarios (defaults to ctx.withdrawer) + pub override_signer: Option<&'a Pubkey>, +} + +impl InstructionConfig for WithdrawConfig<'_> { + fn build_instruction(&self, ctx: &StakeTestContext) -> Instruction { + let signer = self.override_signer.unwrap_or(&ctx.withdrawer); + ixn::withdraw(self.stake.0, signer, self.recipient.0, self.amount, None) + } + + fn build_accounts(&self) -> Vec<(Pubkey, AccountSharedData)> { + vec![ + (*self.stake.0, self.stake.1.clone()), + (*self.recipient.0, self.recipient.1.clone()), + ] + } +} diff --git a/program/tests/helpers/lifecycle.rs b/program/tests/helpers/lifecycle.rs index 231958f0..67aed6a9 100644 --- a/program/tests/helpers/lifecycle.rs +++ b/program/tests/helpers/lifecycle.rs @@ -1,10 +1,20 @@ use { - super::utils::STAKE_RENT_EXEMPTION, solana_account::AccountSharedData, - solana_stake_interface::state::StakeStateV2, solana_stake_program::id, + super::{ + stake_tracker::MolluskStakeExt, + utils::{add_sysvars, create_vote_account, STAKE_RENT_EXEMPTION}, + }, + crate::helpers::stake_tracker::StakeTracker, + mollusk_svm::Mollusk, + solana_account::{Account, AccountSharedData, WritableAccount}, + solana_pubkey::Pubkey, + solana_stake_interface::{ + instruction as ixn, + state::{Authorized, Lockup, StakeStateV2}, + }, + solana_stake_program::id, }; /// Lifecycle states for stake accounts in tests -#[allow(dead_code)] // TODO: remove once tests include all lifecycles #[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord)] pub enum StakeLifecycle { Uninitialized = 0, @@ -17,14 +27,140 @@ pub enum StakeLifecycle { } impl StakeLifecycle { - /// Create an uninitialized stake account - pub fn create_uninitialized_account(self) -> AccountSharedData { - AccountSharedData::new_data_with_space( - STAKE_RENT_EXEMPTION, - &StakeStateV2::Uninitialized, - StakeStateV2::size_of(), - &id(), + /// Helper to create tracker with appropriate background stake for tests + /// Returns a tracker seeded with background cluster stake + pub fn create_tracker_for_test(minimum_delegation: u64) -> StakeTracker { + // Use a moderate background stake amount + // This mimics Banks' cluster-wide effective stake from all validators + // Calculation: needs to be >> test stakes to provide stable warmup base + let background_stake = minimum_delegation.saturating_mul(100); + StakeTracker::with_background_stake(background_stake) + } + + /// Create a stake account with full specification of authorities and lockup + #[allow(clippy::too_many_arguments)] + pub fn create_stake_account_fully_specified( + self, + mollusk: &mut Mollusk, + tracker: &mut StakeTracker, + stake_pubkey: &Pubkey, + vote_account: &Pubkey, + staked_amount: u64, + staker: &Pubkey, + withdrawer: &Pubkey, + lockup: &Lockup, + ) -> AccountSharedData { + let is_closed = self == StakeLifecycle::Closed; + + // Create base account + let mut stake_account = if is_closed { + let mut account = Account::create(STAKE_RENT_EXEMPTION, vec![], id(), false, u64::MAX); + // Add staked_amount even for closed accounts (matches program-test behavior) + if staked_amount > 0 { + account.lamports += staked_amount; + } + account.into() + } else { + Account::create( + STAKE_RENT_EXEMPTION + staked_amount, + vec![0; StakeStateV2::size_of()], + id(), + false, + u64::MAX, + ) + .into() + }; + + if is_closed { + return stake_account; + } + + let authorized = Authorized { + staker: *staker, + withdrawer: *withdrawer, + }; + + // Initialize if needed + if self >= StakeLifecycle::Initialized { + let stake_state = StakeStateV2::Initialized(solana_stake_interface::state::Meta { + rent_exempt_reserve: STAKE_RENT_EXEMPTION, + authorized, + lockup: *lockup, + }); + bincode::serialize_into(stake_account.data_as_mut_slice(), &stake_state).unwrap(); + } + + // Delegate if needed + if self >= StakeLifecycle::Activating { + let instruction = ixn::delegate_stake(stake_pubkey, staker, vote_account); + + let accounts = vec![ + (*stake_pubkey, stake_account.clone()), + (*vote_account, create_vote_account()), + ]; + + // Use add_sysvars to provide clock, stake history, and config accounts + let accounts_with_sysvars = add_sysvars(mollusk, &instruction, accounts); + let result = mollusk.process_instruction(&instruction, &accounts_with_sysvars); + stake_account = result.resulting_accounts[0].1.clone().into(); + + // Track delegation in the tracker + let activation_epoch = mollusk.sysvars.clock.epoch; + tracker.track_delegation(stake_pubkey, staked_amount, activation_epoch, vote_account); + } + + // Advance epoch to activate if needed (Active and beyond) + if self >= StakeLifecycle::Active { + // With background stake in tracker, just warp 1 epoch + // The background stake provides baseline for instant partial activation + let slots_per_epoch = mollusk.sysvars.epoch_schedule.slots_per_epoch; + let current_slot = mollusk.sysvars.clock.slot; + let target_slot = current_slot + slots_per_epoch; + + mollusk.warp_to_slot_with_stake_tracking(tracker, target_slot, Some(0)); + } + + // Deactivate if needed + if self >= StakeLifecycle::Deactivating { + let instruction = ixn::deactivate_stake(stake_pubkey, staker); + + let accounts = vec![(*stake_pubkey, stake_account.clone())]; + + // Use add_sysvars to provide clock account + let accounts_with_sysvars = add_sysvars(mollusk, &instruction, accounts); + let result = mollusk.process_instruction(&instruction, &accounts_with_sysvars); + stake_account = result.resulting_accounts[0].1.clone().into(); + + // Track deactivation in the tracker + let deactivation_epoch = mollusk.sysvars.clock.epoch; + tracker.track_deactivation(stake_pubkey, deactivation_epoch); + } + + // Advance epoch to fully deactivate if needed (Deactive lifecycle) + // Matches program_test.rs line 978-983: advance_epoch once to fully deactivate + if self == StakeLifecycle::Deactive { + // With background stake, advance 1 epoch for deactivation + // Background provides the baseline for instant partial deactivation + let slots_per_epoch = mollusk.sysvars.epoch_schedule.slots_per_epoch; + let current_slot = mollusk.sysvars.clock.slot; + let target_slot = current_slot + slots_per_epoch; + + mollusk.warp_to_slot_with_stake_tracking(tracker, target_slot, Some(0)); + } + + stake_account + } + + /// Whether this lifecycle stage enforces minimum delegation for split + pub fn split_minimum_enforced(&self) -> bool { + matches!( + self, + Self::Activating | Self::Active | Self::Deactivating | Self::Deactive ) - .unwrap() + } + + /// Whether this lifecycle stage enforces minimum delegation for withdraw + pub fn withdraw_minimum_enforced(&self) -> bool { + matches!(self, Self::Activating | Self::Active | Self::Deactivating) } } diff --git a/program/tests/helpers/mod.rs b/program/tests/helpers/mod.rs index fac852a0..7b85f51d 100644 --- a/program/tests/helpers/mod.rs +++ b/program/tests/helpers/mod.rs @@ -1,6 +1,8 @@ #![allow(clippy::arithmetic_side_effects)] +#![allow(dead_code)] pub mod context; pub mod instruction_builders; pub mod lifecycle; +pub mod stake_tracker; pub mod utils; diff --git a/program/tests/helpers/stake_tracker.rs b/program/tests/helpers/stake_tracker.rs new file mode 100644 index 00000000..6885ef51 --- /dev/null +++ b/program/tests/helpers/stake_tracker.rs @@ -0,0 +1,168 @@ +use { + mollusk_svm::Mollusk, + solana_clock::Epoch, + solana_pubkey::Pubkey, + solana_stake_interface::{ + stake_history::{StakeHistory, StakeHistoryEntry}, + state::Delegation, + }, + std::collections::HashMap, +}; + +// This replicates solana-runtime's Banks behavior where stake history is automatically +// updated at epoch boundaries by aggregating all stake delegations. + +/// Tracks stake delegations for automatic stake history management +#[derive(Default, Clone)] +pub struct StakeTracker { + /// Map of stake account pubkey to its delegation info + pub(crate) delegations: HashMap, +} + +#[derive(Clone)] +pub(crate) struct TrackedDelegation { + pub(crate) stake: u64, + pub(crate) activation_epoch: Epoch, + pub(crate) deactivation_epoch: Epoch, + pub(crate) voter_pubkey: Pubkey, +} + +impl StakeTracker { + pub fn new() -> Self { + Self::default() + } + + /// Create a tracker with background cluster stake (like Banks has) + /// This provides the baseline effective stake that enables instant activation/deactivation + pub fn with_background_stake(background_stake: u64) -> Self { + let mut tracker = Self::new(); + + // Add a synthetic background stake that's been active forever (bootstrap stake) + // This mimics Banks' cluster-wide effective stake + tracker.delegations.insert( + Pubkey::new_unique(), // Synthetic background stake pubkey + TrackedDelegation { + stake: background_stake, + activation_epoch: u64::MAX, // Bootstrap = instantly effective + deactivation_epoch: u64::MAX, + voter_pubkey: Pubkey::new_unique(), + }, + ); + + tracker + } + + /// Track a new stake delegation (called after delegate instruction) + pub fn track_delegation( + &mut self, + stake_pubkey: &Pubkey, + stake_amount: u64, + activation_epoch: Epoch, + voter_pubkey: &Pubkey, + ) { + self.delegations.insert( + *stake_pubkey, + TrackedDelegation { + stake: stake_amount, + activation_epoch, + deactivation_epoch: u64::MAX, + voter_pubkey: *voter_pubkey, + }, + ); + } + + /// Mark a stake as deactivating (called after deactivate instruction) + pub fn track_deactivation(&mut self, stake_pubkey: &Pubkey, deactivation_epoch: Epoch) { + if let Some(delegation) = self.delegations.get_mut(stake_pubkey) { + delegation.deactivation_epoch = deactivation_epoch; + } + } + + /// Calculate aggregate stake history for an epoch (replicates Stakes::activate_epoch) + fn calculate_epoch_entry( + &self, + epoch: Epoch, + stake_history: &StakeHistory, + new_rate_activation_epoch: Option, + ) -> StakeHistoryEntry { + self.delegations + .values() + .map(|tracked| { + let delegation = Delegation { + voter_pubkey: tracked.voter_pubkey, + stake: tracked.stake, + activation_epoch: tracked.activation_epoch, + deactivation_epoch: tracked.deactivation_epoch, + ..Delegation::default() + }; + + delegation.stake_activating_and_deactivating( + epoch, + stake_history, + new_rate_activation_epoch, + ) + }) + .fold(StakeHistoryEntry::default(), |acc, status| { + StakeHistoryEntry { + effective: acc.effective + status.effective, + activating: acc.activating + status.activating, + deactivating: acc.deactivating + status.deactivating, + } + }) + } +} + +/// Extension trait that adds stake-aware warping to Mollusk +pub trait MolluskStakeExt { + /// Warp to a slot and automatically update stake history at epoch boundaries + /// + /// This replicates Banks' behavior from solana-runtime: + /// - Bank::warp_from_parent() advances slot + /// - Stakes::activate_epoch() aggregates delegations + /// - Bank::update_stake_history() writes sysvar + fn warp_to_slot_with_stake_tracking( + &mut self, + tracker: &StakeTracker, + target_slot: u64, + new_rate_activation_epoch: Option, + ); +} + +impl MolluskStakeExt for Mollusk { + fn warp_to_slot_with_stake_tracking( + &mut self, + tracker: &StakeTracker, + target_slot: u64, + new_rate_activation_epoch: Option, + ) { + let current_epoch = self.sysvars.clock.epoch; + let current_slot = self.sysvars.clock.slot; + + if target_slot <= current_slot { + panic!( + "Cannot warp backwards: current_slot={}, target_slot={}", + current_slot, target_slot + ); + } + + // Advance the clock (Mollusk's warp_to_slot only updates Clock sysvar) + self.warp_to_slot(target_slot); + + let new_epoch = self.sysvars.clock.epoch; + + // If we crossed epoch boundaries, update stake history for EACH epoch + // StakeHistorySysvar requires contiguous history with no gaps + // This replicates Bank::update_stake_history() + Stakes::activate_epoch() + if new_epoch != current_epoch { + for epoch in current_epoch..new_epoch { + let entry = tracker.calculate_epoch_entry( + epoch, + &self.sysvars.stake_history, + new_rate_activation_epoch, + ); + + self.sysvars.stake_history.add(epoch, entry); + } + } + } +} diff --git a/program/tests/helpers/utils.rs b/program/tests/helpers/utils.rs index 3bc0659b..ee55f2ff 100644 --- a/program/tests/helpers/utils.rs +++ b/program/tests/helpers/utils.rs @@ -1,11 +1,13 @@ use { mollusk_svm::Mollusk, - solana_account::{Account, AccountSharedData}, + solana_account::{Account, AccountSharedData, ReadableAccount, WritableAccount}, + solana_clock::Epoch, solana_instruction::Instruction, solana_pubkey::Pubkey, solana_rent::Rent, solana_stake_interface::{stake_history::StakeHistory, state::StakeStateV2}, solana_sysvar_id::SysvarId, + solana_vote_interface::state::{VoteStateV4, VoteStateVersions}, std::collections::HashMap, }; @@ -63,3 +65,64 @@ pub fn add_sysvars( result } + +/// Create a vote account with VoteStateV4 +pub fn create_vote_account() -> AccountSharedData { + let space = VoteStateV4::size_of(); + let lamports = Rent::default().minimum_balance(space); + let vote_state = VoteStateVersions::new_v4(VoteStateV4::default()); + let data = bincode::serialize(&vote_state).unwrap(); + + Account::create(lamports, data, solana_sdk_ids::vote::id(), false, u64::MAX).into() +} + +/// Parse a stake account into (Meta, Option, lamports) +pub fn parse_stake_account( + stake_account: &AccountSharedData, +) -> ( + solana_stake_interface::state::Meta, + Option, + u64, +) { + let lamports = stake_account.lamports(); + let stake_state: StakeStateV2 = bincode::deserialize(stake_account.data()).unwrap(); + + match stake_state { + StakeStateV2::Initialized(meta) => (meta, None, lamports), + StakeStateV2::Stake(meta, stake, _) => (meta, Some(stake), lamports), + _ => panic!("Expected initialized or staked account"), + } +} + +/// Increment vote account credits +pub fn increment_vote_account_credits( + vote_account: &mut AccountSharedData, + epoch: Epoch, + credits: u64, +) { + let mut vote_state: VoteStateVersions = bincode::deserialize(vote_account.data()).unwrap(); + + if let VoteStateVersions::V4(ref mut v4) = vote_state { + v4.epoch_credits.push((epoch, credits, 0)); + } + + vote_account.set_data(bincode::serialize(&vote_state).unwrap()); +} + +/// Get the effective stake for an account +pub fn get_effective_stake(mollusk: &Mollusk, stake_account: &AccountSharedData) -> u64 { + let stake_state: StakeStateV2 = bincode::deserialize(stake_account.data()).unwrap(); + + if let StakeStateV2::Stake(_, stake, _) = stake_state { + stake + .delegation + .stake_activating_and_deactivating( + mollusk.sysvars.clock.epoch, + &mollusk.sysvars.stake_history, + Some(0), + ) + .effective + } else { + 0 + } +} diff --git a/program/tests/program_test.rs b/program/tests/program_test.rs index 3b60aaee..e50891c8 100644 --- a/program/tests/program_test.rs +++ b/program/tests/program_test.rs @@ -9,7 +9,6 @@ use { solana_program_error::ProgramError, solana_program_test::*, solana_pubkey::Pubkey, - solana_rent::Rent, solana_sdk_ids::system_program, solana_signer::Signer, solana_stake_interface::{ @@ -17,7 +16,7 @@ use { instruction::{self as ixn, LockupArgs}, program::id, stake_history::StakeHistory, - state::{Authorized, Delegation, Lockup, Meta, Stake, StakeAuthorize, StakeStateV2}, + state::{Authorized, Lockup, Meta, Stake, StakeAuthorize, StakeStateV2}, }, solana_system_interface::instruction as system_instruction, solana_transaction::{Signers, Transaction, TransactionError}, @@ -25,7 +24,7 @@ use { instruction as vote_instruction, state::{VoteInit, VoteStateV4}, }, - test_case::{test_case, test_matrix}, + test_case::test_matrix, }; pub const USER_STARTING_LAMPORTS: u64 = 10_000_000_000_000; // 10k sol @@ -630,148 +629,6 @@ async fn program_test_authorize() { } } -#[tokio::test] -async fn program_test_stake_delegate() { - let mut context = program_test().start_with_context().await; - let accounts = Accounts::default(); - accounts.initialize(&mut context).await; - - let vote_account2 = Keypair::new(); - create_vote( - &mut context, - &Keypair::new(), - &Pubkey::new_unique(), - &Pubkey::new_unique(), - &vote_account2, - ) - .await; - - let staker_keypair = Keypair::new(); - let withdrawer_keypair = Keypair::new(); - - let staker = staker_keypair.pubkey(); - let withdrawer = withdrawer_keypair.pubkey(); - - let authorized = Authorized { staker, withdrawer }; - - let vote_state_credits = 100; - context.increment_vote_account_credits(&accounts.vote_account.pubkey(), vote_state_credits); - let minimum_delegation = get_minimum_delegation(&mut context).await; - - let stake = - create_independent_stake_account(&mut context, &authorized, minimum_delegation).await; - let instruction = ixn::delegate_stake(&stake, &staker, &accounts.vote_account.pubkey()); - - process_instruction_test_missing_signers(&mut context, &instruction, &vec![&staker_keypair]) - .await; - - // verify that delegate() looks right - let clock = context.banks_client.get_sysvar::().await.unwrap(); - let (_, stake_data, _) = get_stake_account(&mut context.banks_client, &stake).await; - assert_eq!( - stake_data.unwrap(), - Stake { - delegation: Delegation { - voter_pubkey: accounts.vote_account.pubkey(), - stake: minimum_delegation, - activation_epoch: clock.epoch, - deactivation_epoch: u64::MAX, - ..Delegation::default() - }, - credits_observed: vote_state_credits, - } - ); - - // verify that delegate fails as stake is active and not deactivating - advance_epoch(&mut context).await; - let instruction = ixn::delegate_stake(&stake, &staker, &accounts.vote_account.pubkey()); - let e = process_instruction(&mut context, &instruction, &vec![&staker_keypair]) - .await - .unwrap_err(); - assert_eq!(e, StakeError::TooSoonToRedelegate.into()); - - // deactivate - let instruction = ixn::deactivate_stake(&stake, &staker); - process_instruction(&mut context, &instruction, &vec![&staker_keypair]) - .await - .unwrap(); - - // verify that delegate to a different vote account fails during deactivation - let instruction = ixn::delegate_stake(&stake, &staker, &vote_account2.pubkey()); - let e = process_instruction(&mut context, &instruction, &vec![&staker_keypair]) - .await - .unwrap_err(); - assert_eq!(e, StakeError::TooSoonToRedelegate.into()); - - // verify that delegate succeeds to same vote account when stake is deactivating - refresh_blockhash(&mut context).await; - let instruction = ixn::delegate_stake(&stake, &staker, &accounts.vote_account.pubkey()); - process_instruction(&mut context, &instruction, &vec![&staker_keypair]) - .await - .unwrap(); - - // verify that deactivation has been cleared - let (_, stake_data, _) = get_stake_account(&mut context.banks_client, &stake).await; - assert_eq!(stake_data.unwrap().delegation.deactivation_epoch, u64::MAX); - - // verify that delegate to a different vote account fails if stake is still - // active - let instruction = ixn::delegate_stake(&stake, &staker, &vote_account2.pubkey()); - let e = process_instruction(&mut context, &instruction, &vec![&staker_keypair]) - .await - .unwrap_err(); - assert_eq!(e, StakeError::TooSoonToRedelegate.into()); - - // delegate still fails after stake is fully activated; redelegate is not - // supported - advance_epoch(&mut context).await; - let instruction = ixn::delegate_stake(&stake, &staker, &vote_account2.pubkey()); - let e = process_instruction(&mut context, &instruction, &vec![&staker_keypair]) - .await - .unwrap_err(); - assert_eq!(e, StakeError::TooSoonToRedelegate.into()); - - // delegate to spoofed vote account fails (not owned by vote program) - let mut fake_vote_account = - get_account(&mut context.banks_client, &accounts.vote_account.pubkey()).await; - fake_vote_account.owner = Pubkey::new_unique(); - let fake_vote_address = Pubkey::new_unique(); - context.set_account(&fake_vote_address, &fake_vote_account.into()); - - let stake = - create_independent_stake_account(&mut context, &authorized, minimum_delegation).await; - let instruction = ixn::delegate_stake(&stake, &staker, &fake_vote_address); - - let e = process_instruction(&mut context, &instruction, &vec![&staker_keypair]) - .await - .unwrap_err(); - assert_eq!(e, ProgramError::IncorrectProgramId); - - // delegate stake program-owned non-stake account fails - let rewards_pool_address = Pubkey::new_unique(); - let rewards_pool = SolanaAccount { - lamports: get_stake_account_rent(&mut context.banks_client).await, - data: bincode::serialize(&StakeStateV2::RewardsPool) - .unwrap() - .to_vec(), - owner: id(), - executable: false, - rent_epoch: u64::MAX, - }; - context.set_account(&rewards_pool_address, &rewards_pool.into()); - - let instruction = ixn::delegate_stake( - &rewards_pool_address, - &staker, - &accounts.vote_account.pubkey(), - ); - - let e = process_instruction(&mut context, &instruction, &vec![&staker_keypair]) - .await - .unwrap_err(); - assert_eq!(e, ProgramError::InvalidAccountData); -} - #[derive(Copy, Clone, Debug, PartialEq, Eq, PartialOrd, Ord)] pub enum StakeLifecycle { Uninitialized = 0, @@ -893,405 +750,6 @@ impl StakeLifecycle { } } -#[test_case(StakeLifecycle::Uninitialized; "uninitialized")] -#[test_case(StakeLifecycle::Initialized; "initialized")] -#[test_case(StakeLifecycle::Activating; "activating")] -#[test_case(StakeLifecycle::Active; "active")] -#[test_case(StakeLifecycle::Deactivating; "deactivating")] -#[test_case(StakeLifecycle::Deactive; "deactive")] -#[tokio::test] -async fn program_test_split(split_source_type: StakeLifecycle) { - let mut context = program_test().start_with_context().await; - let accounts = Accounts::default(); - accounts.initialize(&mut context).await; - - let rent_exempt_reserve = get_stake_account_rent(&mut context.banks_client).await; - let minimum_delegation = get_minimum_delegation(&mut context).await; - let staked_amount = minimum_delegation * 2; - - let (split_source_keypair, staker_keypair, _) = split_source_type - .new_stake_account(&mut context, &accounts.vote_account.pubkey(), staked_amount) - .await; - - let split_source = split_source_keypair.pubkey(); - let split_dest = create_blank_stake_account(&mut context).await; - - let signers = match split_source_type { - StakeLifecycle::Uninitialized => vec![&split_source_keypair], - _ => vec![&staker_keypair], - }; - - // fail, split more than available (even if not active, would kick source out of - // rent exemption) - let instruction = &ixn::split( - &split_source, - &signers[0].pubkey(), - staked_amount + 1, - &split_dest, - )[2]; - - let e = process_instruction(&mut context, instruction, &signers) - .await - .unwrap_err(); - assert_eq!(e, ProgramError::InsufficientFunds); - - // an active or transitioning stake account cannot have less than the minimum - // delegation note this is NOT dependent on the minimum delegation feature. - // there was ALWAYS a minimum. it was one lamport! - if split_source_type.split_minimum_enforced() { - // zero split fails - let instruction = &ixn::split(&split_source, &signers[0].pubkey(), 0, &split_dest)[2]; - let e = process_instruction(&mut context, instruction, &signers) - .await - .unwrap_err(); - assert_eq!(e, ProgramError::InsufficientFunds); - - // underfunded destination fails - let instruction = &ixn::split( - &split_source, - &signers[0].pubkey(), - minimum_delegation - 1, - &split_dest, - )[2]; - - let e = process_instruction(&mut context, instruction, &signers) - .await - .unwrap_err(); - assert_eq!(e, ProgramError::InsufficientFunds); - - // underfunded source fails - let instruction = &ixn::split( - &split_source, - &signers[0].pubkey(), - minimum_delegation + 1, - &split_dest, - )[2]; - - let e = process_instruction(&mut context, instruction, &signers) - .await - .unwrap_err(); - assert_eq!(e, ProgramError::InsufficientFunds); - } - - // split to non-owned account fails - let mut fake_split_dest_account = get_account(&mut context.banks_client, &split_dest).await; - fake_split_dest_account.owner = Pubkey::new_unique(); - let fake_split_dest = Pubkey::new_unique(); - context.set_account(&fake_split_dest, &fake_split_dest_account.into()); - - let instruction = &ixn::split( - &split_source, - &signers[0].pubkey(), - staked_amount / 2, - &fake_split_dest, - )[2]; - - let e = process_instruction(&mut context, instruction, &signers) - .await - .unwrap_err(); - assert_eq!(e, ProgramError::InvalidAccountOwner); - - // success - let instruction = &ixn::split( - &split_source, - &signers[0].pubkey(), - staked_amount / 2, - &split_dest, - )[2]; - process_instruction_test_missing_signers(&mut context, instruction, &signers).await; - - // source lost split amount - let source_lamports = get_account(&mut context.banks_client, &split_source) - .await - .lamports; - assert_eq!(source_lamports, staked_amount / 2 + rent_exempt_reserve); - - // destination gained split amount - let dest_lamports = get_account(&mut context.banks_client, &split_dest) - .await - .lamports; - assert_eq!(dest_lamports, staked_amount / 2 + rent_exempt_reserve); - - // destination meta has been set properly if ever delegated - if split_source_type >= StakeLifecycle::Initialized { - let (source_meta, source_stake, _) = - get_stake_account(&mut context.banks_client, &split_source).await; - let (dest_meta, dest_stake, _) = - get_stake_account(&mut context.banks_client, &split_dest).await; - assert_eq!(dest_meta, source_meta); - - // delegations are set properly if activating or active - if split_source_type >= StakeLifecycle::Activating - && split_source_type < StakeLifecycle::Deactive - { - assert_eq!(source_stake.unwrap().delegation.stake, staked_amount / 2); - assert_eq!(dest_stake.unwrap().delegation.stake, staked_amount / 2); - } - } - - // nothing has been deactivated if active - if split_source_type >= StakeLifecycle::Active && split_source_type < StakeLifecycle::Deactive { - assert_eq!( - get_effective_stake(&mut context.banks_client, &split_source).await, - staked_amount / 2, - ); - - assert_eq!( - get_effective_stake(&mut context.banks_client, &split_dest).await, - staked_amount / 2, - ); - } -} - -#[test_case(StakeLifecycle::Uninitialized; "uninitialized")] -#[test_case(StakeLifecycle::Initialized; "initialized")] -#[test_case(StakeLifecycle::Activating; "activating")] -#[test_case(StakeLifecycle::Active; "active")] -#[test_case(StakeLifecycle::Deactivating; "deactivating")] -#[test_case(StakeLifecycle::Deactive; "deactive")] -#[test_case(StakeLifecycle::Closed; "closed")] -#[tokio::test] -async fn program_test_withdraw_stake(withdraw_source_type: StakeLifecycle) { - let mut context = program_test().start_with_context().await; - let accounts = Accounts::default(); - accounts.initialize(&mut context).await; - - let stake_rent_exempt_reserve = get_stake_account_rent(&mut context.banks_client).await; - let minimum_delegation = get_minimum_delegation(&mut context).await; - let staked_amount = minimum_delegation; - - let wallet_rent_exempt_reserve = context - .banks_client - .get_rent() - .await - .unwrap() - .minimum_balance(0); - - let (withdraw_source_keypair, _, withdrawer_keypair) = withdraw_source_type - .new_stake_account(&mut context, &accounts.vote_account.pubkey(), staked_amount) - .await; - let withdraw_source = withdraw_source_keypair.pubkey(); - - let recipient = Pubkey::new_unique(); - transfer(&mut context, &recipient, wallet_rent_exempt_reserve).await; - - let signers = match withdraw_source_type { - StakeLifecycle::Uninitialized | StakeLifecycle::Closed => vec![&withdraw_source_keypair], - _ => vec![&withdrawer_keypair], - }; - - // withdraw that would end rent-exemption always fails - let rent_spillover = if withdraw_source_type == StakeLifecycle::Closed { - stake_rent_exempt_reserve - Rent::default().minimum_balance(0) + 1 - } else { - 1 - }; - - let instruction = ixn::withdraw( - &withdraw_source, - &signers[0].pubkey(), - &recipient, - staked_amount + rent_spillover, - None, - ); - let e = process_instruction(&mut context, &instruction, &signers) - .await - .unwrap_err(); - assert_eq!(e, ProgramError::InsufficientFunds); - - if withdraw_source_type.withdraw_minimum_enforced() { - // withdraw active or activating stake fails - let instruction = ixn::withdraw( - &withdraw_source, - &signers[0].pubkey(), - &recipient, - staked_amount, - None, - ); - let e = process_instruction(&mut context, &instruction, &signers) - .await - .unwrap_err(); - assert_eq!(e, ProgramError::InsufficientFunds); - - // grant rewards - let reward_amount = 10; - transfer(&mut context, &withdraw_source, reward_amount).await; - - // withdraw in excess of rewards is not allowed - let instruction = ixn::withdraw( - &withdraw_source, - &signers[0].pubkey(), - &recipient, - reward_amount + 1, - None, - ); - let e = process_instruction(&mut context, &instruction, &signers) - .await - .unwrap_err(); - assert_eq!(e, ProgramError::InsufficientFunds); - - // withdraw rewards is allowed - let instruction = ixn::withdraw( - &withdraw_source, - &signers[0].pubkey(), - &recipient, - reward_amount, - None, - ); - process_instruction_test_missing_signers(&mut context, &instruction, &signers).await; - - let recipient_lamports = get_account(&mut context.banks_client, &recipient) - .await - .lamports; - assert_eq!( - recipient_lamports, - reward_amount + wallet_rent_exempt_reserve, - ); - } else { - // withdraw that leaves rent behind is allowed - let instruction = ixn::withdraw( - &withdraw_source, - &signers[0].pubkey(), - &recipient, - staked_amount, - None, - ); - process_instruction_test_missing_signers(&mut context, &instruction, &signers).await; - - let recipient_lamports = get_account(&mut context.banks_client, &recipient) - .await - .lamports; - assert_eq!( - recipient_lamports, - staked_amount + wallet_rent_exempt_reserve, - ); - - // full withdraw is allowed - refresh_blockhash(&mut context).await; - transfer(&mut context, &withdraw_source, staked_amount).await; - - let recipient = Pubkey::new_unique(); - transfer(&mut context, &recipient, wallet_rent_exempt_reserve).await; - - let instruction = ixn::withdraw( - &withdraw_source, - &signers[0].pubkey(), - &recipient, - staked_amount + stake_rent_exempt_reserve, - None, - ); - process_instruction_test_missing_signers(&mut context, &instruction, &signers).await; - - let recipient_lamports = get_account(&mut context.banks_client, &recipient) - .await - .lamports; - assert_eq!( - recipient_lamports, - staked_amount + stake_rent_exempt_reserve + wallet_rent_exempt_reserve, - ); - } - - // withdraw from program-owned non-stake not allowed - let rewards_pool_address = Pubkey::new_unique(); - let rewards_pool = SolanaAccount { - lamports: get_stake_account_rent(&mut context.banks_client).await + staked_amount, - data: bincode::serialize(&StakeStateV2::RewardsPool) - .unwrap() - .to_vec(), - owner: id(), - executable: false, - rent_epoch: u64::MAX, - }; - context.set_account(&rewards_pool_address, &rewards_pool.into()); - - let instruction = ixn::withdraw( - &rewards_pool_address, - &signers[0].pubkey(), - &recipient, - staked_amount, - None, - ); - let e = process_instruction(&mut context, &instruction, &signers) - .await - .unwrap_err(); - assert_eq!(e, ProgramError::InvalidAccountData); -} - -#[test_case(false; "activating")] -#[test_case(true; "active")] -#[tokio::test] -async fn program_test_deactivate(activate: bool) { - let mut context = program_test().start_with_context().await; - let accounts = Accounts::default(); - accounts.initialize(&mut context).await; - - let minimum_delegation = get_minimum_delegation(&mut context).await; - - let staker_keypair = Keypair::new(); - let withdrawer_keypair = Keypair::new(); - - let staker = staker_keypair.pubkey(); - let withdrawer = withdrawer_keypair.pubkey(); - - let authorized = Authorized { staker, withdrawer }; - - let stake = - create_independent_stake_account(&mut context, &authorized, minimum_delegation).await; - - // deactivating an undelegated account fails - let instruction = ixn::deactivate_stake(&stake, &staker); - let e = process_instruction(&mut context, &instruction, &vec![&staker_keypair]) - .await - .unwrap_err(); - assert_eq!(e, ProgramError::InvalidAccountData); - - // delegate - let instruction = ixn::delegate_stake(&stake, &staker, &accounts.vote_account.pubkey()); - process_instruction(&mut context, &instruction, &vec![&staker_keypair]) - .await - .unwrap(); - - if activate { - advance_epoch(&mut context).await; - } else { - refresh_blockhash(&mut context).await; - } - - // deactivate with withdrawer fails - let instruction = ixn::deactivate_stake(&stake, &withdrawer); - let e = process_instruction(&mut context, &instruction, &vec![&withdrawer_keypair]) - .await - .unwrap_err(); - assert_eq!(e, ProgramError::MissingRequiredSignature); - - // deactivate succeeds - let instruction = ixn::deactivate_stake(&stake, &staker); - process_instruction_test_missing_signers(&mut context, &instruction, &vec![&staker_keypair]) - .await; - - let clock = context.banks_client.get_sysvar::().await.unwrap(); - let (_, stake_data, _) = get_stake_account(&mut context.banks_client, &stake).await; - assert_eq!( - stake_data.unwrap().delegation.deactivation_epoch, - clock.epoch - ); - - // deactivate again fails - refresh_blockhash(&mut context).await; - - let e = process_instruction(&mut context, &instruction, &vec![&staker_keypair]) - .await - .unwrap_err(); - assert_eq!(e, StakeError::AlreadyDeactivated.into()); - - advance_epoch(&mut context).await; - - let e = process_instruction(&mut context, &instruction, &vec![&staker_keypair]) - .await - .unwrap_err(); - assert_eq!(e, StakeError::AlreadyDeactivated.into()); -} - // XXX the original test_merge is a stupid test // the real thing is test_merge_active_stake which actively controls clock and // stake_history but im just trying to smoke test rn so lets do something diff --git a/program/tests/split.rs b/program/tests/split.rs new file mode 100644 index 00000000..69db3435 --- /dev/null +++ b/program/tests/split.rs @@ -0,0 +1,190 @@ +#![allow(clippy::arithmetic_side_effects)] + +mod helpers; + +use { + helpers::{ + context::StakeTestContext, + instruction_builders::SplitConfig, + lifecycle::StakeLifecycle, + utils::{get_effective_stake, parse_stake_account}, + }, + mollusk_svm::result::Check, + solana_account::{AccountSharedData, WritableAccount}, + solana_program_error::ProgramError, + solana_pubkey::Pubkey, + solana_stake_interface::state::StakeStateV2, + solana_stake_program::id, + test_case::test_case, +}; + +#[test_case(StakeLifecycle::Uninitialized; "uninitialized")] +#[test_case(StakeLifecycle::Initialized; "initialized")] +#[test_case(StakeLifecycle::Activating; "activating")] +#[test_case(StakeLifecycle::Active; "active")] +#[test_case(StakeLifecycle::Deactivating; "deactivating")] +#[test_case(StakeLifecycle::Deactive; "deactive")] +fn test_split(split_source_type: StakeLifecycle) { + let mut ctx = StakeTestContext::new(); + let staked_amount = ctx.minimum_delegation.unwrap() * 2; + + // Create source stake account at the specified lifecycle stage + let (split_source, mut split_source_account) = ctx + .stake_account(split_source_type) + .staked_amount(staked_amount) + .build(); + + // Create destination stake account matching what create_blank_stake_account does: + // rent-exempt lamports, correct size, stake program owner, uninitialized data + let split_dest = Pubkey::new_unique(); + let split_dest_account = + AccountSharedData::new(ctx.rent_exempt_reserve, StakeStateV2::size_of(), &id()); + + // Determine signer based on lifecycle stage + let signer = if split_source_type == StakeLifecycle::Uninitialized { + split_source + } else { + ctx.staker + }; + + // Fail: split more than available (would violate rent exemption) + // Note: Behavior differs between program-test and Mollusk: + // - program-test: Transaction-level rent check returns InsufficientFunds before program runs + // - Mollusk: Program succeeds for uninitialized (no program-level check), but violates rent + // For initialized+ accounts, the program itself checks and returns InsufficientFunds + if split_source_type == StakeLifecycle::Uninitialized { + // Mollusk: Program succeeds, but resulting accounts violate rent exemption + let result = std::panic::catch_unwind(std::panic::AssertUnwindSafe(|| { + ctx.process_with(SplitConfig { + source: (&split_source, &split_source_account), + destination: (&split_dest, &split_dest_account), + signer: &signer, + amount: staked_amount + 1, + }) + .checks(&[Check::success(), Check::all_rent_exempt()]) + .execute() + })); + assert!( + result.is_err(), + "Expected rent exemption check to fail for uninitialized split" + ); + } else { + // Program-level check returns InsufficientFunds for initialized+ accounts + ctx.process_with(SplitConfig { + source: (&split_source, &split_source_account), + destination: (&split_dest, &split_dest_account), + signer: &signer, + amount: staked_amount + 1, + }) + .checks(&[Check::err(ProgramError::InsufficientFunds)]) + .test_missing_signers(false) + .execute(); + } + + // Test minimum delegation enforcement for active/transitioning stakes + if split_source_type.split_minimum_enforced() { + // Zero split fails + ctx.process_with(SplitConfig { + source: (&split_source, &split_source_account), + destination: (&split_dest, &split_dest_account), + signer: &signer, + amount: 0, + }) + .checks(&[Check::err(ProgramError::InsufficientFunds)]) + .test_missing_signers(false) + .execute(); + + // Underfunded destination fails + ctx.process_with(SplitConfig { + source: (&split_source, &split_source_account), + destination: (&split_dest, &split_dest_account), + signer: &signer, + amount: ctx.minimum_delegation.unwrap() - 1, + }) + .checks(&[Check::err(ProgramError::InsufficientFunds)]) + .test_missing_signers(false) + .execute(); + + // Underfunded source fails + ctx.process_with(SplitConfig { + source: (&split_source, &split_source_account), + destination: (&split_dest, &split_dest_account), + signer: &signer, + amount: ctx.minimum_delegation.unwrap() + 1, + }) + .checks(&[Check::err(ProgramError::InsufficientFunds)]) + .test_missing_signers(false) + .execute(); + } + + // Split to account with wrong owner fails + let fake_split_dest = Pubkey::new_unique(); + let mut fake_split_dest_account = split_dest_account.clone(); + fake_split_dest_account.set_owner(Pubkey::new_unique()); + + ctx.process_with(SplitConfig { + source: (&split_source, &split_source_account), + destination: (&fake_split_dest, &fake_split_dest_account), + signer: &signer, + amount: staked_amount / 2, + }) + .checks(&[Check::err(ProgramError::InvalidAccountOwner)]) + .test_missing_signers(false) + .execute(); + + // Success: split half + let result = ctx + .process_with(SplitConfig { + source: (&split_source, &split_source_account), + destination: (&split_dest, &split_dest_account), + signer: &signer, + amount: staked_amount / 2, + }) + .checks(&[ + Check::success(), + Check::all_rent_exempt(), + Check::account(&split_source) + .lamports(staked_amount / 2 + ctx.rent_exempt_reserve) + .owner(&id()) + .space(StakeStateV2::size_of()) + .build(), + Check::account(&split_dest) + .lamports(staked_amount / 2 + ctx.rent_exempt_reserve) + .owner(&id()) + .space(StakeStateV2::size_of()) + .build(), + ]) + .test_missing_signers(true) + .execute(); + + split_source_account = result.resulting_accounts[0].1.clone().into(); + let split_dest_account: AccountSharedData = result.resulting_accounts[1].1.clone().into(); + + // Verify metadata is copied for initialized and above + if split_source_type >= StakeLifecycle::Initialized { + let (source_meta, source_stake, _) = parse_stake_account(&split_source_account); + let (dest_meta, dest_stake, _) = parse_stake_account(&split_dest_account); + assert_eq!(dest_meta, source_meta); + + // Verify delegations are set properly for activating/active/deactivating + if split_source_type >= StakeLifecycle::Activating + && split_source_type < StakeLifecycle::Deactive + { + assert_eq!(source_stake.unwrap().delegation.stake, staked_amount / 2); + assert_eq!(dest_stake.unwrap().delegation.stake, staked_amount / 2); + } + } + + // Verify nothing has been deactivated for active stakes + if split_source_type >= StakeLifecycle::Active && split_source_type < StakeLifecycle::Deactive { + assert_eq!( + get_effective_stake(&ctx.mollusk, &split_source_account), + staked_amount / 2, + ); + + assert_eq!( + get_effective_stake(&ctx.mollusk, &split_dest_account), + staked_amount / 2, + ); + } +} diff --git a/program/tests/stake_tracker_equivalence.rs b/program/tests/stake_tracker_equivalence.rs new file mode 100644 index 00000000..e3f5ed33 --- /dev/null +++ b/program/tests/stake_tracker_equivalence.rs @@ -0,0 +1,1329 @@ +#![allow(clippy::arithmetic_side_effects)] + +//! Equivalence tests proving StakeTracker (Mollusk) matches BanksClient (solana-program-test) +//! +//! These tests run identical stake operations through both implementations and compare results +//! to ensure 1:1 behavioral equivalence in stake history tracking. +use { + mollusk_svm::Mollusk, + solana_account::{AccountSharedData, ReadableAccount, WritableAccount}, + solana_clock::Clock, + solana_keypair::Keypair, + solana_program_test::{ProgramTest, ProgramTestContext}, + solana_pubkey::Pubkey, + solana_signer::Signer, + solana_stake_interface::{ + instruction as ixn, + stake_history::StakeHistory, + state::{Authorized, Lockup, StakeStateV2}, + }, + solana_stake_program::id, + solana_system_interface::instruction as system_instruction, + solana_transaction::Transaction, + test_case::test_case, +}; + +mod helpers; +use helpers::{ + stake_tracker::{MolluskStakeExt, StakeTracker}, + utils::{add_sysvars, create_vote_account, STAKE_RENT_EXEMPTION}, +}; + +// Constants for testing +const MINIMUM_DELEGATION: u64 = 1; + +/// Dual context holding both BanksClient and Mollusk paths +struct DualContext { + // BanksClient path + program_test_ctx: ProgramTestContext, + + // Mollusk path + mollusk: Mollusk, + tracker: StakeTracker, + + // Shared test data + vote_account: Pubkey, + vote_account_data: AccountSharedData, + background_stake: u64, +} + +impl DualContext { + /// Create both contexts with matching initial state + async fn new() -> Self { + // Initialize program test (BanksClient path) + let mut program_test = ProgramTest::default(); + program_test.prefer_bpf(true); + program_test.add_upgradeable_program_to_genesis("solana_stake_program", &id()); + let mut program_test_ctx = program_test.start_with_context().await; + + // Warp to first normal slot on Banks + let slot = program_test_ctx + .genesis_config() + .epoch_schedule + .first_normal_slot + + 1; + program_test_ctx.warp_to_slot(slot).unwrap(); + + // Initialize Mollusk and sync to the same epoch as Banks + let mut mollusk = Mollusk::new(&id(), "solana_stake_program"); + // Banks and Mollusk have different epoch schedules, so we need to get to same epoch + // Get the epoch Banks is at after warping + let banks_clock = program_test_ctx + .banks_client + .get_sysvar::() + .await + .unwrap(); + let banks_epoch = banks_clock.epoch; + + // Warp Mollusk to the same epoch by calculating the corresponding slot + let mollusk_slot_for_epoch = mollusk + .sysvars + .epoch_schedule + .get_first_slot_in_epoch(banks_epoch); + mollusk.warp_to_slot(mollusk_slot_for_epoch); + + // Extract BanksClient's actual background stake from its genesis stake history + // This ensures Mollusk uses the EXACT same background stake for identical history + let banks_stake_history = program_test_ctx + .banks_client + .get_sysvar::() + .await + .unwrap(); + + let epoch_0_entry = banks_stake_history.get(0).cloned().unwrap_or_default(); + let background_stake = epoch_0_entry.effective; + + let tracker = StakeTracker::with_background_stake(background_stake); + + // Initialize Mollusk's stake history to match BanksClient + // Banks may not have history for all intermediate epochs when warped directly, + // so we'll generate them using the tracker (which only has background stake initially) + for epoch in 0..banks_epoch { + if let Some(entry) = banks_stake_history.get(epoch).cloned() { + // Use Banks' actual entry if it exists + mollusk.sysvars.stake_history.add(epoch, entry); + } else { + // Generate entry with just background stake for missing epochs + // This matches what would happen if we had advanced through these epochs naturally + mollusk + .sysvars + .stake_history + .add(epoch, epoch_0_entry.clone()); + } + } + + // Create shared vote account + let vote_account = Pubkey::new_unique(); + let vote_account_data = create_vote_account(); + + // Add vote account to BanksClient (clone to keep original) + program_test_ctx.set_account(&vote_account, &vote_account_data.clone()); + + Self { + program_test_ctx, + mollusk, + tracker, + vote_account, + vote_account_data, + background_stake, + } + } + + /// Create a blank stake account on both paths + async fn create_blank_stake_account(&mut self) -> Pubkey { + let stake_keypair = Keypair::new(); + let stake = stake_keypair.pubkey(); + + // BanksClient path + let transaction = Transaction::new_signed_with_payer( + &[system_instruction::create_account( + &self.program_test_ctx.payer.pubkey(), + &stake, + STAKE_RENT_EXEMPTION, + StakeStateV2::size_of() as u64, + &id(), + )], + Some(&self.program_test_ctx.payer.pubkey()), + &[&self.program_test_ctx.payer, &stake_keypair], + self.program_test_ctx.last_blockhash, + ); + self.program_test_ctx + .banks_client + .process_transaction(transaction) + .await + .unwrap(); + + // Mollusk path - just track that we'll add it when needed + // (Mollusk accounts are passed per-instruction, not stored globally) + + stake + } + + /// Initialize a stake account on both paths + async fn initialize_stake_account( + &mut self, + stake: &Pubkey, + authorized: &Authorized, + lockup: &Lockup, + ) -> AccountSharedData { + let instruction = ixn::initialize(stake, authorized, lockup); + + // BanksClient path + let transaction = Transaction::new_signed_with_payer( + &[instruction.clone()], + Some(&self.program_test_ctx.payer.pubkey()), + &[&self.program_test_ctx.payer], + self.program_test_ctx.last_blockhash, + ); + self.program_test_ctx + .banks_client + .process_transaction(transaction) + .await + .unwrap(); + + // Get account from BanksClient + let banks_account = self + .program_test_ctx + .banks_client + .get_account(*stake) + .await + .unwrap() + .unwrap(); + + // Mollusk path - create matching account + let mut mollusk_account = + AccountSharedData::new(STAKE_RENT_EXEMPTION, StakeStateV2::size_of(), &id()); + + let accounts = vec![(*stake, mollusk_account.clone())]; + let accounts_with_sysvars = add_sysvars(&self.mollusk, &instruction, accounts); + let result = self + .mollusk + .process_instruction(&instruction, &accounts_with_sysvars); + assert!(result.program_result.is_ok()); + mollusk_account = result.resulting_accounts[0].1.clone().into(); + + // Verify accounts match + assert_eq!(banks_account.data, mollusk_account.data()); + assert_eq!(banks_account.lamports, mollusk_account.lamports()); + + mollusk_account + } + + /// Delegate stake on both paths to a specific vote account + async fn delegate_stake_to( + &mut self, + stake: &Pubkey, + stake_account: &mut AccountSharedData, + staker_keypair: &Keypair, + vote_account: &Pubkey, + vote_account_data: &AccountSharedData, + ) { + let instruction = ixn::delegate_stake(stake, &staker_keypair.pubkey(), vote_account); + + // BanksClient path + let transaction = Transaction::new_signed_with_payer( + &[instruction.clone()], + Some(&self.program_test_ctx.payer.pubkey()), + &[&self.program_test_ctx.payer, staker_keypair], + self.program_test_ctx.last_blockhash, + ); + self.program_test_ctx + .banks_client + .process_transaction(transaction) + .await + .unwrap(); + + // Mollusk path + let accounts = vec![ + (*stake, stake_account.clone()), + (*vote_account, vote_account_data.clone()), + ]; + let accounts_with_sysvars = add_sysvars(&self.mollusk, &instruction, accounts); + let result = self + .mollusk + .process_instruction(&instruction, &accounts_with_sysvars); + assert!(result.program_result.is_ok()); + *stake_account = result.resulting_accounts[0].1.clone().into(); + + // Track delegation in Mollusk tracker + let stake_state: StakeStateV2 = bincode::deserialize(stake_account.data()).unwrap(); + if let StakeStateV2::Stake(_, stake_data, _) = stake_state { + self.tracker.track_delegation( + stake, + stake_data.delegation.stake, + stake_data.delegation.activation_epoch, + vote_account, + ); + } + } + + /// Delegate stake on both paths (uses default vote account) + async fn delegate_stake( + &mut self, + stake: &Pubkey, + stake_account: &mut AccountSharedData, + staker_keypair: &Keypair, + ) { + let vote_account = self.vote_account; + let vote_account_data = self.vote_account_data.clone(); + self.delegate_stake_to( + stake, + stake_account, + staker_keypair, + &vote_account, + &vote_account_data, + ) + .await; + } + + /// Deactivate stake on both paths + async fn deactivate_stake( + &mut self, + stake: &Pubkey, + stake_account: &mut AccountSharedData, + staker_keypair: &Keypair, + ) { + let instruction = ixn::deactivate_stake(stake, &staker_keypair.pubkey()); + + // BanksClient path + let transaction = Transaction::new_signed_with_payer( + &[instruction.clone()], + Some(&self.program_test_ctx.payer.pubkey()), + &[&self.program_test_ctx.payer, staker_keypair], + self.program_test_ctx.last_blockhash, + ); + self.program_test_ctx + .banks_client + .process_transaction(transaction) + .await + .unwrap(); + + // Mollusk path + let accounts = vec![(*stake, stake_account.clone())]; + let accounts_with_sysvars = add_sysvars(&self.mollusk, &instruction, accounts); + let result = self + .mollusk + .process_instruction(&instruction, &accounts_with_sysvars); + assert!(result.program_result.is_ok()); + *stake_account = result.resulting_accounts[0].1.clone().into(); + + // Track deactivation + let deactivation_epoch = self.mollusk.sysvars.clock.epoch; + self.tracker.track_deactivation(stake, deactivation_epoch); + } + + /// Advance epoch on both paths with default new_rate_activation_epoch (Some(0)) + async fn advance_epoch(&mut self) { + self.advance_epoch_with_rate(Some(0)).await; + } + + /// Advance epoch on both paths with custom new_rate_activation_epoch + /// Pass None to use old warmup rate behavior + async fn advance_epoch_with_rate(&mut self, new_rate_activation_epoch: Option) { + // Refresh blockhash for BanksClient by advancing slot slightly first + let current_slot = self + .program_test_ctx + .banks_client + .get_root_slot() + .await + .unwrap(); + self.program_test_ctx + .warp_to_slot(current_slot + 1) + .unwrap(); + self.program_test_ctx.last_blockhash = self + .program_test_ctx + .banks_client + .get_latest_blockhash() + .await + .unwrap(); + + // BanksClient path - advance epoch + let root_slot = self + .program_test_ctx + .banks_client + .get_root_slot() + .await + .unwrap(); + let slots_per_epoch = self + .program_test_ctx + .genesis_config() + .epoch_schedule + .slots_per_epoch; + self.program_test_ctx + .warp_to_slot(root_slot + slots_per_epoch) + .unwrap(); + + // Mollusk path - advance epoch with stake tracking + let current_slot = self.mollusk.sysvars.clock.slot; + let mollusk_slots_per_epoch = self.mollusk.sysvars.epoch_schedule.slots_per_epoch; + let target_slot = current_slot + mollusk_slots_per_epoch; + self.mollusk.warp_to_slot_with_stake_tracking( + &self.tracker, + target_slot, + new_rate_activation_epoch, + ); + } + + /// Get stake history from BanksClient + async fn get_banks_stake_history(&mut self) -> StakeHistory { + self.program_test_ctx + .banks_client + .get_sysvar::() + .await + .unwrap() + } + + /// Get stake history from Mollusk + fn get_mollusk_stake_history(&self) -> &StakeHistory { + &self.mollusk.sysvars.stake_history + } + + /// Get effective stake from BanksClient + async fn get_banks_effective_stake(&mut self, stake: &Pubkey) -> u64 { + let clock = self + .program_test_ctx + .banks_client + .get_sysvar::() + .await + .unwrap(); + let stake_history = self.get_banks_stake_history().await; + let account = self + .program_test_ctx + .banks_client + .get_account(*stake) + .await + .unwrap() + .unwrap(); + + match bincode::deserialize::(&account.data).unwrap() { + StakeStateV2::Stake(_, stake_data, _) => { + stake_data + .delegation + .stake_activating_and_deactivating(clock.epoch, &stake_history, Some(0)) + .effective + } + _ => 0, + } + } + + /// Get effective stake from Mollusk + fn get_mollusk_effective_stake(&self, stake_account: &AccountSharedData) -> u64 { + let clock = &self.mollusk.sysvars.clock; + let stake_history = &self.mollusk.sysvars.stake_history; + + match bincode::deserialize::(stake_account.data()).unwrap() { + StakeStateV2::Stake(_, stake_data, _) => { + stake_data + .delegation + .stake_activating_and_deactivating(clock.epoch, stake_history, Some(0)) + .effective + } + _ => 0, + } + } + + /// Compare stake history entries between both implementations + /// Verifies all components: effective, activating, and deactivating + async fn compare_stake_history(&mut self, epoch: u64) { + let banks_history = self + .program_test_ctx + .banks_client + .get_sysvar::() + .await + .unwrap(); + let mollusk_history = &self.mollusk.sysvars.stake_history; + + let banks_entry = banks_history.get(epoch); + let mollusk_entry = mollusk_history.get(epoch); + + assert_eq!( + banks_entry, mollusk_entry, + "Stake history mismatch at epoch {}: BanksClient={:?}, Mollusk={:?}", + epoch, banks_entry, mollusk_entry + ); + } + + /// Verify background stake is preserved in stake history across implementations + async fn verify_background_stake_preservation(&mut self, epoch: u64, expected_background: u64) { + let banks_history = self + .program_test_ctx + .banks_client + .get_sysvar::() + .await + .unwrap(); + let mollusk_history = &self.mollusk.sysvars.stake_history; + + if let Some(banks_entry) = banks_history.get(epoch) { + assert!( + banks_entry.effective >= expected_background, + "Epoch {}: Banks effective stake {} should include background {}", + epoch, + banks_entry.effective, + expected_background + ); + } + + if let Some(mollusk_entry) = mollusk_history.get(epoch) { + assert!( + mollusk_entry.effective >= expected_background, + "Epoch {}: Mollusk effective stake {} should include background {}", + epoch, + mollusk_entry.effective, + expected_background + ); + } + } + + /// Compare account state between both paths + async fn compare_account_state(&mut self, stake: &Pubkey, mollusk_account: &AccountSharedData) { + let banks_account = self + .program_test_ctx + .banks_client + .get_account(*stake) + .await + .unwrap() + .unwrap(); + + assert_eq!( + banks_account.lamports, + mollusk_account.lamports(), + "Lamports mismatch" + ); + + let banks_state: StakeStateV2 = bincode::deserialize(&banks_account.data).unwrap(); + let mollusk_state: StakeStateV2 = bincode::deserialize(mollusk_account.data()).unwrap(); + + match (banks_state, mollusk_state) { + (StakeStateV2::Stake(b_meta, b_stake, _), StakeStateV2::Stake(m_meta, m_stake, _)) => { + assert_eq!(b_meta.authorized, m_meta.authorized); + assert_eq!(b_meta.lockup, m_meta.lockup); + assert_eq!(b_stake.delegation.stake, m_stake.delegation.stake); + assert_eq!( + b_stake.delegation.activation_epoch, + m_stake.delegation.activation_epoch + ); + assert_eq!( + b_stake.delegation.deactivation_epoch, + m_stake.delegation.deactivation_epoch + ); + } + (StakeStateV2::Initialized(b_meta), StakeStateV2::Initialized(m_meta)) => { + assert_eq!(b_meta.authorized, m_meta.authorized); + assert_eq!(b_meta.lockup, m_meta.lockup); + } + _ => { + panic!( + "State type mismatch: banks={:?}, mollusk={:?}", + banks_state, mollusk_state + ); + } + } + } + + /// Advance one epoch and compare stake account state between implementations + async fn advance_and_compare_stake( + &mut self, + stake: &Pubkey, + stake_account: &AccountSharedData, + ) { + self.advance_and_compare_stake_with_rate(stake, stake_account, Some(0)) + .await; + } + + /// Advance one epoch with custom warmup rate and compare stake account state + async fn advance_and_compare_stake_with_rate( + &mut self, + stake: &Pubkey, + stake_account: &AccountSharedData, + new_rate_activation_epoch: Option, + ) { + self.advance_epoch_with_rate(new_rate_activation_epoch) + .await; + let epoch = self.mollusk.sysvars.clock.epoch; + self.compare_stake_history(epoch - 1).await; + + let banks_effective = self.get_banks_effective_stake(stake).await; + let mollusk_effective = self.get_mollusk_effective_stake(stake_account); + assert_eq!( + banks_effective, mollusk_effective, + "Epoch {}: effective stake mismatch", + epoch + ); + self.compare_account_state(stake, stake_account).await; + } + + /// Advance epochs until stake is fully activated (effective == delegated amount) + /// Returns number of epochs advanced. Panics if not activated within max_epochs. + async fn advance_until_fully_activated( + &mut self, + stake: &Pubkey, + stake_account: &AccountSharedData, + max_epochs: u64, + ) -> u64 { + let stake_state: StakeStateV2 = bincode::deserialize(stake_account.data()).unwrap(); + let target_amount = match stake_state { + StakeStateV2::Stake(_, stake_data, _) => stake_data.delegation.stake, + _ => panic!("Stake account not in delegated state"), + }; + + let mut epochs_advanced = 0; + loop { + self.advance_and_compare_stake(stake, stake_account).await; + epochs_advanced += 1; + + let banks_effective = self.get_banks_effective_stake(stake).await; + if banks_effective == target_amount { + return epochs_advanced; + } + + assert!( + epochs_advanced < max_epochs, + "Stake did not fully activate within {} epochs", + max_epochs + ); + } + } + + /// Advance epochs until stake is fully deactivated (effective == 0) + /// Returns number of epochs advanced. Panics if not deactivated within max_epochs. + async fn advance_until_fully_deactivated( + &mut self, + stake: &Pubkey, + stake_account: &AccountSharedData, + max_epochs: u64, + ) -> u64 { + let mut epochs_advanced = 0; + loop { + self.advance_and_compare_stake(stake, stake_account).await; + epochs_advanced += 1; + + let banks_effective = self.get_banks_effective_stake(stake).await; + if banks_effective == 0 { + return epochs_advanced; + } + + assert!( + epochs_advanced < max_epochs, + "Stake did not fully deactivate within {} epochs", + max_epochs + ); + } + } + + /// Advance one epoch and compare multiple stake accounts between implementations + async fn advance_and_compare_stakes(&mut self, stakes: &[(&Pubkey, &AccountSharedData)]) { + self.advance_and_compare_stakes_with_rate(stakes, Some(0)) + .await; + } + + /// Advance one epoch with custom warmup rate and compare multiple stake accounts + async fn advance_and_compare_stakes_with_rate( + &mut self, + stakes: &[(&Pubkey, &AccountSharedData)], + new_rate_activation_epoch: Option, + ) { + self.advance_epoch_with_rate(new_rate_activation_epoch) + .await; + let epoch = self.mollusk.sysvars.clock.epoch; + self.compare_stake_history(epoch - 1).await; + + for (stake, stake_account) in stakes { + let banks_effective = self.get_banks_effective_stake(stake).await; + let mollusk_effective = self.get_mollusk_effective_stake(stake_account); + assert_eq!( + banks_effective, mollusk_effective, + "Epoch {}: stake {} mismatch", + epoch, stake + ); + self.compare_account_state(stake, stake_account).await; + } + } + + /// Fund a stake account on BanksClient side + async fn fund_stake_account(&mut self, stake: &Pubkey, amount: u64) { + let fund_ix = + system_instruction::transfer(&self.program_test_ctx.payer.pubkey(), stake, amount); + let transaction = Transaction::new_signed_with_payer( + &[fund_ix], + Some(&self.program_test_ctx.payer.pubkey()), + &[&self.program_test_ctx.payer], + self.program_test_ctx.last_blockhash, + ); + self.program_test_ctx + .banks_client + .process_transaction(transaction) + .await + .unwrap(); + } + + /// Create, initialize, and fund a stake account + /// Returns (stake pubkey, stake account, staker keypair) + async fn create_and_fund_stake( + &mut self, + staked_amount: u64, + lockup: &Lockup, + ) -> (Pubkey, AccountSharedData, Keypair) { + let stake = self.create_blank_stake_account().await; + let staker = Keypair::new(); + let authorized = Authorized { + staker: staker.pubkey(), + withdrawer: Pubkey::new_unique(), + }; + + let mut stake_account = self + .initialize_stake_account(&stake, &authorized, lockup) + .await; + + stake_account.set_lamports(stake_account.lamports() + staked_amount); + self.fund_stake_account(&stake, staked_amount).await; + + (stake, stake_account, staker) + } +} + +// Test single delegation activation over various epoch counts +// Including large numbers representing years of operation (assuming ~182 epochs/year @ 2 days/epoch) +// Also tests stake history depth limit (512 epochs) to ensure old entries drop correctly +#[test_case(5 ; "five_epochs")] +#[test_case(10 ; "ten_epochs")] +#[test_case(20 ; "twenty_epochs")] +#[test_case(50 ; "fifty_epochs")] +#[test_case(100 ; "hundred_epochs")] +#[test_case(182 ; "one_year")] +#[test_case(500 ; "five_hundred_epochs")] +#[test_case(512 ; "stake_history_depth_boundary")] +#[test_case(515 ; "beyond_stake_history_depth")] +#[test_case(1800 ; "ten_years")] +#[tokio::test] +async fn test_single_delegation_activation(num_epochs: u64) { + let mut ctx = DualContext::new().await; + + // Create, initialize, and fund stake account + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(MINIMUM_DELEGATION, &Lockup::default()) + .await; + + // Delegate stake + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + for i in 0..num_epochs { + ctx.advance_and_compare_stake(&stake, &stake_account).await; + + let epoch = ctx.mollusk.sysvars.clock.epoch; + if num_epochs >= 100 && (i % 100 == 0 || i == num_epochs - 1) { + ctx.verify_background_stake_preservation(epoch - 1, ctx.background_stake) + .await; + } + } +} + +// Test deactivation at different points in lifecycle: immediately, during warmup, or after activation +#[test_case(0 ; "immediate_same_epoch")] +#[test_case(1 ; "early_warmup")] +#[test_case(2 ; "mid_warmup")] +#[test_case(3 ; "late_warmup")] +#[test_case(5 ; "after_activation")] +#[tokio::test] +async fn test_deactivation_timing(epochs_before_deactivate: u64) { + let mut ctx = DualContext::new().await; + + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(MINIMUM_DELEGATION, &Lockup::default()) + .await; + + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + for _ in 0..epochs_before_deactivate { + ctx.advance_and_compare_stake(&stake, &stake_account).await; + } + + ctx.deactivate_stake(&stake, &mut stake_account, &staker) + .await; + + ctx.advance_until_fully_deactivated(&stake, &stake_account, 20) + .await; +} + +// Test various stake amounts: minimum, small, large +#[test_case(MINIMUM_DELEGATION ; "minimum_delegation")] +#[test_case(MINIMUM_DELEGATION * 100 ; "small_amount")] +#[test_case(250_000 * 1_000_000_000 ; "large_amount")] +#[tokio::test] +async fn test_stake_amounts_activation(staked_amount: u64) { + let mut ctx = DualContext::new().await; + + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(staked_amount, &Lockup::default()) + .await; + + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + ctx.advance_until_fully_activated(&stake, &stake_account, 50) + .await; +} + +// Test deactivation with various stake amounts to verify warmup rates match during cooldown +#[test_case(MINIMUM_DELEGATION ; "minimum_delegation")] +#[test_case(MINIMUM_DELEGATION * 100 ; "small_amount")] +#[test_case(250_000 * 1_000_000_000 ; "large_amount")] +#[tokio::test] +async fn test_stake_amounts_deactivation(staked_amount: u64) { + let mut ctx = DualContext::new().await; + + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(staked_amount, &Lockup::default()) + .await; + + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + ctx.advance_until_fully_activated(&stake, &stake_account, 50) + .await; + + ctx.deactivate_stake(&stake, &mut stake_account, &staker) + .await; + + ctx.advance_until_fully_deactivated(&stake, &stake_account, 50) + .await; +} + +// Test multiple stakes delegating simultaneously, optionally to multiple vote accounts +#[test_case(5, 1 ; "five_stakes_one_vote")] +#[test_case(10, 1 ; "ten_stakes_one_vote")] +#[test_case(20, 1 ; "twenty_stakes_one_vote")] +#[test_case(4, 2 ; "four_stakes_two_votes")] +#[tokio::test] +async fn test_multiple_simultaneous_delegations(num_stakes: usize, num_vote_accounts: usize) { + let mut ctx = DualContext::new().await; + + // Create additional vote accounts if needed + let mut vote_accounts = vec![(ctx.vote_account, ctx.vote_account_data.clone())]; + for _ in 1..num_vote_accounts { + let vote_account = Pubkey::new_unique(); + let vote_account_data = create_vote_account(); + ctx.program_test_ctx + .set_account(&vote_account, &vote_account_data.clone()); + vote_accounts.push((vote_account, vote_account_data)); + } + + let mut stakes = Vec::new(); + let mut stake_accounts = Vec::new(); + let mut stakers = Vec::new(); + + for i in 0..num_stakes { + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(MINIMUM_DELEGATION, &Lockup::default()) + .await; + + // Delegate to vote account (alternate if multiple) + let (vote_account, vote_account_data) = &vote_accounts[i % num_vote_accounts]; + ctx.delegate_stake_to( + &stake, + &mut stake_account, + &staker, + vote_account, + vote_account_data, + ) + .await; + + stakes.push(stake); + stake_accounts.push(stake_account); + stakers.push(staker); + } + + for _ in 0..10 { + let stake_refs: Vec<_> = stakes.iter().zip(stake_accounts.iter()).collect(); + ctx.advance_and_compare_stakes(&stake_refs).await; + } +} + +#[tokio::test] +async fn test_concurrent_activation_and_deactivation() { + let mut ctx = DualContext::new().await; + + let (stake_a, mut stake_account_a, staker_a) = ctx + .create_and_fund_stake(MINIMUM_DELEGATION, &Lockup::default()) + .await; + + let (stake_b, mut stake_account_b, staker_b) = ctx + .create_and_fund_stake(MINIMUM_DELEGATION, &Lockup::default()) + .await; + + ctx.delegate_stake(&stake_b, &mut stake_account_b, &staker_b) + .await; + ctx.advance_epoch().await; + + ctx.deactivate_stake(&stake_b, &mut stake_account_b, &staker_b) + .await; + ctx.delegate_stake(&stake_a, &mut stake_account_a, &staker_a) + .await; + + let epoch = ctx.mollusk.sysvars.clock.epoch; + + for _ in 0..5 { + ctx.advance_epoch().await; + let current_epoch = ctx.mollusk.sysvars.clock.epoch; + ctx.compare_stake_history(current_epoch - 1).await; + + let banks_effective_a = ctx.get_banks_effective_stake(&stake_a).await; + let mollusk_effective_a = ctx.get_mollusk_effective_stake(&stake_account_a); + assert_eq!(banks_effective_a, mollusk_effective_a); + ctx.compare_account_state(&stake_a, &stake_account_a).await; + + let banks_effective_b = ctx.get_banks_effective_stake(&stake_b).await; + let mollusk_effective_b = ctx.get_mollusk_effective_stake(&stake_account_b); + assert_eq!(banks_effective_b, mollusk_effective_b); + ctx.compare_account_state(&stake_b, &stake_account_b).await; + } + + let banks_history = ctx.get_banks_stake_history().await; + let mollusk_history = ctx.get_mollusk_stake_history(); + + let banks_entry = banks_history.get(epoch).unwrap(); + let mollusk_entry = mollusk_history.get(epoch).unwrap(); + + assert_eq!(banks_entry.activating, mollusk_entry.activating); + assert_eq!(banks_entry.deactivating, mollusk_entry.deactivating); +} + +#[tokio::test] +async fn test_reactivation_after_deactivation() { + let mut ctx = DualContext::new().await; + + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(MINIMUM_DELEGATION, &Lockup::default()) + .await; + + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + for _ in 0..2 { + ctx.advance_and_compare_stake(&stake, &stake_account).await; + } + + ctx.deactivate_stake(&stake, &mut stake_account, &staker) + .await; + + ctx.advance_until_fully_deactivated(&stake, &stake_account, 20) + .await; + + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + ctx.advance_until_fully_activated(&stake, &stake_account, 20) + .await; +} + +#[tokio::test] +async fn test_staggered_delegations_over_epochs() { + let mut ctx = DualContext::new().await; + + let num_stakes = 5; + let mut stakes = Vec::new(); + let mut stake_accounts = Vec::new(); + let mut stakers = Vec::new(); + + // Create all stakes + for _ in 0..num_stakes { + let stake = ctx.create_blank_stake_account().await; + let staker = Keypair::new(); + let authorized = Authorized { + staker: staker.pubkey(), + withdrawer: Pubkey::new_unique(), + }; + + let stake_account = ctx + .initialize_stake_account(&stake, &authorized, &Lockup::default()) + .await; + + stakes.push(stake); + stake_accounts.push(stake_account); + stakers.push(staker); + } + + // Stagger delegations across epochs (0, 5, 10, 15, 20) + for (idx, i) in [0, 5, 10, 15, 20].iter().enumerate() { + // Advance to target epoch + while ctx.mollusk.sysvars.clock.epoch < *i { + ctx.advance_epoch().await; + } + + // Fund and delegate + let staked_amount = MINIMUM_DELEGATION; + let current_lamports = stake_accounts[idx].lamports(); + stake_accounts[idx].set_lamports(current_lamports + staked_amount); + ctx.fund_stake_account(&stakes[idx], staked_amount).await; + + ctx.delegate_stake(&stakes[idx], &mut stake_accounts[idx], &stakers[idx]) + .await; + } + + while ctx.mollusk.sysvars.clock.epoch < 30 { + let stake_refs: Vec<_> = stakes.iter().zip(stake_accounts.iter()).collect(); + ctx.advance_and_compare_stakes(&stake_refs).await; + } +} + +#[tokio::test] +async fn test_mixed_lifecycle_stress() { + let mut ctx = DualContext::new().await; + + let total_stakes = 20; + let mut stakes = Vec::new(); + let mut stake_accounts = Vec::new(); + let mut stakers = Vec::new(); + + // Create all stakes + for _ in 0..total_stakes { + let stake = ctx.create_blank_stake_account().await; + let staker = Keypair::new(); + let authorized = Authorized { + staker: staker.pubkey(), + withdrawer: Pubkey::new_unique(), + }; + + let mut stake_account = ctx + .initialize_stake_account(&stake, &authorized, &Lockup::default()) + .await; + + let staked_amount = MINIMUM_DELEGATION; + stake_account.set_lamports(stake_account.lamports() + staked_amount); + ctx.fund_stake_account(&stake, staked_amount).await; + + stakes.push(stake); + stake_accounts.push(stake_account); + stakers.push(staker); + } + + // Create various lifecycle states: + // 5 activating (delegate in epoch 0) + for i in 0..5 { + ctx.delegate_stake(&stakes[i], &mut stake_accounts[i], &stakers[i]) + .await; + } + + ctx.advance_epoch().await; // Epoch 1 + + // 5 active (delegated in epoch 0, now partially active) + // Already done above, just advancing + + // 5 more activating (delegate in epoch 1) + for i in 5..10 { + ctx.delegate_stake(&stakes[i], &mut stake_accounts[i], &stakers[i]) + .await; + } + + ctx.advance_epoch().await; // Epoch 2 + + // 5 deactivating (deactivate some active ones) + for i in 0..5 { + ctx.deactivate_stake(&stakes[i], &mut stake_accounts[i], &stakers[i]) + .await; + } + + for _ in 0..10 { + let stake_refs: Vec<_> = stakes.iter().zip(stake_accounts.iter()).collect(); + ctx.advance_and_compare_stakes(&stake_refs).await; + } +} + +// Test repeated cycles of delegation and deactivation with full transitions +#[test_case(2 ; "two_cycles")] +#[test_case(3 ; "three_cycles")] +#[tokio::test] +async fn test_repeated_delegation_cycles(num_cycles: usize) { + let mut ctx = DualContext::new().await; + + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(MINIMUM_DELEGATION, &Lockup::default()) + .await; + + for _ in 0..num_cycles { + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + ctx.advance_until_fully_activated(&stake, &stake_account, 20) + .await; + + ctx.deactivate_stake(&stake, &mut stake_account, &staker) + .await; + ctx.advance_until_fully_deactivated(&stake, &stake_account, 20) + .await; + } +} + +#[tokio::test] +async fn test_delegate_after_full_deactivation_different_validator() { + let mut ctx = DualContext::new().await; + + let vote_account_b = Pubkey::new_unique(); + let vote_account_b_data = create_vote_account(); + ctx.program_test_ctx + .set_account(&vote_account_b, &vote_account_b_data.clone()); + + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(MINIMUM_DELEGATION, &Lockup::default()) + .await; + + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + ctx.advance_until_fully_activated(&stake, &stake_account, 20) + .await; + + ctx.deactivate_stake(&stake, &mut stake_account, &staker) + .await; + + ctx.advance_until_fully_deactivated(&stake, &stake_account, 20) + .await; + + ctx.delegate_stake_to( + &stake, + &mut stake_account, + &staker, + &vote_account_b, + &vote_account_b_data, + ) + .await; + + ctx.advance_until_fully_activated(&stake, &stake_account, 20) + .await; +} + +#[tokio::test] +async fn test_delegate_after_deactivation_during_warmup() { + let mut ctx = DualContext::new().await; + + let vote_account_b = Pubkey::new_unique(); + let vote_account_b_data = create_vote_account(); + ctx.program_test_ctx + .set_account(&vote_account_b, &vote_account_b_data.clone()); + + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(MINIMUM_DELEGATION, &Lockup::default()) + .await; + + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + ctx.advance_and_compare_stake(&stake, &stake_account).await; + ctx.advance_and_compare_stake(&stake, &stake_account).await; + + ctx.deactivate_stake(&stake, &mut stake_account, &staker) + .await; + + ctx.advance_until_fully_deactivated(&stake, &stake_account, 20) + .await; + + ctx.delegate_stake_to( + &stake, + &mut stake_account, + &staker, + &vote_account_b, + &vote_account_b_data, + ) + .await; + + ctx.advance_until_fully_activated(&stake, &stake_account, 20) + .await; +} + +#[tokio::test] +async fn test_delegate_during_deactivation_cooldown() { + let mut ctx = DualContext::new().await; + + let vote_account_b = Pubkey::new_unique(); + let vote_account_b_data = create_vote_account(); + ctx.program_test_ctx + .set_account(&vote_account_b, &vote_account_b_data.clone()); + + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(MINIMUM_DELEGATION, &Lockup::default()) + .await; + + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + ctx.advance_until_fully_activated(&stake, &stake_account, 20) + .await; + + ctx.deactivate_stake(&stake, &mut stake_account, &staker) + .await; + + ctx.advance_and_compare_stake(&stake, &stake_account).await; + ctx.advance_and_compare_stake(&stake, &stake_account).await; + + ctx.delegate_stake_to( + &stake, + &mut stake_account, + &staker, + &vote_account_b, + &vote_account_b_data, + ) + .await; + + ctx.compare_account_state(&stake, &stake_account).await; + + let banks_effective = ctx.get_banks_effective_stake(&stake).await; + let mollusk_effective = ctx.get_mollusk_effective_stake(&stake_account); + assert_eq!( + banks_effective, mollusk_effective, + "Effective stake mismatch immediately after delegating during deactivation" + ); + + ctx.advance_until_fully_activated(&stake, &stake_account, 20) + .await; +} + +#[tokio::test] +async fn test_old_warmup_rate_activation() { + let mut ctx = DualContext::new().await; + + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(MINIMUM_DELEGATION, &Lockup::default()) + .await; + + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + for _ in 0..10 { + ctx.advance_and_compare_stake_with_rate(&stake, &stake_account, None) + .await; + } +} + +#[tokio::test] +async fn test_old_warmup_rate_deactivation() { + let mut ctx = DualContext::new().await; + + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(MINIMUM_DELEGATION, &Lockup::default()) + .await; + + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + for _ in 0..5 { + ctx.advance_and_compare_stake_with_rate(&stake, &stake_account, None) + .await; + } + + ctx.deactivate_stake(&stake, &mut stake_account, &staker) + .await; + + for _ in 0..10 { + ctx.advance_and_compare_stake_with_rate(&stake, &stake_account, None) + .await; + } +} + +#[tokio::test] +async fn test_warmup_rate_transition() { + let mut ctx = DualContext::new().await; + + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(MINIMUM_DELEGATION, &Lockup::default()) + .await; + + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + for _ in 0..5 { + ctx.advance_and_compare_stake_with_rate(&stake, &stake_account, None) + .await; + } + + let transition_epoch = ctx.mollusk.sysvars.clock.epoch; + + for _ in 0..10 { + ctx.advance_and_compare_stake_with_rate(&stake, &stake_account, Some(transition_epoch)) + .await; + } +} + +#[tokio::test] +async fn test_massive_stake_relative_to_background() { + let mut ctx = DualContext::new().await; + + // Get the background stake - use epoch 0's effective stake as the baseline + let banks_stake_history = ctx.get_banks_stake_history().await; + let background_effective = banks_stake_history + .get(0) + .map(|entry| entry.effective) + .expect("Epoch 0 must have background stake"); + + // Stake 50% of background effective stake - this is MASSIVE and will significantly + // affect warmup rates (warmup is limited by new_stake relative to existing effective) + let staked_amount = background_effective / 2; + + if staked_amount < MINIMUM_DELEGATION { + panic!( + "Bad test: background stake {} too small (need at least {})", + background_effective, + MINIMUM_DELEGATION * 2 + ); + } + + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(staked_amount, &Lockup::default()) + .await; + + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + let epochs_to_activate = ctx + .advance_until_fully_activated(&stake, &stake_account, 50) + .await; + + for epoch_offset in 0..epochs_to_activate { + let epoch = ctx.mollusk.sysvars.clock.epoch - epochs_to_activate + epoch_offset; + ctx.verify_background_stake_preservation(epoch, background_effective) + .await; + } +} + +#[tokio::test] +async fn test_massive_stake_deactivation() { + let mut ctx = DualContext::new().await; + + let banks_stake_history = ctx.get_banks_stake_history().await; + let background_effective = banks_stake_history + .get(0) + .map(|entry| entry.effective) + .expect("Epoch 0 must have background stake"); + + let staked_amount = background_effective / 2; + + if staked_amount < MINIMUM_DELEGATION { + panic!( + "Bad test: background stake {} too small (need at least {})", + background_effective, + MINIMUM_DELEGATION * 2 + ); + } + + let (stake, mut stake_account, staker) = ctx + .create_and_fund_stake(staked_amount, &Lockup::default()) + .await; + + ctx.delegate_stake(&stake, &mut stake_account, &staker) + .await; + + ctx.advance_until_fully_activated(&stake, &stake_account, 50) + .await; + + ctx.deactivate_stake(&stake, &mut stake_account, &staker) + .await; + + let epochs_to_deactivate = ctx + .advance_until_fully_deactivated(&stake, &stake_account, 50) + .await; + + for epoch_offset in 0..epochs_to_deactivate { + let epoch = ctx.mollusk.sysvars.clock.epoch - epochs_to_deactivate + epoch_offset; + ctx.verify_background_stake_preservation(epoch, background_effective) + .await; + } +} diff --git a/program/tests/withdraw.rs b/program/tests/withdraw.rs new file mode 100644 index 00000000..a9226855 --- /dev/null +++ b/program/tests/withdraw.rs @@ -0,0 +1,209 @@ +#![allow(clippy::arithmetic_side_effects)] + +mod helpers; + +use { + helpers::{ + context::StakeTestContext, instruction_builders::WithdrawConfig, lifecycle::StakeLifecycle, + }, + mollusk_svm::result::Check, + solana_account::{AccountSharedData, WritableAccount}, + solana_program_error::ProgramError, + solana_pubkey::Pubkey, + solana_rent::Rent, + solana_stake_interface::state::StakeStateV2, + solana_stake_program::id, + test_case::test_case, +}; + +#[test_case(StakeLifecycle::Uninitialized; "uninitialized")] +#[test_case(StakeLifecycle::Initialized; "initialized")] +#[test_case(StakeLifecycle::Activating; "activating")] +#[test_case(StakeLifecycle::Active; "active")] +#[test_case(StakeLifecycle::Deactivating; "deactivating")] +#[test_case(StakeLifecycle::Deactive; "deactive")] +#[test_case(StakeLifecycle::Closed; "closed")] +fn test_withdraw_stake(withdraw_source_type: StakeLifecycle) { + let mut ctx = StakeTestContext::new(); + let staked_amount = ctx.minimum_delegation; + let wallet_rent_exempt_reserve = Rent::default().minimum_balance(0); + + // Create source stake account at the specified lifecycle stage + let (withdraw_source, mut withdraw_source_account) = ctx + .stake_account(withdraw_source_type) + .staked_amount(staked_amount.unwrap()) + .build(); + + // Create recipient account + let recipient = Pubkey::new_unique(); + let mut recipient_account = AccountSharedData::default(); + recipient_account.set_lamports(wallet_rent_exempt_reserve); + + // Determine signer based on lifecycle stage + let signer = if withdraw_source_type == StakeLifecycle::Uninitialized + || withdraw_source_type == StakeLifecycle::Closed + { + withdraw_source // Self-signed for uninitialized/closed + } else { + ctx.withdrawer + }; + + // Withdraw that would end rent-exemption always fails + let rent_spillover = if withdraw_source_type == StakeLifecycle::Closed { + ctx.rent_exempt_reserve - Rent::default().minimum_balance(0) + 1 + } else { + 1 + }; + + // For initialized/delegated accounts, the program itself checks and fails with InsufficientFunds + // For uninitialized/closed accounts, the program succeeds but leaves accounts below rent exemption + if withdraw_source_type == StakeLifecycle::Uninitialized + || withdraw_source_type == StakeLifecycle::Closed + { + // Expect program success but rent check should fail - catch the panic + let result = std::panic::catch_unwind(std::panic::AssertUnwindSafe(|| { + ctx.process_with(WithdrawConfig { + stake: (&withdraw_source, &withdraw_source_account), + override_signer: Some(&signer), + recipient: (&recipient, &recipient_account), + amount: staked_amount.unwrap() + rent_spillover, + }) + .checks(&[Check::success(), Check::all_rent_exempt()]) + .execute() + })); + // The rent exemption check should panic + assert!( + result.is_err(), + "Expected rent exemption check to fail for uninitialized/closed withdraw" + ); + } else { + // Program fails with InsufficientFunds + ctx.process_with(WithdrawConfig { + stake: (&withdraw_source, &withdraw_source_account), + override_signer: Some(&signer), + recipient: (&recipient, &recipient_account), + amount: staked_amount.unwrap() + rent_spillover, + }) + .checks(&[Check::err(ProgramError::InsufficientFunds)]) + .execute(); + } + + if withdraw_source_type.withdraw_minimum_enforced() { + // Withdraw active or activating stake fails + ctx.process_with(WithdrawConfig { + stake: (&withdraw_source, &withdraw_source_account), + override_signer: Some(&signer), + recipient: (&recipient, &recipient_account), + amount: staked_amount.unwrap(), + }) + .checks(&[Check::err(ProgramError::InsufficientFunds)]) + .execute(); + + // Grant rewards + let reward_amount = 10; + withdraw_source_account + .checked_add_lamports(reward_amount) + .unwrap(); + + // Withdraw in excess of rewards is not allowed + ctx.process_with(WithdrawConfig { + stake: (&withdraw_source, &withdraw_source_account), + override_signer: Some(&signer), + recipient: (&recipient, &recipient_account), + amount: reward_amount + 1, + }) + .checks(&[Check::err(ProgramError::InsufficientFunds)]) + .execute(); + + // Withdraw rewards is allowed + ctx.process_with(WithdrawConfig { + stake: (&withdraw_source, &withdraw_source_account), + override_signer: Some(&signer), + recipient: (&recipient, &recipient_account), + amount: reward_amount, + }) + .checks(&[ + Check::success(), + Check::account(&recipient) + .lamports(reward_amount + wallet_rent_exempt_reserve) + .build(), + ]) + .test_missing_signers(true) + .execute(); + } else { + // Withdraw that leaves rent behind is allowed + let result = ctx + .process_with(WithdrawConfig { + stake: (&withdraw_source, &withdraw_source_account), + override_signer: Some(&signer), + recipient: (&recipient, &recipient_account), + amount: staked_amount.unwrap(), + }) + .checks(&[ + Check::success(), + Check::account(&recipient) + .lamports(staked_amount.unwrap() + wallet_rent_exempt_reserve) + .build(), + ]) + .test_missing_signers(true) + .execute(); + + withdraw_source_account = result.resulting_accounts[0].1.clone().into(); + + // Full withdraw is allowed (add back staked_amount) + withdraw_source_account + .checked_add_lamports(staked_amount.unwrap()) + .unwrap(); + + let recipient2 = Pubkey::new_unique(); + let mut recipient2_account = AccountSharedData::default(); + recipient2_account.set_lamports(wallet_rent_exempt_reserve); + + ctx.process_with(WithdrawConfig { + stake: (&withdraw_source, &withdraw_source_account), + override_signer: Some(&signer), + recipient: (&recipient2, &recipient2_account), + amount: staked_amount.unwrap() + ctx.rent_exempt_reserve, + }) + .checks(&[ + Check::success(), + Check::account(&recipient2) + .lamports( + staked_amount.unwrap() + ctx.rent_exempt_reserve + wallet_rent_exempt_reserve, + ) + .build(), + ]) + .test_missing_signers(true) + .execute(); + } +} + +#[test] +fn test_withdraw_from_rewards_pool() { + let ctx = StakeTestContext::new(); + let staked_amount = ctx.minimum_delegation; + + // Create a rewards pool account + let rewards_pool_address = Pubkey::new_unique(); + let rewards_pool_data = AccountSharedData::new_data_with_space( + ctx.rent_exempt_reserve + staked_amount.unwrap(), + &StakeStateV2::RewardsPool, + StakeStateV2::size_of(), + &id(), + ) + .unwrap(); + + let recipient = Pubkey::new_unique(); + let recipient_account = AccountSharedData::default(); + + // Withdraw from program-owned non-stake accounts is not allowed + ctx.process_with(WithdrawConfig { + stake: (&rewards_pool_address, &rewards_pool_data), + recipient: (&recipient, &recipient_account), + amount: staked_amount.unwrap(), + override_signer: None, + }) + .checks(&[Check::err(ProgramError::InvalidAccountData)]) + .test_missing_signers(false) + .execute(); +}