This is an automated email from the ASF dual-hosted git repository.

paulk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy-geb.git


The following commit(s) were added to refs/heads/master by this push:
     new b2670901 minor refactor: more fine-grained headless testing
b2670901 is described below

commit b267090171906e9ffa43058868d6278b3d2094dd
Author: Paul King <[email protected]>
AuthorDate: Thu Dec 26 17:01:23 2024 +1000

    minor refactor: more fine-grained headless testing
---
 .../src/test/groovy/configuration/DriverConfigSpec.groovy    | 12 +++++++++++-
 .../src/main/groovy/geb/fixture/HeadlessTestSupport.groovy   |  2 +-
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git 
a/doc/manual-snippets/src/test/groovy/configuration/DriverConfigSpec.groovy 
b/doc/manual-snippets/src/test/groovy/configuration/DriverConfigSpec.groovy
index d1b01226..16943259 100644
--- a/doc/manual-snippets/src/test/groovy/configuration/DriverConfigSpec.groovy
+++ b/doc/manual-snippets/src/test/groovy/configuration/DriverConfigSpec.groovy
@@ -38,10 +38,12 @@ class DriverConfigSpec extends Specification implements 
InlineConfigurationLoade
 
     def setupSpec() {
         CachingDriverFactory.clearCacheAndQuitDriver()
-        Assumptions.assumeFalse(HeadlessTestSupport.headless)
     }
 
     def "configuring driver using closure"() {
+        given:
+        Assumptions.assumeFalse(HeadlessTestSupport.headless)
+
         when:
         configScript """
             // tag::configuring_driver[]
@@ -59,6 +61,9 @@ class DriverConfigSpec extends Specification implements 
InlineConfigurationLoade
     }
 
     def "configuring driver using class name"() {
+        given:
+        Assumptions.assumeFalse(HeadlessTestSupport.headless)
+
         when:
         configScript """
             // tag::configuring_driver_using_class_name[]
@@ -74,6 +79,9 @@ class DriverConfigSpec extends Specification implements 
InlineConfigurationLoade
     }
 
     def "configuring driver using driver name"() {
+        given:
+        Assumptions.assumeFalse(HeadlessTestSupport.headless)
+
         when:
         configScript """
             // tag::configuring_driver_using_driver_name[]
@@ -92,6 +100,8 @@ class DriverConfigSpec extends Specification implements 
InlineConfigurationLoade
     def "environment sensitive driver config"() {
         when:
         configScript(env, """
+            !env || Assumptions.assumeFalse(HeadlessTestSupport.headless)
+
             // tag::env_sensitive_driver_config[]
             import org.openqa.selenium.htmlunit.HtmlUnitDriver
 
diff --git 
a/internal/test-support/src/main/groovy/geb/fixture/HeadlessTestSupport.groovy 
b/internal/test-support/src/main/groovy/geb/fixture/HeadlessTestSupport.groovy
index 0af6cea8..46f4b404 100644
--- 
a/internal/test-support/src/main/groovy/geb/fixture/HeadlessTestSupport.groovy
+++ 
b/internal/test-support/src/main/groovy/geb/fixture/HeadlessTestSupport.groovy
@@ -45,7 +45,7 @@ abstract class HeadlessTestSupport {
 
     static {
         try {
-            final Class jframe = ClassLoader.loadClass('javax.swing.JFrame')
+            final Class jframe = 
HeadlessTestSupport.classLoader.loadClass('javax.swing.JFrame')
             final Constructor constructor = jframe.getConstructor([String] as 
Class[])
             constructor.newInstance(['testing'] as String[])
             headless = false

Reply via email to