added UTs for remoteConfig and client

Author: Varun Rathore

src/test/java/com/google/firebase/remoteconfig/FirebaseRemoteConfigTest.java

@@ -22,6 +22,7 @@
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.fail;
 
+import com.google.firebase.testing.TestUtils;
 import com.google.firebase.ErrorCode;
 import com.google.firebase.FirebaseApp;
 import com.google.firebase.FirebaseOptions;
@@ -44,6 +45,10 @@ public class FirebaseRemoteConfigTest {
   private static final FirebaseRemoteConfigException TEST_EXCEPTION =
           new FirebaseRemoteConfigException(ErrorCode.INTERNAL, "Test error message");
 
+  private static final String MOCK_SERVER_TEMPLATE_RESPONSE = TestUtils
+          .loadResource("getServerRemoteConfig.json");
+
+
   @After
   public void tearDown() {
     TestOnlyImplFirebaseTrampolines.clearInstancesForTest();
@@ -80,6 +85,19 @@ public void testDefaultRemoteConfigClient() {
     assertEquals(expectedUrl, ((FirebaseRemoteConfigClientImpl) client).getRemoteConfigUrl());
   }
 
+  @Test
+  public void testDefaultServerRemoteConfigClient() {
+    FirebaseApp app = FirebaseApp.initializeApp(TEST_OPTIONS, "custom-app");
+    FirebaseRemoteConfig remoteConfig = FirebaseRemoteConfig.getInstance(app);
+
+    FirebaseRemoteConfigClient client = remoteConfig.getRemoteConfigClient();
+
+    assertTrue(client instanceof FirebaseRemoteConfigClientImpl);
+    assertSame(client, remoteConfig.getRemoteConfigClient());
+    String expectedUrl = "https://firebaseremoteconfig.googleapis.com/v1/projects/test-project/namespaces/firebase-server/serverRemoteConfig";
+    assertEquals(expectedUrl, ((FirebaseRemoteConfigClientImpl) client).getServerRemoteConfigUrl());
+  }
+
   @Test
   public void testAppDelete() {
     FirebaseApp app = FirebaseApp.initializeApp(TEST_OPTIONS, "custom-app");
@@ -597,4 +615,76 @@ private FirebaseRemoteConfig getRemoteConfig(FirebaseRemoteConfigClient client)
     FirebaseApp app = FirebaseApp.initializeApp(TEST_OPTIONS);
     return new FirebaseRemoteConfig(app, client);
   }
+
+
+  // Get Server template tests
+
+  @Test
+  public void testGetServerTemplate() throws FirebaseRemoteConfigException {
+    String TEST_SERVER_TEMPLATE = "{\n" +
+          "  \"etag\": \"etag-123456789012-1\",\n" +
+          "  \"parameters\": {},\n" +
+          "  \"serverConditions\": [],\n" +
+          "  \"parameterGroups\": {}\n" +
+          "}";
+    MockRemoteConfigClient client = MockRemoteConfigClient.fromServerTemplate(
+      new ServerTemplateData().setETag(TEST_ETAG).toJSON());
+    FirebaseRemoteConfig remoteConfig = getRemoteConfig(client);
+
+    ServerTemplate template = remoteConfig.getServerTemplate();
+    String templateData = template.toJson();
+    assertEquals(TEST_SERVER_TEMPLATE, templateData);
+  }
+
+  @Test
+  public void testGetServerTemplate1() throws FirebaseRemoteConfigException {
+    MockRemoteConfigClient client = MockRemoteConfigClient.fromTemplate(
+            new Template().setETag(TEST_ETAG));
+    FirebaseRemoteConfig remoteConfig = getRemoteConfig(client);
+
+    Template template = remoteConfig.getTemplate();
+
+    assertEquals(TEST_ETAG, template.getETag());
+  }
+
+  @Test
+  public void testGetServerTemplateFailure() {
+    MockRemoteConfigClient client = MockRemoteConfigClient.fromException(TEST_EXCEPTION);
+    FirebaseRemoteConfig remoteConfig = getRemoteConfig(client);
+
+    try {
+      remoteConfig.getServerTemplate();
+    } catch (FirebaseRemoteConfigException e) {
+      assertSame(TEST_EXCEPTION, e);
+    }
+  }
+
+  @Test
+  public void testGetServerTemplateAsync() throws Exception {
+    String TEST_SERVER_TEMPLATE = "{\n" +
+          "  \"etag\": \"etag-123456789012-1\",\n" +
+          "  \"parameters\": {},\n" +
+          "  \"serverConditions\": [],\n" +
+          "  \"parameterGroups\": {}\n" +
+          "}";
+    MockRemoteConfigClient client = MockRemoteConfigClient.fromServerTemplate(
+      new ServerTemplateData().setETag(TEST_ETAG).toJSON());
+    FirebaseRemoteConfig remoteConfig = getRemoteConfig(client);
+
+    ServerTemplate template = remoteConfig.getServerTemplateAsync().get();
+    String templateData = template.toJson();
+    assertEquals(TEST_SERVER_TEMPLATE, templateData);
+  }
+
+  @Test
+  public void testGetServerTemplateAsyncFailure() throws InterruptedException {
+    MockRemoteConfigClient client = MockRemoteConfigClient.fromException(TEST_EXCEPTION);
+    FirebaseRemoteConfig remoteConfig = getRemoteConfig(client);
+
+    try {
+      remoteConfig.getServerTemplateAsync().get();
+    } catch (ExecutionException e) {
+      assertSame(TEST_EXCEPTION, e.getCause());
+    }
+  }
 }

src/test/java/com/google/firebase/remoteconfig/MockRemoteConfigClient.java

@@ -21,28 +21,35 @@
 public class MockRemoteConfigClient implements FirebaseRemoteConfigClient{
 
   private final Template resultTemplate;
+  private final String resultServerTemplate;
   private final FirebaseRemoteConfigException exception;
   private final ListVersionsResponse listVersionsResponse;
 
   private MockRemoteConfigClient(Template resultTemplate,
+                                 String resultServerTemplate,
                                  ListVersionsResponse listVersionsResponse,
                                  FirebaseRemoteConfigException exception) {
     this.resultTemplate = resultTemplate;
+    this.resultServerTemplate = resultServerTemplate;
     this.listVersionsResponse = listVersionsResponse;
     this.exception = exception;
   }
 
   static MockRemoteConfigClient fromTemplate(Template resultTemplate) {
-    return new MockRemoteConfigClient(resultTemplate, null, null);
+    return new MockRemoteConfigClient(resultTemplate,null, null, null);
+  }
+
+  static MockRemoteConfigClient fromServerTemplate(String resultTemplate) {
+    return new MockRemoteConfigClient(null, resultTemplate,null, null);
   }
 
   static MockRemoteConfigClient fromListVersionsResponse(
           ListVersionsResponse listVersionsResponse) {
-    return new MockRemoteConfigClient(null, listVersionsResponse, null);
+    return new MockRemoteConfigClient(null,null, listVersionsResponse, null);
   }
 
   static MockRemoteConfigClient fromException(FirebaseRemoteConfigException exception) {
-    return new MockRemoteConfigClient(null, null, exception);
+    return new MockRemoteConfigClient(null,null, null, exception);
   }
 
   @Override
@@ -54,19 +61,19 @@ public Template getTemplate() throws FirebaseRemoteConfigException {
   }
 
   @Override
-  public Template getTemplateAtVersion(String versionNumber) throws FirebaseRemoteConfigException {
+  public String getServerTemplate() throws FirebaseRemoteConfigException {
     if (exception != null) {
       throw exception;
     }
-    return resultTemplate;
+    return resultServerTemplate;
   }
 
   @Override
-  public String getServerTemplate() throws FirebaseRemoteConfigException {
+  public Template getTemplateAtVersion(String versionNumber) throws FirebaseRemoteConfigException {
     if (exception != null) {
       throw exception;
     }
-    return resultTemplate.toString();
+    return resultTemplate;
   }
 
   @Override