close the voting round if the total votes are done (#269)

* close the voting round if the total votes are done

* cargo fmt

---------

Co-authored-by: samelamin <[email protected]>

Author: mshankarrao

node/src/cli.rs

@@ -37,8 +37,8 @@ pub enum Subcommand {
     Benchmark(frame_benchmarking_cli::BenchmarkCmd),
 
     /// Try-runtime has migrated to a standalone
-	/// [CLI](<https://github.com/paritytech/try-runtime-cli>). The subcommand exists as a stub and
-	/// deprecation notice. It will be removed entirely some time after Janurary 2024.
+    /// [CLI](<https://github.com/paritytech/try-runtime-cli>). The subcommand exists as a stub and
+    /// deprecation notice. It will be removed entirely some time after Janurary 2024.
     TryRuntime,
 }
 

pallets/fellowship/src/impls.rs

@@ -3,10 +3,7 @@ use crate::*;
 use common_traits::MaybeConvert;
 use frame_support::{ensure, traits::Get};
 use orml_traits::MultiReservableCurrency;
-use sp_runtime::{
-    traits::BadOrigin,
-    DispatchError,
-};
+use sp_runtime::{traits::BadOrigin, DispatchError};
 use sp_std::{vec, vec::Vec};
 /// Ensure that a account is of a given role.
 /// Used in other pallets like an ensure origin.

pallets/fellowship/src/tests/ensure_role.rs

@@ -27,7 +27,6 @@ fn ensure_role_in_works() {
     });
 }
 
-
 #[test]
 fn ensure_role_in_works_with_rank() {
     new_test_ext().execute_with(|| {

pallets/fellowship/src/tests/test_utils.rs

@@ -1,6 +1,5 @@
 use super::*;
 
-
 // Saves a bit of typing.
 pub(crate) fn add_to_fellowship_take_deposit(
     who: &AccountIdOf<Test>,

pallets/proposals/src/impls/pallet_impls.rs

@@ -92,6 +92,9 @@ impl<T: Config> Pallet<T> {
             Ok::<(), DispatchError>(())
         })?;
 
+        let funding_threshold: BalanceOf<T> =
+            T::PercentRequiredForVoteToPass::get().mul_floor(project.raised_funds);
+
         let vote: Vote<BalanceOf<T>> =
             MilestoneVotes::<T>::try_mutate(project_key, |vote_btree| {
                 if let Some(vote) = vote_btree.get_mut(&milestone_key) {
@@ -100,15 +103,19 @@ impl<T: Config> Pallet<T> {
                     } else {
                         vote.nay = vote.nay.saturating_add(contribution_amount);
                     }
+
+                    //check if the everyone has voted and its still less than the
+                    // funding threshold just reject it
+                    if vote.yay + vote.nay == project.raised_funds && vote.yay < funding_threshold {
+                        Self::close_voting_round(project_key, user_has_voted_key)?;
+                        Self::deposit_event(Event::MilestoneRejected(project_key, milestone_key));
+                    }
                     Ok::<Vote<BalanceOf<T>>, DispatchError>(vote.clone())
                 } else {
                     Err(Error::<T>::VotingRoundNotStarted.into())
                 }
             })?;
 
-        let funding_threshold: BalanceOf<T> =
-            T::PercentRequiredForVoteToPass::get().mul_floor(project.raised_funds);
-
         Self::try_auto_finalise_milestone_voting(
             project_key,
             &vote,

runtime/integration-tests/src/xcm_transfers.rs

@@ -27,9 +27,7 @@ use crate::kusama_test_net::{
 use crate::setup::{ksm_amount, mgx_amount, native_amount, PARA_ID_DEVELOPMENT, PARA_ID_SIBLING};
 use common_runtime::Balance;
 use common_types::{CurrencyId, FundingType, TreasuryOrigin};
-use imbue_kusama_runtime::{
-    OrmlTokens, Runtime as R, RuntimeOrigin, XTokens,
-};
+use imbue_kusama_runtime::{OrmlTokens, Runtime as R, RuntimeOrigin, XTokens};
 use orml_traits::MultiCurrency;
 use pallet_proposals::traits::RefundHandler;