fslev / jtest-utils / #945

94%

Build Type

push

Committed by web-flow

Commit Message

Big refactoring (#266)

* Bump version to 7.0-SNAPSHOT and Java baseline to 17

Raise the main-code Java release from 8 to 17, drop the now-redundant
testRelease 11, and bump the in-development version. Unlocks records,
sealed types, switch expressions, and pattern matching for the upcoming
refactor pass.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* Remove deprecated Polling API

The Polling class and the polling-based ObjectMatcher overloads were
deprecated in 5.14 (Dec 2023) with a recommendation to use Awaitility.
Drop them now as part of the 7.0 cut. Removes Polling, PollingTimeoutException,
the five deprecated polling overloads in ObjectMatcher, the private polling
helper, all polling-coupled tests, and the polling sections of the README
and _config.yml description.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* Convert PlainHttpResponse to a record

Reduces ~110 lines of class boilerplate to a 4-component record while
preserving the public Builder, ParseException, and the custom toString
that downstream tests assert on. The compact constructor takes a
defensive copy of the headers via List.copyOf, replacing the previous
unmodifiableList wrap that didn't actually copy.

Public API break (7.0): accessors getStatus()/getReasonPhrase()/
getEntity()/getHeaders() are now status()/reasonPhrase()/entity()/
headers(). HttpResponseMatcher updated to match.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* Refactor CustomXmlDiffEvaluator dispatch with switch + pattern matching

Replace the if-else-if chain over ComparisonType with a single switch,
and use pattern matching for instanceof to remove the redundant casts
on Attr and Text nodes. The expected-node null guard is split out from
the type-based short-circuit cases for clarity.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* Use switch + EnumSet for JsonMatcher.jsonCompareModes

Replace the HashSet + ... (continued)

Coverage Stats

129 of 133 new or added lines in 8 files covered. (96.99%)

2 existing lines in 1 file now uncovered.

509 of 544 relevant lines covered (93.57%)

0.94 hits per line