You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ap...@apache.org on 2026/02/09 13:53:49 UTC
(camel) branch main updated (0ceeb7d7c0f3 -> ec17beca0a0f)
This is an automated email from the ASF dual-hosted git repository.
apupier pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git
from 0ceeb7d7c0f3 chore(deps): Bump opentelemetry-version from 1.58.0 to 1.59.0 (#21354)
new a992cf63ae71 Remove public identifier for test classes and methods in qdrant
new 3c52761cd7e8 use isEmpty() instead of hasSize(0) assertions in qdrant tests
new ec17beca0a0f Join multiple assertions subject to one assertion chain
The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.../camel/component/qdrant/QdrantCommonTest.java | 4 ++--
.../qdrant/QdrantCreateCollectionTest.java | 4 ++--
.../camel/component/qdrant/QdrantDeleteTest.java | 4 ++--
.../camel/component/qdrant/QdrantRetrieveTest.java | 4 ++--
.../camel/component/qdrant/QdrantUpsertTest.java | 4 ++--
.../component/qdrant/it/QdrantComponentIT.java | 27 +++++++++++-----------
.../qdrant/it/QdrantDeleteCollectionIT.java | 15 ++++++------
.../component/qdrant/it/QdrantDeletePointsIT.java | 14 +++++------
8 files changed, 39 insertions(+), 37 deletions(-)
(camel) 03/03: Join multiple assertions subject to one assertion chain
Posted by ap...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
apupier pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git
commit ec17beca0a0f4ee2f487a9ded8fbc8b53f9cdf38
Author: Aurélien Pupier <ap...@ibm.com>
AuthorDate: Mon Feb 9 14:12:16 2026 +0100
Join multiple assertions subject to one assertion chain
Signed-off-by: Aurélien Pupier <ap...@ibm.com>
---
.../java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java | 5 +++--
.../apache/camel/component/qdrant/it/QdrantDeleteCollectionIT.java | 5 +++--
2 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java
index 6c8a0de382c7..f99b1161c677 100644
--- a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java
+++ b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java
@@ -58,8 +58,9 @@ class QdrantComponentIT extends QdrantTestSupport {
final QdrantActionException exception = result.getException(QdrantActionException.class);
final Throwable cause = exception.getCause();
- assertThat(cause).isNotNull();
- assertThat(cause).isInstanceOf(StatusRuntimeException.class);
+ assertThat(cause)
+ .isNotNull()
+ .isInstanceOf(StatusRuntimeException.class);
StatusRuntimeException statusRuntimeException = (StatusRuntimeException) cause;
assertThat(statusRuntimeException.getStatus().getCode()).isEqualTo(Status.NOT_FOUND.getCode());
diff --git a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeleteCollectionIT.java b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeleteCollectionIT.java
index e37e47752fe3..9a80dba023b9 100644
--- a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeleteCollectionIT.java
+++ b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeleteCollectionIT.java
@@ -89,8 +89,9 @@ class QdrantDeleteCollectionIT extends QdrantTestSupport {
final QdrantActionException exception = result.getException(QdrantActionException.class);
final Throwable cause = exception.getCause();
- assertThat(cause).isNotNull();
- assertThat(cause).isInstanceOf(StatusRuntimeException.class);
+ assertThat(cause)
+ .isNotNull()
+ .isInstanceOf(StatusRuntimeException.class);
StatusRuntimeException statusRuntimeException = (StatusRuntimeException) cause;
assertThat(statusRuntimeException.getStatus().getCode()).isEqualTo(Status.NOT_FOUND.getCode());
(camel) 01/03: Remove public identifier for test classes and methods in qdrant
Posted by ap...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
apupier pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git
commit a992cf63ae71a6eeaf27987f28ab0b61a8dbcaa7
Author: Aurélien Pupier <ap...@ibm.com>
AuthorDate: Mon Feb 9 14:08:59 2026 +0100
Remove public identifier for test classes and methods in qdrant
Signed-off-by: Aurélien Pupier <ap...@ibm.com>
---
.../camel/component/qdrant/QdrantCommonTest.java | 4 ++--
.../component/qdrant/QdrantCreateCollectionTest.java | 4 ++--
.../camel/component/qdrant/QdrantDeleteTest.java | 4 ++--
.../camel/component/qdrant/QdrantRetrieveTest.java | 4 ++--
.../camel/component/qdrant/QdrantUpsertTest.java | 4 ++--
.../camel/component/qdrant/it/QdrantComponentIT.java | 20 ++++++++++----------
.../qdrant/it/QdrantDeleteCollectionIT.java | 10 +++++-----
.../component/qdrant/it/QdrantDeletePointsIT.java | 10 +++++-----
8 files changed, 30 insertions(+), 30 deletions(-)
diff --git a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantCommonTest.java b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantCommonTest.java
index 916e8953c2f1..bf2f70c5858d 100644
--- a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantCommonTest.java
+++ b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantCommonTest.java
@@ -26,12 +26,12 @@ import org.junit.jupiter.params.provider.EnumSource;
import static org.assertj.core.api.Assertions.assertThat;
-public final class QdrantCommonTest extends CamelTestSupport {
+final class QdrantCommonTest extends CamelTestSupport {
@DisplayName("Tests that trying to run actions with a null body triggers a failure")
@ParameterizedTest
@EnumSource(QdrantAction.class)
- public void upsertWithNullBody(QdrantAction action) {
+ void upsertWithNullBody(QdrantAction action) {
Exchange result = fluentTemplate.to("qdrant:test")
.withHeader(QdrantHeaders.ACTION, action)
.withBody(null)
diff --git a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantCreateCollectionTest.java b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantCreateCollectionTest.java
index 0fb6b8e6d19f..521056298289 100644
--- a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantCreateCollectionTest.java
+++ b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantCreateCollectionTest.java
@@ -25,11 +25,11 @@ import org.junit.jupiter.api.Test;
import static org.assertj.core.api.Assertions.assertThat;
-public class QdrantCreateCollectionTest extends QdrantTestSupport {
+class QdrantCreateCollectionTest extends QdrantTestSupport {
@DisplayName("Tests that trying to create a collection without passing the action name triggers a failure")
@Test
- public void createCollectionWithoutRequiredParameters() {
+ void createCollectionWithoutRequiredParameters() {
Exchange result = fluentTemplate.to("qdrant:createCollection")
.withBody(
Collections.VectorParams.newBuilder()
diff --git a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantDeleteTest.java b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantDeleteTest.java
index 9cdc8556f42f..c9b135982090 100644
--- a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantDeleteTest.java
+++ b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantDeleteTest.java
@@ -25,11 +25,11 @@ import org.junit.jupiter.api.Test;
import static io.qdrant.client.ConditionFactory.matchKeyword;
import static org.assertj.core.api.Assertions.assertThat;
-public class QdrantDeleteTest extends QdrantTestSupport {
+class QdrantDeleteTest extends QdrantTestSupport {
@DisplayName("Tests that trying to delete without passing the action name triggers a failure")
@Test
- public void deleteWithoutRequiredParameters() {
+ void deleteWithoutRequiredParameters() {
Exchange result = fluentTemplate.to("qdrant:delete")
.withBody(Common.Filter.newBuilder()
.addMust(matchKeyword("foo", "hello"))
diff --git a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantRetrieveTest.java b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantRetrieveTest.java
index 0b85b4e3c411..bdc540438ed2 100644
--- a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantRetrieveTest.java
+++ b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantRetrieveTest.java
@@ -25,11 +25,11 @@ import org.junit.jupiter.api.Test;
import static io.qdrant.client.PointIdFactory.id;
import static org.assertj.core.api.Assertions.assertThat;
-public class QdrantRetrieveTest extends QdrantTestSupport {
+class QdrantRetrieveTest extends QdrantTestSupport {
@DisplayName("Tests that trying to retrieve without passing the action name triggers a failure")
@Test
- public void retrieveWithoutRequiredParameters() {
+ void retrieveWithoutRequiredParameters() {
Exchange result = fluentTemplate.to("qdrant:retrieve")
.withBody(id(8))
.request(Exchange.class);
diff --git a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantUpsertTest.java b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantUpsertTest.java
index d3e8940f95d5..11f1201f9059 100644
--- a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantUpsertTest.java
+++ b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/QdrantUpsertTest.java
@@ -31,11 +31,11 @@ import static io.qdrant.client.PointIdFactory.id;
import static io.qdrant.client.ValueFactory.value;
import static org.assertj.core.api.Assertions.assertThat;
-public class QdrantUpsertTest extends QdrantTestSupport {
+class QdrantUpsertTest extends QdrantTestSupport {
@DisplayName("Tests that trying to upsert without passing the action name triggers a failure")
@Test
- public void upsertWithoutRequiredParameters() {
+ void upsertWithoutRequiredParameters() {
Exchange result = fluentTemplate.to("qdrant:upsert")
.withBody(
Points.PointStruct.newBuilder()
diff --git a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java
index d291001adc83..887d44acb69e 100644
--- a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java
+++ b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java
@@ -43,11 +43,11 @@ import org.junit.jupiter.params.provider.EnumSource;
import static org.assertj.core.api.Assertions.assertThat;
@TestMethodOrder(MethodOrderer.OrderAnnotation.class)
-public class QdrantComponentIT extends QdrantTestSupport {
+class QdrantComponentIT extends QdrantTestSupport {
@Test
@Order(0)
- public void collectionInfoNonExistent() {
+ void collectionInfoNonExistent() {
Exchange result = fluentTemplate.to("qdrant:testComponent")
.withHeader(QdrantHeaders.ACTION, QdrantAction.COLLECTION_INFO)
.request(Exchange.class);
@@ -67,7 +67,7 @@ public class QdrantComponentIT extends QdrantTestSupport {
@Test
@Order(1)
- public void createCollection() {
+ void createCollection() {
Exchange result = fluentTemplate.to("qdrant:testComponent")
.withHeader(QdrantHeaders.ACTION, QdrantAction.CREATE_COLLECTION)
.withBody(
@@ -82,7 +82,7 @@ public class QdrantComponentIT extends QdrantTestSupport {
@Test
@Order(2)
- public void collectionInfoExistent() {
+ void collectionInfoExistent() {
Exchange result = fluentTemplate.to("qdrant:testComponent")
.withHeader(QdrantHeaders.ACTION, QdrantAction.COLLECTION_INFO)
.request(Exchange.class);
@@ -94,7 +94,7 @@ public class QdrantComponentIT extends QdrantTestSupport {
@Test
@Order(3)
- public void upsert() {
+ void upsert() {
Exchange result = fluentTemplate.to("qdrant:testComponent")
.withHeader(QdrantHeaders.ACTION, QdrantAction.UPSERT)
.withBody(
@@ -124,7 +124,7 @@ public class QdrantComponentIT extends QdrantTestSupport {
@Test
@Order(4)
@SuppressWarnings({ "unchecked" })
- public void retrieve() {
+ void retrieve() {
Exchange result = fluentTemplate.to("qdrant:testComponent")
.withHeader(QdrantHeaders.ACTION, QdrantAction.RETRIEVE)
.withBody(PointIdFactory.id(8))
@@ -142,7 +142,7 @@ public class QdrantComponentIT extends QdrantTestSupport {
@ParameterizedTest
@EnumSource(TestData.class)
@Order(5)
- public void upsertOtherVectors(TestData testData) {
+ void upsertOtherVectors(TestData testData) {
Exchange result = fluentTemplate.to("qdrant:testComponent")
.withHeader(QdrantHeaders.ACTION, QdrantAction.UPSERT)
.withBody(
@@ -165,7 +165,7 @@ public class QdrantComponentIT extends QdrantTestSupport {
@Test
@Order(6)
- public void similaritySeach() {
+ void similaritySeach() {
Exchange result = fluentTemplate.to("qdrant:testComponent")
.withHeader(QdrantHeaders.ACTION, QdrantAction.SIMILARITY_SEARCH)
.withHeader(QdrantHeaders.INCLUDE_VECTORS, true)
@@ -184,7 +184,7 @@ public class QdrantComponentIT extends QdrantTestSupport {
@Test
@Order(7)
- public void delete() {
+ void delete() {
Exchange result = fluentTemplate.to("qdrant:testComponent")
.withHeader(QdrantHeaders.ACTION, QdrantAction.DELETE)
.withBody(ConditionFactory.matchKeyword("foo", "hello"))
@@ -206,7 +206,7 @@ public class QdrantComponentIT extends QdrantTestSupport {
@Test
@Order(8)
- public void retrieveAfterDelete() {
+ void retrieveAfterDelete() {
Exchange result = fluentTemplate.to("qdrant:testComponent")
.withHeader(QdrantHeaders.ACTION, QdrantAction.RETRIEVE)
.withBody(PointIdFactory.id(8))
diff --git a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeleteCollectionIT.java b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeleteCollectionIT.java
index 78da32e39221..e37e47752fe3 100644
--- a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeleteCollectionIT.java
+++ b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeleteCollectionIT.java
@@ -34,13 +34,13 @@ import org.junit.jupiter.api.TestMethodOrder;
import static org.assertj.core.api.Assertions.assertThat;
@TestMethodOrder(MethodOrderer.OrderAnnotation.class)
-public class QdrantDeleteCollectionIT extends QdrantTestSupport {
+class QdrantDeleteCollectionIT extends QdrantTestSupport {
@EndpointInject("qdrant:collectionForDeletion")
QdrantEndpoint qdrantEndpoint;
@Test
@Order(1)
- public void createCollection() {
+ void createCollection() {
Exchange result = fluentTemplate.to(qdrantEndpoint)
.withHeader(QdrantHeaders.ACTION, QdrantAction.CREATE_COLLECTION)
.withBody(
@@ -55,7 +55,7 @@ public class QdrantDeleteCollectionIT extends QdrantTestSupport {
@Test
@Order(2)
- public void collectionInfoExistent() {
+ void collectionInfoExistent() {
Exchange result = fluentTemplate.to(qdrantEndpoint)
.withHeader(QdrantHeaders.ACTION, QdrantAction.COLLECTION_INFO)
.request(Exchange.class);
@@ -67,7 +67,7 @@ public class QdrantDeleteCollectionIT extends QdrantTestSupport {
@Test
@Order(3)
- public void deleteCollection() {
+ void deleteCollection() {
Exchange result = fluentTemplate.to(qdrantEndpoint)
.withHeader(QdrantHeaders.ACTION, QdrantAction.DELETE_COLLECTION)
.request(Exchange.class);
@@ -78,7 +78,7 @@ public class QdrantDeleteCollectionIT extends QdrantTestSupport {
@Test
@Order(4)
- public void collectionInfoNonExistent() {
+ void collectionInfoNonExistent() {
Exchange result = fluentTemplate.to(qdrantEndpoint)
.withHeader(QdrantHeaders.ACTION, QdrantAction.COLLECTION_INFO)
.request(Exchange.class);
diff --git a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeletePointsIT.java b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeletePointsIT.java
index b1eabae7968c..7c3a713e0026 100644
--- a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeletePointsIT.java
+++ b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeletePointsIT.java
@@ -39,10 +39,10 @@ import org.junit.jupiter.api.TestMethodOrder;
import static org.assertj.core.api.Assertions.assertThat;
@TestMethodOrder(MethodOrderer.OrderAnnotation.class)
-public class QdrantDeletePointsIT extends QdrantTestSupport {
+class QdrantDeletePointsIT extends QdrantTestSupport {
@Test
@Order(1)
- public void createCollection() {
+ void createCollection() {
Exchange result = fluentTemplate.to("qdrant:testDelete")
.withHeader(QdrantHeaders.ACTION, QdrantAction.CREATE_COLLECTION)
.withBody(
@@ -57,7 +57,7 @@ public class QdrantDeletePointsIT extends QdrantTestSupport {
@Test
@Order(2)
- public void upsert() {
+ void upsert() {
Exchange result1 = fluentTemplate.to("qdrant:testDelete")
.withHeader(QdrantHeaders.ACTION, QdrantAction.UPSERT)
.withBody(
@@ -87,7 +87,7 @@ public class QdrantDeletePointsIT extends QdrantTestSupport {
@Test
@Order(3)
- public void deleteWithCondition() {
+ void deleteWithCondition() {
Exchange deleteResult = fluentTemplate.to("qdrant:testDelete")
.withHeader(QdrantHeaders.ACTION, QdrantAction.DELETE)
.withBody(ConditionFactory.matchKeyword("foo", "hello1"))
@@ -111,7 +111,7 @@ public class QdrantDeletePointsIT extends QdrantTestSupport {
@Test
@Order(4)
- public void deleteWithFilter() {
+ void deleteWithFilter() {
Exchange deleteResult = fluentTemplate.to("qdrant:testDelete")
.withHeader(QdrantHeaders.ACTION, QdrantAction.DELETE)
.withBody(
(camel) 02/03: use isEmpty() instead of hasSize(0) assertions in qdrant tests
Posted by ap...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
apupier pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git
commit 3c52761cd7e8536cd8849c0cf065362e2e0e1412
Author: Aurélien Pupier <ap...@ibm.com>
AuthorDate: Mon Feb 9 14:10:23 2026 +0100
use isEmpty() instead of hasSize(0) assertions in qdrant tests
Signed-off-by: Aurélien Pupier <ap...@ibm.com>
---
.../java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java | 2 +-
.../org/apache/camel/component/qdrant/it/QdrantDeletePointsIT.java | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java
index 887d44acb69e..6c8a0de382c7 100644
--- a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java
+++ b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantComponentIT.java
@@ -216,7 +216,7 @@ class QdrantComponentIT extends QdrantTestSupport {
assertThat(result.getException()).isNull();
assertThat(result.getIn().getBody()).isInstanceOfSatisfying(Collection.class, c -> {
- assertThat(c).hasSize(0);
+ assertThat(c).isEmpty();
});
}
diff --git a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeletePointsIT.java b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeletePointsIT.java
index 7c3a713e0026..8e31f722a244 100644
--- a/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeletePointsIT.java
+++ b/components/camel-ai/camel-qdrant/src/test/java/org/apache/camel/component/qdrant/it/QdrantDeletePointsIT.java
@@ -105,7 +105,7 @@ class QdrantDeletePointsIT extends QdrantTestSupport {
assertThat(result.getException()).isNull();
assertThat(result.getIn().getBody()).isInstanceOfSatisfying(Collection.class, c -> {
- assertThat(c).hasSize(0);
+ assertThat(c).isEmpty();
});
}
@@ -132,7 +132,7 @@ class QdrantDeletePointsIT extends QdrantTestSupport {
assertThat(result.getException()).isNull();
assertThat(result.getIn().getBody()).isInstanceOfSatisfying(Collection.class, c -> {
- assertThat(c).hasSize(0);
+ assertThat(c).isEmpty();
});
}