Skip to content

Commit 931644b

Browse files
gnodetclaude
andcommitted
CAMEL-22524: Fix embedded test services requiring Docker unnecessarily
Remove ContainerTestService from service interfaces that only have embedded (in-memory) implementations. ContainerTestService implements ExecutionCondition which checks for Docker availability and silently skips tests when Docker is not available, even though these services do not need Docker at all. Affected services: - HazelcastService: only has HazelcastEmbeddedService (in-memory) - IgniteService: only has IgniteEmbeddedService (in-memory) - FtpService: only has embedded services (FtpEmbeddedService, SftpEmbeddedService, FtpsEmbeddedService) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 798c294 commit 931644b

File tree

3 files changed

+3
-6
lines changed
  • test-infra
    • camel-test-infra-ftp/src/main/java/org/apache/camel/test/infra/ftp/services
    • camel-test-infra-hazelcast/src/main/java/org/apache/camel/test/infra/hazelcast/services
    • camel-test-infra-ignite/src/main/java/org/apache/camel/test/infra/ignite/services

3 files changed

+3
-6
lines changed

test-infra/camel-test-infra-ftp/src/main/java/org/apache/camel/test/infra/ftp/services/FtpService.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,13 @@
1616
*/
1717
package org.apache.camel.test.infra.ftp.services;
1818

19-
import org.apache.camel.test.infra.common.services.ContainerTestService;
2019
import org.apache.camel.test.infra.common.services.TestService;
2120
import org.junit.jupiter.api.extension.AfterEachCallback;
2221
import org.junit.jupiter.api.extension.BeforeEachCallback;
2322

2423
/**
2524
* Test infra service for Ftp
2625
*/
27-
public interface FtpService extends ContainerTestService, FtpInfraService, TestService, BeforeEachCallback, AfterEachCallback {
26+
public interface FtpService extends FtpInfraService, TestService, BeforeEachCallback, AfterEachCallback {
2827

2928
}

test-infra/camel-test-infra-hazelcast/src/main/java/org/apache/camel/test/infra/hazelcast/services/HazelcastService.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,10 @@
1616
*/
1717
package org.apache.camel.test.infra.hazelcast.services;
1818

19-
import org.apache.camel.test.infra.common.services.ContainerTestService;
2019
import org.apache.camel.test.infra.common.services.TestService;
2120

2221
/**
2322
* Test infra service for Hazelcast
2423
*/
25-
public interface HazelcastService extends TestService, HazelcastInfraService, ContainerTestService {
24+
public interface HazelcastService extends TestService, HazelcastInfraService {
2625
}

test-infra/camel-test-infra-ignite/src/main/java/org/apache/camel/test/infra/ignite/services/IgniteService.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,10 @@
1616
*/
1717
package org.apache.camel.test.infra.ignite.services;
1818

19-
import org.apache.camel.test.infra.common.services.ContainerTestService;
2019
import org.apache.camel.test.infra.common.services.TestService;
2120

2221
/**
2322
* Test infra service for Ignite
2423
*/
25-
public interface IgniteService extends TestService, IgniteInfraService, ContainerTestService {
24+
public interface IgniteService extends TestService, IgniteInfraService {
2625
}

0 commit comments

Comments
 (0)