add common errors update integration tests

Author: ananas-block

COMMON_ERRORS.md

@@ -0,0 +1,22 @@
+1. │ ERROR: Not enough accounts. Requested 'burn change account owner' at index 3 but only 0 accounts available. programs/compressed-token/program/src/mint_action/burn.rs:142:41
+   - means that packed accounts doesn't contain enough accounts
+   -
+2. `NotEnoughSigners`
+   - `create_and_send_transaction(&[instruction], &payer.pubkey(), &signers)`
+   - needs more signers
+   - signers must be unique you must not pass the same twice it will result in an error
+3. `CompressedAccountError::ZeroCopyExpectedAddress => 12017`
+   - when setting output compressed accounts in a zero copy we expect an address to be provided the address is allocated as Some by the ZeroCopyConfig but None is provided
+   - any error that contains Expected and is an CompressedAccountError means this for the specied compressed account field
+4. `Signer/Program cannot write into an account it doesn't own.`
+    ```mode Small
+      │ Signer/Program cannot write into an account it doesn't own. Write access check failed, compressed account owner [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0] !=  invoking_program_id [9, 21, 163, 87, 35, 121, 78, 143, 182, 93, 7, 91, 107, 114, 105, 156, 56, 221, 2, 229, 148, 139, 117, 176, 229, 160, 65, 142, 128, 151, 91, 68].
+      │ Program SySTEM1eSU2p4BGQfQpimFEWWSC1XDFeun3Nqzz3rT7 consumed 17422 of 1186879 compute units
+      │ Program SySTEM1eSU2p4BGQfQpimFEWWSC1XDFeun3Nqzz3rT7 failed: custom program error: 0x177d
+      │ Program cTokenmWW8bLPjZEBAUgYy3zKxQZW6VKi7bqNFEVv3m consumed 230543 of 1400000 compute units
+      │ Program cTokenmWW8bLPjZEBAUgYy3zKxQZW6VKi7bqNFEVv3m failed: custom program error: 0x177d
+    ```
+    - the compressed output account owner is not set
+5. ` Program SySTEM1eSU2p4BGQfQpimFEWWSC1XDFeun3Nqzz3rT7 failed: custom program error: 0x179a`
+    -
+    - the index for a state Merkle tree in the packed accounts is wrong

INTEGRATION_TESTING.md

@@ -31,7 +31,7 @@ program-tests/
 **All programs in `programs/**` have corresponding integration test programs:**
 
 - **Account Compression Program** (`programs/account-compression/`) → `program-tests/account-compression-test/`
-- **Compressed Token Program** (`programs/compressed-token/`) → `program-tests/compressed-token-test/` 
+- **Compressed Token Program** (`programs/compressed-token/`) → `program-tests/compressed-token-test/`
 - **Registry Program** (`programs/registry/`) → `program-tests/registry-test/`
 - **System Program** (`programs/system/`) → `program-tests/system-test/`
 
@@ -86,7 +86,7 @@ pub async fn assert_mint_operation(
 ) {
     // Get actual state
     let actual = get_actual_state(rpc, operation_params).await;
-    
+
     // Single comprehensive assertion
     assert_eq!(actual, expected_output);
 }
@@ -102,14 +102,14 @@ pub async fn assert_mint_operation(
 #[tokio::test]
 async fn test_complete_token_lifecycle() {
     // 1. Create mint
-    // 2. Mint tokens  
+    // 2. Mint tokens
     // 3. Transfer tokens
     // 4. Compress/decompress
     // Each step verified with assertions
 }
 ```
 
-### Error Test Coverage  
+### Error Test Coverage
 **Every error condition must have a failing test**
 
 ```rust
@@ -139,14 +139,14 @@ assert_eq!(actual_state, expected_end_state);
 {
     // Parse complete state before operation
     let mut expected_after_state = parse_state_before(&state_data_before);
-    
+
     // Apply the expected changes to the before state
     expected_after_state.field1 = new_value;  // Only change what should change
     expected_after_state.amount -= transfer_amount;
-    
+
     // Parse actual state after operation
     let actual_after_state = parse_state_after(&state_data_after);
-    
+
     // Single comprehensive assertion: after = before + changes
     assert_eq!(actual_after_state, expected_after_state);
 }
@@ -165,7 +165,7 @@ From `/program-tests/utils/src/assert_decompressed_token_transfer.rs`:
         spl_token_2022::state::Account::unpack(&recipient_data_before[..165]).unwrap();
     recipient_token_before.amount += transfer_amount;
 
-    // Parse as SPL token accounts first  
+    // Parse as SPL token accounts first
     let sender_account_after =
         spl_token_2022::state::Account::unpack(&sender_account_data.data[..165]).unwrap();
     let recipient_account_after =
@@ -181,6 +181,12 @@ This pattern ensures you're testing **exact state transitions** rather than arbi
 
 ### **❌ Assertion Anti-Patterns to Avoid**
 
+The test indexer in combination with litesvm LightProgram does not need time to catch up it is local.
+```rust
+// Give test indexer time to catch up
+tokio::time::sleep(tokio::time::Duration::from_millis(500)).await;
+```
+
 ```rust
 // ❌ WRONG: Individual field assertions
 assert_eq!(actual.field1, expected_field1);
@@ -265,10 +271,10 @@ async fn create_mint_helper(rpc: &mut RPC) -> Result<Signature, RpcError> {
 **✅ CORRECT: Use borsh deserialization for easier type handling**
 ```rust
 // Parse complete structs using borsh for easier handling
-let mint_data: CompressedMint = 
+let mint_data: CompressedMint =
     BorshDeserialize::deserialize(&mut account_data.as_slice())
         .expect("Failed to deserialize CompressedMint");
-        
+
 // Work with the complete struct
 assert_eq!(actual_mint, expected_mint);
 ```
@@ -296,29 +302,29 @@ use light_test_utils::assert_operation_result;
 #[serial]
 async fn test_functional_flow() {
     let mut rpc = setup_test_environment().await;
-    
+
     // Execute operation
     let result = perform_operation(&mut rpc, test_params).await.unwrap();
-    
+
     // Assert complete expected outcome
     let expected = ExpectedOperationResult {
         transaction_signature: result.signature,
         modified_accounts: expected_account_changes,
         emitted_events: expected_events,
         // ... all expected outputs
     };
-    
+
     assert_operation_result(&mut rpc, &expected).await;
 }
 
 #[tokio::test]
-#[serial] 
+#[serial]
 async fn test_operation_fails_with_invalid_input() {
     let mut rpc = setup_test_environment().await;
     let invalid_params = create_invalid_test_params();
-    
+
     let result = perform_operation(&mut rpc, invalid_params).await;
-    
+
     assert!(result.is_err());
     assert_eq!(
         result.unwrap_err().to_string(),
@@ -337,11 +343,11 @@ async fn test_operation_fails_with_invalid_input() {
 ## Key Principles
 
 ### 1. Comprehensive Coverage
-- **Integration tests**: Every user workflow  
+- **Integration tests**: Every user workflow
 - **Error tests**: Every error condition
 - **Edge cases**: Boundary conditions and invalid inputs
 
-### 2. Clear Test Structure  
+### 2. Clear Test Structure
 - **Arrange**: Set up test data and environment
 - **Act**: Execute the operation under test
 - **Assert**: Verify complete expected outcome using assertion helpers
@@ -409,4 +415,4 @@ Features:
 7. **Include all required signers** in transaction calls
 8. **Handle multi-signer scenarios** correctly
 9. **Test with realistic amounts** not just trivial values
-10. **Verify amount conservation** in transfer operations
+10. **Verify amount conservation** in transfer operations