Skip to content

Commit 5d6cb90

Browse files
committed
Fixed hamcrest problem with Iteratables
1 parent b3ec84c commit 5d6cb90

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

src/test/java/com/kpelykh/docker/client/test/DockerClientTest.java

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
import org.apache.commons.io.IOUtils;
88
import org.apache.commons.io.LineIterator;
99
import org.apache.commons.lang.StringUtils;
10+
import org.hamcrest.Matcher;
1011
import org.slf4j.Logger;
1112
import org.slf4j.LoggerFactory;
1213
import org.testng.Assert;
@@ -124,10 +125,11 @@ public void testDockerInfo() throws DockerClientException {
124125

125126
@Test
126127
public void testDockerSearch() throws DockerClientException {
127-
List dockerSearch = dockerClient.search("busybox");
128+
List<SearchItem> dockerSearch = dockerClient.search("busybox");
128129
LOG.info("Search returned" + dockerSearch.toString());
129130

130-
assertThat(dockerSearch, hasItem(hasField("name", equalTo("busybox"))));
131+
Matcher matcher = hasItem(hasField("name", equalTo("busybox")));
132+
assertThat(dockerSearch, matcher);
131133

132134
assertThat(filter(hasField("name", is("busybox")), dockerSearch).size(), equalTo(1));
133135
}
@@ -176,7 +178,8 @@ public void testListContainers() throws DockerClientException {
176178

177179
List containers2 = dockerClient.listContainers(true);
178180
assertThat(size + 1, is(equalTo(containers2.size())));
179-
assertThat(containers2, hasItem(hasField("id", startsWith(container1.id))));
181+
Matcher matcher = hasItem(hasField("id", startsWith(container1.id)));
182+
assertThat(containers2, matcher);
180183

181184
List<Container> filteredContainers = filter(hasField("id", startsWith(container1.id)), containers2);
182185
assertThat(filteredContainers.size(), is(equalTo(1)));
@@ -411,7 +414,8 @@ public void removeContainer() throws DockerClientException {
411414
dockerClient.removeContainer(container.id);
412415

413416
List containers2 = dockerClient.listContainers(true);
414-
assertThat(containers2, not(hasItem(hasField("id", startsWith(container.id)))));
417+
Matcher matcher = not(hasItem(hasField("id", startsWith(container.id))));
418+
assertThat(containers2, matcher);
415419

416420
}
417421

@@ -515,7 +519,8 @@ public void testRemoveImage() throws DockerClientException {
515519
dockerClient.removeImage(imageId);
516520

517521
List containers = dockerClient.listContainers(true);
518-
assertThat(containers, not(hasItem(hasField("id", startsWith(imageId)))));
522+
Matcher matcher = not(hasItem(hasField("id", startsWith(imageId))));
523+
assertThat(containers, matcher);
519524
}
520525

521526

0 commit comments

Comments
 (0)