Increase test timeouts due to slow github action runners

jdaugherty · jdaugherty · commit 9a731cc13deb · 2025-01-09T09:08:49.000-05:00
diff --git a/grails-async/core/src/test/groovy/grails/async/FutureTaskPromiseFactorySpec.groovy b/grails-async/core/src/test/groovy/grails/async/FutureTaskPromiseFactorySpec.groovy
@@ -52,7 +52,7 @@ class FutureTaskPromiseFactorySpec extends Specification {
             list.onComplete { List l -> result = l }
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result
                 result == [2, 4]
             }
@@ -62,7 +62,7 @@ class FutureTaskPromiseFactorySpec extends Specification {
             list.onComplete { result = it }
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [4, 8]
             }
     }
@@ -92,7 +92,7 @@ class FutureTaskPromiseFactorySpec extends Specification {
 
         then: 'the onComplete handler is invoked and the onError handler is ignored'
             thrown(ExecutionException)
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 !result
                 error
             }
diff --git a/grails-async/core/src/test/groovy/grails/async/PromiseListSpec.groovy b/grails-async/core/src/test/groovy/grails/async/PromiseListSpec.groovy
@@ -37,7 +37,7 @@ class PromiseListSpec extends Specification {
             list.onComplete { result = it }
 
         then: 'then the result from onComplete is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [1, 2, 3]
             }
     }
@@ -53,7 +53,7 @@ class PromiseListSpec extends Specification {
             list.onComplete { result = it }
 
         then: 'then the result from onComplete is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [1, 2, 3]
             }
     }
diff --git a/grails-async/core/src/test/groovy/grails/async/PromiseMapSpec.groovy b/grails-async/core/src/test/groovy/grails/async/PromiseMapSpec.groovy
@@ -33,7 +33,7 @@ class PromiseMapSpec extends Specification {
             map.onComplete { result = it }
 
         then: 'an appropriately populated map is returned to the onComplete event'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result
                 result['one'] == 1
                 result['four'] == 4
@@ -52,7 +52,7 @@ class PromiseMapSpec extends Specification {
             map.onComplete { result = it }
 
         then: 'an appropriately populated map is returned to the onComplete event'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result
                 result['one'] == 1
                 result['four'] == 4
diff --git a/grails-async/core/src/test/groovy/grails/async/PromiseSpec.groovy b/grails-async/core/src/test/groovy/grails/async/PromiseSpec.groovy
@@ -80,7 +80,7 @@ class PromiseSpec extends Specification {
             list.onComplete { result = it }
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [2, 4]
             }
     }
@@ -95,7 +95,7 @@ class PromiseSpec extends Specification {
             list.onComplete { result = it }
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [2, 4]
             }
 
@@ -104,7 +104,7 @@ class PromiseSpec extends Specification {
             list.onComplete { result = it }
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [6, 8]
             }
     }
@@ -119,7 +119,7 @@ class PromiseSpec extends Specification {
             promise.onError { hasError = true }
 
         then: 'the onComplete handler is invoked and the onError handler is ignored'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == 2
                 hasError == false
             }
@@ -135,7 +135,7 @@ class PromiseSpec extends Specification {
             promise.onError { error = it }
 
         then: 'the onComplete handler is invoked and the onError handler is ignored'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 !result
                 error
                 error.message.contains('bad')
diff --git a/grails-async/core/src/test/groovy/grails/async/SynchronousPromiseFactorySpec.groovy b/grails-async/core/src/test/groovy/grails/async/SynchronousPromiseFactorySpec.groovy
@@ -66,7 +66,7 @@ class SynchronousPromiseFactorySpec extends Specification {
             list.onComplete { result = it }
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [2, 4]
             }
 
@@ -75,7 +75,7 @@ class SynchronousPromiseFactorySpec extends Specification {
             list.onComplete { result = it }
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [6, 8]
             }
     }
@@ -90,7 +90,7 @@ class SynchronousPromiseFactorySpec extends Specification {
             promise.onError { hasError = true }
 
         then: 'The onComplete handler is invoked and the onError handler is ignored'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == 2
                 hasError == false
             }
@@ -106,7 +106,7 @@ class SynchronousPromiseFactorySpec extends Specification {
             promise.onError { error = it }
 
         then: 'the onComplete handler is invoked and the onError handler is ignored'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 !result
                 error
                 error.message == 'bad'
diff --git a/grails-async/gpars/src/test/groovy/grails/async/GparsPromiseSpec.groovy b/grails-async/gpars/src/test/groovy/grails/async/GparsPromiseSpec.groovy
@@ -80,7 +80,7 @@ class GparsPromiseSpec extends Specification {
             promiseList.onComplete { List v -> result = v }
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [2, 4]
             }
 
@@ -90,7 +90,7 @@ class GparsPromiseSpec extends Specification {
             promiseList.onComplete { List v -> result = v }
 
         then: 'The result is correct'
-            new PollingConditions() {
+            new PollingConditions(timeout: 5) {
                 result == [2,4]
             }
     }
@@ -105,7 +105,7 @@ class GparsPromiseSpec extends Specification {
             promise.onError { hasError = true }
 
         then: 'the onComplete handler is invoked and the onError handler is ignored'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == 2
                 hasError == false
             }
@@ -121,7 +121,7 @@ class GparsPromiseSpec extends Specification {
             promise.onError { error = it }
 
         then: 'the onError handler is invoked and the onComplete handler is ignored'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 !result
                 error
                 error.message == 'bad'
diff --git a/grails-async/rxjava/src/test/groovy/org/grails/async/factory/rxjava/RxJavaPromiseListSpec.groovy b/grails-async/rxjava/src/test/groovy/org/grails/async/factory/rxjava/RxJavaPromiseListSpec.groovy
@@ -21,7 +21,7 @@ class RxJavaPromiseListSpec extends Specification{
             list.onComplete { result = it }
 
         then: 'then the result from onComplete is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [1,2,3]
             }
     }
@@ -37,7 +37,7 @@ class RxJavaPromiseListSpec extends Specification{
             list.onComplete { result = it }
 
         then: 'then the result from onComplete is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [1,2,3]
             }
     }
diff --git a/grails-async/rxjava/src/test/groovy/org/grails/async/factory/rxjava/RxJavaPromiseMapSpec.groovy b/grails-async/rxjava/src/test/groovy/org/grails/async/factory/rxjava/RxJavaPromiseMapSpec.groovy
@@ -18,7 +18,7 @@ class RxJavaPromiseMapSpec extends Specification{
             map.onComplete { result = it }
 
         then: 'an appropriately populated map is returned to the onComplete event'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result
                 result['one'] == 1
                 result['four'] == 4
@@ -37,7 +37,7 @@ class RxJavaPromiseMapSpec extends Specification{
             map.onComplete { result = it }
 
         then: 'an appropriately populated map is returned to the onComplete event'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result
                 result['one'] == 1
                 result['four'] == 4
@@ -56,7 +56,7 @@ class RxJavaPromiseMapSpec extends Specification{
             map.onComplete { result = it }
 
         then: 'an appropriately populated map is returned to the onComplete event'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result
                 result['one'] == 1
                 result['four'] == 4
@@ -79,7 +79,7 @@ class RxJavaPromiseMapSpec extends Specification{
         map.onError { error = it }
 
         then: 'an appropriately populated map is returned to the onComplete event'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 !result
                 error
                 error.message == 'bad'
diff --git a/grails-async/rxjava/src/test/groovy/org/grails/async/factory/rxjava/RxJavaPromiseSpec.groovy b/grails-async/rxjava/src/test/groovy/org/grails/async/factory/rxjava/RxJavaPromiseSpec.groovy
@@ -62,7 +62,7 @@ class RxJavaPromiseSpec extends Specification {
             list.onComplete { result = it }
 
         then: 'The result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [2,4]
             }
 
@@ -71,7 +71,7 @@ class RxJavaPromiseSpec extends Specification {
             list.onComplete { result = it }
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [4,8]
             }
     }
@@ -86,7 +86,7 @@ class RxJavaPromiseSpec extends Specification {
             promise.onError { hasError = true }
 
         then: 'the onComplete handler is invoked and the onError handler is ignored'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == 2
                 hasError == false
             }
@@ -102,7 +102,7 @@ class RxJavaPromiseSpec extends Specification {
             promise.onError { error = it }
 
         then: 'the onComplete handler is invoked and the onError handler is ignored'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 !result
                 error
                 error.message == 'bad'
diff --git a/grails-async/rxjava2/src/test/groovy/org/grails/async/factory/rxjava2/RxPromiseListSpec.groovy b/grails-async/rxjava2/src/test/groovy/org/grails/async/factory/rxjava2/RxPromiseListSpec.groovy
@@ -20,7 +20,7 @@ class RxPromiseListSpec extends Specification {
             list.onComplete { result = it }
 
         then: 'then the result from onComplete is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [1,2,3]
             }
     }
@@ -36,7 +36,7 @@ class RxPromiseListSpec extends Specification {
             list.onComplete { result = it }
 
         then: 'then the result from onComplete is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [1,2,3]
             }
     }
diff --git a/grails-async/rxjava2/src/test/groovy/org/grails/async/factory/rxjava2/RxPromiseSpec.groovy b/grails-async/rxjava2/src/test/groovy/org/grails/async/factory/rxjava2/RxPromiseSpec.groovy
@@ -64,7 +64,7 @@ class RxPromiseSpec extends Specification {
             list.onComplete { result = it }
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [2,4]
             }
 
@@ -73,7 +73,7 @@ class RxPromiseSpec extends Specification {
             list.onComplete { result = it }
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == [4,8]
             }
     }
@@ -88,7 +88,7 @@ class RxPromiseSpec extends Specification {
             promise.onError { hasError = true }
 
         then: 'the onComplete handler is invoked and the onError handler is ignored'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == 2
                 hasError == false
             }
@@ -104,7 +104,7 @@ class RxPromiseSpec extends Specification {
             promise.onError { error = it }
 
         then: 'the onComplete handler is ignored and the onError handler is invoked'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 !result
                 error
                 error.message == 'bad'
diff --git a/grails-events/gpars/src/test/groovy/org/grails/events/gpars/ActorEventBusSpec.groovy b/grails-events/gpars/src/test/groovy/org/grails/events/gpars/ActorEventBusSpec.groovy
@@ -18,7 +18,7 @@ class ActorEventBusSpec extends Specification {
             eventBus.notify('test', 'bar')
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == 'foo bar'
             }
     }
@@ -36,7 +36,7 @@ class ActorEventBusSpec extends Specification {
         eventBus.notify('test', 'bar', 'baz')
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == 'foo [bar, baz]'
             }
     }
@@ -54,7 +54,7 @@ class ActorEventBusSpec extends Specification {
             eventBus.notify('test', 'bar', 'baz')
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == 'foo bar baz'
             }
     }
