On Tue, 28 Apr 2026 11:21:35 GMT, Daniel Fuchs <[email protected]> wrote:
>> test/jdk/java/util/BitSet/stream/BitSetStreamTest.java line 124:
>>
>>> 122: for (int i = bs.nextSetBit(0); i >= 0; i = bs.nextSetBit(i +
>>> 1)) {
>>> 123: assertTrue(it.hasNext());
>>> 124: assertEquals(it.nextInt(), i);
>>
>> I believe the arguments need to be swapped here: it should be
>> `assertEquals(expected, actual)`
>
> Could you double check whether other places where assertEquals is called need
> their arguments to be swapped?
Thank you for the review.
I went through all `assertEquals` calls manually and updated them where it was
needed.
I ran the tier1 and tier2 tests and they passed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30954#discussion_r3161406692
