[geode] branch develop updated: GEODE-7453: Fix Flaky Test (#4326)

[jjramos]

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

jjramos 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 c9e75e6  GEODE-7453: Fix Flaky Test (#4326)
c9e75e6 is described below

commit c9e75e6f8f82a99b66faa293d0ad7c19e47c22b8
Author: Juan José Ramos <jujora...@users.noreply.github.com>
AuthorDate: Fri Nov 15 13:58:03 2019 +0000

    GEODE-7453: Fix Flaky Test (#4326)
    
    - Fixed flakiness in SessionExpirationCacheListenerIntegrationTest.
---
 .../callback/SessionExpirationCacheListenerIntegrationTest.java   | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/extensions/geode-modules/src/integrationTest/java/org/apache/geode/modules/session/catalina/callback/SessionExpirationCacheListenerIntegrationTest.java
 
b/extensions/geode-modules/src/integrationTest/java/org/apache/geode/modules/session/catalina/callback/SessionExpirationCacheListenerIntegrationTest.java
index c7c4752..9bd32c7 100644
--- 
a/extensions/geode-modules/src/integrationTest/java/org/apache/geode/modules/session/catalina/callback/SessionExpirationCacheListenerIntegrationTest.java
+++ 
b/extensions/geode-modules/src/integrationTest/java/org/apache/geode/modules/session/catalina/callback/SessionExpirationCacheListenerIntegrationTest.java
@@ -69,8 +69,8 @@ public class SessionExpirationCacheListenerIntegrationTest {
         .create(REGION_NAME);
     httpSessionRegion.put(deltaSession.getId(), deltaSession);
 
-    await().untilAsserted(() -> 
assertThat(httpSessionRegion.isEmpty()).isTrue());
-    assertThat(deltaSession.processExpiredCalls.get()).isEqualTo(1);
+    await().untilAsserted(() -> 
assertThat(deltaSession.processExpiredCalls.get()).isEqualTo(1));
+    assertThat(httpSessionRegion.isEmpty()).isTrue();
   }
 
   @Test
@@ -91,8 +91,8 @@ public class SessionExpirationCacheListenerIntegrationTest {
     httpSessionRegion.put(deltaSession.getId(), deltaSession);
 
     deltaSession.expire(true);
-    await().untilAsserted(() -> 
assertThat(httpSessionRegion.isEmpty()).isTrue());
-    assertThat(deltaSession.processExpiredCalls.get()).isEqualTo(1);
+    await().untilAsserted(() -> 
assertThat(deltaSession.processExpiredCalls.get()).isEqualTo(1));
+    assertThat(httpSessionRegion.isEmpty()).isTrue();
   }
 
   static class TestCustomExpiry implements CustomExpiry<String, HttpSession> {