Merge pull request #106 from jdaugherty/5.0.x

fix setup() methods being called
grails · Dec 17, 2024 · 6559b50 · 6559b50
2 parents 3139fc6 + 9e7d760
commit 6559b50
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 ->