More logging to try to diagnose SIGSEGV issues

Author: kcooney

buildSrc/src/main/groovy/java-common-conventions.gradle

@@ -1,3 +1,6 @@
+import org.gradle.api.tasks.testing.logging.TestExceptionFormat
+import org.gradle.api.tasks.testing.logging.TestLogEvent
+
 plugins {
     // Apply the java-library plugin for API and implementation separation.
     id 'java-library'
@@ -9,11 +12,85 @@ java {
     targetCompatibility = JavaVersion.VERSION_17
 }
 
-tasks.withType(JavaCompile) {
+tasks.withType(JavaCompile).configureEach {
     // Configure string concat to always inline compile
     options.compilerArgs.add '-XDstringConcat=inline'
 }
 
+project.ext {
+    runningTestDescriptions = [] as Set
+}
+
+tasks.withType(Test).configureEach {
+
+    testLogging {
+        // set options for log level LIFECYCLE
+        events TestLogEvent.FAILED,
+                TestLogEvent.PASSED,
+                TestLogEvent.SKIPPED,
+                TestLogEvent.STANDARD_OUT
+        exceptionFormat TestExceptionFormat.FULL
+        showExceptions true
+        showCauses true
+        showStackTraces true
+
+        // set options for log level DEBUG and INFO
+        debug {
+            events TestLogEvent.STARTED,
+                    TestLogEvent.FAILED,
+                    TestLogEvent.PASSED,
+                    TestLogEvent.SKIPPED,
+                    TestLogEvent.STANDARD_ERROR,
+                    TestLogEvent.STANDARD_OUT
+            exceptionFormat TestExceptionFormat.FULL
+        }
+        info.events = debug.events
+        info.exceptionFormat = debug.exceptionFormat
+
+        beforeTest { desc ->
+            project.ext.runningTestDescriptions.add(desc.displayName)
+        }
+
+        afterTest { desc, result ->
+            project.ext.runningTestDescriptions.remove(desc.displayName )
+        }
+
+        afterSuite { desc, result ->
+            if (!desc.parent) {  // will match the outermost suite
+                def name = desc.name
+                def idx = name.indexOf(":")
+                if (idx >= 0) {
+                    name = name.substring(idx)
+                }
+
+                def rows = [
+                        "Results for ${name} - ${result.resultType} (${result.testCount} tests, ${result.successfulTestCount} passed, ${result.failedTestCount} failed, ${result.skippedTestCount} skipped)"
+                ]
+                def running = project.ext.runningTestDescriptions
+                if (!running.isEmpty()) {
+                    rows.add("Interrupted tests: ${running}")
+                }
+                def maxContentLen = rows.stream()
+                        .map { it.length() }
+                        .max(Comparator.naturalOrder())
+                        .orElse(0)
+                def startItem = '│  ', endItem = '  │'
+                def repeatLength = startItem.length() + maxContentLen + endItem.length() - 2
+
+                def printRow = { String row ->
+                    println(startItem + row + (' ' * (maxContentLen - row.length())) + endItem)
+                }
+
+                println('\n┌' + ('─' * repeatLength) + '┐')
+                for (row in rows) {
+                    printRow row
+                }
+                println('└' + ('─' * repeatLength) + '┘\n')
+            }
+        }
+    }
+}
+
 tasks.named('jar') {
     manifest {
         attributes(

gradle.properties

@@ -1 +1 @@
-org.gradle.logging.level=debug
+org.gradle.logging.level=info

lib/build.gradle

@@ -51,16 +51,6 @@ tasks.named('test') {
     systemProperty 'junit.jupiter.extensions.autodetection.enabled', 'true'
 }
 
-test {
-    testLogging {
-        debug {
-            showStandardStreams = true
-            exceptionFormat = "full"
-            displayGranularity = -1
-        }
-    }
-}
-
 javadoc {
     options.tags = [
         // Converts `@apiNote` paragraph to "Note: " section in the generated javadoc

lib/src/test/java/com/team2813/lib2813/preferences/PersistedConfigurationTest.java

@@ -5,7 +5,6 @@
 import static java.util.stream.Collectors.toMap;
 import static org.junit.Assert.assertThrows;
 
-import com.team2813.lib2813.util.PrintingWatcher;
 import edu.wpi.first.networktables.NetworkTable;
 import edu.wpi.first.networktables.NetworkTableEntry;
 import edu.wpi.first.networktables.NetworkTableInstance;
@@ -19,12 +18,10 @@
 import java.util.function.*;
 import org.junit.After;
 import org.junit.Before;
-import org.junit.ClassRule;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.experimental.runners.Enclosed;
 import org.junit.rules.ErrorCollector;
-import org.junit.rules.TestWatcher;
 import org.junit.runner.RunWith;
 import org.junit.runners.Parameterized;
 import org.junit.runners.Parameterized.Parameters;
@@ -35,13 +32,12 @@ public final class PersistedConfigurationTest {
   private static final double EPSILON = 0.001;
 
   /** Base class for all nested classes of {@link PersistedConfigurationTest}. */
-  public abstract static class PreferencesRegistryTestCase<T extends Record> {
+  abstract static class PreferencesRegistryTestCase<T extends Record> {
     private final String preferenceName;
     private final Class<T> recordClass;
 
     @Rule public final IsolatedPreferences isolatedPreferences = new IsolatedPreferences();
     @Rule public final ErrorCollector errorCollector = new ErrorCollector();
-    @ClassRule @Rule public static final TestWatcher printingWatcher = new PrintingWatcher();
 
     protected PreferencesRegistryTestCase(String preferenceName, Class<T> recordClass) {
       this.preferenceName = preferenceName;

lib/src/test/java/com/team2813/lib2813/util/InputValidationTest.java

@@ -4,16 +4,11 @@
 import static com.google.common.truth.Truth.assertWithMessage;
 import static org.junit.Assert.assertThrows;
 
-import org.junit.ClassRule;
-import org.junit.Rule;
 import org.junit.Test;
-import org.junit.rules.TestWatcher;
 
 public class InputValidationTest {
   // Tests for the `InputValidation.checkCanId(...)` method.
   public static class CheckCanIdTest {
-    @ClassRule @Rule public static final TestWatcher printingWatcher = new PrintingWatcher();
-
     @Test
     public void invalidCanId() {
       // Can IDs can only valid in the range [0, 62].