Listing/unlisting reserved to Root

substrate/frame/opf/src/lib.rs

@@ -23,6 +23,12 @@ mod types;
 pub use pallet_scheduler as Schedule;
 pub use types::*;
 
+#[cfg(test)]
+mod mock;
+
+#[cfg(test)]
+mod tests;
+
 #[frame_support::pallet(dev_mode)]
 pub mod pallet {
 	use super::*;
@@ -113,10 +119,6 @@ pub mod pallet {
 	pub type WhiteListedProjectAccounts<T: Config> =
 		StorageValue<_, BoundedVec<ProjectInfo<T>, T::MaxProjects>, ValueQuery>;
 
-	/// Whitelisted Projects registration counter
-	#[pallet::storage]
-	pub type WhiteListedProjectCounter<T: Config> = StorageValue<_, u32, ValueQuery>;
-
 	/// Returns (positive_funds,negative_funds) of Whitelisted Project accounts
 	#[pallet::storage]
 	pub type ProjectFunds<T: Config> = StorageMap<
@@ -238,21 +240,20 @@ pub mod pallet {
 		///
 		/// ## Details
 		///
-		/// From this extrinsic any user can register project.
+		/// From this extrinsic only Root can register project.
 		///
 		/// ### Parameters
 		/// - `project_id`: The account that will receive the reward.
 		///
 		/// ### Errors
-		/// - [`Error::<T>::SubmittedProjectId`]: Project already submitted under this project_id
 		/// - [`Error::<T>::MaximumProjectsNumber`]: Maximum number of project subscriptions reached
 		///  
 		/// ## Events
 		/// Emits [`Event::<T>::Projectlisted`].
 		#[pallet::call_index(0)]
 		#[transactional]
 		pub fn register_project(origin: OriginFor<T>, project_id: ProjectId<T>) -> DispatchResult {
-			let _caller = ensure_signed(origin)?;
+			let _caller = ensure_root(origin)?;
 			let when = T::BlockNumberProvider::current_block_number();
 			ProjectInfo::<T>::new(project_id.clone());
 			Self::deposit_event(Event::Projectlisted { when, project_id });

substrate/frame/opf/src/tests.rs

@@ -49,7 +49,9 @@ pub fn run_to_block(n: BlockNumberFor<Test>) {
 fn project_registration_works(){
     new_test_ext().execute_with(|| {
 
-
-
+      assert_ok!(Opf::register_project(RawOrigin::Root.into(), BOB));
+      let project_list = WhiteListedProjectAccounts::<Test>::get().to_vec();
+      assert_eq!(project_list.len(),1);
+
     })
 }

substrate/frame/opf/src/types.rs

@@ -158,7 +158,7 @@ pub struct VoteInfo<T: Config> {
 // If no conviction, user's funds are released at the end of the voting round
 impl<T: Config> VoteInfo<T> {
 	pub fn funds_unlock(&mut self) {
-		let conviction_coeff = <u8 as From<Conviction>>::from(self.conviction);
+		//let conviction_coeff = <u8 as From<Conviction>>::from(self.conviction);
 		let funds_unlock_block = self.round.round_ending_block;
 		self.funds_unlock_block = funds_unlock_block;
 	}