Add featutre flag for nav state

Author: bidetofevil

embrace-android-config-fakes/src/main/kotlin/io/embrace/android/embracesdk/fakes/behavior/FakeAutoDataCaptureBehavior.kt

@@ -17,6 +17,7 @@ class FakeAutoDataCaptureBehavior(
     private val endStartupWithAppReady: Boolean = false,
     private val enableStateCapture: Boolean = false,
     private val networkCallbackConnectivityServiceEnabled: Boolean = false,
+    private val navigationStateCaptureEnabled: Boolean = true,
 ) : AutoDataCaptureBehavior {
 
     override fun isThermalStatusCaptureEnabled(): Boolean = thermalStatusCaptureEnabled
@@ -33,4 +34,5 @@ class FakeAutoDataCaptureBehavior(
     override fun isEndStartupWithAppReadyEnabled(): Boolean = endStartupWithAppReady
     override fun isStateCaptureEnabled(): Boolean = enableStateCapture
     override fun isNetworkCallbackConnectivityServiceEnabled(): Boolean = networkCallbackConnectivityServiceEnabled
+    override fun isNavigationStateCaptureEnabled(): Boolean = navigationStateCaptureEnabled
 }

embrace-android-config/src/main/kotlin/io/embrace/android/embracesdk/internal/config/behavior/AutoDataCaptureBehavior.kt

@@ -74,4 +74,9 @@ interface AutoDataCaptureBehavior {
      * Whether the NetworkCallback-based connectivity service implementation is enabled
      */
     fun isNetworkCallbackConnectivityServiceEnabled(): Boolean
+
+    /**
+     * Whether navigation state capture is enabled
+     */
+    fun isNavigationStateCaptureEnabled(): Boolean
 }

embrace-android-config/src/main/kotlin/io/embrace/android/embracesdk/internal/config/behavior/AutoDataCaptureBehaviorImpl.kt

@@ -51,4 +51,7 @@ class AutoDataCaptureBehaviorImpl(
 
     override fun isNetworkCallbackConnectivityServiceEnabled(): Boolean =
         thresholdCheck.isBehaviorEnabled(remote?.pctNetworkCallbackConnectivityServiceEnabled) ?: false
+
+    override fun isNavigationStateCaptureEnabled(): Boolean =
+        thresholdCheck.isBehaviorEnabled(remote?.pctNavigationStateCaptureEnabled) ?: true
 }

embrace-android-config/src/test/kotlin/io/embrace/android/embracesdk/internal/config/behavior/AutoDataCaptureBehaviorImplTest.kt

@@ -40,6 +40,7 @@ internal class AutoDataCaptureBehaviorImplTest {
             assertFalse(isEndStartupWithAppReadyEnabled())
             assertFalse(isStateCaptureEnabled())
             assertFalse(isNetworkCallbackConnectivityServiceEnabled())
+            assertTrue(isNavigationStateCaptureEnabled())
         }
     }
 
@@ -187,9 +188,34 @@ internal class AutoDataCaptureBehaviorImplTest {
         )
     }
 
+    @Test
+    fun `navigation state capture enabled by default when remote pct is null`() {
+        assertTrue(
+            createBehavior(remote = RemoteConfig()).isNavigationStateCaptureEnabled()
+        )
+    }
+
+    @Test
+    fun `navigation state capture enabled when pct is 100`() {
+        assertTrue(
+            createBehavior(
+                remote = RemoteConfig(pctNavigationStateCaptureEnabled = 100.0f)
+            ).isNavigationStateCaptureEnabled()
+        )
+    }
+
+    @Test
+    fun `navigation state capture disabled when pct is 0`() {
+        assertFalse(
+            createBehavior(
+                remote = RemoteConfig(pctNavigationStateCaptureEnabled = 0.0f)
+            ).isNavigationStateCaptureEnabled()
+        )
+    }
+
     private fun createBehavior(
-        localUiLoadTracingEnabled: Boolean,
-        localUiLoadTracingTraceAllEnabled: Boolean,
+        localUiLoadTracingEnabled: Boolean = true,
+        localUiLoadTracingTraceAllEnabled: Boolean = true,
         stateCaptureEnabled: Boolean = false,
         remote: RemoteConfig,
     ) = AutoDataCaptureBehaviorImpl(
@@ -198,7 +224,7 @@ internal class AutoDataCaptureBehaviorImplTest {
             enabledFeatures = FakeEnabledFeatureConfig(
                 uiLoadTracingTraceAll = localUiLoadTracingTraceAllEnabled,
                 uiLoadTracingEnabled = localUiLoadTracingEnabled,
-                stateCaptureEnabled = stateCaptureEnabled
+                stateCaptureEnabled = stateCaptureEnabled,
             )
         ),
         remote = remote

embrace-android-payload/src/main/kotlin/io/embrace/android/embracesdk/internal/config/instrumented/schema/EnabledFeatureConfig.kt

@@ -173,4 +173,5 @@ interface EnabledFeatureConfig {
      * Gates whether the state feature is enabled. This is never set to true in production.
      */
     fun isStateCaptureEnabled(): Boolean = false
+
 }

embrace-android-payload/src/main/kotlin/io/embrace/android/embracesdk/internal/config/remote/RemoteConfig.kt

@@ -103,4 +103,7 @@ data class RemoteConfig(
      */
     @Json(name = "pct_network_callback_connectivity_service_enabled")
     val pctNetworkCallbackConnectivityServiceEnabled: Float? = null,
+
+    @Json(name = "pct_navigation_state_capture_enabled")
+    val pctNavigationStateCaptureEnabled: Float? = null,
 )