Merge pull request #120 from kitakkun/feature/fix_android_test_fails

Fix Android Tests
kitakkun · Jul 16, 2024 · 895cbd3 · 895cbd3
2 parents 4d489e5 + fd72ea4
commit 895cbd3
Show file tree

Hide file tree

Showing 9 changed files with 54 additions and 49 deletions.
diff --git a/test/build.gradle.kts b/test/build.gradle.kts
@@ -43,6 +43,11 @@ kotlin {
                 implementation(libs.androidx.compose.runtime)
             }
         }
+        val androidUnitTest by getting {
+            dependencies {
+                implementation(libs.robolectric)
+            }
+        }
     }
 }
 

diff --git a/...roidDebugUnitTest/kotlin/io/github/kitakkun/backintime/test/BackInTimeDebugServiceTest.kt b/...roidDebugUnitTest/kotlin/io/github/kitakkun/backintime/test/BackInTimeDebugServiceTest.kt
diff --git a/...c/androidUnitTest/kotlin/io/github/kitakkun/backintime/test/BackInTimeDebugServiceTest.kt b/...c/androidUnitTest/kotlin/io/github/kitakkun/backintime/test/BackInTimeDebugServiceTest.kt
@@ -0,0 +1,38 @@
+package io.github.kitakkun.backintime.test
+
+import io.github.kitakkun.backintime.runtime.BackInTimeDebugService
+import io.github.kitakkun.backintime.runtime.getBackInTimeDebugService
+import io.github.kitakkun.backintime.runtime.internal.BackInTimeCompilerInternalApi
+import io.github.kitakkun.backintime.test.base.MockConnector
+import io.github.kitakkun.backintime.websocket.event.BackInTimeDebugServiceEvent
+import org.junit.runner.RunWith
+import org.robolectric.RobolectricTestRunner
+import kotlin.test.AfterTest
+import kotlin.test.BeforeTest
+
+/**
+ * copy of BackInTimeDebugServiceTest.kt in jvmTest
+ */
+@RunWith(RobolectricTestRunner::class)
+abstract class BackInTimeDebugServiceTest {
+    @OptIn(BackInTimeCompilerInternalApi::class)
+    private val service: BackInTimeDebugService = getBackInTimeDebugService(useInUnitTest = true)
+    private val mockConnector = MockConnector()
+    private val events: List<BackInTimeDebugServiceEvent> get() = mockConnector.eventsFromService
+
+    val registerInstanceEvents get() = events.filterIsInstance<BackInTimeDebugServiceEvent.RegisterInstance>()
+    val propertyValueChangeEvents get() = events.filterIsInstance<BackInTimeDebugServiceEvent.NotifyValueChange>()
+    val methodCallEvents get() = events.filterIsInstance<BackInTimeDebugServiceEvent.NotifyMethodCall>()
+    val registerRelationShipEvents get() = events.filterIsInstance<BackInTimeDebugServiceEvent.RegisterRelationship>()
+
+    @BeforeTest
+    fun setup() {
+        service.setConnector(mockConnector)
+        service.startService()
+    }
+
+    @AfterTest
+    fun teardown() {
+        service.stopService()
+    }
+}
diff --git a/...un/backintime/test/LambdaViewModelTest.kt → ...un/backintime/test/LambdaViewModelTest.kt b/...un/backintime/test/LambdaViewModelTest.kt → ...un/backintime/test/LambdaViewModelTest.kt
diff --git a/...kintime/test/MutableStateViewModelTest.kt → ...kintime/test/MutableStateViewModelTest.kt b/...kintime/test/MutableStateViewModelTest.kt → ...kintime/test/MutableStateViewModelTest.kt
diff --git a/...st/PlusAssignMutableStateViewModelTest.kt → ...st/PlusAssignMutableStateViewModelTest.kt b/...st/PlusAssignMutableStateViewModelTest.kt → ...st/PlusAssignMutableStateViewModelTest.kt
diff --git a/...n/backintime/test/StateFlowCaptureTest.kt → ...n/backintime/test/StateFlowCaptureTest.kt b/...n/backintime/test/StateFlowCaptureTest.kt → ...n/backintime/test/StateFlowCaptureTest.kt
@@ -10,8 +10,15 @@ class StateFlowCaptureTest : BackInTimeDebugServiceTest() {
 
         viewModel.updateValues()
 
-        val values = propertyValueChangeEvents.map { it.propertyValue }
-        val expectedValues = listOf("Hoge", "Fuga", "Piyo", "Foo", "Bar", "Baz")
+        val values = propertyValueChangeEvents.map { it.value }
+        val expectedValues = listOf(
+            "\"Hoge\"",
+            "\"Fuga\"",
+            "\"Piyo\"",
+            "\"Foo\"",
+            "\"Bar\"",
+            "\"Baz\""
+        )
         assertEquals(expectedValues, values)
     }
 }
diff --git a/...ntime/test/WeirdCodeStyleViewModelTest.kt → ...ntime/test/WeirdCodeStyleViewModelTest.kt b/...ntime/test/WeirdCodeStyleViewModelTest.kt → ...ntime/test/WeirdCodeStyleViewModelTest.kt
diff --git a/test/src/commonTest/kotlin/io/github/kitakkun/backintime/test/base/MockConnector.kt b/test/src/commonTest/kotlin/io/github/kitakkun/backintime/test/base/MockConnector.kt
@@ -3,6 +3,7 @@ package io.github.kitakkun.backintime.test.base
 import io.github.kitakkun.backintime.runtime.connector.BackInTimeWebSocketConnector
 import io.github.kitakkun.backintime.websocket.event.BackInTimeDebugServiceEvent
 import io.github.kitakkun.backintime.websocket.event.BackInTimeDebuggerEvent
+import kotlinx.coroutines.awaitCancellation
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.MutableSharedFlow
 
@@ -19,7 +20,7 @@ class MockConnector : BackInTimeWebSocketConnector {
     }
 
     override suspend fun awaitCloseSession() {
-        // no-op
+        awaitCancellation()
     }
 
     override suspend fun sendEventToDebugger(event: BackInTimeDebugServiceEvent) {