|
From: Vest <no...@gi...> - 2026-06-06 20:54:02
|
Branch: refs/heads/master Home: https://github.com/PCGen/pcgen Commit: 4fa2b89165e0dec4b8578fe5545382fd5ceb8128 https://github.com/PCGen/pcgen/commit/4fa2b89165e0dec4b8578fe5545382fd5ceb8128 Author: Vest <Ve...@us...> Date: 2026-06-07 (Sun, 07 Jun 2026) Changed paths: M build.gradle M code/src/test/plugin/PluginBuildTest.java Log Message: ----------- Build/test cleanup: drop duplicated test JVM args, refresh stale Javadoc (#7587) * build.gradle: drop duplicated test-task JVM args tasks.named("test", Test) re-added six args that tasks.withType(Test).configureEach already applies to every Test: -Dtestfx.robot=glass, -Dtestfx.headless=true, -Dprism.order=sw, --add-exports for javafx.graphics/com.sun.javafx.util and javafx.base/com.sun.javafx.logging, and --add-opens for javafx.graphics/com.sun.glass.ui. Each was landing on the test JVM command line twice. Kept --add-opens javafx.graphics/com.sun.javafx.application: withType has --add-exports (not --add-opens) for that package, and TestFX's @AfterEach cleanup uses setAccessible(true) on a private field of ParametersImpl, which exports alone don't satisfy. Verified by running pcgen.gui3.dialog.AboutDialogTest after the cleanup. * PluginBuildTest: refresh stale class Javadoc The class Javadoc still referenced the deleted Ant pluginbuild.xml. Now describes what the test actually does today: every plugin source file under code/src/java/plugin/<category>/ is packaged into the matching <category>plugins.jar produced by code/gradle/plugins.gradle. To unsubscribe from these emails, change your notification settings at https://github.com/PCGen/pcgen/settings/notifications |