added syncAfterSetUser feature setting (#445)

* added syncAfterSetUser feature setting

* fixed unit tests

* reverted whitespace changes

* simplified trackOnce calls

* bumped version number

* bumped version number

Author: cameron-morrow-toptal

sdk/build.gradle

@@ -10,7 +10,7 @@ apply plugin: "org.jetbrains.dokka"
 apply plugin: 'io.radar.mvnpublish'
 
 ext {
-    radarVersion = '3.21.4'
+    radarVersion = '3.21.6'
 }
 
 String buildNumber = ".${System.currentTimeMillis()}"

sdk/src/main/java/io/radar/sdk/Radar.kt

@@ -624,6 +624,10 @@ object Radar {
         }
 
         RadarSettings.setUserId(context, userId)
+
+        if (RadarSettings.getSdkConfiguration(context).syncAfterSetUser) {
+            trackOnce()
+        }
     }
 
     /**
@@ -688,6 +692,10 @@ object Radar {
         }
 
         RadarSettings.setMetadata(context, metadata)
+
+        if (RadarSettings.getSdkConfiguration(context).syncAfterSetUser) {
+            trackOnce()
+        }
     }
 
     /**

sdk/src/main/java/io/radar/sdk/model/RadarSdkConfiguration.kt

@@ -23,6 +23,7 @@ internal data class RadarSdkConfiguration(
     val useOpenedAppConversion: Boolean = false,
     val useForegroundLocationUpdatedAtMsDiff: Boolean = false,
     val locationManagerTimeout: Int = 0,
+    val syncAfterSetUser: Boolean = false,
 ) {
     companion object {
         private const val MAX_CONCURRENT_JOBS = "maxConcurrentJobs"
@@ -39,6 +40,7 @@ internal data class RadarSdkConfiguration(
         private const val USE_OPENED_APP_CONVERSION = "useOpenedAppConversion"
         private const val USE_FOREGROUND_LOCATION_UPDATED_AT_MS_DIFF = "useForegroundLocationUpdatedAtMsDiff"
         private const val LOCATION_MANAGER_TIMEOUT = "locationManagerTimeout"
+        private const val SYNC_AFTER_SET_USER = "syncAfterSetUser"
 
         fun fromJson(json: JSONObject?): RadarSdkConfiguration {
             // set json as empty object if json is null, which uses fallback values
@@ -58,6 +60,7 @@ internal data class RadarSdkConfiguration(
                 config.optBoolean(USE_OPENED_APP_CONVERSION, true),
                 config.optBoolean(USE_FOREGROUND_LOCATION_UPDATED_AT_MS_DIFF, false),
                 config.optInt(LOCATION_MANAGER_TIMEOUT, 0),
+                config.optBoolean(SYNC_AFTER_SET_USER, false),
             )
         }
 
@@ -89,6 +92,7 @@ internal data class RadarSdkConfiguration(
             putOpt(USE_OPENED_APP_CONVERSION, useOpenedAppConversion)
             putOpt(USE_FOREGROUND_LOCATION_UPDATED_AT_MS_DIFF, useForegroundLocationUpdatedAtMsDiff)
             putOpt(LOCATION_MANAGER_TIMEOUT, locationManagerTimeout)
+            putOpt(SYNC_AFTER_SET_USER, syncAfterSetUser)
         }
     }
 }

sdk/src/test/java/io/radar/sdk/model/RadarSdkConfigurationTest.kt

@@ -31,6 +31,7 @@ class RadarSdkConfigurationTest {
     private var useOpenedAppConversion = false
     private var useForegroundLocationUpdatedAtMsDiff = false
     private var locationManagerTimeout = 123456
+    private var syncAfterSetUser = false
 
     @Before
     fun setUp() {
@@ -50,7 +51,8 @@ class RadarSdkConfigurationTest {
             "useLocationMetadata":$useLocationMetadata,
             "useOpenedAppConversion":$useOpenedAppConversion,
             "useForegroundLocationUpdatedAtMsDiff":$useForegroundLocationUpdatedAtMsDiff,
-            "locationManagerTimeout":$locationManagerTimeout
+            "locationManagerTimeout":$locationManagerTimeout,
+            "syncAfterSetUser":$syncAfterSetUser,
         }""".trimIndent()
     }
 
@@ -71,7 +73,8 @@ class RadarSdkConfigurationTest {
                 useLocationMetadata,
                 useOpenedAppConversion,
                 useForegroundLocationUpdatedAtMsDiff,
-                locationManagerTimeout
+                locationManagerTimeout,
+                syncAfterSetUser,
             ).toJson().toString()
         )
     }
@@ -92,6 +95,7 @@ class RadarSdkConfigurationTest {
         assertEquals(useOpenedAppConversion, settings.useOpenedAppConversion)
         assertEquals(useForegroundLocationUpdatedAtMsDiff, settings.useForegroundLocationUpdatedAtMsDiff)
         assertEquals(locationManagerTimeout, settings.locationManagerTimeout)
+        assertEquals(syncAfterSetUser, settings.syncAfterSetUser)
     }
 
     @Test