Prevent removal of ErrorSpecNode from execution hierarchy

spock-core/src/main/java/org/spockframework/runtime/ErrorSpecNode.java

@@ -24,6 +24,13 @@ public void prune() {
     // without this empty override, the node is thrown away and the error is not reported
   }
 
+  @Override
+  public void removeFromHierarchy() {
+    // prevent removal of this node
+    // As the ErrorSpecNode does not have children it would get removed when trying to select specific tests methods.
+    // For example, gradle will report that no test were found, and not report the actual error.
+  }
+
   @Override
   public SpockExecutionContext prepare(SpockExecutionContext context) throws Exception {
     return ExceptionUtil.sneakyThrow(error);

spock-specs/src/test/groovy/org/spockframework/runtime/ErrorSpecNodeSpec.groovy

@@ -3,22 +3,34 @@ package org.spockframework.runtime
 import spock.lang.Specification
 
 import org.junit.platform.engine.*
+import spock.lang.Subject
 
 class ErrorSpecNodeSpec extends Specification {
+  TestDescriptor tdParent = Mock()
+
+  @Subject
+  def errorSpecNode = new ErrorSpecNode(
+    UniqueId.forEngine("test"),
+    null,
+    new SpecInfoBuilder(getClass()).build(),
+    null).tap {
+    parent = tdParent
+  }
+
+
   def 'should not be pruned'() {
-    given:
-    TestDescriptor parent = Mock()
-    def testee = new ErrorSpecNode(
-      UniqueId.forEngine("test"),
-      null,
-      new SpecInfoBuilder(getClass()).build(),
-      null)
-    testee.setParent(parent)
+    when:
+    errorSpecNode.prune()
+
+    then:
+    0 * tdParent.removeChild(*_)
+  }
 
+  def 'should prevent removal of ErrorSpecNode'() {
     when:
-    testee.prune()
+    errorSpecNode.removeFromHierarchy()
 
     then:
-    0 * parent.removeChild(*_)
+    0 * tdParent.removeChild(*_)
   }
 }