@@ -72,7 +72,7 @@ class ActorEventBusSpec extends Specification {
         eventBus.sendAndReceive('test', 'bar') { result = it }
 
         then: 'the result is correct'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result == 'foo bar'
             }
     }
@@ -92,7 +92,7 @@ class ActorEventBusSpec extends Specification {
             eventBus.sendAndReceive('test', 'bar') { result = it }
 
         then: 'the result is a throwable'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 result instanceof Throwable
             }
     }
diff --git a/grails-events/rxjava/src/test/groovy/org/grails/events/rxjava/PublishSubscribeSpringSpec.groovy b/grails-events/rxjava/src/test/groovy/org/grails/events/rxjava/PublishSubscribeSpringSpec.groovy
@@ -35,7 +35,7 @@ class PublishSubscribeSpringSpec extends Specification {
             publisher.sum(1, 2)
 
         then: 'the subscriber is notified'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 !subscriber.error
                 subscriber.total == 3
                 subscriber.events.size() == 1
@@ -47,7 +47,7 @@ class PublishSubscribeSpringSpec extends Specification {
             publisher.wrongType()
 
         then: 'the subscriber is not notified'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 !subscriber.error
                 subscriber.total == 3
                 subscriber.events.size() == 2
@@ -59,7 +59,7 @@ class PublishSubscribeSpringSpec extends Specification {
         then: 'an exception is thrown'
             def e = thrown(RuntimeException)
             e.message == 'bad'
-            new PollingConditions().eventually {
+            new PollingConditions(timeout: 5).eventually {
                 subscriber.error == e
                 subscriber.events.size() == 3
                 subscriber.total == 3
diff --git a/grails-events/transform/src/test/groovy/grails/events/annotation/PublishSubscribeSpringSpec.groovy b/grails-events/transform/src/test/groovy/grails/events/annotation/PublishSubscribeSpringSpec.groovy
diff --git a/grails-test-examples/async-events-pubsub-demo/src/integration-test/groovy/pubsub/demo/PubSubSpec.groovy b/grails-test-examples/async-events-pubsub-demo/src/integration-test/groovy/pubsub/demo/PubSubSpec.groovy

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ class FutureTaskPromiseFactorySpec extends Specification {`
`52`	`52`	`list.onComplete { List l -> result = l }`
`53`	`53`
`54`	`54`	`then: 'the result is correct'`
`55`		`- new PollingConditions().eventually {`
	`55`	`+ new PollingConditions(timeout: 5).eventually {`
`56`	`56`	`result`
`57`	`57`	`result == [2, 4]`
`58`	`58`	`}`
`@@ -62,7 +62,7 @@ class FutureTaskPromiseFactorySpec extends Specification {`
`62`	`62`	`list.onComplete { result = it }`
`63`	`63`
`64`	`64`	`then: 'the result is correct'`
`65`		`- new PollingConditions().eventually {`
	`65`	`+ new PollingConditions(timeout: 5).eventually {`
`66`	`66`	`result == [4, 8]`
`67`	`67`	`}`
`68`	`68`	`}`
`@@ -92,7 +92,7 @@ class FutureTaskPromiseFactorySpec extends Specification {`
`92`	`92`
`93`	`93`	`then: 'the onComplete handler is invoked and the onError handler is ignored'`
`94`	`94`	`thrown(ExecutionException)`
`95`		`- new PollingConditions().eventually {`
	`95`	`+ new PollingConditions(timeout: 5).eventually {`
`96`	`96`	`!result`
`97`	`97`	`error`
`98`	`98`	`}`
Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ class PromiseListSpec extends Specification {`
`37`	`37`	`list.onComplete { result = it }`
`38`	`38`
`39`	`39`	`then: 'then the result from onComplete is correct'`
`40`		`- new PollingConditions().eventually {`
	`40`	`+ new PollingConditions(timeout: 5).eventually {`
`41`	`41`	`result == [1, 2, 3]`
`42`	`42`	`}`
`43`	`43`	`}`
`@@ -53,7 +53,7 @@ class PromiseListSpec extends Specification {`
`53`	`53`	`list.onComplete { result = it }`
`54`	`54`
`55`	`55`	`then: 'then the result from onComplete is correct'`
`56`		`- new PollingConditions().eventually {`
	`56`	`+ new PollingConditions(timeout: 5).eventually {`
`57`	`57`	`result == [1, 2, 3]`
`58`	`58`	`}`
`59`	`59`	`}`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ class RxJavaPromiseListSpec extends Specification{`
`21`	`21`	`list.onComplete { result = it }`
`22`	`22`
`23`	`23`	`then: 'then the result from onComplete is correct'`
`24`		`- new PollingConditions().eventually {`
	`24`	`+ new PollingConditions(timeout: 5).eventually {`
`25`	`25`	`result == [1,2,3]`
`26`	`26`	`}`
`27`	`27`	`}`
`@@ -37,7 +37,7 @@ class RxJavaPromiseListSpec extends Specification{`
`37`	`37`	`list.onComplete { result = it }`
`38`	`38`
`39`	`39`	`then: 'then the result from onComplete is correct'`
`40`		`- new PollingConditions().eventually {`
	`40`	`+ new PollingConditions(timeout: 5).eventually {`
`41`	`41`	`result == [1,2,3]`
`42`	`42`	`}`
`43`	`43`	`}`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ class RxPromiseListSpec extends Specification {`
`20`	`20`	`list.onComplete { result = it }`
`21`	`21`
`22`	`22`	`then: 'then the result from onComplete is correct'`
`23`		`- new PollingConditions().eventually {`
	`23`	`+ new PollingConditions(timeout: 5).eventually {`
`24`	`24`	`result == [1,2,3]`
`25`	`25`	`}`
`26`	`26`	`}`
`@@ -36,7 +36,7 @@ class RxPromiseListSpec extends Specification {`
`36`	`36`	`list.onComplete { result = it }`
`37`	`37`
`38`	`38`	`then: 'then the result from onComplete is correct'`
`39`		`- new PollingConditions().eventually {`
	`39`	`+ new PollingConditions(timeout: 5).eventually {`
`40`	`40`	`result == [1,2,3]`
`41`	`41`	`}`
`42`	`42`	`}`