[GitHub] [johnzon] rmannibucau commented on a change in pull request #73: Fix InaccessibleObjectException on JDK16+

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org