Repository: kafka Updated Branches: refs/heads/trunk f5596a851 -> 6b1a92c89
KAFKA-2587; Increasing timeout for the test verification. Author: Parth Brahmbhatt <[email protected]> Reviewers: Ismael Juma <[email protected]>, Sriharsha Chintalapani <[email protected]> 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 <[email protected]> Authored: Tue Sep 29 15:17:09 2015 -0700 Committer: Sriharsha Chintalapani <[email protected]> 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 }
