gnodet opened a new pull request, #22409:
URL: https://github.com/apache/camel/pull/22409

   ## Summary
   
   Several test-infra service interfaces extend `ContainerTestService`, which 
implements JUnit 5's `ExecutionCondition` and disables tests when Docker is 
unavailable. However, some of these services only have embedded (in-memory) 
implementations that don't need Docker at all. This causes their tests to be 
silently skipped in environments without Docker.
   
   This PR removes `ContainerTestService` from the following service interfaces 
that only have embedded implementations:
   
   - **`HazelcastService`** - only has `HazelcastEmbeddedService` (in-memory 
Hazelcast)
   - **`IgniteService`** - only has `IgniteEmbeddedService` (in-memory Ignite)
   - **`FtpService`** - only has embedded services (`FtpEmbeddedService`, 
`SftpEmbeddedService`, `FtpsEmbeddedService`)
   
   ## Test plan
   
   - [ ] Verify the three affected modules compile: `mvn install -B -pl 
test-infra/camel-test-infra-hazelcast,test-infra/camel-test-infra-ignite,test-infra/camel-test-infra-ftp
 -DskipTests`
   - [ ] Verify Hazelcast tests run without Docker
   - [ ] Verify Ignite tests run without Docker
   - [ ] Verify FTP/SFTP tests run without Docker
   - [ ] Verify no regressions in CI
   
   _Claude Code on behalf of Guillaume Nodet_


-- 
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]

Reply via email to