Disable tests with real source of the time on Java's Windows to preve… (#3516)

* Disable tests with the real source of the time on Java's Windows to prevent flakiness

We still want to test real code, not virtualized mocked clocks, but Windows itself is
virtualized on our TeamCity, making Java's parker resolution really low and unstable for such tests

Co-authored-by: dkhalanskyjb <[email protected]>

Author: qwwdfsad

kotlinx-coroutines-core/common/test/JobTest.kt

@@ -103,11 +103,11 @@ class JobTest : TestBase() {
         }
         assertTrue(job.isActive)
         for (i in 0 until n) assertEquals(0, fireCount[i])
-        val tryCancel = Try { job.cancel() }
+        val cancelResult = runCatching { job.cancel() }
         assertTrue(!job.isActive)
         for (i in 0 until n) assertEquals(1, fireCount[i])
-        assertTrue(tryCancel.exception is CompletionHandlerException)
-        assertTrue(tryCancel.exception!!.cause is TestException)
+        assertTrue(cancelResult.exceptionOrNull() is CompletionHandlerException)
+        assertTrue(cancelResult.exceptionOrNull()!!.cause is TestException)
     }
 
     @Test

kotlinx-coroutines-core/common/test/TestBase.common.kt

@@ -104,3 +104,5 @@ class BadClass {
     override fun hashCode(): Int = error("hashCode")
     override fun toString(): String = error("toString")
 }
+
+public expect val isJavaAndWindows: Boolean

kotlinx-coroutines-core/common/test/Try.kt

@@ -131,6 +131,7 @@ class WithTimeoutOrNullDurationTest : TestBase() {
 
     @Test
     fun testOuterTimeout() = runTest {
+        if (isJavaAndWindows) return@runTest
         var counter = 0
         val result = withTimeoutOrNull(320.milliseconds) {
             while (true) {

kotlinx-coroutines-core/common/test/WithTimeoutOrNullDurationTest.kt

@@ -128,6 +128,7 @@ class WithTimeoutOrNullTest : TestBase() {
 
     @Test
     fun testOuterTimeout() = runTest {
+        if (isJavaAndWindows) return@runTest
         var counter = 0
         val result = withTimeoutOrNull(320) {
             while (true) {

kotlinx-coroutines-core/common/test/WithTimeoutOrNullTest.kt

@@ -138,3 +138,5 @@ public actual open class TestBase actual constructor() {
 
 private fun <T> Promise<T>.finally(block: () -> Unit): Promise<T> =
     then(onFulfilled = { value -> block(); value }, onRejected = { ex -> block(); throw ex })
+
+public actual val isJavaAndWindows: Boolean get() = false

kotlinx-coroutines-core/js/test/TestBase.kt

@@ -254,3 +254,10 @@ public actual open class TestBase(private var disableOutCheck: Boolean)  {
 
     protected suspend fun currentDispatcher() = coroutineContext[ContinuationInterceptor]!!
 }
+
+/*
+ * We ignore tests that test **real** non-virtualized tests with time on Windows, because
+ * our CI Windows is virtualized itself (oh, the irony) and its clock resolution is dozens of ms,
+ * which makes such tests flaky.
+ */
+public actual val isJavaAndWindows: Boolean = System.getProperty("os.name")!!.contains("Windows")

kotlinx-coroutines-core/jvm/test/TestBase.kt

@@ -107,3 +107,5 @@ public actual open class TestBase actual constructor() {
             error("Too few unhandled exceptions $exCount, expected ${unhandled.size}")
     }
 }
+
+public actual val isJavaAndWindows: Boolean get() = false