test: add multi-step transaction sequence test

fix_tests.py

@@ -0,0 +1,32 @@
+import os
+import re
+
+tests_dir = "src/tests"
+
+for root, _, files in os.walk(tests_dir):
+    for file in files:
+        if file.endswith(".rs"):
+            filepath = os.path.join(root, file)
+            with open(filepath, "r") as f:
+                content = f.read()
+
+            new_content = content
+
+            # Fix CreateCampaignParams
+            new_content = re.sub(
+                r'(max_contribution_per_user:\s*[^,}]+,?)(\s*)}',
+                r'\1\n\2    token: crate::types::MaybeAddress::None,\n\2    uses_milestones: false,\n\2}',
+                new_content
+            )
+
+            # Fix Campaign
+            new_content = re.sub(
+                r'(effective_amount_raised:\s*[^,}]+,?)(\s*)}',
+                r'\1\n\2    token: soroban_sdk::Address::generate(&env),\n\2    uses_milestones: false,\n\2}',
+                new_content
+            )
+
+            if new_content != content:
+                with open(filepath, "w") as f:
+                    f.write(new_content)
+                print(f"Fixed {filepath}")

src/tests/helpers.rs

@@ -29,6 +29,8 @@ pub(crate) fn make_params(
         has_revenue_sharing,
         revenue_share_percentage,
         max_contribution_per_user,
+        token: crate::types::MaybeAddress::None,
+        uses_milestones: false,
     }
 }
 
@@ -87,3 +89,19 @@ pub(crate) fn setup_env<'a>() -> (
     });
     setup
 }
+
+pub(crate) fn setup_token<'a>(env: &Env, admin: &Address) -> TokenClient<'a> {
+    let token_address = env.register_stellar_asset_contract(admin.clone());
+    TokenClient::new(env, &token_address)
+}
+
+pub(crate) fn setup_contract<'a>(
+    env: &Env,
+    admin: &Address,
+    token_address: &Address,
+) -> ProofOfHeartClient<'a> {
+    let contract_id = env.register_contract(None, ProofOfHeart);
+    let client = ProofOfHeartClient::new(env, &contract_id);
+    client.init(admin, token_address, &300);
+    client
+}

src/tests/mod.rs

@@ -17,6 +17,7 @@ mod test_init;
 mod test_issues;
 mod test_lifecycle_events;
 mod test_listing;
+mod test_multi_step;
 mod test_purge_voting;
 mod test_refund;
 mod test_refund_edge;

src/tests/test_benchmark.rs

@@ -19,6 +19,10 @@ fn make_revenue_campaign(
         has_revenue_sharing: true,
         revenue_share_percentage: 1000,
         max_contribution_per_user: 0,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     }
 }
 
@@ -108,6 +112,10 @@ fn test_get_campaigns_by_category_bucketed_pagination_budget() {
             has_revenue_sharing: false,
             revenue_share_percentage: 0,
             max_contribution_per_user: 0,
+
+            token: crate::types::MaybeAddress::None,
+
+            uses_milestones: false,
         };
         client.create_campaign(&params);
     }

src/tests/test_campaign_update.rs

