[ 
https://issues.apache.org/jira/browse/KAFKA-19097?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17941661#comment-17941661
 ] 

Matthias J. Sax commented on KAFKA-19097:
-----------------------------------------

There is actually many tests with this issue. Would be great to clean it up. We 
did this as best-effort side cleanup only so far... So if anybody want to check 
the test code across the board and fix, would be great (maybe not one single 
huge PR, but many smaller PRs to keep reviewing simple)

> Fix order of arguments to assertEquals
> --------------------------------------
>
>                 Key: KAFKA-19097
>                 URL: https://issues.apache.org/jira/browse/KAFKA-19097
>             Project: Kafka
>          Issue Type: Improvement
>          Components: streams, unit tests
>            Reporter: Lorcan
>            Assignee: HongYi Chen
>            Priority: Minor
>              Labels: beginner-friendly, newbie,
>
> There are examples of tests using assertEquals where the arguments are in the 
> wrong order.
> Example from InternalStreamsBuilderTest:
> assertEquals(count.get(), 3);
> The expected value should be the first argument.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to