rmannibucau commented on a change in pull request #73:
URL: https://github.com/apache/johnzon/pull/73#discussion_r559389372
##########
File path:
johnzon-jsonb/src/test/java/org/apache/johnzon/jsonb/JsonbWriteTest.java
##########
@@ -92,6 +94,14 @@ public void list() {
assertEquals("[\"a\",\"b\"]",
JsonbProvider.provider().create().build().toJson(map));
}
+ @Test
+ public void listOfSimple() {
Review comment:
Interesting, just read on the phone but our list detection is something
along "is parameterized and raw type is a collection or is an array" so passing
List.class was moving it to a fallback handling and lead to the unexpected
stack you get.
Guess it is what we must fix and probably for maps too.
Hope it makes sense
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
