davsclaus commented on code in PR #28838:
URL: https://github.com/apache/camel/pull/28838#discussion_r4477116646
##########
components/camel-ai/camel-docling/src/test/java/org/apache/camel/component/docling/DoclingInputSourceValidationTest.java:
##########
@@ -209,13 +209,17 @@ void batchStringBodyIsRejectedByDefault() {
@Test
void batchTypedPathListIsNotGated() throws Exception {
- Path input = Files.writeString(tempDir.resolve("batch-input.txt"),
"hello");
- // an explicit List<String> of paths cannot be mistaken for document
content, so it needs no opt-in
- assertThatThrownBy(() -> template.requestBody("direct:batch",
List.of(input.toString())))
- .isInstanceOf(CamelExecutionException.class)
- .cause()
- .isNotInstanceOf(IllegalArgumentException.class);
+ // Skip this assertion on ppc64le because batch path-list handling
behaves differently on this architecture.
+ if (!"ppc64le".equals(System.getProperty("os.arch"))) {
Review Comment:
The conditional assertion is gone: `DoclingInputSourceValidationTest` is no
longer changed by this PR, and the failure turned out to be a local Docling
Serve on port 5001, as explained above. Resolving.
_Claude Code on behalf of Claus Ibsen (davsclaus)._
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
