fix: add members

Author: brech1

src/group.rs

@@ -108,9 +108,9 @@ impl Group {
     }
 
     /// Adds a set of members to the group
-    pub fn add_members(&mut self, members: Vec<Element>) -> Result<(), SemaphoreError> {
-        for member in &members {
-            if *member == EMPTY_ELEMENT {
+    pub fn add_members(&mut self, members: &[Element]) -> Result<(), SemaphoreError> {
+        for &member in members {
+            if member == EMPTY_ELEMENT {
                 return Err(SemaphoreError::EmptyLeaf);
             }
         }
@@ -282,7 +282,7 @@ mod tests {
         let member1 = [1; 32];
         let member2 = [2; 32];
 
-        group.add_members(vec![member1, member2]).unwrap();
+        group.add_members(&[member1, member2]).unwrap();
 
         assert_eq!(group.size(), 2);
     }
@@ -293,7 +293,7 @@ mod tests {
         let member1 = [1; 32];
         let zero = [0u8; ELEMENT_SIZE];
 
-        let result = group.add_members(vec![member1, zero]);
+        let result = group.add_members(&[member1, zero]);
 
         assert!(result.is_err());
         assert_eq!(result, Err(SemaphoreError::EmptyLeaf));
@@ -305,7 +305,7 @@ mod tests {
         let member2 = [2; 32];
         let mut group = Group::default();
 
-        group.add_members(vec![member1, member2]).unwrap();
+        group.add_members(&[member1, member2]).unwrap();
         let index = group.index_of(member2);
 
         assert_eq!(index, Some(1));
@@ -317,7 +317,7 @@ mod tests {
         let member2 = [2; 32];
         let mut group = Group::default();
 
-        group.add_members(vec![member1, member2]).unwrap();
+        group.add_members(&[member1, member2]).unwrap();
 
         group.update_member(0, member1).unwrap();
         assert_eq!(group.size(), 2);
@@ -332,7 +332,7 @@ mod tests {
         let member2 = [2; 32];
         let mut group = Group::default();
 
-        group.add_members(vec![member1, member2]).unwrap();
+        group.add_members(&[member1, member2]).unwrap();
         group.remove_member(0).unwrap();
 
         let result = group.update_member(0, member1);
@@ -346,7 +346,7 @@ mod tests {
         let member2 = [2; 32];
         let mut group = Group::default();
 
-        group.add_members(vec![member1, member2]).unwrap();
+        group.add_members(&[member1, member2]).unwrap();
         group.remove_member(0).unwrap();
 
         let members = group.members();
@@ -360,7 +360,7 @@ mod tests {
         let member2 = [2; 32];
         let mut group = Group::default();
 
-        group.add_members(vec![member1, member2]).unwrap();
+        group.add_members(&[member1, member2]).unwrap();
         group.remove_member(0).unwrap();
 
         let result = group.remove_member(0);
@@ -375,7 +375,7 @@ mod tests {
         let member2 = [2; 32];
         let mut group = Group::default();
 
-        group.add_members(vec![member1, member2]).unwrap();
+        group.add_members(&[member1, member2]).unwrap();
 
         let proof = group.generate_proof(0).unwrap();
         assert_eq!(proof.leaf, member1);
@@ -387,7 +387,7 @@ mod tests {
         let member2 = [2; 32];
         let mut group = Group::default();
 
-        group.add_members(vec![member1, member2]).unwrap();
+        group.add_members(&[member1, member2]).unwrap();
 
         let proof_0 = group.generate_proof(0).unwrap();
         assert_eq!(Group::verify_proof(&proof_0), true);

tests/group.rs

@@ -71,7 +71,7 @@ mod group {
     use ark_ed_on_bn254::Fq;
     use ark_ff::{BigInteger, PrimeField};
     use num_bigint::BigInt;
-    use semaphore_rs::group::{EMPTY_ELEMENT, Element, Group};
+    use semaphore_rs::group::{Element, Group, EMPTY_ELEMENT};
     use std::str::FromStr;
 
     fn str_to_element(s: &str) -> Element {
@@ -123,7 +123,7 @@ mod group {
     fn add_members() {
         let mut group = Group::default();
         let elements: Vec<Element> = ADDED_MEMBERS.iter().map(|s| str_to_element(s)).collect();
-        group.add_members(elements).unwrap();
+        group.add_members(&elements).unwrap();
 
         let root = group.root().unwrap();
         assert_eq!(leaf_to_str(&root), ADD_MEMBERS_ROOT_STR);
@@ -215,7 +215,7 @@ mod group {
             .map(|s| str_to_element(s))
             .collect();
 
-        group.add_members(initial_elements).unwrap();
+        group.add_members(&initial_elements).unwrap();
         group
             .add_member(str_to_element("400000000000000000000000000000"))
             .unwrap();