RFR: 8340493: Fix some Asserts failure messages [v2]
Weijun Wang
weijun at openjdk.org
Tue Dec 17 15:07:27 UTC 2024
On Tue, 17 Dec 2024 12:57:08 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:
>> Are those calls from me? I know I haven't followed this pattern and I'm thinking about fixing them later.
>>
>> The reason I want to make this change is to make it consistent with the current `assertEquals` method that shows " expected: LEFT but was: RIGHT". There are quite a lot of calls like `assertEquals(variable, "literal")`, but I think that's the callers' problem instead of the method's.
>
> Could you rename `lhs` and `rhs` to `expected` and `actual`? Just to avoid the problems mentioned in https://github.com/openjdk/jdk/pull/17952#issuecomment-1973131407
I added a new commit with the new names. Only methods I touched in this PR were updated.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21101#discussion_r1888668343
More information about the core-libs-dev
mailing list