You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by je...@apache.org on 2018/06/20 14:52:12 UTC
[geode] branch develop updated: Fix test failure related to #2067
This is an automated email from the ASF dual-hosted git repository.
jensdeppe pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git
The following commit(s) were added to refs/heads/develop by this push:
new 7d7a651 Fix test failure related to #2067
7d7a651 is described below
commit 7d7a65169e2123168efacf089996fd45a2e35b00
Author: Jens Deppe <jd...@pivotal.io>
AuthorDate: Wed Jun 20 07:50:14 2018 -0700
Fix test failure related to #2067
---
.../test/java/org/apache/geode/security/ClientAuthDUnitTest.java | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/geode-core/src/test/java/org/apache/geode/security/ClientAuthDUnitTest.java b/geode-core/src/test/java/org/apache/geode/security/ClientAuthDUnitTest.java
index a50daf0..ae0313e 100644
--- a/geode-core/src/test/java/org/apache/geode/security/ClientAuthDUnitTest.java
+++ b/geode-core/src/test/java/org/apache/geode/security/ClientAuthDUnitTest.java
@@ -18,8 +18,6 @@ import static org.apache.geode.distributed.ConfigurationProperties.SECURITY_MANA
import static org.apache.geode.test.dunit.Disconnect.disconnectAllFromDS;
import static org.assertj.core.api.AssertionsForClassTypes.assertThatThrownBy;
-import java.io.Serializable;
-
import org.junit.After;
import org.junit.Rule;
import org.junit.Test;
@@ -29,6 +27,7 @@ import org.apache.geode.cache.client.ClientCache;
import org.apache.geode.cache.client.ClientRegionFactory;
import org.apache.geode.cache.client.ClientRegionShortcut;
import org.apache.geode.test.dunit.IgnoredException;
+import org.apache.geode.test.dunit.SerializableRunnableIF;
import org.apache.geode.test.dunit.rules.ClusterStartupRule;
import org.apache.geode.test.junit.categories.DistributedTest;
import org.apache.geode.test.junit.categories.FlakyTest;
@@ -53,7 +52,7 @@ public class ClientAuthDUnitTest {
@Test
public void authWithCorrectPasswordShouldPass() throws Exception {
lsRule.startClientVM(0, "test", "test", true, server.getPort(),
- () -> new ClientCacheHook(lsRule));
+ new ClientCacheHook(lsRule));
}
@Test
@@ -61,11 +60,11 @@ public class ClientAuthDUnitTest {
IgnoredException.addIgnoredException(AuthenticationFailedException.class.getName());
assertThatThrownBy(() -> lsRule.startClientVM(0, "test", "invalidPassword", true,
- server.getPort(), () -> new ClientCacheHook(lsRule)))
+ server.getPort(), new ClientCacheHook(lsRule)))
.isInstanceOf(AuthenticationFailedException.class);
}
- static class ClientCacheHook implements Runnable, Serializable {
+ static class ClientCacheHook implements SerializableRunnableIF {
final ClusterStartupRule lsRule;
ClientCacheHook(ClusterStartupRule lsRule) {