Repository: kafka
Updated Branches:
  refs/heads/trunk 5aedde5b2 -> ac434a4eb


MINOR: Fix checkstyle failure in `StreamsConfigTest`

I removed the hamcrest matcher to unbreak the build, but we probably want to 
tweak the `import-control.xml` as it currently only allows it for `<subpackage 
name="integration">`, which is weird.

Author: Ismael Juma <[email protected]>

Reviewers: Guozhang Wang <[email protected]>

Closes #1380 from ijuma/fix-streams-config-test-checkstyle


Project: http://git-wip-us.apache.org/repos/asf/kafka/repo
Commit: http://git-wip-us.apache.org/repos/asf/kafka/commit/ac434a4e
Tree: http://git-wip-us.apache.org/repos/asf/kafka/tree/ac434a4e
Diff: http://git-wip-us.apache.org/repos/asf/kafka/diff/ac434a4e

Branch: refs/heads/trunk
Commit: ac434a4ebdc36338ff707c38e189f9d1668ff423
Parents: 5aedde5
Author: Ismael Juma <[email protected]>
Authored: Thu May 12 20:36:47 2016 +0100
Committer: Ismael Juma <[email protected]>
Committed: Thu May 12 20:36:47 2016 +0100

----------------------------------------------------------------------
 .../test/java/org/apache/kafka/streams/StreamsConfigTest.java    | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kafka/blob/ac434a4e/streams/src/test/java/org/apache/kafka/streams/StreamsConfigTest.java
----------------------------------------------------------------------
diff --git 
a/streams/src/test/java/org/apache/kafka/streams/StreamsConfigTest.java 
b/streams/src/test/java/org/apache/kafka/streams/StreamsConfigTest.java
index d7def70..17d6b4b 100644
--- a/streams/src/test/java/org/apache/kafka/streams/StreamsConfigTest.java
+++ b/streams/src/test/java/org/apache/kafka/streams/StreamsConfigTest.java
@@ -30,10 +30,8 @@ import java.util.List;
 import java.util.Map;
 import java.util.Properties;
 
-import static org.hamcrest.CoreMatchers.equalTo;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertThat;
 
 public class StreamsConfigTest {
 
@@ -101,7 +99,7 @@ public class StreamsConfigTest {
         StreamsConfig config = new StreamsConfig(props);
 
         List<String> actualBootstrapServers = 
config.getList(StreamsConfig.BOOTSTRAP_SERVERS_CONFIG);
-        assertThat(actualBootstrapServers, equalTo(expectedBootstrapServers));
+        assertEquals(expectedBootstrapServers, actualBootstrapServers);
     }
 
 }

Reply via email to