Merge pull request #107 from jdaugherty/4.1.x

4.1.x - backport setup() methods being called
grails · Dec 18, 2024 · b4f35e6 · b4f35e6
2 parents 02efb4c + 3986649
commit b4f35e6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/src/testFixtures/groovy/grails/plugin/geb/GebRecordingTestListener.groovy b/src/testFixtures/groovy/grails/plugin/geb/GebRecordingTestListener.groovy
@@ -19,9 +19,6 @@ import groovy.transform.CompileStatic
 import org.spockframework.runtime.AbstractRunListener
 import org.spockframework.runtime.model.ErrorInfo
 import org.spockframework.runtime.model.IterationInfo
-import org.spockframework.runtime.model.SpecInfo
-
-import java.time.LocalDateTime
 
 /**
  * A test listener that reports the test result to {@link org.testcontainers.containers.BrowserWebDriverContainer} so

diff --git a/src/testFixtures/groovy/grails/plugin/geb/GrailsContainerGebExtension.groovy b/src/testFixtures/groovy/grails/plugin/geb/GrailsContainerGebExtension.groovy
@@ -86,9 +86,11 @@ class GrailsContainerGebExtension implements IGlobalExtension {
                 invocation.proceed()
             }
 
-            spec.addSetupInterceptor {
+            spec.addSetupInterceptor { invocation ->
                 // Grails will be initialized by this point, so setup the browser url correctly
-                holder.setupBrowserUrl(it)
+                holder.setupBrowserUrl(invocation)
+
+                invocation.proceed()
             }
 
             spec.addInterceptor { invocation ->