Megha-Megha13 commented on code in PR #28838:
URL: https://github.com/apache/camel/pull/28838#discussion_r4441756727
##########
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:
I investigated the ppc64le test failures and found that a Docling Serve
instance already running on "localhost:5001" was affecting the expected
exception.
After stopping the service, the test passed successfully, confirming that
this was an environment-related issue rather than an architecture-specific one.
I have removed the architecture-specific skip and all tests are now passing.
Latest test logs are attached for reference.
[docling_serve_updated_ppc64le_logs.txt](https://github.com/user-attachments/files/34269293/docling_serve_updated_ppc64le_logs.txt)
--
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]
