testkit-backend: fix TestkitListDeserializer (#893)

Enabling `list of maps` deserialization by adding a special case for treating this data type.

The extras `nextToken` calls were removed because this was making the deserializer leaves the parent object, this way breaking the `map of lists` deserialization.

Author: bigmontz

testkit-backend/src/main/java/neo4j/org/testkit/backend/messages/requests/deserializer/TestkitListDeserializer.java

@@ -32,10 +32,12 @@
 
 public class TestkitListDeserializer extends StdDeserializer<List<?>>
 {
+    private final TestkitCypherParamDeserializer mapDeserializer;
 
     public TestkitListDeserializer()
     {
         super( List.class );
+        mapDeserializer = new TestkitCypherParamDeserializer();
     }
 
     @Override
@@ -88,14 +90,17 @@ public List<?> deserialize( JsonParser p, DeserializationContext ctxt ) throws I
                     }
                     else
                     {
-                        result.add( p.readValueAs( mapValueType ) );
+                        if ( paramType.equals( "CypherMap" ) ) // special recursive case for maps
+                        {
+                            result.add( mapDeserializer.deserialize( p, ctxt ) );
+                        } else {
+                            result.add( p.readValueAs( mapValueType ) );
+                        }
                     }
                 }
             }
             nextToken = p.nextToken();
         }
-        p.nextToken();
-        p.nextToken();
         return result;
     }
 }