From fa6201ffd60930bf975363fe6a0cae0ab07b24e6 Mon Sep 17 00:00:00 2001
From: Mattias Reichel <mattias.reichel@gmail.com>
Date: Tue, 24 Dec 2024 23:16:12 +0100
Subject: [PATCH] Skip Starting Recording Container When Recording Mode is Set
 to `SKIP` (#118)

* fix: disable recording container

Closes gh-117

* refactor: set recording mode directly on created container
---
 .../plugin/geb/WebDriverContainerHolder.groovy      | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/src/testFixtures/groovy/grails/plugin/geb/WebDriverContainerHolder.groovy b/src/testFixtures/groovy/grails/plugin/geb/WebDriverContainerHolder.groovy
index 6ee8a01..de3d367 100644
--- a/src/testFixtures/groovy/grails/plugin/geb/WebDriverContainerHolder.groovy
+++ b/src/testFixtures/groovy/grails/plugin/geb/WebDriverContainerHolder.groovy
@@ -98,14 +98,11 @@ class WebDriverContainerHolder {
         }
 
         currentConfiguration = specConfiguration
-        currentContainer = new BrowserWebDriverContainer()
-        if (grailsGebSettings.recordingEnabled) {
-            currentContainer = currentContainer.withRecordingMode(
-                    grailsGebSettings.recordingMode,
-                    grailsGebSettings.recordingDirectory,
-                    grailsGebSettings.recordingFormat
-            )
-        }
+        currentContainer = new BrowserWebDriverContainer().withRecordingMode(
+                grailsGebSettings.recordingMode,
+                grailsGebSettings.recordingDirectory,
+                grailsGebSettings.recordingFormat
+        )
         currentContainer.tap {
             withAccessToHost(true)
             start()