This is an automated email from the ASF dual-hosted git repository.

jinmeiliao 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 465f2cd  GEODE-1683: temp trial to fix clientAuthorizationDUnitTest 
(#1112)
465f2cd is described below

commit 465f2cdd55232ecdab16ad7431ecd93217276474
Author: jinmeiliao <[email protected]>
AuthorDate: Fri Dec 1 08:57:31 2017 -0800

    GEODE-1683: temp trial to fix clientAuthorizationDUnitTest (#1112)
---
 .../security/ClientAuthorizationDUnitTest.java     | 32 +++++++++++++++-------
 1 file changed, 22 insertions(+), 10 deletions(-)

diff --git 
a/geode-core/src/test/java/org/apache/geode/security/ClientAuthorizationDUnitTest.java
 
b/geode-core/src/test/java/org/apache/geode/security/ClientAuthorizationDUnitTest.java
index 3275c87..a16600e 100644
--- 
a/geode-core/src/test/java/org/apache/geode/security/ClientAuthorizationDUnitTest.java
+++ 
b/geode-core/src/test/java/org/apache/geode/security/ClientAuthorizationDUnitTest.java
@@ -14,11 +14,25 @@
  */
 package org.apache.geode.security;
 
-import static org.apache.geode.internal.AvailablePort.*;
-import static org.apache.geode.security.SecurityTestUtils.*;
-import static org.apache.geode.test.dunit.Assert.*;
-import static org.apache.geode.test.dunit.IgnoredException.*;
-import static org.apache.geode.test.dunit.LogWriterUtils.*;
+import static org.apache.geode.internal.AvailablePort.SOCKET;
+import static org.apache.geode.internal.AvailablePort.getRandomAvailablePort;
+import static org.apache.geode.security.SecurityTestUtils.AUTHFAIL_EXCEPTION;
+import static org.apache.geode.security.SecurityTestUtils.NOTAUTHZ_EXCEPTION;
+import static org.apache.geode.security.SecurityTestUtils.NO_EXCEPTION;
+import static org.apache.geode.security.SecurityTestUtils.OTHER_EXCEPTION;
+import static org.apache.geode.security.SecurityTestUtils.closeCache;
+import static org.apache.geode.security.SecurityTestUtils.concatProperties;
+import static org.apache.geode.security.SecurityTestUtils.createCacheClient;
+import static 
org.apache.geode.security.SecurityTestUtils.createCacheClientWithDynamicRegion;
+import static org.apache.geode.security.SecurityTestUtils.doGets;
+import static org.apache.geode.security.SecurityTestUtils.doNGets;
+import static org.apache.geode.security.SecurityTestUtils.doNPuts;
+import static org.apache.geode.security.SecurityTestUtils.doPutAllP;
+import static org.apache.geode.security.SecurityTestUtils.doPuts;
+import static org.apache.geode.security.SecurityTestUtils.getLocatorPort;
+import static org.apache.geode.test.dunit.Assert.fail;
+import static org.apache.geode.test.dunit.IgnoredException.addIgnoredException;
+import static org.apache.geode.test.dunit.LogWriterUtils.getLogWriter;
 
 import java.util.ArrayList;
 import java.util.Collection;
@@ -32,17 +46,13 @@ import org.junit.runner.RunWith;
 import org.junit.runners.Parameterized;
 
 import org.apache.geode.cache.operations.OperationContext.OperationCode;
-import org.apache.geode.distributed.ConfigurationProperties;
 import org.apache.geode.internal.AvailablePortHelper;
-import org.apache.geode.internal.Version;
 import org.apache.geode.internal.cache.tier.sockets.ServerConnection;
 import org.apache.geode.security.generator.AuthzCredentialGenerator;
 import org.apache.geode.security.generator.CredentialGenerator;
 import org.apache.geode.security.generator.DummyCredentialGenerator;
 import org.apache.geode.security.generator.XmlAuthzCredentialGenerator;
 import org.apache.geode.security.templates.UserPasswordAuthInit;
-import org.apache.geode.security.templates.UsernamePrincipal;
-import org.apache.geode.test.dunit.Host;
 import org.apache.geode.test.dunit.VM;
 import org.apache.geode.test.dunit.standalone.VersionManager;
 import org.apache.geode.test.junit.categories.DistributedTest;
@@ -63,7 +73,9 @@ import 
org.apache.geode.test.junit.runners.CategoryWithParameterizedRunnerFactor
 public class ClientAuthorizationDUnitTest extends ClientAuthorizationTestCase {
   @Parameterized.Parameters
   public static Collection<String> data() {
-    List<String> result = VersionManager.getInstance().getVersions();
+    List<String> result = 
VersionManager.getInstance().getVersionsWithoutCurrent();
+    result.add("000");
+    result.remove("130");
     if (result.size() < 1) {
       throw new RuntimeException("No older versions of Geode were found to 
test against");
     } else {

-- 
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].

Reply via email to