Minor - fix failing test

ljacqu · Jul 16, 2016 · 02bb68c · 02bb68c
1 parent 56c87d2
commit 02bb68c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/test/java/ch/jalu/injector/utils/ReflectionUtilsTest.java b/src/test/java/ch/jalu/injector/utils/ReflectionUtilsTest.java
@@ -5,7 +5,6 @@
 import ch.jalu.injector.annotations.NoMethodScan;
 import ch.jalu.injector.exceptions.InjectorException;
 import ch.jalu.injector.exceptions.InjectorReflectionException;
-import ch.jalu.injector.samples.PostConstructTestClass;
 import org.hamcrest.Matcher;
 import org.junit.Test;
 
@@ -22,10 +21,10 @@
 import java.util.Set;
 
 import static org.hamcrest.Matchers.arrayContaining;
-import static org.hamcrest.Matchers.arrayWithSize;
 import static org.hamcrest.Matchers.contains;
 import static org.hamcrest.Matchers.emptyArray;
 import static org.hamcrest.Matchers.equalTo;
+import static org.hamcrest.Matchers.greaterThanOrEqualTo;
 import static org.hamcrest.Matchers.instanceOf;
 import static org.hamcrest.Matchers.nullValue;
 import static org.junit.Assert.assertThat;
@@ -293,13 +292,15 @@ public void shouldBeWellFormedUtilsClass() {
     @Test
     public void shouldReturnEmptyMethodListForNoMethodScanClass() {
         assertThat(ReflectionUtils.safeGetDeclaredMethods(NoMethodScanClass.class), emptyArray());
-        assertThat(ReflectionUtils.safeGetDeclaredMethods(PostConstructTestClass.class), arrayWithSize(3));
+        // Note: We need to check the size >= expected because plugins like Jacoco may add additional members
+        assertThat(ReflectionUtils.safeGetDeclaredMethods(ReflectionsTestClass.class).length, greaterThanOrEqualTo(4));
     }
 
     @Test
     public void shouldReturnEmptyFieldListForNoFieldScanClass() {
         assertThat(ReflectionUtils.safeGetDeclaredFields(NoFieldScanClass.class), emptyArray());
-        assertThat(ReflectionUtils.safeGetDeclaredFields(PostConstructTestClass.class), arrayWithSize(3));
+        // Note: We need to check the size >= expected because plugins like Jacoco may add additional members
+        assertThat(ReflectionUtils.safeGetDeclaredFields(ReflectionsTestClass.class).length, greaterThanOrEqualTo(4));
     }
 
     private static Field getField(String name) {

diff --git a/src/test/java/ch/jalu/injector/utils/ReflectionsTestClass.java b/src/test/java/ch/jalu/injector/utils/ReflectionsTestClass.java
@@ -1,7 +1,7 @@
 package ch.jalu.injector.utils;
 
 /**
- * Test class with various fields, methods and
+ * Test class with various fields, methods and constructors.
  */
 public class ReflectionsTestClass {