Isolate flaky timeout tests

spockframework · Dec 20, 2024 · 2436da9 · 2436da9
1 parent 9a3ab97
commit 2436da9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/spock-specs/src/test/groovy/org/spockframework/smoke/extension/GlobalTimeoutExtension.groovy b/spock-specs/src/test/groovy/org/spockframework/smoke/extension/GlobalTimeoutExtension.groovy
@@ -1,12 +1,13 @@
 package org.spockframework.smoke.extension
 
-
+import spock.lang.Isolated
 import spock.timeout.BaseTimeoutExtensionSpecification
 
 import java.time.Duration
 
 import org.spockframework.runtime.SpockTimeoutError
 
+@Isolated("The timings are quite tight and it can get flaky on weak machines if run in parallel.")
 class GlobalTimeoutExtension extends BaseTimeoutExtensionSpecification {
   def "applies timeout to features"() {
     given:
@@ -151,7 +152,7 @@ class GlobalTimeoutExtension extends BaseTimeoutExtensionSpecification {
 
   def "can exclude fixture methods from global timeout"() {
     given:
-    enableGlobalTimeout(false, Duration.ofMillis(5))
+    enableGlobalTimeout(false, Duration.ofMillis(20))
 
     when:
     runner.runSpecBody("""

diff --git a/spock-specs/src/test/groovy/spock/util/concurrent/BlockingVariableSpec.groovy b/spock-specs/src/test/groovy/spock/util/concurrent/BlockingVariableSpec.groovy
@@ -19,9 +19,10 @@ import spock.lang.*
 import java.util.concurrent.TimeUnit
 import org.spockframework.runtime.SpockTimeoutError
 
+@Isolated("The test can get flaky on weak machines if run in parallel.")
 class BlockingVariableSpec extends Specification {
   def "variable is read after it is written"() {
-    def list = new BlockingVariable<List<String>>()
+    def list = new BlockingVariable<List<Integer>>()
 
     when:
     Thread.start {
@@ -35,7 +36,7 @@ class BlockingVariableSpec extends Specification {
   }
 
   def "variable is read before it is written"() {
-    def list = new BlockingVariable<List<String>>()
+    def list = new BlockingVariable<List<Integer>>()
 
     when:
     Thread.start {
@@ -49,7 +50,7 @@ class BlockingVariableSpec extends Specification {
   }
 
   def "read times out if no write occurs"() {
-    def list = new BlockingVariable<String>()
+    def list = new BlockingVariable<Integer>()
 
     when:
     list.get()