Browse Source

membership module: update docs for methods on GenesisConfigBuilder

Mokhtar Naamani 4 years ago
parent
commit
14abbc420e
1 changed files with 22 additions and 15 deletions
  1. 22 15
      runtime-modules/membership/src/genesis.rs

+ 22 - 15
runtime-modules/membership/src/genesis.rs

@@ -31,6 +31,7 @@ impl<T: Trait> Default for GenesisConfigBuilder<T> {
 }
 
 impl<T: Trait> GenesisConfigBuilder<T> {
+    /// Assign default paid membeship fee
     pub fn default_paid_membership_fee(
         mut self,
         default_paid_membership_fee: BalanceOf<T>,
@@ -38,30 +39,36 @@ impl<T: Trait> GenesisConfigBuilder<T> {
         self.default_paid_membership_fee = default_paid_membership_fee;
         self
     }
+
+    /// Assign a collection of MemberId and AccountId pairs, used to derive mock member at genesis
     pub fn members(mut self, members: Vec<(T::MemberId, T::AccountId)>) -> Self {
         self.members = members;
         self
     }
 
+    /// Generates a Vec of `Member`s from pairs of MemberId and AccountId
+    fn generate_mock_members(&self) -> Vec<Member<T::MemberId, T::AccountId, T::Moment>> {
+        self.members
+            .iter()
+            .enumerate()
+            .map(|(ix, (ref member_id, ref account_id))| Member {
+                member_id: *member_id,
+                root_account: account_id.clone(),
+                controller_account: account_id.clone(),
+                // hack to get min handle length to 5
+                handle: (10000 + ix).to_string(),
+                avatar_uri: "".into(),
+                about: "".into(),
+                registered_at_time: T::Moment::from(0),
+            })
+            .collect()
+    }
+
     /// Construct GenesisConfig for mocked testing purposes only
     pub fn build(&self) -> GenesisConfig<T> {
         GenesisConfig::<T> {
             default_paid_membership_fee: self.default_paid_membership_fee,
-            members: self
-                .members
-                .iter()
-                .enumerate()
-                .map(|(ix, (ref member_id, ref account_id))| Member {
-                    member_id: *member_id,
-                    root_account: account_id.clone(),
-                    controller_account: account_id.clone(),
-                    // hack to get min handle length to 5
-                    handle: (10000 + ix).to_string(),
-                    avatar_uri: "".into(),
-                    about: "".into(),
-                    registered_at_time: T::Moment::from(0),
-                })
-                .collect(),
+            members: self.generate_mock_members(),
         }
     }
 }