Allow registering listeners in Groovy DSL

Theres a few things I've identified that need to be looked into:
 - Generic bounds in both the mapping model and Groovy DSL
 - Removing usage of the Groovy DSL in the standard Lorenz tests
 - Splitting up the Groovy DSL unit test

Author: jamierocks

lorenz-dsl-groovy/src/main/java/org/cadixdev/lorenz/dsl/groovy/ClassMappingDsl.java

@@ -43,7 +43,7 @@
  * @author Jamie Mansfield
  * @since 0.6.0
  */
-public class ClassMappingDsl<T extends ClassMapping<?, ?>> extends MappingDsl<T> {
+public class ClassMappingDsl<T extends ClassMapping<T, P>, P> extends MappingDsl<T, P> {
 
     public ClassMappingDsl(final T mapping) {
         super(mapping);

lorenz-dsl-groovy/src/main/java/org/cadixdev/lorenz/dsl/groovy/MappingDsl.java

@@ -27,6 +27,7 @@
 
 import org.cadixdev.lorenz.model.ExtensionKey;
 import org.cadixdev.lorenz.model.Mapping;
+import org.cadixdev.lorenz.util.MappingChangedListener;
 
 /**
  * A DSL to simplify the manipulation of {@link Mapping}s in Groovy.
@@ -35,7 +36,7 @@
  * @author Jamie Mansfield
  * @since 0.6.0
  */
-public class MappingDsl<T extends Mapping<?, ?>> {
+public class MappingDsl<T extends Mapping<T, P>, P> {
 
     /**
      * The mapping manipulated by this DSL.
@@ -68,4 +69,14 @@ public <K> void extension(final ExtensionKey<K> key, final K value) {
         this.mapping.set(key, value);
     }
 
+    /**
+     * Adds the given listener to the mapping.
+     *
+     * @param listener The mapping listener
+     * @see Mapping#addListener(MappingChangedListener)
+     */
+    public void listener(final MappingChangedListener<T, P> listener) {
+        this.mapping.addListener(listener);
+    }
+
 }

lorenz-dsl-groovy/src/main/java/org/cadixdev/lorenz/dsl/groovy/MethodMappingDsl.java

@@ -27,6 +27,7 @@
 
 import groovy.lang.Closure;
 import groovy.lang.DelegatesTo;
+import org.cadixdev.lorenz.model.ClassMapping;
 import org.cadixdev.lorenz.model.MethodMapping;
 import org.cadixdev.lorenz.model.MethodParameterMapping;
 
@@ -36,7 +37,7 @@
  * @author Jamie Mansfield
  * @since 0.6.0
  */
-public class MethodMappingDsl extends MappingDsl<MethodMapping> {
+public class MethodMappingDsl extends MappingDsl<MethodMapping, ClassMapping> {
 
     public MethodMappingDsl(final MethodMapping mapping) {
         super(mapping);

lorenz/src/test/groovy/org/cadixdev/lorenz/test/MappingObserveSpec.groovy

@@ -36,9 +36,10 @@ class MappingObserveSpec extends Specification {
         given:
         final MappingSet mappings = MappingSetDsl.create {
             klass('a') {
-            }.addListener({ TopLevelClassMapping mapping, String newName ->
-                throw new IllegalArgumentException("beep boop")
-            })
+                listener { TopLevelClassMapping mapping, String newName ->
+                    throw new IllegalArgumentException("beep boop")
+                }
+            }
         }
 
         when: