Refactor :core:network DI

- Delete unused `NetworkModule` from benchmark sourceSet (contains unknowns references)
- Extract common bindings into a single `NetworkModule`
- Expose "flavored" bindings through `FlavoredNetworkModule`s

Author: SimonMarquis

…android/core/network/di/NetworkModule.kt …core/network/di/FlavoredNetworkModule.ktcore/network/src/demo/java/com/google/samples/apps/nowinandroid/core/network/di/NetworkModule.kt renamed to core/network/src/demo/java/com/google/samples/apps/nowinandroid/core/network/di/FlavoredNetworkModule.kt core/network/src/demo/java/com/google/samples/apps/nowinandroid/core/network/di/NetworkModule.kt renamed to core/network/src/demo/java/com/google/samples/apps/nowinandroid/core/network/di/FlavoredNetworkModule.kt

@@ -16,39 +16,17 @@
 
 package com.google.samples.apps.nowinandroid.core.network.di
 
-import android.content.Context
 import com.google.samples.apps.nowinandroid.core.network.NiaNetworkDataSource
-import com.google.samples.apps.nowinandroid.core.network.fake.FakeAssetManager
 import com.google.samples.apps.nowinandroid.core.network.fake.FakeNiaNetworkDataSource
 import dagger.Binds
 import dagger.Module
-import dagger.Provides
 import dagger.hilt.InstallIn
-import dagger.hilt.android.qualifiers.ApplicationContext
 import dagger.hilt.components.SingletonComponent
-import javax.inject.Singleton
-import kotlinx.serialization.json.Json
 
 @Module
 @InstallIn(SingletonComponent::class)
-interface NetworkModule {
+interface FlavoredNetworkModule {
 
     @Binds
-    fun bindsNiaNetwork(
-        niANetwork: FakeNiaNetworkDataSource
-    ): NiaNetworkDataSource
-
-    companion object {
-        @Provides
-        @Singleton
-        fun providesNetworkJson(): Json = Json {
-            ignoreUnknownKeys = true
-        }
-
-        @Provides
-        @Singleton
-        fun providesFakeAssetManager(
-            @ApplicationContext context: Context,
-        ): FakeAssetManager = FakeAssetManager(context.assets::open)
-    }
+    fun FakeNiaNetworkDataSource.binds(): NiaNetworkDataSource
 }

…android/core/network/di/NetworkModule.kt …android/core/network/di/NetworkModule.ktcore/network/src/benchmark/java/com/google/samples/apps/nowinandroid/core/network/di/NetworkModule.kt renamed to core/network/src/main/java/com/google/samples/apps/nowinandroid/core/network/di/NetworkModule.kt core/network/src/benchmark/java/com/google/samples/apps/nowinandroid/core/network/di/NetworkModule.kt renamed to core/network/src/main/java/com/google/samples/apps/nowinandroid/core/network/di/NetworkModule.kt

@@ -17,38 +17,28 @@
 package com.google.samples.apps.nowinandroid.core.network.di
 
 import android.content.Context
-import com.google.samples.apps.nowinandroid.core.network.NiANetwork
 import com.google.samples.apps.nowinandroid.core.network.fake.FakeAssetManager
-import com.google.samples.apps.nowinandroid.core.network.fake.FakeNiANetwork
-import dagger.Binds
 import dagger.Module
 import dagger.Provides
-import dagger.hilt.android.qualifiers.ApplicationContext
 import dagger.hilt.InstallIn
+import dagger.hilt.android.qualifiers.ApplicationContext
 import dagger.hilt.components.SingletonComponent
 import javax.inject.Singleton
 import kotlinx.serialization.json.Json
 
 @Module
 @InstallIn(SingletonComponent::class)
-interface NetworkModule {
-
-    @Binds
-    fun bindsNiANetwork(
-        niANetwork: FakeNiANetwork
-    ): NiANetwork
+object NetworkModule {
 
-    companion object {
-        @Provides
-        @Singleton
-        fun providesNetworkJson(): Json = Json {
-            ignoreUnknownKeys = true
-        }
-
-        @Provides
-        @Singleton
-        fun providesFakeAssetManager(
-            @ApplicationContext context: Context
-        ): FakeAssetManager = FakeAssetManager(context.assets::open)
+    @Provides
+    @Singleton
+    fun providesNetworkJson(): Json = Json {
+        ignoreUnknownKeys = true
     }
+
+    @Provides
+    @Singleton
+    fun providesFakeAssetManager(
+        @ApplicationContext context: Context,
+    ): FakeAssetManager = FakeAssetManager(context.assets::open)
 }

…android/core/network/di/NetworkModule.kt …core/network/di/FlavoredNetworkModule.ktcore/network/src/prod/java/com/google/samples/apps/nowinandroid/core/network/di/NetworkModule.kt renamed to core/network/src/prod/java/com/google/samples/apps/nowinandroid/core/network/di/FlavoredNetworkModule.kt core/network/src/prod/java/com/google/samples/apps/nowinandroid/core/network/di/NetworkModule.kt renamed to core/network/src/prod/java/com/google/samples/apps/nowinandroid/core/network/di/FlavoredNetworkModule.kt

@@ -16,39 +16,17 @@
 
 package com.google.samples.apps.nowinandroid.core.network.di
 
-import android.content.Context
-import com.google.samples.apps.nowinandroid.core.network.fake.FakeAssetManager
 import com.google.samples.apps.nowinandroid.core.network.NiaNetworkDataSource
 import com.google.samples.apps.nowinandroid.core.network.retrofit.RetrofitNiaNetwork
 import dagger.Binds
 import dagger.Module
-import dagger.Provides
-import dagger.hilt.android.qualifiers.ApplicationContext
 import dagger.hilt.InstallIn
 import dagger.hilt.components.SingletonComponent
-import javax.inject.Singleton
-import kotlinx.serialization.json.Json
 
 @Module
 @InstallIn(SingletonComponent::class)
-interface NetworkModule {
+interface FlavoredNetworkModule {
 
     @Binds
-    fun bindsNiaNetwork(
-        niANetwork: RetrofitNiaNetwork
-    ): NiaNetworkDataSource
-
-    companion object {
-        @Provides
-        @Singleton
-        fun providesNetworkJson(): Json = Json {
-            ignoreUnknownKeys = true
-        }
-
-        @Provides
-        @Singleton
-        fun providesFakeAssetManager(
-            @ApplicationContext context: Context,
-        ): FakeAssetManager = FakeAssetManager(context.assets::open)
-    }
+    fun RetrofitNiaNetwork.binds(): NiaNetworkDataSource
 }