ci: fail hanging tests by name instead of holding the job

Nightly #302 and the PR check for #147 hung in :app-desktop:test for
over an hour, and the log named only the task. Thirteen reruns of the
same commit passed, so the hang is rare and still unidentified.

Every test now gets a two-minute JUnit timeout in a separate thread, so
even an uninterruptible hang fails with the test's name and the stack
where it stood. The test jobs are capped at 20 minutes for a hang
outside any test.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

.github/workflows/nightly-artifacts.yml

@@ -111,10 +111,8 @@ jobs:
             exit 1
           fi
 
-      # Also on cancelled(): the job timeout cancels a hanging run, and the reports of the
-      # classes that did finish carry the timed-out tests' full stacks.
       - name: Keep the test reports on a failure
-        if: failure() || cancelled()
+        if: failure()
         uses: actions/upload-artifact@v7
         with:
           name: test-results

.github/workflows/pr-compile-check.yml

@@ -286,10 +286,8 @@ jobs:
           if-no-files-found: ignore
           retention-days: 14
 
-      # Also on cancelled(): the job timeout cancels a hanging run, and the reports of the
-      # classes that did finish carry the timed-out tests' full stacks.
       - name: Keep the test reports on a failure
-        if: failure() || cancelled()
+        if: failure()
         uses: actions/upload-artifact@v7
         with:
           name: test-results

.github/workflows/tagged-release.yml

@@ -130,10 +130,8 @@ jobs:
             exit 1
           fi
 
-      # Also on cancelled(): the job timeout cancels a hanging run, and the reports of the
-      # classes that did finish carry the timed-out tests' full stacks.
       - name: Keep the test reports on a failure
-        if: failure() || cancelled()
+        if: failure()
         uses: actions/upload-artifact@v7
         with:
           name: test-results

build.gradle.kts

@@ -151,11 +151,6 @@ subprojects {
          */
         systemProperty("junit.jupiter.execution.timeout.default", "2m")
         systemProperty("junit.jupiter.execution.timeout.thread.mode.default", "SEPARATE_THREAD")
-        // The default short format prints only the first frame of each cause, which for a
-        // timed-out test is the line it hung in but not the call that blocked there.
-        testLogging {
-            exceptionFormat = org.gradle.api.tasks.testing.logging.TestExceptionFormat.FULL
-        }
         finalizedBy(checkTestUserHome)
     }
 

packaging/aur/kst4contest-git/PKGBUILD

@@ -7,22 +7,12 @@ arch=('x86_64')
 url="https://github.com/HamRadioOnline/kst4contest"
 license=('GPL-3.0-only')
 depends=('gst-plugins-base' 'gst-plugins-good')
-makedepends=('git')
+makedepends=('java-environment=21' 'git')
 provides=('kst4contest')
 conflicts=('kst4contest' 'kst4contest-bin')
 source=("kst4contest::git+https://github.com/HamRadioOnline/kst4contest.git")
 sha256sums=('SKIP')
 
