Discussions

Author: igordmn

compose/runtime/runtime/src/jvmTest/kotlin/androidx/compose/runtime/internal/Sleep.jvm.kt

@@ -16,7 +16,5 @@
 
 package androidx.compose.runtime.internal
 
-actual fun isSleepAvailable() = true
-
-actual fun sleep(millis: UInt) =
+internal actual fun sleep(millis: UInt) =
     Thread.sleep(millis.toLong())

compose/runtime/runtime/src/nativeTest/kotlin/androidx/compose/runtime/internal/Sleep.native.kt

@@ -16,8 +16,6 @@
 
 package androidx.compose.runtime.internal
 
-actual fun isSleepAvailable() = true
-
-actual fun sleep(millis: UInt) {
+internal actual fun sleep(millis: UInt) {
     platform.posix.usleep(millis)
 }

compose/runtime/runtime/src/nonEmulatorCommonTest/kotlin/androidx/compose/runtime/BroadcastFrameClockTest.kt

@@ -17,7 +17,6 @@
 package androidx.compose.runtime
 
 import androidx.compose.runtime.internal.AtomicInt
-import androidx.compose.runtime.internal.isSleepAvailable
 import androidx.compose.runtime.internal.sleep
 import kotlin.test.Test
 import kotlin.test.assertEquals
@@ -36,6 +35,8 @@ import kotlinx.coroutines.launch
 import kotlinx.coroutines.test.UnconfinedTestDispatcher
 import kotlinx.coroutines.test.runTest
 import kotlinx.coroutines.yield
+import kotlinx.test.IgnoreJsTarget
+import kotlinx.test.IgnoreWasmTarget
 
 @ExperimentalCoroutinesApi
 class BroadcastFrameClockTest {
@@ -87,9 +88,10 @@ class BroadcastFrameClockTest {
         }
 
     @OptIn(InternalCoroutinesApi::class)
+    @IgnoreJsTarget
+    @IgnoreWasmTarget
     @Test
     fun locklessCancellation() = runTest(timeout = 5_000.milliseconds) {
-        if (!isSleepAvailable()) return@runTest
 
         val clock = BroadcastFrameClock()
         val cancellationGate = AtomicInt(1)

compose/runtime/runtime/src/nonEmulatorCommonTest/kotlin/androidx/compose/runtime/internal/Sleep.kt

@@ -16,6 +16,4 @@
 
 package androidx.compose.runtime.internal
 
-internal expect fun isSleepAvailable(): Boolean
-
 internal expect fun sleep(millis: UInt)

compose/runtime/runtime/src/webTest/kotlin/androidx/compose/runtime/internal/Sleep.web.kt

@@ -16,7 +16,5 @@
 
 package androidx.compose.runtime.internal
 
-actual fun isSleepAvailable() = false
-
-actual fun sleep(millis: UInt): Unit =
+internal actual fun sleep(millis: UInt): Unit =
     throw UnsupportedOperationException("Sleep is not supported")