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

Ksolves India Limited commented on KAFKA-19097:
-----------------------------------------------

Hi [~kj455] ,

Sorry for the {*}late response{*}. Thank you for handling this one.

We are currently *actively reviewing* other packages within Kafka for which PRs 
{*}have not been created yet{*}.
 * For the {*}merged/resolved tickets{*}, please *create a PR* and update the 
resolved tickets with the text "{*}Part 2{*}."

 * For the {*}non-merged PRs/tickets{*}, could you please *review the 
subpackage* and update us if we {*}missed any{*}?

> 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: Ksolves India Limited
>            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