A detailed description for iterable values has been added.

Maverick-Crank-GRey · Maverick-Crank-GRey · commit 5098600e4c88 · 2014-09-18T06:34:08.000-07:00
diff --git a/hamcrest-core/src/main/java/org/hamcrest/BaseDescription.java b/hamcrest-core/src/main/java/org/hamcrest/BaseDescription.java
@@ -49,6 +49,9 @@ public Description appendValue(Object value) {
             append("F>");
         } else if (value.getClass().isArray()) {
             appendValueList("[",", ","]", new ArrayIterator(value));
+        } else if (value instanceof Iterable) {
+            final Iterable iterable = (Iterable) value;
+            appendValueList("<[",", ","]>", iterable.iterator());
         } else {
             append('<');
             append(descriptionOf(value));
diff --git a/hamcrest-core/src/test/java/org/hamcrest/BaseDescriptionTest.java b/hamcrest-core/src/test/java/org/hamcrest/BaseDescriptionTest.java
@@ -4,6 +4,9 @@
 
 import org.junit.Test;
 
+import java.util.Arrays;
+import java.util.Iterator;
+
 public final class BaseDescriptionTest {
 
     private final StringBuilder result = new StringBuilder();
@@ -56,7 +59,18 @@ public final class BaseDescriptionTest {
         assertEquals("[\"2\", \"3\"]", result.toString());
     }
 
-    @Test public void 
+    @Test public void
+    describesAppendedIterableValue() {
+        baseDescription.appendValue(new Iterable<String>() {
+            @Override
+            public Iterator<String> iterator() {
+                return Arrays.asList("2", "3").iterator();
+            }
+        });
+        assertEquals("<[\"2\", \"3\"]>", result.toString());
+    }
+
+    @Test public void
     bracketsAppendedObjectValue() {
         final Object value = new Object();
         baseDescription.appendValue(value);
diff --git a/hamcrest-library/src/test/java/org/hamcrest/collection/IsEmptyCollectionTest.java b/hamcrest-library/src/test/java/org/hamcrest/collection/IsEmptyCollectionTest.java
@@ -22,7 +22,7 @@ public void testMatchesAnEmptyCollection() {
     }
 
     public void testDoesNotMatchACollectionWithAnItem() {
-        assertMismatchDescription("<[one, three]>", is(createMatcher()), collectionOfValues());
+        assertMismatchDescription("<[\"one\", \"three\"]>", is(createMatcher()), collectionOfValues());
     }
 
     public void testHasAReadableDescription() {

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ public void testMatchesAnEmptyCollection() {`
`22`	`22`	`}`
`23`	`23`
`24`	`24`	`public void testDoesNotMatchACollectionWithAnItem() {`
`25`		`- assertMismatchDescription("<[one, three]>", is(createMatcher()), collectionOfValues());`
	`25`	`+ assertMismatchDescription("<[\"one\", \"three\"]>", is(createMatcher()), collectionOfValues());`
`26`	`26`	`}`
`27`	`27`
`28`	`28`	`public void testHasAReadableDescription() {`