Skip to content

[codex] Support Java 11 build and test cleanup#381

Draft
NirBY wants to merge 1 commit into
Top-Q:masterfrom
NirBY:Java11
Draft

[codex] Support Java 11 build and test cleanup#381
NirBY wants to merge 1 commit into
Top-Q:masterfrom
NirBY:Java11

Conversation

@NirBY
Copy link
Copy Markdown

@NirBY NirBY commented May 8, 2026

What changed

This PR collects the local Java 11 migration and test-suite cleanup work into a single clean commit based on Top-Q/jsystem:master.

Build and dependency updates

  • Set Maven compiler source/target to Java 11 in the parent build.
  • Updated module and archetype POM versions/references to the Java 11 branch version line.
  • Updated selected dependency coordinates/versions needed for the Java 11 build, including Commons Net and Difido/Jackson-related changes.
  • Removed the obsolete explicit Maven deploy plugin configuration.
  • Added explicit Surefire 3.2.5 plugin declarations where module-level test filtering was introduced.

Test-suite cleanup

  • Added/adjusted skips for tests that need external services, live SUT/device files, an Agent/RMI runtime, mail services, or runner-specific context.
  • Added a bug-triage note in BUG_REPORT_ISSUE.md documenting the failing Maven test areas and why some tests need isolation or skips.
  • Added META-INF/jsystemCore.build.properties test/build metadata and report8.html test resource needed by the adjusted tests.
  • Hardened utility/reporting code used during Maven test execution, including null guards and Java 11 compatibility fixes.

Runtime/source changes

  • Added TsvDataProvider for tab-separated data-driven scenario input.
  • Updated data-driven task registration so TSV can be discovered/used.
  • Updated Difido/reporting code for the adjusted dependency/API behavior.
  • Updated runner batch scripts and Java version handling for Java 11.
  • Added the runner icon resource.

UI and misc compatibility updates

  • Adjusted several Tree UI classes for Java 11 compatibility and type/API cleanup.
  • Updated .gitignore to exclude VS Code workspace files and rejected patch artifacts (*.rej) so IDE/temp files are not included.

Security and privacy check

  • Rebuilt the PR as a single clean commit from upstream master; the earlier local history containing internal Maven repository config is not part of this PR branch.
  • Removed internal Nexus repository/pluginRepository entries from the final branch.
  • Scanned the final diff and branch commit for passwords, tokens, private keys, internal Nexus hostnames, local C:\Users paths, and personal identifiers. No such data was found.
  • The only added URLs are public Top-Q Maven repository references already aligned with the project and a StackOverflow reference in an existing comment.

Validation

  • Pulled latest Top-Q/jsystem:master and latest fork branch before rebuilding the PR branch.
  • Ran git diff --check github/master...HEAD successfully.
  • Ran full build/test verification successfully:
    • mvn -f jsystem-parent\pom.xml test

Notes: validation ran on this workstation with Maven 3.9.9 and Java 21 while compiling with target/source 11. The build completed with exit code 0; existing deprecation/runtime warnings are still printed by legacy tests/reporting code, but the test run finishes successfully.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants