You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by sr...@apache.org on 2015/09/30 00:17:16 UTC
kafka git commit: KAFKA-2587;
Increasing timeout for the test verification.
Repository: kafka
Updated Branches:
refs/heads/trunk f5596a851 -> 6b1a92c89
KAFKA-2587; Increasing timeout for the test verification.
Author: Parth Brahmbhatt <br...@gmail.com>
Reviewers: Ismael Juma <is...@juma.me.uk>, Sriharsha Chintalapani <ha...@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 <br...@gmail.com>
Authored: Tue Sep 29 15:17:09 2015 -0700
Committer: Sriharsha Chintalapani <ha...@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
}