Skip to content

Commit 6e6da86

Browse files
committed
Remove debug statement from test (accidentally left); add a comment
1 parent c806f26 commit 6e6da86

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/main/java/com/fasterxml/jackson/databind/deser/std/CollectionDeserializer.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -312,6 +312,8 @@ protected Collection<Object> _deserializeFromString(JsonParser p, Deserializatio
312312
return (Collection<Object>) _deserializeFromEmptyString(
313313
p, ctxt, act, rawTargetType, "empty String (\"\")");
314314
}
315+
// note: `CoercionAction.Fail` falls through because we may need to allow
316+
// `ACCEPT_SINGLE_VALUE_AS_ARRAY` handling later on
315317
}
316318
// 26-Mar-2021, tatu: Some day is today; as per [dataformat-xml#460],
317319
// we do need to support blank String too...
@@ -322,6 +324,8 @@ else if (_isBlank(value)) {
322324
return (Collection<Object>) _deserializeFromEmptyString(
323325
p, ctxt, act, rawTargetType, "blank String (all whitespace)");
324326
}
327+
// note: `CoercionAction.Fail` falls through because we may need to allow
328+
// `ACCEPT_SINGLE_VALUE_AS_ARRAY` handling later on
325329
}
326330
return handleNonArray(p, ctxt, createDefaultInstance(ctxt));
327331
}

src/test/java/com/fasterxml/jackson/databind/convert/CoerceContainersTest.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,7 @@ public void testScalarCollections() throws Exception
4141
// verifyException(e, "Cannot coerce empty String");
4242
verifyException(e, "Cannot deserialize value of type");
4343
verifyException(e, "from String value");
44-
e.printStackTrace();
4544
}
46-
4745

4846
List<Double> result = _readWithCoercion(listType);
4947
assertNotNull(result);

0 commit comments

Comments
 (0)