rmannibucau commented on a change in pull request #73:
URL: https://github.com/apache/johnzon/pull/73#discussion_r560041836
##########
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:
Hi @reta,
I like this version more but it still fixes the consequence instead of the
source of the issue which is an unexpected branch in the mapper (class for
collection/primitive/map/throwable/jdk-class) so even if this flavor can be
kept I still think we should fix first the cause then maybe review this flavor.
Wdyt?
----------------------------------------------------------------
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]
