kafka-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From srihar...@apache.org
Subject kafka git commit: KAFKA-2587; Increasing timeout for the test verification.
Date Tue, 29 Sep 2015 22:17:16 GMT
Repository: kafka
Updated Branches:
  refs/heads/trunk f5596a851 -> 6b1a92c89


KAFKA-2587; Increasing timeout for the test verification.

Author: Parth Brahmbhatt <brahmbhatt.parth@gmail.com>

Reviewers: Ismael Juma <ismael@juma.me.uk>, Sriharsha Chintalapani <harsha@hortonworks.com>

Closes #260 from Parth-Brahmbhatt/KAFKA-2587


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

Branch: refs/heads/trunk
Commit: 6b1a92c89fce443902b7ef85e1f4b7eaa9bd33c3
Parents: f5596a8
Author: Parth Brahmbhatt <brahmbhatt.parth@gmail.com>
Authored: Tue Sep 29 15:17:09 2015 -0700
Committer: Sriharsha Chintalapani <harsha@hortonworks.com>
Committed: Tue Sep 29 15:17:09 2015 -0700

----------------------------------------------------------------------
 .../unit/kafka/security/auth/SimpleAclAuthorizerTest.scala      | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kafka/blob/6b1a92c8/core/src/test/scala/unit/kafka/security/auth/SimpleAclAuthorizerTest.scala
----------------------------------------------------------------------
diff --git a/core/src/test/scala/unit/kafka/security/auth/SimpleAclAuthorizerTest.scala b/core/src/test/scala/unit/kafka/security/auth/SimpleAclAuthorizerTest.scala
index d3efc36..bd37ce2 100644
--- a/core/src/test/scala/unit/kafka/security/auth/SimpleAclAuthorizerTest.scala
+++ b/core/src/test/scala/unit/kafka/security/auth/SimpleAclAuthorizerTest.scala
@@ -188,7 +188,7 @@ class SimpleAclAuthorizerTest extends ZooKeeperTestHarness {
 
     //test removing last acl also deletes zookeeper path
     acls = changeAclAndVerify(Set.empty[Acl], Set(acl1), Set.empty[Acl])
-    changeAclAndVerify(Set.empty[Acl], Set.empty[Acl], acls)
+    changeAclAndVerify(acls, Set.empty[Acl], acls)
     assertTrue(!ZkUtils.pathExists(zkClient, simpleAclAuthorizer.toResourcePath(resource)))
   }
 
@@ -226,7 +226,8 @@ class SimpleAclAuthorizerTest extends ZooKeeperTestHarness {
       acls --=removedAcls
     }
 
-    TestUtils.waitUntilTrue(() => simpleAclAuthorizer.getAcls(resource) == acls, "changes
not propagated in timeout period.")
+    TestUtils.waitUntilTrue(() => simpleAclAuthorizer.getAcls(resource) == acls,
+      s"changes not propagated in timeout period. expected $acls but got ${simpleAclAuthorizer.getAcls(resource)}",
waitTime = 10000)
 
     acls
   }


Mime
View raw message