You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by la...@apache.org on 2017/08/09 22:02:40 UTC

geode git commit: GEODE-2226: SessionReplicationIntegrationTests do not run on windows

Repository: geode
Updated Branches:
  refs/heads/feature/GEODE-2226-2 [created] 1efbf58f3


GEODE-2226: SessionReplicationIntegrationTests do not run on windows

- moved Assume(not windows) until tester variable set to avoid NPE on tearDown


Project: http://git-wip-us.apache.org/repos/asf/geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/geode/commit/1efbf58f
Tree: http://git-wip-us.apache.org/repos/asf/geode/tree/1efbf58f
Diff: http://git-wip-us.apache.org/repos/asf/geode/diff/1efbf58f

Branch: refs/heads/feature/GEODE-2226-2
Commit: 1efbf58f3a254a6e27bd63c2b676f216c35c9d9f
Parents: b06f69f
Author: Lynn Hughes-Godfrey <lh...@pivotal.io>
Authored: Wed Aug 9 14:59:34 2017 -0700
Committer: Lynn Hughes-Godfrey <lh...@pivotal.io>
Committed: Wed Aug 9 14:59:34 2017 -0700

----------------------------------------------------------------------
 .../internal/filter/SessionReplicationIntegrationJUnitTest.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/geode/blob/1efbf58f/extensions/geode-modules-session/src/test/java/org/apache/geode/modules/session/internal/filter/SessionReplicationIntegrationJUnitTest.java
----------------------------------------------------------------------
diff --git a/extensions/geode-modules-session/src/test/java/org/apache/geode/modules/session/internal/filter/SessionReplicationIntegrationJUnitTest.java b/extensions/geode-modules-session/src/test/java/org/apache/geode/modules/session/internal/filter/SessionReplicationIntegrationJUnitTest.java
index 5a0add2..2679ecc 100644
--- a/extensions/geode-modules-session/src/test/java/org/apache/geode/modules/session/internal/filter/SessionReplicationIntegrationJUnitTest.java
+++ b/extensions/geode-modules-session/src/test/java/org/apache/geode/modules/session/internal/filter/SessionReplicationIntegrationJUnitTest.java
@@ -85,8 +85,6 @@ public class SessionReplicationIntegrationJUnitTest {
   @Before
   public void setUp() throws Exception {
 
-    Assume.assumeFalse(System.getProperty("os.name").toLowerCase().contains("win"));
-
     File gemfireLogFile = new File(tmpdir.newFolder(), "gemfire_modules.log");
 
     request = HttpTester.newRequest();
@@ -94,6 +92,8 @@ public class SessionReplicationIntegrationJUnitTest {
     tester = new MyServletTester();
     tester.setContextPath("/test");
 
+    Assume.assumeFalse(System.getProperty("os.name").toLowerCase().contains("win"));
+
     filterHolder =
         tester.addFilter(SessionCachingFilter.class, "/*", EnumSet.of(DispatcherType.REQUEST));
     filterHolder.setInitParameter(DistributionConfig.GEMFIRE_PREFIX + "property.mcast-port", "0");