fix tests

ananas-block · ananas-block · commit 5bc72d580ed0 · 2025-06-15T16:53:26.000+01:00
diff --git a/program-libs/account-checks/tests/tests.rs b/program-libs/account-checks/tests/tests.rs
@@ -170,7 +170,7 @@ fn test_check_account_info_mut() {
         set_discriminator::<TestStruct>(&mut account.data).unwrap();
         assert_eq!(
             check_account_info_mut::<TestStruct, _>(&owner.to_bytes(), &account.get_account_info()),
-            Err(AccountError::AccountMutable)
+            Err(AccountError::AccountNotMutable)
         );
     }
 
@@ -193,7 +193,7 @@ fn test_check_account_info_mut() {
         account_info_init::<TestStruct, _>(&account).unwrap();
         assert_eq!(
             check_account_info_mut::<TestStruct, _>(&owner, &account),
-            Err(AccountError::AccountMutable)
+            Err(AccountError::AccountNotMutable)
         );
     }
 }
diff --git a/programs/system/src/invoke/instruction.rs b/programs/system/src/invoke/instruction.rs
@@ -1,3 +1,4 @@
+use light_account_checks::checks::check_signer;
 use pinocchio::{account_info::AccountInfo, program_error::ProgramError};
 
 use crate::{
@@ -46,7 +47,8 @@ impl<'info> InvokeInstruction<'info> {
         let fee_payer = check_fee_payer(accounts.next())?;
 
         // Fee payer and authority can be the same account in case of invoke.
-        let authority = check_fee_payer(accounts.next())?;
+        let authority = accounts.next().ok_or(ProgramError::NotEnoughAccountKeys)?;
+        check_signer(authority).map_err(ProgramError::from)?;
 
         let registered_program_pda = check_non_mut_account_info(accounts.next())?;
 

Original file line number	Diff line number	Diff line change
`@@ -170,7 +170,7 @@ fn test_check_account_info_mut() {`
`170`	`170`	`set_discriminator::<TestStruct>(&mut account.data).unwrap();`
`171`	`171`	`assert_eq!(`
`172`	`172`	`check_account_info_mut::<TestStruct, _>(&owner.to_bytes(), &account.get_account_info()),`
`173`		`- Err(AccountError::AccountMutable)`
	`173`	`+ Err(AccountError::AccountNotMutable)`
`174`	`174`	`);`
`175`	`175`	`}`
`176`	`176`
`@@ -193,7 +193,7 @@ fn test_check_account_info_mut() {`
`193`	`193`	`account_info_init::<TestStruct, _>(&account).unwrap();`
`194`	`194`	`assert_eq!(`
`195`	`195`	`check_account_info_mut::<TestStruct, _>(&owner, &account),`
`196`		`- Err(AccountError::AccountMutable)`
	`196`	`+ Err(AccountError::AccountNotMutable)`
`197`	`197`	`);`
`198`	`198`	`}`
`199`	`199`	`}`