gradle-extensions-plugin: fix output of exceptions with multiple nested suppressed exceptions

Author: vlsi

README.md

@@ -191,6 +191,7 @@ Change log
 ----------
 v1.72
 * jandex-plugin: add `maxErrors` option to show several errors rather than stop on the first one
+* gradle-extensions-plugin: fix output of exceptions with multiple nested suppressed exceptions
 
 v1.71
 * Add CI jobs with nightly and RC Gradle versions

plugins/gradle-extensions-plugin/src/main/kotlin/com/github/vlsi/gradle/ThrowablePrinter.kt

@@ -207,27 +207,31 @@ class ThrowablePrinter {
             }
 
             val nextIndent = if (hideThrowable) indent else "    $indent"
-            throwable.suppressed.forEach {
-                queue += Work(it, nextIndent, "Suppressed: ", ourStack)
-            }
 
             fun addNextExceptions(prefix: String, causes: Iterable<Throwable>) {
                 val skipMessage = causes is List<*> && causes.size == 1
+                val errors = mutableListOf<Work>()
                 for ((i, cause) in causes.withIndex()) {
                     val causeIndex = if (skipMessage) "" else "$prefix ${i + 1}: "
-                    queue += Work(cause, nextIndent, causeIndex, ourStack)
+                    errors += Work(cause, nextIndent, causeIndex, ourStack)
+                }
+                for(work in errors.asReversed()) {
+                    queue.addFirst(work)
                 }
             }
             if (throwable is MultiCauseException) {
                 addNextExceptions("Cause", throwable.causes)
             } else {
                 throwable.cause?.let {
-                    queue += Work(it, nextIndent, "Caused by: ", ourStack)
+                    queue.addFirst(Work(it, nextIndent, "Caused by: ", ourStack))
                 }
             }
             if (throwable is SQLException) {
                 addNextExceptions("Next exception", Iterable { throwable.iterator() })
             }
+            throwable.suppressed.asList().asReversed().forEach {
+                queue.addFirst(Work(it, nextIndent, "Suppressed: ", ourStack))
+            }
             val lastFrame = if (hideStacktrace || ourStack?.isEmpty() == null) {
                 continue
             } else if (causedBy?.isNotEmpty() == true) {