Skip to content

Commit 02e237d

Browse files
committed
Fix RedisVectorStoreIT
- Add missing org.apache.commons:commons-lang3 dependency in test scope which seems required for testcontainers 2.0 - Fix the RedisVectorStoreIT's test configuration include specifying `spring.data.redis.client-type=jedis` Signed-off-by: Ilayaperumal Gopinathan <ilayaperumal.gopinathan@broadcom.com>
1 parent f3eb842 commit 02e237d

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

vector-stores/spring-ai-redis-store/pom.xml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,12 +97,18 @@
9797
<scope>test</scope>
9898
</dependency>
9999

100-
<dependency>
100+
<dependency>
101101
<groupId>io.micrometer</groupId>
102102
<artifactId>micrometer-observation-test</artifactId>
103103
<scope>test</scope>
104104
</dependency>
105105

106+
<dependency>
107+
<groupId>org.apache.commons</groupId>
108+
<artifactId>commons-lang3</artifactId>
109+
<scope>test</scope>
110+
</dependency>
111+
106112
</dependencies>
107113

108114
</project>

vector-stores/spring-ai-redis-store/src/test/java/org/springframework/ai/vectorstore/redis/RedisVectorStoreIT.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,8 @@ class RedisVectorStoreIT extends BaseVectorStoreTests {
6868
private final ApplicationContextRunner contextRunner = new ApplicationContextRunner()
6969
.withConfiguration(AutoConfigurations.of(DataRedisAutoConfiguration.class))
7070
.withUserConfiguration(TestApplication.class)
71-
.withPropertyValues("spring.data.redis.url=" + redisContainer.getRedisURI());
71+
.withPropertyValues("spring.data.redis.url=" + redisContainer.getRedisURI())
72+
.withPropertyValues("spring.data.redis.client-type=jedis");
7273

7374
List<Document> documents = List.of(
7475
new Document("1", getText("classpath:/test/data/spring.ai.txt"), Map.of("meta1", "meta1")),

0 commit comments

Comments
 (0)