Add tests for ApolloConfig

spotify · Jan 19, 2016 · c360cf4 · c360cf4
1 parent 94566ed
commit c360cf4
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/apollo-environment/src/main/java/com/spotify/apollo/environment/ApolloConfig.java b/apollo-environment/src/main/java/com/spotify/apollo/environment/ApolloConfig.java
@@ -62,7 +62,6 @@ public boolean enableOutgoingRequestLogging() {
     return optionalBoolean(apolloNode, "logOutgoingRequests").orElse(true);
   }
 
-
   public boolean enableMetaApi() {
     return optionalBoolean(apolloNode, "metaApi").orElse(true);
   }

diff --git a/apollo-environment/src/test/java/com/spotify/apollo/environment/ApolloConfigTest.java b/apollo-environment/src/test/java/com/spotify/apollo/environment/ApolloConfigTest.java
@@ -87,6 +87,24 @@ public void testEnableIncomingRequestLoggingDefault() throws Exception {
     assertThat(sut.enableIncomingRequestLogging(), is(true));
   }
 
+  @Test
+  public void testEnableOutgoingRequestLogging() throws Exception {
+    final Config config = ConfigFactory.parseMap(ImmutableMap.of(
+        "apollo.logOutgoingRequests", false
+    ));
+    final ApolloConfig sut = new ApolloConfig(config);
+
+    assertThat(sut.enableOutgoingRequestLogging(), is(false));
+  }
+
+  @Test
+  public void testEnableOutgoingRequestLoggingDefault() throws Exception {
+    final Config config = ConfigFactory.parseMap(ImmutableMap.of());
+    final ApolloConfig sut = new ApolloConfig(config);
+
+    assertThat(sut.enableOutgoingRequestLogging(), is(true));
+  }
+
   @Test
   public void testEnableMetaApi() throws Exception {
     final Config config = ConfigFactory.parseMap(ImmutableMap.of(