[geode] branch develop updated: GEODE-9490: Correctly ignore RedisProxy exceptions in NativeRedisClusterTestRule (#6745)

[jensdeppe]

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 a1df4b3  GEODE-9490: Correctly ignore RedisProxy exceptions in 
NativeRedisClusterTestRule (#6745)
a1df4b3 is described below

commit a1df4b3be86c6ac8668770a88210c3cb5f15abef
Author: Jens Deppe <jde...@vmware.com>
AuthorDate: Wed Aug 18 07:08:17 2021 -0700

    GEODE-9490: Correctly ignore RedisProxy exceptions in 
NativeRedisClusterTestRule (#6745)
---
 .../java/org/apache/geode/redis/NativeRedisClusterTestRule.java        | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/geode-apis-compatible-with-redis/src/commonTest/java/org/apache/geode/redis/NativeRedisClusterTestRule.java
 
b/geode-apis-compatible-with-redis/src/commonTest/java/org/apache/geode/redis/NativeRedisClusterTestRule.java
index 2ad2721..cd677c2 100644
--- 
a/geode-apis-compatible-with-redis/src/commonTest/java/org/apache/geode/redis/NativeRedisClusterTestRule.java
+++ 
b/geode-apis-compatible-with-redis/src/commonTest/java/org/apache/geode/redis/NativeRedisClusterTestRule.java
@@ -57,7 +57,6 @@ public class NativeRedisClusterTestRule extends 
ExternalResource implements Seri
         // Docker compose does not work on windows in CI. Ignore this test on 
windows
         // Using a RuleChain to make sure we ignore the test before the rule 
comes into play
         .outerRule(new IgnoreOnWindowsRule());
-    IgnoredException.addIgnoredException("RedisProxy");
   }
 
   public List<Integer> getExposedPorts() {
@@ -116,6 +115,8 @@ public class NativeRedisClusterTestRule extends 
ExternalResource implements Seri
         proxies.forEach(p -> p.configure(translationMappings));
 
         logger.info("Started redis cluster with mapped ports: {}", 
translationMappings);
+        IgnoredException.addIgnoredException("RedisProxy");
+
         try {
           base.evaluate(); // This will run the test.
         } finally {