Skip to content

Y20240122-1000

@HeikoKlare HeikoKlare tagged this 22 Jan 13:39
In order to reduce/avoid usage of ambiguous assertArrayEquals (hard to
distinguish expected/actual) and prepare for a migration to JUnit 5 with
swapped expected/actual parameters in the assertEquals signature, this
change replaces all calls of assertArrayEquals with calls:
* Replace assertArrayEquals(MESSAGE, EXPECTED, ACTUAL) with
assertThat(MESSAGE, ACTUAL, is(EXPECTED)) and remove obsolete
message where possible
* Remove unnecessary array conversion when lists can be compared
* Remove unnecessary non-null assertions that are already covered by
subsequent array validations
* Fixes some assertions in which actual and expected were in wrong order
Assets 2
Loading