@@ -488,6 +488,10 @@ fn test_unpause_clears_auto_pause_when_resume_campaign_blocked() {
         has_revenue_sharing: false,
         revenue_share_percentage: 0,
         max_contribution_per_user: 0,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
 
@@ -508,6 +512,10 @@ fn test_unpause_clears_auto_pause_when_resume_campaign_blocked() {
         has_revenue_sharing: false,
         revenue_share_percentage: 0,
         max_contribution_per_user: 0,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     assert_eq!(res.unwrap_err().unwrap(), Error::ContractPaused);
 
@@ -536,6 +544,10 @@ fn test_unpause_clears_auto_pause_when_resume_campaign_blocked() {
         has_revenue_sharing: false,
         revenue_share_percentage: 0,
         max_contribution_per_user: 0,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     assert!(new_id > 1);
 }

src/tests/test_cancel_after_goal_met.rs

@@ -21,6 +21,10 @@ fn test_cancel_campaign_blocked_after_goal_met() {
         has_revenue_sharing: false,
         revenue_share_percentage: 0,
         max_contribution_per_user: 0i128,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
 
@@ -52,6 +56,10 @@ fn test_cancel_campaign_allowed_when_goal_not_met() {
         has_revenue_sharing: false,
         revenue_share_percentage: 0,
         max_contribution_per_user: 0i128,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
 
@@ -82,6 +90,10 @@ fn test_cancel_campaign_blocked_when_amount_exceeds_goal() {
         has_revenue_sharing: false,
         revenue_share_percentage: 0,
         max_contribution_per_user: 0i128,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
 

src/tests/test_cancel_revenue_orphan.rs

@@ -22,6 +22,10 @@ fn test_cancel_campaign_refunds_revenue_pool() {
         has_revenue_sharing: true,
         revenue_share_percentage: 2000, // 20% to contributors
         max_contribution_per_user: 0i128,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
 
@@ -89,6 +93,10 @@ fn test_cannot_claim_revenue_after_cancel() {
         has_revenue_sharing: true,
         revenue_share_percentage: 2000,
         max_contribution_per_user: 0i128,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
 
@@ -142,6 +150,10 @@ fn test_cancel_with_multiple_contributors_and_revenue() {
         has_revenue_sharing: true,
         revenue_share_percentage: 3000, // 30% to contributors
         max_contribution_per_user: 0i128,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
 
@@ -208,6 +220,10 @@ fn test_cancel_campaign_emits_revenue_refund_event() {
         has_revenue_sharing: true,
         revenue_share_percentage: 2000,
         max_contribution_per_user: 0i128,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
     client.contribute(&campaign_id, &contributor1, &1000);
@@ -250,6 +266,10 @@ fn test_cancel_campaign_with_no_revenue() {
         has_revenue_sharing: true,
         revenue_share_percentage: 2000,
         max_contribution_per_user: 0i128,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
 

src/tests/test_contribute_caps.rs

@@ -185,6 +185,10 @@ fn test_huge_contribution_triggers_auto_pause() {
         has_revenue_sharing: false,
         revenue_share_percentage: 0,
         max_contribution_per_user: 0,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
 

src/tests/test_issues.rs

@@ -115,6 +115,10 @@ fn make_campaign_params_simple(env: &Env, creator: &Address) -> CreateCampaignPa
         has_revenue_sharing: false,
         revenue_share_percentage: 0,
         max_contribution_per_user: 0,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     }
 }
 
@@ -260,6 +264,10 @@ fn test_set_personal_cap_cannot_exceed_max_contribution_per_user() {
         has_revenue_sharing: false,
         revenue_share_percentage: 0,
         max_contribution_per_user: 500,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     };
     let campaign_id = client.create_campaign(&params);
 
@@ -371,6 +379,10 @@ fn test_pending_creator_none_round_trip() {
         fee_override: None,
         deadline_extended: false,
         effective_amount_raised: 0,
+
+        token: soroban_sdk::Address::generate(&env),
+
+        uses_milestones: false,
     };
 
     env.as_contract(&contract_id, || {
@@ -411,6 +423,10 @@ fn test_pending_creator_some_round_trip() {
         fee_override: None,
         deadline_extended: false,
         effective_amount_raised: 0,
+
+        token: soroban_sdk::Address::generate(&env),
+
+        uses_milestones: false,
     };
 
     env.as_contract(&contract_id, || {
@@ -576,6 +592,10 @@ fn test_creator_claim_does_not_absorb_contributor_rounding() {
         has_revenue_sharing: true,
         revenue_share_percentage: 5000, // 50%
         max_contribution_per_user: 0i128,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
     client.contribute(&campaign_id, &contributor1, &10_001);

src/tests/test_lifecycle_events.rs

@@ -30,6 +30,10 @@ fn test_full_lifecycle_event_sequence() {
         has_revenue_sharing: true,
         revenue_share_percentage: 1000,
         max_contribution_per_user: 0,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
 
     assert!(
@@ -98,6 +102,10 @@ fn test_cancel_lifecycle_event_sequence() {
         has_revenue_sharing: false,
         revenue_share_percentage: 0,
         max_contribution_per_user: 0,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
 
     client.verify_campaign(&id);
@@ -133,6 +141,10 @@ fn test_campaign_cancelled_event_includes_creator_and_amount() {
         has_revenue_sharing: false,
         revenue_share_percentage: 0,
         max_contribution_per_user: 0,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
 
     client.verify_campaign(&id);
@@ -169,6 +181,10 @@ fn test_campaign_created_event_includes_category() {
         has_revenue_sharing: false,
         revenue_share_percentage: 0,
         max_contribution_per_user: 0,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
 
     let events = env.events().all();

src/tests/test_multi_step.rs

@@ -0,0 +1,32 @@
+#![cfg(test)]
+
+use crate::tests::helpers::{setup_contract, setup_token};
+use soroban_sdk::{testutils::Address as _, Address, Env, String};
+
+#[test]
+fn test_multi_step_sequence() {
+    let env = Env::default();
+    env.mock_all_auths();
+
+    let admin = Address::generate(&env);
+    let token = setup_token(&env, &admin);
+    let client = setup_contract(&env, &admin, &token.address);
+
+    let creator = Address::generate(&env);
+    let params = crate::types::CreateCampaignParams {
+        creator: creator.clone(),
+        title: String::from_str(&env, "Test"),
+        description: String::from_str(&env, "Desc"),
+        funding_goal: 100_000,
+        duration_days: 10,
+        category: crate::types::Category::Educator,
+        has_revenue_sharing: false,
+        revenue_share_percentage: 0,
+        max_contribution_per_user: 0,
+        token: crate::types::MaybeAddress::None,
+        uses_milestones: false,
+    };
+
+    let id = client.create_campaign(&params);
+    client.cancel_campaign(&id);
+}

src/tests/test_refund_edge.rs

@@ -128,6 +128,10 @@ fn test_claim_refund_clears_existing_revenue_claimed_key() {
         has_revenue_sharing: true,
         revenue_share_percentage: 2000,
         max_contribution_per_user: 0i128,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
     client.contribute(&campaign_id, &contributor1, &1000);

src/tests/test_revenue.rs

@@ -287,6 +287,10 @@ fn test_claim_revenue_blocked_before_funds_withdrawn() {
         has_revenue_sharing: true,
         revenue_share_percentage: 2000,
         max_contribution_per_user: 0i128,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
     client.contribute(&campaign_id, &contributor1, &5000);

src/tests/test_revenue_deposit.rs

@@ -134,6 +134,10 @@ fn test_deposit_revenue_repeated_calls_accumulate_and_emit_events() {
         has_revenue_sharing: true,
         revenue_share_percentage: 2000,
         max_contribution_per_user: 0i128,
+
+        token: crate::types::MaybeAddress::None,
+
+        uses_milestones: false,
     });
     client.verify_campaign(&campaign_id);
     client.contribute(&campaign_id, &contributor1, &1000);