Make integration tests compatible with RE by uglide · Pull Request #4387 · redis/jedis

uglide · 2026-01-02T17:45:41Z

Guard all RE-incompatible integration tests with @ConditionalOnEnv
Fixes all time-based issues when running tests against RE

github-actions · 2026-01-02T17:58:49Z

Test Results

301 files - 2 301 suites - 2 12m 4s ⏱️ -6s
10 908 tests - 18 10 852 ✅ - 16 56 💤 - 2 0 ❌ ±0
5 631 runs - 16 5 622 ✅ - 16 9 💤 ±0 0 ❌ ±0

Results for commit 00523d4. ± Comparison against base commit be47c2f.

This pull request removes 26 and adds 8 tests. Note that renamed tests count towards both.

redis.clients.jedis.commands.jedis.ModuleTest[1] ‑ testModules
redis.clients.jedis.commands.jedis.ModuleTest[2] ‑ testModules
redis.clients.jedis.commands.jedis.ModuleTest[3] ‑ testModules
redis.clients.jedis.tls.ACLRedisClusterClientIT ‑ connectByIpAddress
redis.clients.jedis.tls.ACLRedisClusterClientIT ‑ connectByIpAddressFailsWithSSLParameters
redis.clients.jedis.tls.ACLRedisClusterClientIT ‑ connectToNodesFailsWithSSLParametersAndNoHostMapping
redis.clients.jedis.tls.ACLRedisClusterClientIT ‑ connectToNodesSucceedsWithSSLParametersAndHostMapping
redis.clients.jedis.tls.ACLRedisClusterClientIT ‑ connectWithCustomHostNameVerifier
redis.clients.jedis.tls.ACLRedisClusterClientIT ‑ connectWithCustomSocketFactory
redis.clients.jedis.tls.ACLRedisClusterClientIT ‑ connectWithEmptyTrustStore
…

redis.clients.jedis.tls.RedisClusterClientIT ‑ connectToNodesSucceedsWithSSLParametersAndHostMapping(String, SslOptions)[1]
redis.clients.jedis.tls.RedisClusterClientIT ‑ connectToNodesSucceedsWithSSLParametersAndHostMapping(String, SslOptions)[2]
redis.clients.jedis.tls.RedisClusterClientIT ‑ connectToNodesSucceedsWithSSLParametersAndHostMapping(String, SslOptions)[3]
redis.clients.jedis.tls.RedisClusterClientIT ‑ connectWithCustomHostNameVerifierAndSslOptions
redis.clients.jedis.tls.RedisClusterClientIT ‑ connectWithSslFlag
redis.clients.jedis.tls.RedisClusterClientIT ‑ testSSLDiscoverNodesAutomatically(String, SslOptions)[1]
redis.clients.jedis.tls.RedisClusterClientIT ‑ testSSLDiscoverNodesAutomatically(String, SslOptions)[2]
redis.clients.jedis.tls.RedisClusterClientIT ‑ testSSLDiscoverNodesAutomatically(String, SslOptions)[3]

This pull request skips 1 test.

redis.clients.jedis.UnavailableConnectionTest ‑ testAvoidQuitInDestroyObjectForBrokenConnection

♻️ This comment has been updated with latest results.

uglide · 2026-01-27T14:57:08Z

augment review

augmentcode · 2026-01-27T15:01:18Z

🤖 Augment PR Summary

Summary: This PR updates Jedis integration tests to run cleanly against Redis Enterprise (RE) environments.

Changes:

Introduces widespread use of @ConditionalOnEnv to skip tests/variants that are incompatible with RE.
Adds/propagates the EnvCondition JUnit 5 extension in several base test classes so env-gated tests are evaluated consistently.
Skips or adjusts RE-incompatible command tests (e.g., blocking list ops, BITOP, SORT STORE, COPY/RENAME-related cases) across commandobjects/jedis/unified/pipeline suites.
Relaxes a few timing-sensitive assertions (e.g., HPTTL tolerance, OBJECT IDLETIME) to reduce flakiness.
Updates control/config tests to use slowlog-max-len instead of maxmemory for broader compatibility.
Improves stability in a few tests via small lifecycle/ordering tweaks (e.g., slowlog entry selection, cluster teardown guarding).

Technical Notes: The new environment gating relies on TestEnvUtil.TEST_ENV_PROVIDER and the EnvCondition execution condition to enable/disable tests at runtime.

_{🤖 Was this summary useful? React with 👍 or 👎}

augmentcode

Review completed. 3 suggestions posted.

Comment augment review to trigger a new review at any time.

augmentcode · 2026-01-27T15:01:19Z

