From abba4122fba2d206a260761241396d478b347d39 Mon Sep 17 00:00:00 2001 From: Hardhat Chad Date: Fri, 16 Feb 2024 20:04:14 +0000 Subject: [PATCH] failed admin test --- src/processor/update_admin.rs | 2 +- src/processor/update_difficulty.rs | 2 +- tests/test_update_admin.rs | 6 ++++++ 3 files changed, 8 insertions(+), 2 deletions(-) diff --git a/src/processor/update_admin.rs b/src/processor/update_admin.rs index 3a91ad9..96931aa 100644 --- a/src/processor/update_admin.rs +++ b/src/processor/update_admin.rs @@ -23,7 +23,7 @@ pub fn process_update_admin<'a, 'info>( // Validate admin signer let mut treasury_data = treasury_info.data.borrow_mut(); let mut treasury = Treasury::try_from_bytes_mut(&mut treasury_data)?; - if !treasury.admin.eq(&signer.key) { + if treasury.admin.ne(&signer.key) { return Err(ProgramError::MissingRequiredSignature); } diff --git a/src/processor/update_difficulty.rs b/src/processor/update_difficulty.rs index 6e32cf4..f1c5952 100644 --- a/src/processor/update_difficulty.rs +++ b/src/processor/update_difficulty.rs @@ -25,7 +25,7 @@ pub fn process_update_difficulty<'a, 'info>( // Validate admin signer let mut treasury_data = treasury_info.data.borrow_mut(); let mut treasury = Treasury::try_from_bytes_mut(&mut treasury_data)?; - if !treasury.admin.eq(&signer.key) { + if treasury.admin.ne(&signer.key) { return Err(ProgramError::MissingRequiredSignature); } diff --git a/tests/test_update_admin.rs b/tests/test_update_admin.rs index f057bf1..c070c09 100644 --- a/tests/test_update_admin.rs +++ b/tests/test_update_admin.rs @@ -40,6 +40,12 @@ async fn test_update_admin() { treasury_.total_claimed_rewards, treasury.total_claimed_rewards ); + + // Submit another update admin ix + let ix = ore::instruction::update_admin(payer.pubkey(), payer.pubkey()); + let tx = Transaction::new_signed_with_payer(&[ix], Some(&payer.pubkey()), &[&payer], blockhash); + let res = banks.process_transaction(tx).await; + assert!(res.is_err()); } async fn setup_program_test_env() -> (BanksClient, Keypair, Hash) {