fix unit tests

OneFiveFour · Dec 23, 2024 · bcf6e58 · bcf6e58
1 parent 4ccfd4d
commit bcf6e58
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 10 deletions.
diff --git a/core/timer/src/main/kotlin/net/onefivefour/sessiontimer/core/timer/SessionTimerImpl.kt b/core/timer/src/main/kotlin/net/onefivefour/sessiontimer/core/timer/SessionTimerImpl.kt
@@ -40,12 +40,15 @@ internal class SessionTimerImpl @Inject constructor(
         if (timerJob?.isActive == true) return
 
         _state.update { it.copy(mode = TimerMode.RUNNING) }
-        timerJob = timerScope.launch {
-            while (isActive) {
-                delay(frameRate)
-                updateProgress(frameRate)
+            .also {
+                timerJob = timerScope.launch {
+                    while (isActive) {
+                        delay(frameRate)
+                        updateProgress(frameRate)
+                    }
+                }
             }
-        }
+
     }
 
     override fun pause() {
@@ -94,11 +97,7 @@ internal class SessionTimerImpl @Inject constructor(
 
     private fun updateTimerStatus() = updateProgress(Duration.ZERO)
 
-    override fun getStatus() = _state.stateIn(
-        scope = timerScope,
-        started = SharingStarted.Eagerly,
-        initialValue = TimerState()
-    )
+    override fun getStatus() = _state
 
     private fun cancelTimer() {
         timerJob?.cancel()

diff --git a/core/timer/src/test/kotlin/net/onefivefour/sessiontimer/core/timer/SessionTimerImplTest.kt b/core/timer/src/test/kotlin/net/onefivefour/sessiontimer/core/timer/SessionTimerImplTest.kt
@@ -7,6 +7,7 @@ import kotlinx.coroutines.ExperimentalCoroutinesApi
 import kotlinx.coroutines.delay
 import kotlinx.coroutines.flow.first
 import kotlinx.coroutines.test.advanceUntilIdle
+import kotlinx.coroutines.test.runCurrent
 import kotlinx.coroutines.test.runTest
 import net.onefivefour.sessiontimer.core.test.StandardTestDispatcherRule
 import net.onefivefour.sessiontimer.core.timer.api.model.TimerMode