From 381d9c770efbaf86ab2f10d731d2dd07251bff4c Mon Sep 17 00:00:00 2001 From: Richard North Date: Sat, 19 Nov 2016 09:26:19 +0000 Subject: [PATCH] Address review comments from @bsideup --- .../java/org/testcontainers/utility/ResourceReaper.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/core/src/main/java/org/testcontainers/utility/ResourceReaper.java b/core/src/main/java/org/testcontainers/utility/ResourceReaper.java index f6faef40e8d..f00d11a3524 100644 --- 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 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; }