refs #161: upgrade dependencies (#162)

* refs #161: upgrade dependencies

* refs #161: Update README.md for dependency versions

Author: JaidenAshmore

README.md

@@ -76,12 +76,12 @@ The framework relies on the following dependencies and therefore it is recommend
 for compatibility.
 - [Core Framework](java-dynamic-sqs-listener-core)
   - JDK 1.8 or higher
-  - [AWS SQS SDK](https://docs.aws.amazon.com/sdk-for-java/v2/developer-guide/welcome.html): 2.12.0
+  - [AWS SQS SDK](https://docs.aws.amazon.com/sdk-for-java/v2/developer-guide/welcome.html): 2.13.7
   - [Guava](https://github.com/google/guava): 29.0-jre
-  - [Jackson Databind](https://github.com/FasterXML/jackson-databind): 2.9.10.3
+  - [Jackson Databind](https://github.com/FasterXML/jackson-databind): 2.11.0
 - [Spring Framework](java-dynamic-sqs-listener-spring)
   - All of the core dependencies
-  - [Spring Boot](https://github.com/spring-projects/spring-boot): 2.2.5.RELEASE
+  - [Spring Boot](https://github.com/spring-projects/spring-boot): 2.3.1.RELEASE
 
 ### How to Mark the message as successfully processed
 When the method executing the message finishes without throwing an exception, the

java-dynamic-sqs-listener-core/src/test/java/com/jashmore/sqs/argument/attribute/MessageAttributeArgumentResolverTest.java

@@ -1,6 +1,6 @@
 package com.jashmore.sqs.argument.attribute;
 
-import static org.assertj.core.api.Java6Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThat;
 
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.google.common.collect.ImmutableMap;
@@ -21,7 +21,7 @@
 
 @Slf4j
 class MessageAttributeArgumentResolverTest {
-    private MessageAttributeArgumentResolver messageAttributeArgumentResolver = new MessageAttributeArgumentResolver(new ObjectMapper());
+    private final MessageAttributeArgumentResolver messageAttributeArgumentResolver = new MessageAttributeArgumentResolver(new ObjectMapper());
 
     @Test
     void stringMessageAttributesCanBeObtainedFromMessage() throws Exception {
@@ -552,6 +552,6 @@ public void consume(@MessageAttribute("bytes") final byte[] b) {
     @Builder
     @SuppressWarnings("WeakerAccess")
     public static class MyPojo {
-        private final String name;
+        String name;
     }
 }

java-dynamic-sqs-listener-core/src/test/java/com/jashmore/sqs/argument/payload/mapper/JacksonPayloadMapperTest.java

@@ -6,6 +6,7 @@
 import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.when;
 
+import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
@@ -58,7 +59,7 @@ void payloadContainerPojoCanBeMappedToObject() throws IOException {
     void errorBuildingPayloadThrowsArgumentResolutionException() throws IOException {
         // arrange
         final Message message = Message.builder().body("test").build();
-        when(objectMapper.readValue(anyString(), eq(Pojo.class))).thenThrow(new IOException());
+        when(objectMapper.readValue(anyString(), eq(Pojo.class))).thenThrow(JsonProcessingException.class);
 
         // act
         final PayloadMappingException exception = assertThrows(PayloadMappingException.class, () -> payloadMapper.map(message, Pojo.class));
@@ -80,5 +81,4 @@ String getField() {
             return field;
         }
     }
-
 }

java-dynamic-sqs-listener-spring/java-dynamic-sqs-listener-spring-core/src/test/java/it/com/jashmore/sqs/proxy/ProxyBeanQueueListenerResolutionIntegrationTest.java

@@ -1,6 +1,6 @@
 package it.com.jashmore.sqs.proxy;
 
-import static org.assertj.core.api.Java6Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThat;
 
 import com.jashmore.sqs.argument.payload.Payload;
 import com.jashmore.sqs.elasticmq.ElasticMqSqsAsyncClient;

pom.xml

@@ -66,32 +66,37 @@
         <checkstyle.config.location>configuration/checkstyle/google_6_18_checkstyle.xml</checkstyle.config.location>
         <spotbugs.config.location>configuration/spotbugs/bugsExcludeFilter.xml</spotbugs.config.location>
 
-        <assertj.version>3.12.2</assertj.version>
+        <assertj.version>3.16.1</assertj.version>
         <avro.version>1.9.2</avro.version>
-        <aws.sdk.version>2.13.3</aws.sdk.version>
+        <aws.sdk.version>2.13.7</aws.sdk.version>
         <builder-helper-maven-plugin.version>3.1.0</builder-helper-maven-plugin.version>
         <checkstyle-plugin.version>3.1.1</checkstyle-plugin.version>
-        <cglib.version>3.2.10</cglib.version>
+        <cglib.version>3.3.0</cglib.version>
         <elasticmq.version>0.15.6</elasticmq.version>
         <guava.version>29.0-jre</guava.version>
-        <hamcrest.verion>2.2</hamcrest.verion>
-        <jackson.version>2.9.10.4</jackson.version>
+        <jackson.version>2.11.0</jackson.version>
         <jacoco.version>0.8.5</jacoco.version>
         <javax.validation.version>2.0.1.Final</javax.validation.version>
-        <junit4.version>4.12</junit4.version>
         <junit-jupiter.version>5.6.2</junit-jupiter.version>
         <logback.version>1.2.3</logback.version>
         <lombok.version>1.18.12</lombok.version>
+
+        <!-- Be wary of upgrading this to 3.0.0-M5. It made tests begin to fail for no reason, specifically
+         ConcurrentMessageBrokerTest#threadInterruptedDuringBackoffShouldStopBroker and I was able to get it to
+         pass by adding a log message before the shutdown. It is like the main thread wasn't being ran after the
+         countdown latch was passed. No idea why... -->
         <maven.failsafe.version>3.0.0-M4</maven.failsafe.version>
         <maven.surefire.version>3.0.0-M4</maven.surefire.version>
+
+
         <mockito.version>3.3.3</mockito.version>
         <mockito.jupiter.version>3.3.3</mockito.jupiter.version>
-        <pitest.version>1.5.1</pitest.version>
+        <pitest.version>1.5.2</pitest.version>
         <slf4j.version>1.7.30</slf4j.version>
         <spotbugs.plugin.version>4.0.0</spotbugs.plugin.version>
         <spotbugs.version>4.0.2</spotbugs.version>
         <spring-cloud-schema-registry-client.version>1.0.3.RELEASE</spring-cloud-schema-registry-client.version>
-        <spring.boot.version>2.2.5.RELEASE</spring.boot.version>
+        <spring.boot.version>2.3.1.RELEASE</spring.boot.version>
         <skipTests>false</skipTests>
         <maven.javadoc.skip>true</maven.javadoc.skip>
     </properties>
@@ -202,12 +207,6 @@
                 <version>${elasticmq.version}</version>
             </dependency>
 
-            <dependency>
-                <groupId>junit</groupId>
-                <artifactId>junit</artifactId>
-                <version>${junit4.version}</version>
-            </dependency>
-
             <dependency>
                 <groupId>org.junit.jupiter</groupId>
                 <artifactId>junit-jupiter</artifactId>
@@ -227,12 +226,6 @@
                 <version>${mockito.version}</version>
             </dependency>
 
-            <dependency>
-                <groupId>org.hamcrest</groupId>
-                <artifactId>hamcrest</artifactId>
-                <version>${hamcrest.verion}</version>
-            </dependency>
-
             <dependency>
                 <groupId>org.assertj</groupId>
                 <artifactId>assertj-core</artifactId>

util/common-utils/src/main/java/com/jashmore/sqs/util/properties/PropertyUtils.java

@@ -103,7 +103,7 @@ private <T> T safelyGetValue(final String propertyName,
                         return true;
                     })
                     .orElse(defaultValue);
-        } catch (final Throwable throwable) {
+        } catch (final RuntimeException throwable) {
             log.error("Error obtaining Property value {} returning default value {}", propertyName, defaultValue, throwable);
             return defaultValue;
         }