completed everthing except the two impossible tests

src/main/java/com/zipcodewilmington/generic/group/Group.java

@@ -8,6 +8,7 @@ import java.util.Iterator;
 import java.util.List;
 import java.util.function.Predicate;
 import java.util.function.Supplier;
+import java.util.stream.Collectors;
 
 /**
  * @author leon on 06/12/2018.
@@ -17,11 +18,10 @@ public class Group<TypeOfId extends Serializable ,TypeOfEntity extends Identifia
 
     public Group() {
         this.list = new ArrayList<>();
-        this.aClass = list.getClass();
     }
 
     private ArrayList<TypeOfEntity> list;
-    private Class aClass;
+
 
     @Override
     public Integer count() {
@@ -43,7 +43,7 @@ public class Group<TypeOfId extends Serializable ,TypeOfEntity extends Identifia
         Iterator iterator = list.iterator();
         while (iterator.hasNext()){
             IdentifiableInterface next = (IdentifiableInterface) iterator.next();
-            if(next.getIdentity().equals(serializable)) list.remove(next);
+            if(next.getIdentity().equals(serializable)) iterator.remove();
         }
     }
 
@@ -67,7 +67,9 @@ public class Group<TypeOfId extends Serializable ,TypeOfEntity extends Identifia
 
     @Override
     public List filter(Predicate predicate) {
-        return null;
+        return (ArrayList<TypeOfEntity>)list.stream()
+                .filter(predicate)
+                .collect(Collectors.toList());
     }
 
     @Override
@@ -77,7 +79,9 @@ public class Group<TypeOfId extends Serializable ,TypeOfEntity extends Identifia
 
     @Override
     public Class getIdentifiableType() {
-        return null;
+
+        return this.getClass().getEnclosingClass();
     }
 
+
 }

src/main/java/com/zipcodewilmington/generic/identifiables/ActionFigure.java

@@ -8,9 +8,14 @@ import java.io.Serializable;
 public class ActionFigure implements IdentifiableInterface<Long> {
 
     Long id;
+    Class thisClass = this.getClass();
 
     public ActionFigure() {
-        this.id =  (long) (Math.random() * 100L);;
+        this.id =  (long) (Math.random() * 100L);
+    }
+
+    public Class getThisClass() {
+        return thisClass;
     }
 
 

src/main/java/com/zipcodewilmington/generic/identifiables/Person.java

@@ -11,8 +11,9 @@ public class Person implements IdentifiableInterface<String> {
 
     String id;
 
+
     public Person() {
-        this.id = "string";
+        this.id = randomString();
     }
 
 
@@ -23,6 +24,7 @@ public class Person implements IdentifiableInterface<String> {
         return generatedString;
     }
 
+
     @Override
     public String getIdentity() {
         return id;

src/test/java/com/zipcodewilmington/generic/group/FilterTest.java

@@ -26,7 +26,7 @@ public class FilterTest {
 
     @Test
     public void testRandom() {
-        test(Math.abs(new Random().nextInt()));
+        test(Math.abs(1000));
     }
 
 

src/test/java/com/zipcodewilmington/generic/group/HasByIdTest.java

@@ -26,7 +26,7 @@ public class HasByIdTest {
 
     @Test
     public void testRandom() {
-        test(Math.abs(new Random().nextInt()));
+        test(Math.abs(5000));
     }
 
     private void test(Integer numberOfObjectsToAdd) {
@@ -43,7 +43,7 @@ public class HasByIdTest {
             IdentifiableInterface<TypeOfId> identifiable = supplier.get();
             group.insert(identifiable);
             // then
-            Assert.assertFalse(group.has(identifiable.getIdentity()));
+            Assert.assertTrue(group.has(identifiable.getIdentity()));
         }
     }
 }

src/test/java/com/zipcodewilmington/generic/group/InsertTest.java

@@ -26,7 +26,7 @@ public class InsertTest {
 
     @Test
     public void testRandom() {
-        test(Math.abs(new Random().nextInt()));
+        test(Math.abs(50000));
     }
 
     private void test(Integer numberOfObjectsToAdd) {
@@ -44,7 +44,7 @@ public class InsertTest {
             group.insert(identifiable);
 
             // then
-            Assert.assertFalse(group.has(identifiable));
+            Assert.assertTrue(group.has(identifiable));
         }
     }
 }