Refactor: Move codeable get free estimate url remote config to wrapper

This is done to make sure that everything 'FirebaseRemoteConfig' related
is only being accessed via the 'FirebaseRemoteConfigWrapper', thus
aligning this with any other such call across the app, which in turn
will make it easier to reason about all Firebase 'Remote Config' related
functionality configured for this project.

Author: ParaskP7

WordPress/src/main/java/org/wordpress/android/ui/jetpack/scan/details/ThreatDetailsNavigationEvents.kt

@@ -1,11 +1,9 @@
 package org.wordpress.android.ui.jetpack.scan.details
 
 import androidx.annotation.StringRes
-import com.google.firebase.remoteconfig.FirebaseRemoteConfig
 import org.wordpress.android.R
 import org.wordpress.android.ui.utils.UiString
-
-private const val CODEABLE_GET_FREE_ESTIMATE_FIREBASE_KEY = "codeable_get_free_estimate_url"
+import org.wordpress.android.util.FirebaseRemoteConfigWrapper
 
 sealed class ThreatDetailsNavigationEvents {
     class OpenThreatActionDialog(
@@ -24,8 +22,10 @@ sealed class ThreatDetailsNavigationEvents {
 
     data class ShowUpdatedFixState(val threatId: Long) : ThreatDetailsNavigationEvents()
 
-    object ShowGetFreeEstimate : ThreatDetailsNavigationEvents() {
-        fun url() = FirebaseRemoteConfig.getInstance().getString(CODEABLE_GET_FREE_ESTIMATE_FIREBASE_KEY)
+    data class ShowGetFreeEstimate(
+        val firebaseRemoteConfigWrapper: FirebaseRemoteConfigWrapper
+    ) : ThreatDetailsNavigationEvents() {
+        fun url() = firebaseRemoteConfigWrapper.getCodeableGetFreeEstimateUrl()
     }
 
     data class ShowJetpackSettings(val url: String) : ThreatDetailsNavigationEvents()

WordPress/src/main/java/org/wordpress/android/ui/jetpack/scan/details/ThreatDetailsViewModel.kt

@@ -29,6 +29,7 @@ import org.wordpress.android.ui.utils.HtmlMessageUtils
 import org.wordpress.android.ui.utils.UiString
 import org.wordpress.android.ui.utils.UiString.UiStringRes
 import org.wordpress.android.ui.utils.UiString.UiStringText
+import org.wordpress.android.util.FirebaseRemoteConfigWrapper
 import org.wordpress.android.util.analytics.ScanTracker
 import org.wordpress.android.viewmodel.Event
 import org.wordpress.android.viewmodel.ResourceProvider
@@ -43,7 +44,8 @@ class ThreatDetailsViewModel @Inject constructor(
     private val builder: ThreatDetailsListItemsBuilder,
     private val htmlMessageUtils: HtmlMessageUtils,
     private val resourceProvider: ResourceProvider,
-    private val scanTracker: ScanTracker
+    private val scanTracker: ScanTracker,
+    private val firebaseRemoteConfigWrapper: FirebaseRemoteConfigWrapper
 ) : ViewModel() {
     private lateinit var site: SiteModel
     private lateinit var threatModel: ThreatModel
@@ -153,7 +155,7 @@ class ThreatDetailsViewModel @Inject constructor(
 
     private fun onGetFreeEstimateButtonClicked() {
         scanTracker.trackOnGetFreeEstimateButtonClicked()
-        updateNavigationEvent(ThreatDetailsNavigationEvents.ShowGetFreeEstimate)
+        updateNavigationEvent(ThreatDetailsNavigationEvents.ShowGetFreeEstimate(firebaseRemoteConfigWrapper))
     }
 
     private fun onEnterServerCredsIconClicked() {

WordPress/src/main/java/org/wordpress/android/util/FirebaseRemoteConfigWrapper.kt

@@ -8,12 +8,16 @@ import javax.inject.Singleton
 class FirebaseRemoteConfigWrapper @Inject constructor() {
     fun getOpenWebLinksWithJetpackFlowFrequency() =
         FirebaseRemoteConfig.getInstance().getLong(OPEN_WEB_LINKS_WITH_JETPACK_FLOW_FREQUENCY_KEY)
+    fun getCodeableGetFreeEstimateUrl() =
+        FirebaseRemoteConfig.getInstance().getString(CODEABLE_GET_FREE_ESTIMATE_FIREBASE_KEY)
     fun getPerformanceMonitoringSampleRate(): Double =
         FirebaseRemoteConfig.getInstance().getDouble(PERFORMANCE_MONITORING_SAMPLE_RATE_KEY)
 
     companion object {
         private const val PERFORMANCE_MONITORING_SAMPLE_RATE_KEY = "wp_android_performance_monitoring_sample_rate"
 
+        private const val CODEABLE_GET_FREE_ESTIMATE_FIREBASE_KEY = "codeable_get_free_estimate_url"
+
         private const val OPEN_WEB_LINKS_WITH_JETPACK_FLOW_FREQUENCY_KEY = "open_web_links_with_jetpack_flow_frequency"
     }
 }

WordPress/src/test/java/org/wordpress/android/ui/jetpack/scan/details/ThreatDetailsViewModelTest.kt

@@ -37,6 +37,7 @@ import org.wordpress.android.ui.pages.SnackbarMessageHolder
 import org.wordpress.android.ui.utils.HtmlMessageUtils
 import org.wordpress.android.ui.utils.UiString.UiStringRes
 import org.wordpress.android.ui.utils.UiString.UiStringText
+import org.wordpress.android.util.FirebaseRemoteConfigWrapper
 import org.wordpress.android.util.analytics.ScanTracker
 import org.wordpress.android.viewmodel.Event
 import org.wordpress.android.viewmodel.ResourceProvider
@@ -78,6 +79,9 @@ class ThreatDetailsViewModelTest : BaseUnitTest() {
     @Mock
     private lateinit var scanTracker: ScanTracker
 
+    @Mock
+    private lateinit var firebaseRemoteConfigWrapper: FirebaseRemoteConfigWrapper
+
     @Mock
     private lateinit var scanStore: ScanStore
 
@@ -99,7 +103,8 @@ class ThreatDetailsViewModelTest : BaseUnitTest() {
             builder,
             htmlMessageUtils,
             resourceProvider,
-            scanTracker
+            scanTracker,
+            firebaseRemoteConfigWrapper
         )
         whenever(site.name).thenReturn(TEST_SITE_NAME)
         whenever(selectedSiteRepository.getSelectedSite()).thenReturn(site)