Added Jacoco, some refactoring and code styling

Author: polysantiago

.gitignore

@@ -1,21 +1,55 @@
+.gradle
+/build/
+!gradle/wrapper/gradle-wrapper.jar
+dependency-reduced-pom.xml
+
+### STS ###
+.apt_generated
+.classpath
+.factorypath
+.project
+.settings
+.springBeans
+
+### IntelliJ IDEA ###
+.idea
+*.iws
+*.ipr
 *.iml
-**/target/
 
-coverage-error.log
+### NetBeans ###
+nbproject/private/
+nbbuild/
+dist/
+nbdist/
+.nb-gradle/
 
-**/build
+### Eclipse ###
+*.eclipse
+*.project
+*.classpath
+.extract*
+.settings*
 
-*.log
-.gradle
-schema
+### Plugins ###
+*.eclipse-pmd
+**.metadata
+*.pmd
+*.checkstyle
+*-result.html
+rebel_project.xml
 
-src/test/generated-java
+### npm ###
+node
+node_modules
+frontend-dest
+data/
 
-.idea/*
-## Videocore shared settings:
-!/.idea/vcs.xml
-!/.idea/codeStyleSettings.xml
-!.idea/checkstyle-idea.xml
-!.idea/encodings.xml
-!.idea/sqldialects.xml
-!.idea/compiler.xml
+.DS_Store
+*.py~
+*.swp
+*.pyc
+*~
+target/
+*#
+*.log

build.gradle

@@ -1,6 +1,7 @@
 plugins {
     id 'java'
     id 'idea'
+    id 'jacoco'
     id "io.spring.dependency-management" version "1.0.2.RELEASE"
     id 'maven-publish'
     id 'signing'
@@ -50,6 +51,7 @@ dependencyManagement {
 dependencies {
     compile("org.springframework.boot:spring-boot-autoconfigure")
     compile("org.springframework.boot:spring-boot-starter-logging")
+    compileOnly('org.springframework.boot:spring-boot-configuration-processor')
     compile("org.springframework:spring-webmvc")
     compile('org.springframework:spring-aop')
     compile('org.aspectj:aspectjweaver')

src/main/java/io/github/polysantiago/spring/rest/RestClientProperties.java

@@ -1,13 +1,14 @@
 package io.github.polysantiago.spring.rest;
 
 import io.github.polysantiago.spring.rest.retry.RetrySettings;
-import lombok.Data;
-import org.springframework.boot.context.properties.ConfigurationProperties;
-
 import java.util.HashMap;
 import java.util.Map;
+import lombok.Getter;
+import lombok.Setter;
+import org.springframework.boot.context.properties.ConfigurationProperties;
 
-@Data
+@Getter
+@Setter
 @ConfigurationProperties(prefix = RestClientProperties.PREFIX)
 public class RestClientProperties {
 

src/main/java/io/github/polysantiago/spring/rest/retry/BackOffSettings.java

@@ -1,8 +1,10 @@
 package io.github.polysantiago.spring.rest.retry;
 
-import lombok.Data;
+import lombok.Getter;
+import lombok.Setter;
 
-@Data
+@Getter
+@Setter
 public class BackOffSettings {
 
     private long delay = 1000;

src/main/java/io/github/polysantiago/spring/rest/retry/RetrySettings.java

@@ -1,8 +1,10 @@
 package io.github.polysantiago.spring.rest.retry;
 
-import lombok.Data;
+import lombok.Getter;
+import lombok.Setter;
 
-@Data
+@Getter
+@Setter
 public class RetrySettings {
 
     private int maxAttempts = 3;

src/main/java/io/github/polysantiago/spring/rest/support/SyntheticParametrizedType.java

@@ -1,13 +1,13 @@
 package io.github.polysantiago.spring.rest.support;
 
-import lombok.Getter;
-import org.springframework.core.ResolvableType;
+import static java.util.stream.Collectors.joining;
 
 import java.io.Serializable;
 import java.lang.reflect.ParameterizedType;
 import java.lang.reflect.Type;
 import java.util.Arrays;
-import java.util.stream.Collectors;
+import lombok.Getter;
+import org.springframework.core.ResolvableType;
 
 @Getter
 public final class SyntheticParametrizedType implements ParameterizedType, Serializable {
@@ -39,6 +39,7 @@ public Type getOwnerType() {
     @Override
     public String toString() {
         return String.format("%s<%s<%s>>", SyntheticParametrizedType.class.getName(), rawType.getTypeName(),
-            Arrays.stream(actualTypeArguments).map(Type::getTypeName).collect(Collectors.joining(",")));
+            Arrays.stream(actualTypeArguments).map(Type::getTypeName).collect(joining(",")));
     }
+
 }

src/main/java/io/github/polysantiago/spring/rest/support/SyntheticParametrizedTypeReference.java

@@ -1,14 +1,15 @@
 package io.github.polysantiago.spring.rest.support;
 
-import lombok.*;
+import java.lang.reflect.Type;
+import lombok.AccessLevel;
+import lombok.EqualsAndHashCode;
+import lombok.Getter;
+import lombok.RequiredArgsConstructor;
 import org.springframework.core.ParameterizedTypeReference;
 import org.springframework.core.ResolvableType;
 
-import java.lang.reflect.Type;
-
 @Getter
 @EqualsAndHashCode(callSuper = false)
-@ToString
 @RequiredArgsConstructor(access = AccessLevel.PRIVATE)
 public class SyntheticParametrizedTypeReference<T> extends ParameterizedTypeReference<T> {
 

src/main/java/io/github/polysantiago/spring/rest/util/OptionalTypeFutureAdapter.java

@@ -1,17 +1,16 @@
 package io.github.polysantiago.spring.rest.util;
 
+import java.util.Optional;
+import java.util.concurrent.ExecutionException;
+import java.util.concurrent.TimeUnit;
+import java.util.concurrent.TimeoutException;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.util.concurrent.FailureCallback;
 import org.springframework.util.concurrent.ListenableFuture;
 import org.springframework.util.concurrent.SuccessCallback;
 import org.springframework.web.client.HttpClientErrorException;
 
-import java.util.Optional;
-import java.util.concurrent.ExecutionException;
-import java.util.concurrent.TimeUnit;
-import java.util.concurrent.TimeoutException;
-
 public class OptionalTypeFutureAdapter<T> extends ResponseFutureAdapter<Optional<T>> {
 
     public OptionalTypeFutureAdapter(ListenableFuture<ResponseEntity<Optional<T>>> delegate) {
@@ -22,7 +21,7 @@ public OptionalTypeFutureAdapter(ListenableFuture<ResponseEntity<Optional<T>>> d
     public void addCallback(SuccessCallback<? super Optional<T>> successCallback, FailureCallback failureCallback) {
         super.addCallback(successCallback, throwable -> {
             if (isNotFoundException(throwable)) {
-                successCallback.onSuccess(Optional.<T>empty());
+                successCallback.onSuccess(Optional.empty());
             } else {
                 failureCallback.onFailure(throwable);
             }

src/main/java/io/github/polysantiago/spring/rest/util/ResolvableTypeUtils.java

@@ -1,19 +1,13 @@
 package io.github.polysantiago.spring.rest.util;
 
-import lombok.experimental.UtilityClass;
-import org.springframework.core.ResolvableType;
-
 import java.lang.reflect.Method;
-import java.lang.reflect.ParameterizedType;
 import java.util.Arrays;
+import lombok.experimental.UtilityClass;
+import org.springframework.core.ResolvableType;
 
 @UtilityClass
 public class ResolvableTypeUtils {
 
-    public static boolean returnTypeIsGeneric(Method method) {
-        return method.getGenericReturnType() instanceof ParameterizedType;
-    }
-
     public static boolean typeIsAnyOf(ResolvableType resolvableType, Class<?>... classes) {
         return Arrays.stream(classes).anyMatch(clazz -> typeIs(resolvableType, clazz));
     }
@@ -23,7 +17,8 @@ public static boolean typeIs(ResolvableType resolvableType, Class<?> clazz) {
     }
 
     public static boolean returnTypeIs(Method method, Class<?> returnType) {
-        return ResolvableType.forClass(returnType).isAssignableFrom(ResolvableType.forMethodReturnType(method));
+        return ResolvableType.forClass(returnType)
+            .isAssignableFrom(ResolvableType.forMethodReturnType(method));
     }
 
     public static boolean returnTypeIsAnyOf(Method method, Class<?>... returnTypes) {