-# The launcher is built with Temurin, not with the system JDK. Arch's openjdk links
-# jpackage's launcher library against the shared libstdc++, and that build aborts
-# with "pure virtual method called" (SIGABRT, core dump) in a static destructor when
-# the application exits; Temurin links libstdc++ statically, like the CI packages.
-# The _x86_64 arrays are deliberate: aur-publish.yml rewrites the plain sha256sums=
-# line on every release and would drop a second entry there.
-_temurin=21.0.13+9
-source_x86_64=("https://github.com/adoptium/temurin21-binaries/releases/download/jdk-${_temurin/+/%2B}/OpenJDK21U-jdk_x64_linux_hotspot_${_temurin/+/_}.tar.gz")
-sha256sums_x86_64=('d7f5d5cf865a2902b70ff549b97586a7d3f234e016f041ab447ea740c7d66d1c')
-
 pkgver() {
     cd "${srcdir}/kst4contest"
     BASE=$(grep -m1 '^version=' gradle.properties \
@@ -33,7 +23,7 @@ pkgver() {
 build() {
     cd "${srcdir}/kst4contest"
 
-    export JAVA_HOME="${srcdir}/jdk-${_temurin}"
+    export JAVA_HOME=$(find /usr/lib/jvm -maxdepth 1 -name 'java-21-*' -type d | head -n 1)
     export PATH="${JAVA_HOME}/bin:${PATH}"
 
     # A Gradle home inside $srcdir keeps the build independent of whatever the
@@ -41,13 +31,7 @@ build() {
     # cache, every dependency added since the last local build broke the package.
     export GRADLE_USER_HOME="${srcdir}/gradle-home"
 
-    # The Java toolchain, and with it jpackage, would otherwise pick any JDK 21 it
-    # finds in /usr/lib/jvm, the system one included.
-    ./gradlew --no-daemon -S \
-        -Porg.gradle.java.installations.auto-detect=false \
-        -Porg.gradle.java.installations.auto-download=false \
-        -Porg.gradle.java.installations.paths="${JAVA_HOME}" \
-        :app-desktop:createDistributable
+    ./gradlew --no-daemon -S :app-desktop:createDistributable
 
     # package() reads dist/KST4Contest. Application name, icon and the JDK
     # module list come from app-desktop/build.gradle.kts.

packaging/aur/kst4contest/PKGBUILD

@@ -7,25 +7,16 @@ arch=('x86_64')
 url="https://github.com/HamRadioOnline/kst4contest"
 license=('GPL-3.0-only')
 depends=('gst-plugins-base' 'gst-plugins-good')
+makedepends=('java-environment=21')
 provides=('kst4contest')
 conflicts=('kst4contest-bin' 'kst4contest-git')
 source=("${pkgname}-${pkgver}.tar.gz::https://github.com/HamRadioOnline/kst4contest/archive/refs/tags/v${pkgver}.tar.gz")
 sha256sums=('e702df24a29e6f914a43934df0095bc4adbd00a9f02df3a632816c832272e914')
 
-# The launcher is built with Temurin, not with the system JDK. Arch's openjdk links
-# jpackage's launcher library against the shared libstdc++, and that build aborts
-# with "pure virtual method called" (SIGABRT, core dump) in a static destructor when
-# the application exits; Temurin links libstdc++ statically, like the CI packages.
-# The _x86_64 arrays are deliberate: aur-publish.yml rewrites the plain sha256sums=
-# line on every release and would drop a second entry there.
-_temurin=21.0.13+9
-source_x86_64=("https://github.com/adoptium/temurin21-binaries/releases/download/jdk-${_temurin/+/%2B}/OpenJDK21U-jdk_x64_linux_hotspot_${_temurin/+/_}.tar.gz")
-sha256sums_x86_64=('d7f5d5cf865a2902b70ff549b97586a7d3f234e016f041ab447ea740c7d66d1c')
-
 build() {
     cd "${srcdir}/kst4contest-${pkgver}"
 
-    export JAVA_HOME="${srcdir}/jdk-${_temurin}"
+    export JAVA_HOME=$(find /usr/lib/jvm -maxdepth 1 -name 'java-21-*' -type d | head -n 1)
     export PATH="${JAVA_HOME}/bin:${PATH}"
 
     # A Gradle home inside $srcdir keeps the build independent of whatever the
@@ -33,13 +24,7 @@ build() {
     # cache, every dependency added since the last local build broke the package.
     export GRADLE_USER_HOME="${srcdir}/gradle-home"
 
-    # The Java toolchain, and with it jpackage, would otherwise pick any JDK 21 it
-    # finds in /usr/lib/jvm, the system one included.
-    ./gradlew --no-daemon -S \
-        -Porg.gradle.java.installations.auto-detect=false \
-        -Porg.gradle.java.installations.auto-download=false \
-        -Porg.gradle.java.installations.paths="${JAVA_HOME}" \
-        :app-desktop:createDistributable
+    ./gradlew --no-daemon -S :app-desktop:createDistributable
 
     # package() reads dist/KST4Contest. Application name, icon and the JDK
     # module list come from app-desktop/build.gradle.kts, so this PKGBUILD no