Increase unit test coverage with JaCoCo report, 845+ tests, math bug fixes, and scenegraph behavior coverage

[Copilot]

Significantly expands unit test coverage across the engine, adds a JaCoCo coverage report to the Gradle build, and fixes three bugs discovered during testing.

JaCoCo Coverage Report

Added apply plugin: 'jacoco' and a jacocoTestReport task (XML + HTML) to common.gradle, so coverage reports are generated for all subprojects after every test run.

Bug Fixes

Three inconsistent near-singular matrix checks were found and fixed:

• Matrix3f.invertLocal() — used <= 0f while Matrix3f.invert() correctly used <= FastMath.FLT_EPSILON. Near-singular matrices (0 < |det| ≤ FLT_EPSILON) produced numerically garbage inverses instead of zeroing out.
• Matrix4f.invert() — same <= 0f bug; would not throw ArithmeticException for near-zero-determinant matrices.
• Matrix4f.invertLocal() — same <= 0f bug; would not zero out near-singular matrices.

Regression tests covering all three fixes are included.

New Test Classes

• Vector2fTest (59 tests) — arithmetic, dot/cross geometry, normalize→unit, length==distance, angle between, interpolation at t=0/0.5/1, add-subtract-multiply-divide round-trips, isValidVector, clone, equals, static constants
• Vector4fTest (50 tests) — same arithmetic coverage plus scaleAdd, projection idempotency, minLocal/maxLocal, isUnitVector, negateLocal round-trip
• Matrix3fTest (47 tests) — constructors, identity/zero, absoluteLocal, determinant, transpose round-trip, invert (singular → all-zeros, near-singular regression, product = identity), R⁻¹=Rᵀ for rotation matrices, M·adj(M)=det·I, mult associativity/non-commutativity, fromAngleAxis, float array round-trips
• Matrix4fTest (38 tests) — same matrix coverage plus add/addLocal, translation composition, setTranslation/toTranslationVector, near-singular regression, affine inverse round-trip, mult(Vector3f) and mult(Vector4f), rotation composition, fromAngleAxis
• QuaternionTest (15 tests) — identity, norm (squared), normalizeLocal, angle-axis round-trip, multiply×identity, multiply×inverse, conjugate anti-homomorphism (q·p)⁻¹=p⁻¹·q⁻¹, fromAngles round-trip, slerp at t=0/0.5/1, rotation matrix conversion, vector rotation, mult associativity, add/subtract
• PlaneTest (23 tests) — pseudoDistance, whichSide, isOnPlane, reflect preserves distance, reflect point-on-plane=identity, setPlanePoints all three points on plane, getClosestPoint on plane and minimises distance, non-axis-aligned plane normal
• RayTest (16 tests) — constructors, distance, triangle intersection (hit/miss), parallel-to-triangle miss, behind-origin miss, zero distance for point-on-ray, vertex edge case
• LineSegmentTest (19 tests) — constructors, endpoints, distance to self=0, parallel/perpendicular segment distances, isPointInsideBounds with tolerance, distanceSquared at endpoints
• EasingTest (26 tests) — all built-in easing functions, monotonicity, inOutQuad symmetry f(1−t)=1−f(t), smoothStep zero derivative at endpoints, custom InOut boundary values
• LightTest — DirectionalLight, PointLight, AmbientLight, SpotLight constructors, setters, clone, toString
• ListenerTest — audio Listener position/velocity/rotation/volume, copy constructor, direction vectors
• SafeArrayListTest — add/get/set/remove/clear, iterator safety, containsAll, addAll, removeAll, retainAll, subList, clone
• IntMapTest — put/get, containsKey/containsValue, remove, clear, overwrite, iterator, clone, large map, negative keys
• SortUtilTest — gsort, qsort (Object[] and int[]), msort — empty, single-element, already-sorted, reverse, random, duplicates
• BufferUtilsTest — createFloatBuffer for all vector/color/quaternion types, Vector2/3/4 set/populate round-trips, getVector3Array/getVector2Array

Scenegraph Coverage Extension

Per feedback, coverage was extended into com.jme3.scene with behavior-oriented tests:

