Fix: Implement Campaign Status Update in Boundless Contract

contracts/boundless/src/logic/campaign.rs

@@ -1,5 +1,5 @@
 use crate::datatypes::{Backer, BoundlessError, Campaign, Milestone, Status};
-use crate::interface::CampaignManagement;
+use crate::interface::{CampaignManagement, ContractManagement};
 use crate::{BoundlessContract, BoundlessContractArgs, BoundlessContractClient};
 use soroban_sdk::{contractimpl, Address, Env, Symbol, Vec};
 
@@ -75,18 +75,32 @@ impl CampaignManagement for BoundlessContract {
         Ok(())
     }
 
-    fn update_campaign_status(
+  fn update_campaign_status(
         env: Env,
         campaign_id: u64,
         status: Status,
         admin: Address,
     ) -> Result<(), BoundlessError> {
-        // TODO: update campaign status logic
-        // - Verify admin authorization
-        // - Get campaign from storage
-        // - Update status
-        // - Store updated campaign
-        // - Emit status update event
+        admin.require_auth();
+        let contract_admin = <BoundlessContract as ContractManagement>::get_admin(&env);
+        if admin != contract_admin {
+            return Err(BoundlessError::Unauthorized);
+        }
+
+        let campaign_key = crate::datatypes::DataKey::Campaign(campaign_id);
+        let mut campaign: Campaign = env
+            .storage()
+            .persistent()
+            .get(&campaign_key)
+            .ok_or(BoundlessError::CampaignNotFound)?;
+
+        campaign.status = status.clone();
+        env.storage().persistent().set(&campaign_key, &campaign);
+        env.events().publish(
+            (Symbol::new(&env, "campaign"), Symbol::new(&env, "status_update")),
+            (campaign_id, status, admin),
+        );
+
         Ok(())
     }
 

contracts/boundless/src/tests/campaign_update_status.rs

@@ -0,0 +1,212 @@
+#![cfg(test)]
+
+use crate::{
+    datatypes::{Campaign, Status},
+    BoundlessContract, BoundlessContractClient,
+};
+use soroban_sdk::{
+    testutils::Address as _,
+    Address, Env, Symbol, Vec,
+};
+
+extern crate std;
+mod boundless {
+    soroban_sdk::contractimport!(file = "../../target/wasm32v1-none/release/boundless.wasm");
+}
+
+#[test]
+fn test_update_campaign_status_success() {
+    let env = Env::default();
+    env.mock_all_auths();
+
+    let admin = Address::generate(&env);
+    let contract_id = env.register(BoundlessContract, ());
+    let contract = BoundlessContractClient::new(&env, &contract_id);
+
+    contract.initialize(&admin);
+
+    let campaign_id = 1u64;
+    let owner = Address::generate(&env);
+    let title = Symbol::new(&env, "TestCampaign");
+    let description = Symbol::new(&env, "TestDescription");
+    let funding_goal = 1000i128;
+    let escrow_contract_id = Address::generate(&env);
+    let milestones = Vec::new(&env);
+    let backers = Vec::new(&env);
+
+    let campaign = Campaign {
+        id: campaign_id,
+        owner: owner.clone(),
+        title: title.clone(),
+        description: description.clone(),
+        funding_goal,
+        escrow_contract_id: escrow_contract_id.clone(),
+        milestones,
+        backers,
+        status: Status::Active,
+    };
+
+    let campaign_key = crate::datatypes::DataKey::Campaign(campaign_id);
+    env.as_contract(&contract_id, || {
+        env.storage().persistent().set(&campaign_key, &campaign);
+    });
+
+    // Update status to Completed
+    contract.update_campaign_status(&campaign_id, &Status::Completed, &admin);
+
+    let updated_campaign: Campaign = env.as_contract(&contract_id, || {
+        env.storage().persistent().get(&campaign_key).unwrap()
+    });
+    assert_eq!(updated_campaign.status, Status::Completed);
+}
+
+#[test]
+#[should_panic]
+fn test_update_campaign_status_unauthorized() {
+    let env = Env::default();
+    env.mock_all_auths();
+
+    let admin = Address::generate(&env);
+    let unauthorized_user = Address::generate(&env);
+    let contract_id = env.register(BoundlessContract, ());
+    let contract = BoundlessContractClient::new(&env, &contract_id);
+
+    contract.initialize(&admin);
+
+    let campaign_id = 1u64;
+    let owner = Address::generate(&env);
+    let title = Symbol::new(&env, "TestCampaign");
+    let description = Symbol::new(&env, "TestDescription");
+    let funding_goal = 1000i128;
+    let escrow_contract_id = Address::generate(&env);
+    let milestones = Vec::new(&env);
+    let backers = Vec::new(&env);
+
+    let campaign = Campaign {
+        id: campaign_id,
+        owner: owner.clone(),
+        title: title.clone(),
+        description: description.clone(),
+        funding_goal,
+        escrow_contract_id: escrow_contract_id.clone(),
+        milestones,
+        backers,
+        status: Status::Active,
+    };
+
+    let campaign_key = crate::datatypes::DataKey::Campaign(campaign_id);
+    env.as_contract(&contract_id, || {
+        env.storage().persistent().set(&campaign_key, &campaign);
+    });
+
+    contract.update_campaign_status(&campaign_id, &Status::Completed, &unauthorized_user);
+}
+
+#[test]
+#[should_panic]
+fn test_update_campaign_status_not_found() {
+    let env = Env::default();
+    env.mock_all_auths();
+
+    let admin = Address::generate(&env);
+    let contract_id = env.register(BoundlessContract, ());
+    let contract = BoundlessContractClient::new(&env, &contract_id);
+
+    contract.initialize(&admin);
+
+    let campaign_id = 999u64; // Non-existent campaign
+
+    contract.update_campaign_status(&campaign_id, &Status::Completed, &admin);
+}
+
+#[test]
+fn test_update_campaign_status_to_failed() {
+    let env = Env::default();
+    env.mock_all_auths();
+
+    let admin = Address::generate(&env);
+    let contract_id = env.register(BoundlessContract, ());
+    let contract = BoundlessContractClient::new(&env, &contract_id);
+
+    contract.initialize(&admin);
+
+    let campaign_id = 1u64;
+    let owner = Address::generate(&env);
+    let title = Symbol::new(&env, "TestCampaign");
+    let description = Symbol::new(&env, "TestDescription");
+    let funding_goal = 1000i128;
+    let escrow_contract_id = Address::generate(&env);
+    let milestones = Vec::new(&env);
+    let backers = Vec::new(&env);
+
+    let campaign = Campaign {
+        id: campaign_id,
+        owner: owner.clone(),
+        title: title.clone(),
+        description: description.clone(),
+        funding_goal,
+        escrow_contract_id: escrow_contract_id.clone(),
+        milestones,
+        backers,
+        status: Status::Active,
+    };
+
+    let campaign_key = crate::datatypes::DataKey::Campaign(campaign_id);
+    env.as_contract(&contract_id, || {
+        env.storage().persistent().set(&campaign_key, &campaign);
+    });
+
+    // Update status to Failed
+    contract.update_campaign_status(&campaign_id, &Status::Failed, &admin);
+
+    let updated_campaign: Campaign = env.as_contract(&contract_id, || {
+        env.storage().persistent().get(&campaign_key).unwrap()
+    });
+    assert_eq!(updated_campaign.status, Status::Failed);
+}
+
+#[test]
+fn test_update_campaign_status_to_pending() {
+    let env = Env::default();
+    env.mock_all_auths();
+
+    let admin = Address::generate(&env);
+    let contract_id = env.register(BoundlessContract, ());
+    let contract = BoundlessContractClient::new(&env, &contract_id);
+
+    contract.initialize(&admin);
+
+    let campaign_id = 1u64;
+    let owner = Address::generate(&env);
+    let title = Symbol::new(&env, "TestCampaign");
+    let description = Symbol::new(&env, "TestDescription");
+    let funding_goal = 1000i128;
+    let escrow_contract_id = Address::generate(&env);
+    let milestones = Vec::new(&env);
+    let backers = Vec::new(&env);
+
+    let campaign = Campaign {
+        id: campaign_id,
+        owner: owner.clone(),
+        title: title.clone(),
+        description: description.clone(),
+        funding_goal,
+        escrow_contract_id: escrow_contract_id.clone(),
+        milestones,
+        backers,
+        status: Status::Active,
+    };
+
+    let campaign_key = crate::datatypes::DataKey::Campaign(campaign_id);
+    env.as_contract(&contract_id, || {
+        env.storage().persistent().set(&campaign_key, &campaign);
+    });
+
+    // Update status to Pending
+    contract.update_campaign_status(&campaign_id, &Status::Pending, &admin);
+
+    let updated_campaign: Campaign = env.as_contract(&contract_id, || {
+        env.storage().persistent().get(&campaign_key).unwrap()
+    });
+    assert_eq!(updated_campaign.status, Status::Pending);
+}

contracts/boundless/src/tests/mod.rs

@@ -2,3 +2,4 @@
 
 pub mod admin;
 pub mod milestone;
+pub mod campaign_update_status;