Remove guava dependency

Author: Caesar Ralf

pom.xml

@@ -125,12 +125,6 @@
       <artifactId>jackson-databind</artifactId>
       <scope>test</scope>
     </dependency>
-    <dependency>
-      <groupId>com.google.guava</groupId>
-      <artifactId>guava</artifactId>
-      <version>17.0</version>
-      <scope>test</scope>
-    </dependency>
   </dependencies>
 
   <build>

src/test/java/com/spotify/logging/LoggingConfiguratorTest.java

@@ -49,7 +49,6 @@
 import ch.qos.logback.classic.net.SyslogAppender;
 import ch.qos.logback.core.ConsoleAppender;
 import ch.qos.logback.core.status.Status;
-import com.google.common.collect.FluentIterable;
 import com.spotify.logging.logback.CustomLogstashEncoder;
 import io.sentry.logback.SentryAppender;
 import net.logstash.logback.composite.loggingevent.ArgumentsJsonProvider;
@@ -214,9 +213,9 @@ private void assertLogstashEncoder(final Level level) {
     final CustomLogstashEncoder encoder = (CustomLogstashEncoder) stdout.getEncoder();
     assertEquals(
         1,
-        FluentIterable.from(encoder.getProviders().getProviders())
-            .filter(ArgumentsJsonProvider.class)
-            .size());
+        encoder.getProviders().getProviders().stream()
+            .filter(ArgumentsJsonProvider.class::isInstance)
+            .count());
   }
 
   private void assertDefault(final String ident, final Level level) {

src/test/java/com/spotify/logging/logback/LogstashEncoderTest.java

@@ -26,10 +26,11 @@
 import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.ObjectMapper;
-import com.google.common.collect.ImmutableList;
-import com.google.common.collect.ImmutableMap;
 import com.spotify.logging.LoggingConfigurator;
 import com.spotify.logging.LoggingConfigurator.Level;
+import java.util.Arrays;
+import java.util.HashMap;
+import java.util.Map;
 import net.logstash.logback.encoder.LogstashEncoder;
 import org.junit.Rule;
 import org.junit.Test;
@@ -52,8 +53,8 @@ public void shouldIncludeStructuredArguments() throws JsonProcessingException {
         "foo={} bar={} list={} map={} thing={}",
         value("foo", 17),
         value("bar", "quux"),
-        value("list", ImmutableList.of(1, 2)),
-        value("map", ImmutableMap.of("a", 3, "b", 4)),
+        value("list", Arrays.asList(1, 2)),
+        value("map", mapOf("a", 3, "b", 4)),
         value("thing", new Thing(5, "6")));
     final String log = systemOutRule.getLog();
     final JsonNode parsedMessage = mapper.readTree(log);
@@ -67,6 +68,14 @@ public void shouldIncludeStructuredArguments() throws JsonProcessingException {
     assertEquals(mapper.createObjectNode().put("v1", 5).put("v2", "6"), parsedMessage.get("thing"));
   }
 
+  private <V> Map<String, V> mapOf(
+      final String keyA, final V valueA, final String keyB, final V valueB) {
+    final Map<String, V> map = new HashMap<>();
+    map.put(keyA, valueA);
+    map.put(keyB, valueB);
+    return map;
+  }
+
   public static class Thing {
 
     private final int v1;