diff --git a/core-java-modules/core-java-collections-list/src/test/java/com/ossez/collections/JavaCollectionCleanupUnitTest.java b/core-java-modules/core-java-collections-list/src/test/java/com/ossez/collections/JavaCollectionCleanupUnitTest.java
index 40aa276080..f5f85ee57d 100644
--- a/core-java-modules/core-java-collections-list/src/test/java/com/ossez/collections/JavaCollectionCleanupUnitTest.java
+++ b/core-java-modules/core-java-collections-list/src/test/java/com/ossez/collections/JavaCollectionCleanupUnitTest.java
@@ -1,9 +1,9 @@
package com.ossez.collections;
+import static org.hamcrest.MatcherAssert.assertThat;
import static org.hamcrest.Matchers.containsInAnyOrder;
import static org.hamcrest.Matchers.containsInRelativeOrder;
import static org.hamcrest.Matchers.hasSize;
-import static org.junit.Assert.assertThat;
import java.util.ArrayList;
import java.util.Collections;
@@ -13,22 +13,28 @@ import java.util.List;
import org.apache.commons.collections4.CollectionUtils;
import org.apache.commons.collections4.PredicateUtils;
-import org.junit.Test;
import com.google.common.base.Predicates;
import com.google.common.collect.Iterables;
import com.google.common.collect.Lists;
import com.google.common.collect.Sets;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.TestInstance;
+/**
+ * Test for list Clean up
+ *
+ * https://www.ossez.com/t/java-list-null/13940
+ *
+ * @author YuCheng
+ */
+@TestInstance(TestInstance.Lifecycle.PER_CLASS)
public class JavaCollectionCleanupUnitTest {
- // tests - removing nulls
-
@Test
public void givenListContainsNulls_whenRemovingNullsWithPlainJava_thenCorrect() {
final List list = Lists.newArrayList(null, 1, null);
- while (list.remove(null))
- ;
+ while (list.remove(null));
assertThat(list, hasSize(1));
}
diff --git a/core-java-modules/pom.xml b/core-java-modules/pom.xml
index c5bf29f281..09d7bee994 100644
--- a/core-java-modules/pom.xml
+++ b/core-java-modules/pom.xml
@@ -16,10 +16,10 @@
core-java-8
-
+ core-java-annotations
core-java-collections-list
-
-
+
+