Avoid loading testcontainers unnecessarily

quarkusio · Feb 2, 2022 · e1f394d · e1f394d
1 parent 1a73092
commit e1f394d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/extensions/apicurio-registry-avro/runtime/pom.xml b/extensions/apicurio-registry-avro/runtime/pom.xml
@@ -46,11 +46,10 @@
             <groupId>io.quarkus</groupId>
             <artifactId>quarkus-vertx</artifactId>
         </dependency>
-
-      <dependency>
-        <groupId>org.apache.commons</groupId>
-        <artifactId>commons-lang3</artifactId>
-      </dependency>
+        <dependency>
+            <groupId>org.apache.commons</groupId>
+            <artifactId>commons-lang3</artifactId>
+        </dependency>
         <dependency>
             <groupId>io.quarkus</groupId>
             <artifactId>quarkus-devservices</artifactId>

diff --git a/...ices/deployment/src/main/java/io/quarkus/devservices/deployment/DevServicesProcessor.java b/...ices/deployment/src/main/java/io/quarkus/devservices/deployment/DevServicesProcessor.java
@@ -132,7 +132,7 @@ private List<DevServiceDescription> buildServiceDescriptions(List<DevServicesRes
     }
 
     private Map<String, Container> fetchContainerInfos(Set<String> containerIds) {
-        if (!isDockerWorking.getAsBoolean() || containerIds.isEmpty()) {
+        if (containerIds.isEmpty() || !isDockerWorking.getAsBoolean()) {
             return Collections.emptyMap();
         }
         return DockerClientFactory.lazyClient().listContainersCmd()