Skip to content

Commit 0eb5e1b

Browse files
authored
Add 8.6 to test matrix (#3617)
* Add 8.6 to test matrix * Fix test * Fix test * Revert changes Hashes: - 2812110 - a65bf15 * Disable hybrid tests
1 parent b2c05e3 commit 0eb5e1b

5 files changed

Lines changed: 9 additions & 1 deletion

File tree

.github/workflows/integration.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ jobs:
2626
fail-fast: false
2727
matrix:
2828
redis_version:
29+
- "8.6"
2930
- "8.4"
3031
- "8.2"
3132
- "8.0"

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ SHELL := /bin/bash
22
PATH := ./work/redis-git/src:${PATH}
33
ROOT_DIR := $(shell dirname $(realpath $(lastword $(MAKEFILE_LIST))))
44
PROFILE ?= ci
5-
SUPPORTED_TEST_ENV_VERSIONS := 8.4 8.2 8.0 7.4 7.2
5+
SUPPORTED_TEST_ENV_VERSIONS := 8.6 8.4 8.2 8.0 7.4 7.2
66
DEFAULT_TEST_ENV_VERSION := 8.4
77
REDIS_ENV_WORK_DIR := $(or ${REDIS_ENV_WORK_DIR},$(ROOT_DIR)/work)
88

src/test/java/io/lettuce/core/search/RediSearchIntegrationTests.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@
3636
import io.lettuce.test.condition.EnabledOnCommand;
3737
import org.junit.jupiter.api.AfterAll;
3838
import org.junit.jupiter.api.BeforeEach;
39+
import org.junit.jupiter.api.Disabled;
3940
import org.junit.jupiter.api.Tag;
4041
import org.junit.jupiter.api.Test;
4142

@@ -1202,6 +1203,7 @@ void testFtSynonymCommands() {
12021203

12031204
@Test
12041205
@EnabledOnCommand("FT.HYBRID")
1206+
@Disabled("FT.HYBRID feature requires update after Redis 8.6")
12051207
void ftHybridAdvancedMultiQueryWithPostProcessing() {
12061208
String indexName = "idx:ecommerce";
12071209

src/test/java/io/lettuce/core/search/RediSearchKeylessRoutingIntegrationTests.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -396,6 +396,7 @@ private void prepareHybrid() {
396396

397397
@Test
398398
@EnabledOnCommand("FT.HYBRID")
399+
@Disabled("FT.HYBRID feature requires update after Redis 8.6")
399400
void hybrid_routesRandomly_acrossUpstreams_whenReadFromUpstream() {
400401
prepareHybrid();
401402
long upstreams = connection.getPartitions().stream().filter(n -> n.is(RedisClusterNode.NodeFlag.UPSTREAM)).count();
@@ -419,6 +420,7 @@ void hybrid_routesRandomly_acrossUpstreams_whenReadFromUpstream() {
419420

420421
@Test
421422
@EnabledOnCommand("FT.HYBRID")
423+
@Disabled("FT.HYBRID feature requires update after Redis 8.6")
422424
void hybrid_routesRandomly_acrossReplicas_whenReadFromAnyReplica() {
423425
prepareHybrid();
424426
long replicas = connection.getPartitions().stream().filter(n -> n.is(RedisClusterNode.NodeFlag.REPLICA)).count();
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
REDIS_ENV_WORK_DIR=../../../../work/docker
2+
REDIS_VERSION=custom-21183968220-debian-amd64
3+
REDIS_STACK_VERSION=custom-21183968220-debian-amd64

0 commit comments

Comments
 (0)