refactor(navigator): log when sending on the MoveSummaryChannel

Author: w2sv

modules/navigator/src/main/kotlin/com/w2sv/navigator/MoveResultCollector.kt

@@ -1,6 +1,6 @@
 package com.w2sv.navigator
 
-import com.w2sv.navigator.di.MoveSummaryChannel
+import com.w2sv.navigator.postmove.MoveSummaryChannel
 import com.w2sv.navigator.postmove.MoveSummaryListener
 import javax.inject.Inject
 import kotlinx.coroutines.flow.consumeAsFlow

modules/navigator/src/main/kotlin/com/w2sv/navigator/di/FileNavigatorModule.kt

@@ -3,7 +3,7 @@ package com.w2sv.navigator.di
 import android.content.ContentResolver
 import android.content.Context
 import com.w2sv.navigator.FileNavigator
-import com.w2sv.navigator.postmove.MoveOperationSummary
+import com.w2sv.navigator.postmove.MoveSummaryChannel
 import dagger.Module
 import dagger.Provides
 import dagger.hilt.InstallIn
@@ -13,8 +13,6 @@ import javax.inject.Singleton
 import kotlinx.coroutines.channels.Channel
 import kotlinx.coroutines.flow.StateFlow
 
-internal typealias MoveSummaryChannel = Channel<MoveOperationSummary>
-
 @InstallIn(SingletonComponent::class)
 @Module
 internal object FileNavigatorModule {

modules/navigator/src/main/kotlin/com/w2sv/navigator/moving/BatchMoveService.kt

@@ -6,7 +6,7 @@ import android.os.Parcelable
 import com.w2sv.androidutils.content.getParcelableCompat
 import com.w2sv.androidutils.content.intent
 import com.w2sv.common.logging.LoggingUnboundService
-import com.w2sv.navigator.di.MoveSummaryChannel
+import com.w2sv.navigator.postmove.MoveSummaryChannel
 import com.w2sv.navigator.domain.moving.MoveDestination
 import com.w2sv.navigator.domain.moving.MoveOperation
 import com.w2sv.navigator.postmove.MoveOperationSummary

modules/navigator/src/main/kotlin/com/w2sv/navigator/moving/FileMover.kt

@@ -2,12 +2,13 @@ package com.w2sv.navigator.moving
 
 import android.content.Context
 import com.w2sv.common.di.ApplicationIoScope
-import com.w2sv.navigator.di.MoveSummaryChannel
 import com.w2sv.navigator.domain.moving.MoveOperation
 import com.w2sv.navigator.postmove.MoveOperationSummary
-import javax.inject.Inject
+import com.w2sv.navigator.postmove.MoveSummaryChannel
+import com.w2sv.navigator.postmove.loggingTrySend
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.launch
+import javax.inject.Inject
 
 internal class FileMover @Inject constructor(
     private val moveSummaryChannel: MoveSummaryChannel,
@@ -16,7 +17,7 @@ internal class FileMover @Inject constructor(
     operator fun invoke(operation: MoveOperation, context: Context) {
         scope.launch {
             operation.file.moveTo(destination = operation.destination, context = context) { result ->
-                moveSummaryChannel.trySend(MoveOperationSummary(result, operation))
+                moveSummaryChannel.loggingTrySend(MoveOperationSummary(result, operation))
             }
         }
     }

modules/navigator/src/main/kotlin/com/w2sv/navigator/moving/activity/MoveActivityFinisher.kt

@@ -1,15 +1,16 @@
 package com.w2sv.navigator.moving.activity
 
 import android.app.Activity
-import com.w2sv.navigator.di.MoveSummaryChannel
-import com.w2sv.navigator.postmove.MoveOperationSummary
 import com.w2sv.navigator.domain.moving.MoveResult
+import com.w2sv.navigator.postmove.MoveOperationSummary
+import com.w2sv.navigator.postmove.MoveSummaryChannel
+import com.w2sv.navigator.postmove.loggingTrySend
 import javax.inject.Inject
 
 class MoveActivityFinisher @Inject constructor(private val moveSummaryChannel: MoveSummaryChannel) {
 
     fun finishOnError(activity: Activity, error: MoveResult) {
-        moveSummaryChannel.trySend(MoveOperationSummary(error))
+        moveSummaryChannel.loggingTrySend(MoveOperationSummary(error))
         activity.finishAndRemoveTask()
     }
 }

modules/navigator/src/main/kotlin/com/w2sv/navigator/postmove/MoveSummaryChannel.kt

@@ -0,0 +1,15 @@
+package com.w2sv.navigator.postmove
+
+import kotlinx.coroutines.channels.Channel
+import kotlinx.coroutines.channels.onFailure
+import slimber.log.e
+import slimber.log.i
+
+internal typealias MoveSummaryChannel = Channel<MoveOperationSummary>
+
+fun MoveSummaryChannel.loggingTrySend(operationSummary: MoveOperationSummary) {
+    i { "Try sending $operationSummary via MoveSummaryChannel" }
+    trySend(operationSummary).onFailure { error ->
+        e { "Failed to send summary: $error" }
+    }
+}