Release 2.2.1: ignore synthetic fields (fixes #10)

Author: f-lopes

README.md

@@ -1,7 +1,7 @@
 # Spring MVC Test utils
 
 [![Codacy Badge](https://api.codacy.com/project/badge/Grade/1e521e339dc446328b7ccc52faaa7648)](https://www.codacy.com/app/f-lopes/spring-mvc-test-utils?utm_source=github.com&utm_medium=referral&utm_content=f-lopes/spring-mvc-test-utils&utm_campaign=badger)
-[![Maven Central](https://maven-badges.herokuapp.com/maven-central/io.florianlopes/spring-mvc-test-utils/badge.svg)](https://search.maven.org/#artifactdetails%7Cio.florianlopes%7Cspring-mvc-test-utils%7C2.2.0%7Cjar)
+[![Maven Central](https://maven-badges.herokuapp.com/maven-central/io.florianlopes/spring-mvc-test-utils/badge.svg)](https://search.maven.org/#artifactdetails%7Cio.florianlopes%7Cspring-mvc-test-utils%7C2.2.1%7Cjar)
 [![CircleCI](https://circleci.com/gh/f-lopes/spring-mvc-test-utils/tree/master.svg?style=svg&circle-token=b34b191c4f183c701aee405ba6dca66c9f277d49)](https://circleci.com/gh/f-lopes/spring-mvc-test-utils/tree/master)
 [![codecov](https://codecov.io/gh/f-lopes/spring-mvc-test-utils/branch/master/graph/badge.svg?token=2yY70RB1tw)](https://codecov.io/gh/f-lopes/spring-mvc-test-utils)
 [![Dependency Status](https://www.versioneye.com/user/projects/57e036e36dfcd0003e7f55c3/badge.svg?style=flat-square)](https://www.versioneye.com/user/projects/57e036e36dfcd0003e7f55c3)
@@ -19,7 +19,7 @@ Add this dependency to your pom.xml file:
 <dependency>
     <groupId>io.florianlopes</groupId>
     <artifactId>spring-mvc-test-utils</artifactId>
-    <version>2.2.0</version>
+    <version>2.2.1</version>
     <scope>test</scope>
 </dependency>
 ```

src/main/java/io/florianlopes/spring/test/web/servlet/request/MockMvcRequestBuilderUtils.java

@@ -6,11 +6,11 @@
 import java.lang.reflect.Type;
 import java.time.temporal.Temporal;
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.Date;
 import java.util.List;
 import java.util.Map;
 import java.util.TreeMap;
+import java.util.stream.Collectors;
 import org.apache.commons.lang3.StringUtils;
 import org.apache.commons.lang3.reflect.FieldUtils;
 import org.apache.commons.logging.Log;
@@ -95,7 +95,7 @@ private static MockHttpServletRequestBuilder buildFormFields(Object form, MockHt
     }
 
     private static Map<String, String> getFormFields(Object form, Map<String, String> formFields, String path) {
-        final List<Field> fields = form != null ? Arrays.asList(FieldUtils.getAllFields(form.getClass())) : new ArrayList<>();
+        final List<Field> fields = form != null ? getAllNonSyntheticFields(form) : new ArrayList<>();
         for (Field field : fields) {
             final Class<?> fieldType = field.getType();
             final Object fieldValue = getFieldValue(form, field);
@@ -134,6 +134,14 @@ private static Map<String, String> getFormFields(Object form, Map<String, String
         return formFields;
     }
 
+    private static List<Field> getAllNonSyntheticFields(Object form) {
+        return FieldUtils.getAllFieldsList(form.getClass())
+                .stream()
+                // Only retrieve non synthetic fields to ignore JaCoCo's ones (https://github.com/f-lopes/spring-mvc-test-utils/issues/10)
+                .filter(field -> !field.isSynthetic())
+                .collect(Collectors.toList());
+    }
+
     private static Map<?, ?> getMap(Object fieldValue, Class<?> type) {
         return Map.class.isAssignableFrom(type) ? (Map) fieldValue : null;
     }