Fix debug probes not being installed

dkhalanskyjb · dkhalanskyjb · commit cd7afaf21699 · 2021-04-23T10:45:27.000+03:00
diff --git a/kotlinx-coroutines-debug/src/junit5/CoroutinesTimeoutExtension.kt b/kotlinx-coroutines-debug/src/junit5/CoroutinesTimeoutExtension.kt
@@ -39,7 +39,7 @@ public class CoroutinesTimeoutExtension: InvocationInterceptor {
             }.orElseGet {
                 throw UnsupportedOperationException("CoroutinesTimeoutExtension should not be used directly; annotate the test class or method with CoroutinesTimeout instead.")
             }
-        invocation.proceed()
+        interceptInvocation(invocation, invocationContext.executable.name, annotation)
     }
 
     override fun interceptAfterAllMethod(
@@ -107,6 +107,7 @@ public class CoroutinesTimeoutExtension: InvocationInterceptor {
             invocation.proceed()
         }
         val testThread = Thread(testResult, "Timeout test thread").apply { isDaemon = true }
+        DebugProbes.install()
         try {
             testThread.start()
             // Await until test is started to take only test execution time into account

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ public class CoroutinesTimeoutExtension: InvocationInterceptor {`
`39`	`39`	`}.orElseGet {`
`40`	`40`	`throw UnsupportedOperationException("CoroutinesTimeoutExtension should not be used directly; annotate the test class or method with CoroutinesTimeout instead.")`
`41`	`41`	`}`
`42`		`- invocation.proceed()`
	`42`	`+ interceptInvocation(invocation, invocationContext.executable.name, annotation)`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`override fun interceptAfterAllMethod(`
`@@ -107,6 +107,7 @@ public class CoroutinesTimeoutExtension: InvocationInterceptor {`
`107`	`107`	`invocation.proceed()`
`108`	`108`	`}`
`109`	`109`	`val testThread = Thread(testResult, "Timeout test thread").apply { isDaemon = true }`
	`110`	`+ DebugProbes.install()`
`110`	`111`	`try {`
`111`	`112`	`testThread.start()`
`112`	`113`	`// Await until test is started to take only test execution time into account`