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

ASF GitHub Bot commented on KAFKA-6104:
---------------------------------------

GitHub user soenkeliebau opened a pull request:

    https://github.com/apache/kafka/pull/4113

    KAFKA-6104: Added unit tests for ClusterConnectionStates.

    Added a few unit tests for ClusterConnectionStates.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/soenkeliebau/kafka KAFKA-6104

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/kafka/pull/4113.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #4113
    
----
commit c42986e2084e19173719e9e554a84c33a460f60f
Author: Soenke Liebau <soenke.lie...@opencore.com>
Date:   2017-10-22T20:16:25Z

    KAFKA-6104: Added unit tests for ClusterConnectionStates.

----


> Add unit tests for ClusterConnectionStates
> ------------------------------------------
>
>                 Key: KAFKA-6104
>                 URL: https://issues.apache.org/jira/browse/KAFKA-6104
>             Project: Kafka
>          Issue Type: Test
>          Components: unit tests
>    Affects Versions: 0.11.0.0
>            Reporter: Sönke Liebau
>            Assignee: Sönke Liebau
>            Priority: Trivial
>
> There are no unit tests for the ClusterConnectionStates object.
> I've created tests to:
> * Cycle through connection states and check correct properties during the 
> process
> * Check authentication exception is correctly stored
> * Check that max reconnect backoff is not exceeded during reconnects
> * Check that removed connections are correctly reset
> There is currently no test that checks whether the reconnect times are 
> correctly increased, as that is still being fixed in KAFKA-6101 .



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to