Address review comments from @bsideup

testcontainers · Nov 19, 2016 · 381d9c7 · 381d9c7
1 parent 7b3edaa
commit 381d9c7
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/core/src/main/java/org/testcontainers/utility/ResourceReaper.java b/core/src/main/java/org/testcontainers/utility/ResourceReaper.java
@@ -88,7 +88,7 @@ private void stopContainer(String containerId, String imageName) {
 
         List<Container> allContainers = dockerClient.listContainersCmd().withShowAll(true).exec();
 
-        if (allContainers.stream().noneMatch(it -> it.getId().equals(containerId))) {
+        if (allContainers.stream().map(Container::getId).noneMatch(containerId::equals)) {
             LOGGER.trace("Was going to clean up container but it apparently no longer exists: {}");
             return;
         }
@@ -116,9 +116,9 @@ private void stopContainer(String containerId, String imageName) {
         }
 
         try {
-            InspectContainerResponse containerInfo = dockerClient.inspectContainerCmd(containerId).exec();
+            dockerClient.inspectContainerCmd(containerId).exec();
         } catch (NotFoundException e) {
-            LOGGER.trace("Was going to kill container but it apparently no longer exists: {}");
+            LOGGER.trace("Was going to remove container but it apparently no longer exists: {}");
             return;
         }