src/test/java/redis/clients/jedis/commands/jedis/SentinelCommandsTest.java

 public class SentinelCommandsTest {

+  @RegisterExtension
+  public  EnvCondition envCondition = new EnvCondition();


Because @ConditionalOnEnv is used at class level and prepare() does endpoint lookups, consider registering EnvCondition as a static @RegisterExtension (or via @ExtendWith) so the condition can disable the container before @BeforeAll runs in Redis Enterprise.

_{🤖 Was this useful? React with 👍 or 👎}

augmentcode · 2026-01-27T15:01:19Z

src/test/java/redis/clients/jedis/commands/jedis/ClientCommandsTest.java

    assertEquals(0, client.clientList(ClientType.MASTER).length());
    assertEquals(1, client.clientList(ClientType.SLAVE).split("\\n").length);
-    assertEquals(1, client.clientList(ClientType.REPLICA).split("\\n").length);
+    if (!TestEnvUtil.getTestEnvProvider().equals(TestEnvUtil.ENV_REDIS_ENTERPRISE)) {


EnvCondition matches environments via equalsIgnoreCase, but this inline check uses equals, so TEST_ENV_PROVIDER=RE/Re would not be treated as Redis Enterprise here. Consider using a case-insensitive comparison for consistency with the rest of the environment gating.

_{🤖 Was this useful? React with 👍 or 👎}

augmentcode · 2026-01-27T15:01:20Z

src/test/java/redis/clients/jedis/commands/jedis/ClusterCommandsTest.java

  @AfterAll
  public static void resetRedisAfter() {
-    removeSlots();
+    if (endpoint != null) {


With the @BeforeAll slot reset removed, a previously “dirty” cluster-unbound could keep slot assignments between runs and impact early assertions. Consider doing a one-time removeSlots() after endpoint is initialized in prepareEndpoints(), in addition to the @AfterAll cleanup.

_{🤖 Was this useful? React with 👍 or 👎}

- Use endpoint in useWithoutConnecting - Disable startWithUri() on RE

…rge data transfer

…e' into im/skip-it-not-compatible-with-re

- Rely on correctly configured Redis Cluster instead of testing outdated workarounds with hostname mapping - Merge ACL, SSLOptions and RedisClusterClientIT into one test - Remove redundant utilities

jit-ci · 2026-02-24T15:02:59Z

🛡️ Jit Security Scan Results

✅ No security findings were detected in this PR

^{Security scan by Jit}

uglide force-pushed the im/skip-it-not-compatible-with-re branch 2 times, most recently from ee72417 to c266e87 Compare January 5, 2026 16:19

uglide force-pushed the im/skip-it-not-compatible-with-re branch 2 times, most recently from 2b77f6b to 74d5ebf Compare January 23, 2026 14:24

augmentcode bot reviewed Jan 27, 2026

View reviewed changes

uglide force-pushed the im/skip-it-not-compatible-with-re branch 2 times, most recently from 5334382 to 659c4d4 Compare February 16, 2026 13:30

uglide added 17 commits February 20, 2026 17:55

Make command tests compatible with RE

3aa6309

Skip more incompatible tests

f25e345

Allow up to 1 sec for HFE assertions

3ea96b8

Use standalone-0 endpoint in ConnectionTest instead of hardcoded 6379

66662ec

Fix ACLJedisTest

78bf293

- Use endpoint in useWithoutConnecting - Disable startWithUri() on RE

Add 5 minutes timeout for Junit tests

de372a2

Skip incompatible tests in TransactionV2Test

22e07c3

Skip handleClientOutputBufferLimitForSubscribeTooSlow on RE due to la…

c0624cd

…rge data transfer

EnvCondition should be static

777901d

Add guards for more incompatible tests

a2c2d91

Skip more tests

f8bec63

Fix TimeSeriesTest

08d924c

Fix UnavailableConnectionTest

151f49a

Fix ConsolidatedConfigurationCommandsTest

1d88f7f

Align property name for tls certificates path

c6d2b87

Check if the path is absolute in TlsUtil

9ec7617

Disable CSC tests with heavy write on RE env

4ae3bd0

uglide force-pushed the im/skip-it-not-compatible-with-re branch from 1e67bce to 4ae3bd0 Compare February 20, 2026 16:55

uglide and others added 2 commits February 23, 2026 12:12

Merge branch 'master' into im/skip-it-not-compatible-with-re

18f6087

Add missing guard for RE

1ae3334

uglide and others added 4 commits February 23, 2026 14:33

Merge remote-tracking branch 'origin/im/skip-it-not-compatible-with-r…

9ca6e65

…e' into im/skip-it-not-compatible-with-re

Refactor Cluster TLS tests to focus on production scenarios

36bfec7

- Rely on correctly configured Redis Cluster instead of testing outdated workarounds with hostname mapping - Merge ACL, SSLOptions and RedisClusterClientIT into one test - Remove redundant utilities

Remove assertions on cluster topology in the TLS test

bec81f8

Merge branch 'master' into im/skip-it-not-compatible-with-re

00523d4

uglide marked this pull request as ready for review February 24, 2026 15:32

uglide requested a review from ggivo February 24, 2026 15:32

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

Make integration tests compatible with RE#4387

Make integration tests compatible with RE#4387
uglide wants to merge 23 commits intomasterfrom
im/skip-it-not-compatible-with-re

uglide commented Jan 2, 2026 •

edited

Loading

Uh oh!

github-actions bot commented Jan 2, 2026 •

edited

Loading

Uh oh!

uglide commented Jan 27, 2026

Uh oh!

augmentcode bot commented Jan 27, 2026

Uh oh!

augmentcode bot left a comment

Uh oh!

augmentcode bot Jan 27, 2026

Uh oh!

augmentcode bot Jan 27, 2026

Uh oh!

augmentcode bot Jan 27, 2026

Uh oh!

jit-ci bot commented Feb 24, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Comments

Conversation

uglide commented Jan 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Jan 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results

Uh oh!

uglide commented Jan 27, 2026

Uh oh!

augmentcode bot commented Jan 27, 2026

Uh oh!

augmentcode bot left a comment

Choose a reason for hiding this comment

Uh oh!

augmentcode bot Jan 27, 2026

Choose a reason for hiding this comment

Uh oh!

augmentcode bot Jan 27, 2026

Choose a reason for hiding this comment

Uh oh!

augmentcode bot Jan 27, 2026

Choose a reason for hiding this comment

Uh oh!

jit-ci bot commented Feb 24, 2026

🛡️ Jit Security Scan Results

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

uglide commented Jan 2, 2026 •

edited

Loading

github-actions bot commented Jan 2, 2026 •

edited

Loading