Fix project transition event

integration-tests/src/tests/ct_migration.rs

@@ -174,7 +174,10 @@ fn create_settled_project() -> (ProjectId, Vec<AccountId>) {
 			default_community_contributions(),
 			default_remainder_contributions(),
 		);
-		assert_eq!(inst.go_to_next_state(project_id), pallet_funding::ProjectStatus::SettlementStarted(FundingOutcome::Success));
+		assert_eq!(
+			inst.go_to_next_state(project_id),
+			pallet_funding::ProjectStatus::SettlementStarted(FundingOutcome::Success)
+		);
 		let mut participants: Vec<AccountId> =
 			pallet_funding::Evaluations::<PolimecRuntime>::iter_prefix_values((project_id,))
 				.map(|eval| eval.evaluator)
@@ -229,7 +232,10 @@ fn create_project_with_unsettled_participation(participation_type: Participation
 			default_remainder_contributions(),
 		);
 
-		assert_eq!(inst.go_to_next_state(project_id), pallet_funding::ProjectStatus::SettlementStarted(FundingOutcome::Success));
+		assert_eq!(
+			inst.go_to_next_state(project_id),
+			pallet_funding::ProjectStatus::SettlementStarted(FundingOutcome::Success)
+		);
 		let evaluations_to_settle =
 			pallet_funding::Evaluations::<PolimecRuntime>::iter_prefix_values((project_id,)).collect_vec();
 		let bids_to_settle = pallet_funding::Bids::<PolimecRuntime>::iter_prefix_values((project_id,)).collect_vec();
@@ -258,13 +264,7 @@ fn create_project_with_unsettled_participation(participation_type: Participation
 
 		let start = if participation_type == ParticipationType::Bid { 1 } else { 0 };
 		for bid in bids_to_settle[start..].iter() {
-			PolimecFunding::settle_bid(
-				RuntimeOrigin::signed(alice()),
-				project_id,
-				bid.bidder.clone(),
-				bid.id,
-			)
-			.unwrap()
+			PolimecFunding::settle_bid(RuntimeOrigin::signed(alice()), project_id, bid.bidder.clone(), bid.id).unwrap()
 		}
 
 		let start = if participation_type == ParticipationType::Contribution { 1 } else { 0 };

pallets/funding/src/functions/2_evaluation.rs

@@ -37,7 +37,7 @@ impl<T: Config> Pallet<T> {
 			project_details,
 			ProjectStatus::Application,
 			ProjectStatus::EvaluationRound,
-			T::EvaluationDuration::get(),
+			Some(T::EvaluationDuration::get()),
 			false,
 		)
 	}
@@ -91,7 +91,7 @@ impl<T: Config> Pallet<T> {
 				project_details,
 				ProjectStatus::EvaluationRound,
 				ProjectStatus::AuctionInitializePeriod,
-				T::AuctionInitializePeriodDuration::get(),
+				Some(T::AuctionInitializePeriodDuration::get()),
 				false,
 			)
 		// Unsuccessful path
@@ -104,7 +104,7 @@ impl<T: Config> Pallet<T> {
 				project_details,
 				ProjectStatus::EvaluationRound,
 				ProjectStatus::FundingFailed,
-				One::one(),
+				Some(One::one()),
 				false,
 			)
 		}

pallets/funding/src/functions/3_auction.rs

@@ -35,7 +35,7 @@ impl<T: Config> Pallet<T> {
 			project_details,
 			ProjectStatus::AuctionInitializePeriod,
 			ProjectStatus::AuctionRound,
-			T::AuctionOpeningDuration::get(),
+			Some(T::AuctionOpeningDuration::get()),
 			skip_round_end_check,
 		)
 	}
@@ -72,7 +72,7 @@ impl<T: Config> Pallet<T> {
 					updated_project_details,
 					ProjectStatus::AuctionRound,
 					ProjectStatus::CommunityRound(now.saturating_add(T::CommunityFundingDuration::get())),
-					T::CommunityFundingDuration::get() + T::RemainderFundingDuration::get(),
+					Some(T::CommunityFundingDuration::get() + T::RemainderFundingDuration::get()),
 					false,
 				)?;
 				Ok(PostDispatchInfo {

pallets/funding/src/functions/5_funding_end.rs

@@ -73,10 +73,14 @@ impl<T: Config> Pallet<T> {
 			)
 		};
 
-		let round_end = now.saturating_add(duration).saturating_sub(One::one());
-		project_details.round_duration.update(Some(now), Some(round_end));
-		project_details.status = next_status;
-		ProjectsDetails::<T>::insert(project_id, project_details);
+		Self::transition_project(
+			project_id,
+			project_details.clone(),
+			project_details.status,
+			next_status,
+			Some(duration),
+			true,
+		)?;
 
 		Ok(PostDispatchInfo { actual_weight: Some(actual_weight), pays_fee: Pays::Yes })
 	}

