From a19fcd9e09e7221445028557db9cf9f4d1eae68a Mon Sep 17 00:00:00 2001
From: Simon Marquis <contact@simon-marquis.fr>
Date: Sun, 14 Sep 2025 14:27:12 +0200
Subject: [PATCH] Move internal modifier to the object to work around build
 errors

`error: cannot find symbol` on:

- `DataStoreModule.INSTANCE.providesUserPreferencesDataStore$datastore_demoDebug`
- `SyncModule.Companion.provideFirebaseMessaging$work_prodDebug()`
---
 .../apps/nowinandroid/core/datastore/di/DataStoreModule.kt  | 4 +++-
 .../google/samples/apps/nowinandroid/sync/di/SyncModule.kt  | 2 ++
 .../google/samples/apps/nowinandroid/sync/di/SyncModule.kt  | 6 ++++--
 3 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/core/datastore/src/main/kotlin/com/google/samples/apps/nowinandroid/core/datastore/di/DataStoreModule.kt b/core/datastore/src/main/kotlin/com/google/samples/apps/nowinandroid/core/datastore/di/DataStoreModule.kt
index 8e0d7d4d8..6c4486880 100644
--- a/core/datastore/src/main/kotlin/com/google/samples/apps/nowinandroid/core/datastore/di/DataStoreModule.kt
+++ b/core/datastore/src/main/kotlin/com/google/samples/apps/nowinandroid/core/datastore/di/DataStoreModule.kt
@@ -33,15 +33,17 @@ import dagger.hilt.android.qualifiers.ApplicationContext
 import dagger.hilt.components.SingletonComponent
 import kotlinx.coroutines.CoroutineDispatcher
 import kotlinx.coroutines.CoroutineScope
+import org.jetbrains.annotations.VisibleForTesting
 import javax.inject.Singleton
 
+@VisibleForTesting
 @Module
 @InstallIn(SingletonComponent::class)
 object DataStoreModule {
 
     @Provides
     @Singleton
-    internal fun providesUserPreferencesDataStore(
+    fun providesUserPreferencesDataStore(
         @ApplicationContext context: Context,
         @Dispatcher(IO) ioDispatcher: CoroutineDispatcher,
         @ApplicationScope scope: CoroutineScope,
diff --git a/sync/work/src/demo/kotlin/com/google/samples/apps/nowinandroid/sync/di/SyncModule.kt b/sync/work/src/demo/kotlin/com/google/samples/apps/nowinandroid/sync/di/SyncModule.kt
index 91ef476f6..054cd913d 100644
--- a/sync/work/src/demo/kotlin/com/google/samples/apps/nowinandroid/sync/di/SyncModule.kt
+++ b/sync/work/src/demo/kotlin/com/google/samples/apps/nowinandroid/sync/di/SyncModule.kt
@@ -24,7 +24,9 @@ import dagger.Binds
 import dagger.Module
 import dagger.hilt.InstallIn
 import dagger.hilt.components.SingletonComponent
+import org.jetbrains.annotations.VisibleForTesting
 
+@VisibleForTesting
 @Module
 @InstallIn(SingletonComponent::class)
 abstract class SyncModule {
diff --git a/sync/work/src/prod/kotlin/com/google/samples/apps/nowinandroid/sync/di/SyncModule.kt b/sync/work/src/prod/kotlin/com/google/samples/apps/nowinandroid/sync/di/SyncModule.kt
index ceeb39548..e59147813 100644
--- a/sync/work/src/prod/kotlin/com/google/samples/apps/nowinandroid/sync/di/SyncModule.kt
+++ b/sync/work/src/prod/kotlin/com/google/samples/apps/nowinandroid/sync/di/SyncModule.kt
@@ -28,8 +28,10 @@ import dagger.Module
 import dagger.Provides
 import dagger.hilt.InstallIn
 import dagger.hilt.components.SingletonComponent
+import org.jetbrains.annotations.VisibleForTesting
 import javax.inject.Singleton
 
+@VisibleForTesting
 @Module
 @InstallIn(SingletonComponent::class)
 abstract class SyncModule {
@@ -43,9 +45,9 @@ abstract class SyncModule {
         syncSubscriber: FirebaseSyncSubscriber,
     ): SyncSubscriber
 
-    companion object {
+    internal companion object {
         @Provides
         @Singleton
-        internal fun provideFirebaseMessaging(): FirebaseMessaging = Firebase.messaging
+        fun provideFirebaseMessaging(): FirebaseMessaging = Firebase.messaging
     }
 }