Fix the rendering of non-assertion-error throwables in verifyEach

Author: Vampire

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

@@ -217,7 +217,7 @@ public static <T> void verifyEach(
   }
 
   private static <T> SpockAssertionError getAssertionFailedError(Function<? super T, ?> namer, InternalItemFailure<T> failure) {
-    SpockAssertionError error = new SpockAssertionError(String.format("Assertions failed for item[%d] %s:\n%s", failure.index, namer.apply(failure.item), failure.throwable.getMessage()));
+    SpockAssertionError error = new SpockAssertionError(String.format("Assertions failed for item[%d] %s:\n%s", failure.index, namer.apply(failure.item), failure.throwable.toString()));
     error.setStackTrace(failure.throwable.getStackTrace());
     return error;
   }

spock-specs/src/test/groovy/org/spockframework/smoke/VerifyEachBlocks.groovy

@@ -83,6 +83,44 @@ class VerifyEachBlocks extends EmbeddedSpecification {
     snap.assertThat(e.message).matchesSnapshot()
   }
 
+  def "verifyEach fails handles exception"(@Snapshot Snapshotter snap) {
+    given:
+    def range = (1..10)
+
+    when:
+    verifyEach(range) {
+      if (it == 5) {
+        throw new RuntimeException("x == $it").tap {
+          // remove stack trace to make snapshot stable
+          it.stackTrace = []
+        }
+      }
+    }
+
+    then:
+    SpockAssertionError e = thrown()
+    snap.assertThat(e.message).matchesSnapshot()
+  }
+
+  def "verifyEach fails handles exceptions"(@Snapshot Snapshotter snap) {
+    given:
+    def range = (1..10)
+
+    when:
+    verifyEach(range) {
+      if (it in (5..6)) {
+        throw new RuntimeException("x == $it").tap {
+          // remove stack trace to make snapshot stable
+          it.stackTrace = []
+        }
+      }
+    }
+
+    then:
+    MultipleFailuresError e = thrown()
+    snap.assertThat(e.message).matchesSnapshot()
+  }
+
   def "verifyEach fails handles nested exceptions"(@Snapshot Snapshotter snap) {
     given:
     def range = (1..10)

spock-specs/src/test/resources/snapshots/org/spockframework/smoke/VerifyEachBlocks/verifyEach_fails_handles_exception.txt

@@ -0,0 +1,2 @@
+Assertions failed for item[4] 5:
+java.lang.RuntimeException: x == 5

spock-specs/src/test/resources/snapshots/org/spockframework/smoke/VerifyEachBlocks/verifyEach_fails_handles_exceptions.txt

@@ -0,0 +1,5 @@
+Multiple Failures (2 failures)
+	org.spockframework.runtime.SpockAssertionError: Assertions failed for item[4] 5:
+java.lang.RuntimeException: x == 5
+	org.spockframework.runtime.SpockAssertionError: Assertions failed for item[5] 6:
+java.lang.RuntimeException: x == 6