pallets/funding/src/functions/6_settlement.rs

@@ -4,8 +4,8 @@ use frame_support::{
 	dispatch::DispatchResult,
 	ensure,
 	traits::{
-		fungible::{MutateHold as FungibleMutateHold},
-		fungibles::{Mutate as FungiblesMutate},
+		fungible::MutateHold as FungibleMutateHold,
+		fungibles::Mutate as FungiblesMutate,
 		tokens::{Fortitude, Precision, Preservation, Restriction},
 		Get,
 	},
@@ -22,22 +22,11 @@ use sp_runtime::{
 impl<T: Config> Pallet<T> {
 	#[transactional]
 	pub fn do_start_settlement(project_id: ProjectId) -> DispatchResultWithPostInfo {
-		// * Get variables *
 		let mut project_details = ProjectsDetails::<T>::get(project_id).ok_or(Error::<T>::ProjectDetailsNotFound)?;
 		let token_information =
 			ProjectsMetadata::<T>::get(project_id).ok_or(Error::<T>::ProjectMetadataNotFound)?.token_information;
 		let now = <frame_system::Pallet<T>>::block_number();
-		let round_end_block = project_details.round_duration.end().ok_or(Error::<T>::ImpossibleState)?;
 
-		// * Validity checks *
-		ensure!(
-			project_details.status == ProjectStatus::FundingSuccessful ||
-				project_details.status == ProjectStatus::FundingFailed,
-			Error::<T>::IncorrectRound
-		);
-		ensure!(now > round_end_block, Error::<T>::TooEarlyForRound);
-
-		// * Calculate new variables *
 		project_details.funding_end_block = Some(now);
 
 		let escrow_account = Self::fund_account_id(project_id);
@@ -72,16 +61,28 @@ impl<T: Config> Pallet<T> {
 				liquidity_pools_ct_amount,
 			)?;
 
-			project_details.status = ProjectStatus::SettlementStarted(FundingOutcome::Success);
-			ProjectsDetails::<T>::insert(project_id, &project_details);
+			Self::transition_project(
+				project_id,
+				project_details,
+				ProjectStatus::FundingSuccessful,
+				ProjectStatus::SettlementStarted(FundingOutcome::Success),
+				None,
+				false,
+			)?;
 
 			Ok(PostDispatchInfo {
 				actual_weight: Some(WeightInfoOf::<T>::start_settlement_funding_success()),
 				pays_fee: Pays::Yes,
 			})
 		} else {
-			project_details.status = ProjectStatus::SettlementStarted(FundingOutcome::Failure);
-			ProjectsDetails::<T>::insert(project_id, &project_details);
+			Self::transition_project(
+				project_id,
+				project_details,
+				ProjectStatus::FundingFailed,
+				ProjectStatus::SettlementStarted(FundingOutcome::Failure),
+				None,
+				false,
+			)?;
 
 			Ok(PostDispatchInfo {
 				actual_weight: Some(WeightInfoOf::<T>::start_settlement_funding_failure()),
@@ -153,8 +154,8 @@ impl<T: Config> Pallet<T> {
 			Error::<T>::SettlementNotStarted
 		);
 
-        // Return either the full amount to refund if bid is rejected/project failed,
-        // or a partial amount when the wap > paid price/bid is partially accepted
+		// Return either the full amount to refund if bid is rejected/project failed,
+		// or a partial amount when the wap > paid price/bid is partially accepted
 		let (final_ct_price, final_ct_amount, refunded_plmc, refunded_funding_asset_amount) =
 			Self::calculate_refund(&bid, funding_success, wap)?;
 
@@ -305,9 +306,9 @@ impl<T: Config> Pallet<T> {
 	}
 
 	pub fn do_mark_project_as_settled(project_id: ProjectId) -> DispatchResult {
-		let mut project_details = ProjectsDetails::<T>::get(project_id).ok_or(Error::<T>::ProjectDetailsNotFound)?;
+		let project_details = ProjectsDetails::<T>::get(project_id).ok_or(Error::<T>::ProjectDetailsNotFound)?;
 		let outcome = match project_details.status {
-			ProjectStatus::SettlementStarted(outcome) => outcome,
+			ProjectStatus::SettlementStarted(ref outcome) => outcome.clone(),
 			_ => return Err(Error::<T>::IncorrectRound.into()),
 		};
 
@@ -323,8 +324,14 @@ impl<T: Config> Pallet<T> {
 		);
 
 		// Mark the project as settled
-		project_details.status = ProjectStatus::SettlementFinished(outcome);
-		ProjectsDetails::<T>::insert(project_id, project_details);
+		Self::transition_project(
+			project_id,
+			project_details,
+			ProjectStatus::SettlementStarted(outcome.clone()),
+			ProjectStatus::SettlementFinished(outcome),
+			None,
+			false,
+		)?;
 
 		Ok(())
 	}

pallets/funding/src/functions/7_ct_migration.rs

@@ -8,18 +8,18 @@ impl<T: Config> Pallet<T> {
 		let mut project_details = ProjectsDetails::<T>::get(project_id).ok_or(Error::<T>::ProjectDetailsNotFound)?;
 
 		ensure!(project_details.issuer_account == caller, Error::<T>::NotIssuer);
-		match project_details.status {
-			ProjectStatus::SettlementFinished(FundingOutcome::Success) => (),
-			ProjectStatus::FundingSuccessful | ProjectStatus::SettlementStarted(FundingOutcome::Success) =>
-				return Err(Error::<T>::SettlementNotComplete.into()),
-			_ => return Err(Error::<T>::IncorrectRound.into()),
-		}
 
 		project_details.migration_type = Some(MigrationType::Offchain);
-		project_details.status = ProjectStatus::CTMigrationStarted;
-		ProjectsDetails::<T>::insert(project_id, project_details);
 
-		// * Emit events *
+		Self::transition_project(
+			project_id,
+			project_details,
+			ProjectStatus::SettlementFinished(FundingOutcome::Success),
+			ProjectStatus::CTMigrationStarted,
+			None,
+			false,
+		)?;
+
 		Ok(())
 	}
 
@@ -72,11 +72,15 @@ impl<T: Config> Pallet<T> {
 			migration_readiness_check: None,
 		};
 		project_details.migration_type = Some(MigrationType::Pallet(parachain_receiver_pallet_info));
-		project_details.status = ProjectStatus::CTMigrationStarted;
-		ProjectsDetails::<T>::insert(project_id, project_details);
 
-		// * Emit events *
-		Self::deposit_event(Event::PalletMigrationStarted { project_id, para_id });
+		Self::transition_project(
+			project_id,
+			project_details,
+			ProjectStatus::SettlementFinished(FundingOutcome::Success),
+			ProjectStatus::CTMigrationStarted,
+			None,
+			false,
+		)?;
 
 		Ok(())
 	}

pallets/funding/src/functions/misc.rs

@@ -393,23 +393,27 @@ impl<T: Config> Pallet<T> {
 		mut project_details: ProjectDetailsOf<T>,
 		current_round: ProjectStatus<BlockNumberFor<T>>,
 		next_round: ProjectStatus<BlockNumberFor<T>>,
-		round_duration: BlockNumberFor<T>,
+		maybe_round_duration: Option<BlockNumberFor<T>>,
 		skip_end_check: bool,
 	) -> DispatchResult {
 		/* Verify */
 		let now = <frame_system::Pallet<T>>::block_number();
 		ensure!(project_details.round_duration.ended(now) || skip_end_check, Error::<T>::TooEarlyForRound);
 		ensure!(project_details.status == current_round, Error::<T>::IncorrectRound);
 
-		let round_end = now.saturating_add(round_duration).saturating_sub(One::one());
-		project_details.round_duration.update(Some(now), Some(round_end));
-		project_details.status = next_round;
+		let round_end = if let Some(round_duration) = maybe_round_duration {
+			Some(now.saturating_add(round_duration).saturating_sub(One::one()))
+		} else {
+			None
+		};
+		project_details.round_duration.update(Some(now), round_end);
+		project_details.status = next_round.clone();
 
 		// * Update storage *
 		ProjectsDetails::<T>::insert(project_id, project_details);
 
-		// // * Emit events *
-		// Self::deposit_event(Event::ProjectPhaseTransition { project_id, phase: next_round });
+		// * Emit events *
+		Self::deposit_event(Event::ProjectPhaseTransition { project_id, phase: next_round });
 
 		Ok(())
 	}

pallets/funding/src/instantiator/mod.rs

@@ -39,9 +39,7 @@ use sp_arithmetic::{
 	traits::{SaturatedConversion, Saturating, Zero},
 	FixedPointNumber, Percent, Perquintill,
 };
-use sp_runtime::{
-	traits::{Convert, Member, One},
-};
+use sp_runtime::traits::{Convert, Member, One};
 use sp_std::{
 	cell::RefCell,
 	collections::{btree_map::*, btree_set::*},

pallets/funding/src/lib.rs

@@ -574,7 +574,7 @@ pub mod pallet {
 		/// Project transitioned to a new phase.
 		ProjectPhaseTransition {
 			project_id: ProjectId,
-			phase: ProjectPhases,
+			phase: ProjectStatus<BlockNumberFor<T>>,
 		},
 		/// A `bonder` bonded an `amount` of PLMC for `project_id`.
 		Evaluation {

pallets/funding/src/runtime_api.rs

@@ -70,9 +70,7 @@ impl<T: Config> Pallet<T> {
 
 	pub fn top_bids(project_id: ProjectId, amount: u32) -> Vec<BidInfoOf<T>> {
 		Bids::<T>::iter_prefix_values((project_id,))
-			.sorted_by(|a, b| {
-				b.final_ct_amount().cmp(&a.final_ct_amount())
-			})
+			.sorted_by(|a, b| b.final_ct_amount().cmp(&a.final_ct_amount()))
 			.take(amount as usize)
 			.collect_vec()
 	}

pallets/funding/src/tests/3_auction.rs

@@ -9,7 +9,7 @@ mod round_flow {
 		use super::*;
 		use frame_support::traits::fungibles::metadata::Inspect;
 		use sp_core::bounded_vec;
-		use std::{collections::HashSet};
+		use std::collections::HashSet;
 
 		#[test]
 		fn auction_round_completed() {

pallets/funding/src/tests/misc.rs

@@ -366,3 +366,50 @@ mod inner_functions {
 		assert_eq!(multiplier_25_duration, FixedU128::from_rational(52008, 1000).saturating_mul_int((DAYS * 7) as u64));
 	}
 }
+
+#[test]
+fn project_state_transition_event() {
+	let mut inst = MockInstantiator::new(Some(RefCell::new(new_test_ext())));
+	let project_id = inst.create_settled_project(
+		default_project_metadata(ISSUER_1),
+		ISSUER_1,
+		None,
+		default_evaluations(),
+		default_bids(),
+		default_community_contributions(),
+		default_remainder_contributions(),
+	);
+
+	let events = inst.execute(|| System::events());
+	let transition_events = events
+		.into_iter()
+		.filter_map(|event| {
+			if let RuntimeEvent::PolimecFunding(e @ crate::Event::ProjectPhaseTransition { .. }) = event.event {
+				Some(e)
+			} else {
+				None
+			}
+		})
+		.collect_vec();
+
+	let mut desired_transitions = vec![
+		ProjectStatus::EvaluationRound,
+		ProjectStatus::AuctionInitializePeriod,
+		ProjectStatus::AuctionRound,
+		ProjectStatus::CommunityRound(
+			EvaluationDuration::get() +
+				AuctionOpeningDuration::get() +
+				AuctionClosingDuration::get() +
+				CommunityRoundDuration::get() +
+				1u64,
+		),
+		ProjectStatus::FundingSuccessful,
+		ProjectStatus::SettlementStarted(FundingOutcome::Success),
+		ProjectStatus::SettlementFinished(FundingOutcome::Success),
+	]
+	.into_iter();
+
+	transition_events.into_iter().for_each(|event| {
+		assert_eq!(event, Event::ProjectPhaseTransition { project_id, phase: desired_transitions.next().unwrap() });
+	});
+}