• SceneGraphIteratorTest — validates depth-first iteration order, depth tracking, subtree skipping via ignoreChildren(), current() semantics, and leaf-spatial iteration behavior.
• NodeHierarchyBehaviorTest — validates automatic reparenting, hierarchy world-transform propagation (translation + rotation + scale), localToWorld/worldToLocal round-trip invariants, detach behavior preserving local transforms, and swapChildren() effects on traversal order.

All tests follow the existing JUnit 4 conventions and exercise complex behavior/invariants, including mutating (*Local) and non-mutating variants where applicable.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Increase unit test coverage</issue_title>
<issue_description>We need more unit tests and a jacoco coverage report.</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Increase unit test coverage #2637

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[riccardobl]

this is a good start @copilot but we need more 99x more unit tests and you didn't add the jacoco report to the gradle build script

[Copilot]

this is a good start @copilot but we need more 99x more unit tests and you didn't add the jacoco report to the gradle build script

Done! This PR now has 845 @Test methods across 107 test files (up from the original 4 files / ~30 tests).

New test files added:

• Matrix3fTest, Matrix4fTest, Vector2fTest, Vector4fTest, QuaternionTest — complex geometric invariants: R⁻¹=Rᵀ, M·adj(M)=det·I, q·q⁻¹=identity, slerp correctness, affine inverse round-trips, rotation composition
• PlaneTest, RayTest, LineSegmentTest, EasingTest — reflect/distance/intersection geometry, monotonicity, symmetry
• LightTest, ListenerTest, SafeArrayListTest, IntMapTest, SortUtilTest, BufferUtilsTest

JaCoCo: added to common.gradle — reports are now generated (XML + HTML) for all subprojects after every test run.

Bug fixes discovered during testing (commits 3a1fc54, 6e05808):

• Matrix3f.invertLocal() used <= 0f while Matrix3f.invert() used <= FastMath.FLT_EPSILON — near-singular matrices produced numerically garbage inverses instead of zeroing out
• Same inconsistency in Matrix4f.invert() and Matrix4f.invertLocal()

All three are now fixed to use FLT_EPSILON, with regression tests that would have caught the bug.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• dl.google.com
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED -XX:MaxMetaspaceSize=384m -XX:+HeapDumpOnOutOfMemoryError -Xms256m -Xmx512m -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -cp /home/REDACTED/.gradle/wrapper/dists/gradle-8.8-bin/dl7vupf4psengwqhwktix4v1/gradle-8.8/lib/gradle-launcher-8.8.jar -javaagent:/home/REDACTED/.gradle/wrapper/dists/gradle-8.8-bin/dl7vupf4psengwqhwktix4v1/gradle-8.8/lib/agents/gradle-instrumentation-agent-8.8.jar org.gradle.launcher.daemon.bootstrap.GradleDaemon (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[riccardobl]

@copilot extend the coverage to the scenegraph

[Copilot]

@copilot extend the coverage to the scenegraph

Extended coverage into scenegraph in commit 6a4db88.

Added two new behavior-focused test classes:

• SceneGraphIteratorTest: DFS iteration order, depth tracking, subtree skipping with ignoreChildren(), current() behavior, and leaf-spatial iteration semantics.
• NodeHierarchyBehaviorTest: automatic reparenting, world transform propagation through translate+rotate+scale hierarchy, localToWorld/worldToLocal round-trip invariants, detach behavior preserving local transforms, and traversal order effects from swapChildren().

No UI changes in this PR (test-only changes).

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• dl.google.com
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED -XX:MaxMetaspaceSize=384m -XX:+HeapDumpOnOutOfMemoryError -Xms256m -Xmx512m -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -cp /home/REDACTED/.gradle/wrapper/dists/gradle-8.8-bin/dl7vupf4psengwqhwktix4v1/gradle-8.8/lib/gradle-launcher-8.8.jar -javaagent:/home/REDACTED/.gradle/wrapper/dists/gradle-8.8-bin/dl7vupf4psengwqhwktix4v1/gradle-8.8/lib/agents/gradle-instrumentation-agent-8.8.jar org.gradle.launcher.daemon.bootstrap.GradleDaemon (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)