makeKey Add restriction for : (#37)

Author: prayansh

src/main/java/com/segment/analytics/android/integrations/firebase/FirebaseIntegration.java

@@ -258,7 +258,7 @@ private static void putValue(Bundle bundle, String key, Object value) {
   }
 
   public static String makeKey(String key) {
-    String[] forbiddenChars = {".", "-", " "};
+    String[] forbiddenChars = {".", "-", " ", ":"};
     for (String forbidden : forbiddenChars) {
       if (key.contains(forbidden)) {
         key = key.trim().replace(forbidden, "_");

src/test/java/com/segment/analytics/android/integration/firebase/FirebaseTest.java

@@ -220,6 +220,12 @@ public void makeKeyWithDashAndDot() {
         verify(firebase).logEvent(eq("test_event_dashed_and_dotted"), bundleEq(new Bundle()));
     }
 
+    @Test
+    public void makeKeyWithColon() {
+        integration.track(new TrackPayload.Builder().anonymousId("12345").event("test:colon").build());
+        verify(firebase).logEvent(eq("test_colon"), bundleEq(new Bundle()));
+    }
+
     /**
      * Uses the string representation of the object. Useful for JSON objects.
      * @param expected Expected object