Apache
/
commons-lang
mirror of https://github.com/apache/commons-lang.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Some reflection tests must account for classes files being instrumented 

by Jacoco.
This commit is contained in:
Gary Gregory 2018-03-29 13:26:59 -06:00
parent 56b7ae44f9
commit 17f9d22f33
1 changed files with 19 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/test/java/org/apache/commons/lang3/reflect/FieldUtilsTest.java
View File

@ -165,7 +165,15 @@ public class FieldUtilsTest {
assertArrayEquals(fieldsNumber, FieldUtils.getAllFields(Number.class)); assertArrayEquals(fieldsNumber, FieldUtils.getAllFields(Number.class));
final Field[] fieldsInteger = Integer.class.getDeclaredFields(); final Field[] fieldsInteger = Integer.class.getDeclaredFields();
assertArrayEquals(ArrayUtils.addAll(fieldsInteger, fieldsNumber), FieldUtils.getAllFields(Integer.class)); assertArrayEquals(ArrayUtils.addAll(fieldsInteger, fieldsNumber), FieldUtils.getAllFields(Integer.class));
assertEquals(5, FieldUtils.getAllFields(PublicChild.class).length); final Field[] allFields = FieldUtils.getAllFields(PublicChild.class);
// Under Jacoco,0.8.1 and Java 10, the field count is 7.
int expected = 5;
for (Field field : allFields) {
if (field.getName().equals("$jacocoData")) {
expected++;
}
}
assertEquals(Arrays.toString(allFields), expected, allFields.length);
} }
@Test @Test
@ -177,7 +185,16 @@ public class FieldUtilsTest {
final List<Field> allFieldsInteger = new ArrayList<>(fieldsInteger); final List<Field> allFieldsInteger = new ArrayList<>(fieldsInteger);
allFieldsInteger.addAll(fieldsNumber); allFieldsInteger.addAll(fieldsNumber);
assertEquals(allFieldsInteger, FieldUtils.getAllFieldsList(Integer.class)); assertEquals(allFieldsInteger, FieldUtils.getAllFieldsList(Integer.class));
assertEquals(5, FieldUtils.getAllFieldsList(PublicChild.class).size()); final List<Field> allFields = FieldUtils.getAllFieldsList(PublicChild.class);
// Under Jacoco,0.8.1 and Java 10, the field count is 7.
int expected = 5;
for (Field field : allFields) {
if (field.getName().equals("$jacocoData")) {
expected++;
}
}
assertEquals(allFields.toString(), expected, allFields.size());
} }
@Test @Test