move related classes, grails.plugin.geb should only be "business logic" (real override parts)

can be dropped

Author: JonasPammer

README.md

@@ -78,15 +78,15 @@ By default, no test recording will be performed.  Here are the system properties
 
 To customize the naming convention for recording files:
 
-1. Create a class that implements [`ContainerGebTestDescription`](./src/testFixtures/groovy/grails/plugin/geb/ContainerGebTestDescription.groovy). 
-2. Specify its fully qualified class name in a `META-INF/services/grails.plugin.geb.ContainerGebTestDescription` file
+1. Create a class that implements [`ContainerGebTestDescription`](./src/testFixtures/groovy/grails/plugin/geb/serviceloader/ContainerGebTestDescription.groovy). 
+2. Specify its fully qualified class name in a `META-INF/services/grails.plugin.geb.serviceloader.ContainerGebTestDescription` file
    on the classpath (e.g., `src/integration-test/resources`).
 
 Once both conditions are met, the class is automatically loaded and replaces the default behavior. \
-You may extend the default implementation, [`DefaultContainerGebTestDescription`](./src/testFixtures/groovy/grails/plugin/geb/DefaultContainerGebTestDescription.groovy), as a base for your custom implementation. 
+You may extend the default implementation, [`DefaultContainerGebTestDescription`](./src/testFixtures/groovy/grails/plugin/geb/serviceloader/DefaultContainerGebTestDescription.groovy), as a base for your custom implementation. 
 To modify the naming convention, override at least the `getTestId()` method.
 
-Optionally, in specific test classes, either call [`ContainerGebTestDescriptionServiceLoader.setInstance()`](./src/testFixtures/groovy/grails/plugin/geb/ContainerGebTestDescriptionServiceLoader.groovy)
+Optionally, in specific test classes, either call [`ContainerGebTestDescriptionServiceLoader.setInstance()`](./src/testFixtures/groovy/grails/plugin/geb/serviceloader/ContainerGebTestDescriptionServiceLoader.groovy)
 in a Spock `setupSpec` method to apply your naming convention (And use a `cleanupSpec` to limit this to one class),
 or set the `testDescription` Property on your ContainerGebConfiguration annotation.
 

spock-container-test-app/src/integration-test/groovy/org/demo/spock/ContainerGebTestDescriptionImpl.groovy

@@ -1,6 +1,6 @@
 package org.demo.spock
 
-import grails.plugin.geb.DefaultContainerGebTestDescription
+import grails.plugin.geb.serviceloader.DefaultContainerGebTestDescription
 import groovy.transform.CompileStatic
 
 @CompileStatic

src/testFixtures/groovy/grails/plugin/geb/ContainerGebConfiguration.groovy

@@ -15,6 +15,8 @@
  */
 package grails.plugin.geb
 
+import grails.plugin.geb.serviceloader.ContainerGebTestDescription
+import grails.plugin.geb.serviceloader.NullContainerGebTestDescription
 import org.testcontainers.containers.GenericContainer
 
 import java.lang.annotation.ElementType

src/testFixtures/groovy/grails/plugin/geb/GebRecordingTestListener.groovy

@@ -15,6 +15,7 @@
  */
 package grails.plugin.geb
 
+import grails.plugin.geb.serviceloader.ContainerGebTestDescription
 import grails.plugin.geb.serviceloader.ContainerGebTestDescriptionServiceLoader
 import groovy.transform.CompileStatic
 import org.spockframework.runtime.AbstractRunListener

src/testFixtures/groovy/grails/plugin/geb/WebDriverContainerHolder.groovy

@@ -20,7 +20,9 @@ import geb.Browser
 import geb.Configuration
 import geb.spock.SpockGebTestManagerBuilder
 import geb.test.GebTestManager
+import grails.plugin.geb.serviceloader.ContainerGebTestDescription
 import grails.plugin.geb.serviceloader.ContainerGebTestDescriptionServiceLoader
+import grails.plugin.geb.serviceloader.NullContainerGebTestDescription
 import groovy.transform.CompileStatic
 import groovy.transform.EqualsAndHashCode
 import groovy.transform.PackageScope

…n/geb/ContainerGebTestDescription.groovy …oader/ContainerGebTestDescription.groovysrc/testFixtures/groovy/grails/plugin/geb/ContainerGebTestDescription.groovy renamed to src/testFixtures/groovy/grails/plugin/geb/serviceloader/ContainerGebTestDescription.groovy src/testFixtures/groovy/grails/plugin/geb/ContainerGebTestDescription.groovy renamed to src/testFixtures/groovy/grails/plugin/geb/serviceloader/ContainerGebTestDescription.groovy

@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package grails.plugin.geb
+package grails.plugin.geb.serviceloader
 
 import groovy.transform.CompileStatic
 import org.spockframework.runtime.model.ErrorInfo
@@ -26,7 +26,7 @@ import org.testcontainers.lifecycle.TestDescription
  * <p>
  * Implementations must provide a zero-argument constructor to ensure compatibility with {@link java.util.ServiceLoader}.
  *
- * @see GebRecordingTestListener
+ * @see grails.plugin.geb.GebRecordingTestListener
  */
 @CompileStatic
 interface ContainerGebTestDescription extends TestDescription {

src/testFixtures/groovy/grails/plugin/geb/serviceloader/ContainerGebTestDescriptionServiceLoader.groovy

@@ -15,8 +15,7 @@
  */
 package grails.plugin.geb.serviceloader
 
-import grails.plugin.geb.ContainerGebTestDescription
-import grails.plugin.geb.DefaultContainerGebTestDescription
+
 import groovy.transform.CompileStatic
 
 /**

…efaultContainerGebTestDescription.groovy …efaultContainerGebTestDescription.groovysrc/testFixtures/groovy/grails/plugin/geb/DefaultContainerGebTestDescription.groovy renamed to src/testFixtures/groovy/grails/plugin/geb/serviceloader/DefaultContainerGebTestDescription.groovy src/testFixtures/groovy/grails/plugin/geb/DefaultContainerGebTestDescription.groovy renamed to src/testFixtures/groovy/grails/plugin/geb/serviceloader/DefaultContainerGebTestDescription.groovy

@@ -13,7 +13,8 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package grails.plugin.geb
+package grails.plugin.geb.serviceloader
+
 
 import groovy.transform.CompileStatic
 import org.spockframework.runtime.model.ErrorInfo
@@ -49,6 +50,6 @@ class DefaultContainerGebTestDescription implements ContainerGebTestDescription
 }
 
 /**
- * Used by {@link ContainerGebConfiguration#testDescription()} interface to represent a null value.
+ * Used by {@link grails.plugin.geb.ContainerGebConfiguration#testDescription()} interface to represent a null value.
  */
 class NullContainerGebTestDescription extends DefaultContainerGebTestDescription {}