Sync featured tools alongside tool order in ToolsPresenter

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: frett

app/src/main/kotlin/org/cru/godtools/ui/dashboard/tools/ToolsPresenter.kt

@@ -22,7 +22,9 @@ import dagger.assisted.AssistedFactory
 import dagger.assisted.AssistedInject
 import dagger.hilt.components.SingletonComponent
 import java.util.Locale
+import kotlinx.coroutines.coroutineScope
 import kotlinx.coroutines.flow.first
+import kotlinx.coroutines.launch
 import org.ccci.gto.android.common.sync.SyncTracker
 import org.cru.godtools.analytics.model.OpenAnalyticsActionEvent
 import org.cru.godtools.analytics.model.OpenAnalyticsActionEvent.Companion.ACTION_OPEN_TOOL_DETAILS
@@ -197,7 +199,10 @@ class ToolsPresenter @AssistedInject internal constructor(
 
     private fun SyncTracker.syncData(locale: Locale, force: Boolean = false) = launchSync {
         val country = settings.getCountrySettingFlow().first()
-        syncService.syncToolOrder(locale, country, force)
+        coroutineScope {
+            launch { syncService.syncFeaturedTools(locale, country, force) }
+            launch { syncService.syncToolOrder(locale, country, force) }
+        }
     }
 
     @AssistedFactory

app/src/testDebug/kotlin/org/cru/godtools/ui/dashboard/tools/ToolsPresenterTest.kt

@@ -73,6 +73,7 @@ class ToolsPresenterTest {
     }
     private val syncService: GodToolsSyncService = mockk {
         coEvery { syncToolOrder(any(), any(), any()) } coAnswers { toolOrderSync.receive() }
+        coEvery { syncFeaturedTools(any(), any(), any()) } returns true
     }
     private val favoriteToolsBannerPresenter = FakeBannerPresenter<FavoriteToolsBannerPresenter.UiState>(null)
     private val featuredToolsFlowProducer: FeaturedToolsFlowProducer = mockk {
@@ -265,7 +266,10 @@ class ToolsPresenterTest {
         presenter.test {
             awaitInitialItem()
             toolOrderSync.send(true)
-            coVerifyAll { syncService.syncToolOrder(Locale.ENGLISH, "US", false) }
+            coVerifyAll {
+                syncService.syncToolOrder(Locale.ENGLISH, "US", false)
+                syncService.syncFeaturedTools(Locale.ENGLISH, "US", false)
+            }
         }
     }
 
@@ -278,7 +282,10 @@ class ToolsPresenterTest {
         presenter.test {
             awaitInitialItem()
             toolOrderSync.send(true)
-            coVerifyAll { syncService.syncToolOrder(Locale.FRENCH, "US", false) }
+            coVerifyAll {
+                syncService.syncToolOrder(Locale.FRENCH, "US", false)
+                syncService.syncFeaturedTools(Locale.FRENCH, "US", false)
+            }
         }
     }