fix: Prevent NullPointerException in cleanupSpec for non-@Integration tests (#66)

matrei · web-flow · commit bfe9f34c807f · 2024-10-26T18:23:28.000+02:00
If a test is not annotated with `@Integration`, the `webDriverContainer` is null in `cleanupSpec`. This fix ensures that `cleanupSpec` checks for a null `webDriverContainer`, preventing a NullPointerException during cleanup for non-`@Integration` tests.
diff --git a/src/testFixtures/groovy/grails/plugin/geb/ContainerGebSpec.groovy b/src/testFixtures/groovy/grails/plugin/geb/ContainerGebSpec.groovy
@@ -79,7 +79,7 @@ class ContainerGebSpec extends GebSpec {
     }
 
     def cleanupSpec() {
-        webDriverContainer.stop()
+        webDriverContainer?.stop()
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ class ContainerGebSpec extends GebSpec {`
`79`	`79`	`}`
`80`	`80`
`81`	`81`	`def cleanupSpec() {`
`82`		`- webDriverContainer.stop()`
	`82`	`+ webDriverContainer?.stop()`
`83`	`83`	`}`
`84`	`84`
`85`	`85`	`/**`