On Fri, 6 Mar 2026 09:21:56 GMT, Raffaello Giulietti <[email protected]> 
wrote:

> A migration from TestNG to JUnit

test/jdk/java/util/Arrays/largeMemory/ParallelPrefix.java line 160:

> 158:         int[] parallelRangeResult = Arrays.copyOfRange(data, fromIndex, 
> toIndex);
> 159:         Arrays.parallelPrefix(parallelRangeResult, op);
> 160:         assertArraysEqual(parallelRangeResult, 
> Arrays.copyOfRange(sequentialResult, fromIndex, toIndex));

With JUnit, the first parameter to assertArraysEqual is the "expected" as 
migration from TestNG will usually means transposing these parameters. It's 
really only an issue if there is failure of course, and only leads to a 
confusing message.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/30111#discussion_r2895212798

Reply via email to