[Pcgen-github] [PCGen/pcgen] 4fa2b8: Build/test cleanup: drop duplicated test JVM args, ...

[Vest <no...@gi...>]

  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