You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@kafka.apache.org by "Andrii Biletskyi (JIRA)" <ji...@apache.org> on 2015/03/22 18:46:11 UTC

[jira] [Commented] (KAFKA-2021) Consolidate test classes for KafkaConfig

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

Andrii Biletskyi commented on KAFKA-2021:
-----------------------------------------

Created reviewboard https://reviews.apache.org/r/32376/diff/
 against branch origin/trunk

> Consolidate test classes for KafkaConfig
> ----------------------------------------
>
>                 Key: KAFKA-2021
>                 URL: https://issues.apache.org/jira/browse/KAFKA-2021
>             Project: Kafka
>          Issue Type: Task
>            Reporter: Gwen Shapira
>            Assignee: Andrii Biletskyi
>            Priority: Minor
>         Attachments: KAFKA-2021.patch
>
>
> We have kafka.server.KafkaConfigTest, KafkaConfigConfigDefTest and kafka.unit.KafkaTest (in a file called KafkaConfigTest.scala)
> I think consolidating them into one test class (or at list renaming so it will be clear how they are different) will make a lot of sense.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)