Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

testkit-backend: fix TestkitListDeserializer #893

Merged

Conversation

bigmontz
Copy link
Contributor

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.

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.
@bigmontz bigmontz requested a review from gjmwoods May 18, 2021 10:40
@bigmontz bigmontz merged commit 47e6f0d into neo4j:4.3 May 18, 2021
@bigmontz bigmontz deleted the 4.3-tk-backend-fix-TestkitListDeserializer branch May 18, 2021 13:05
gjmwoods pushed a commit to gjmwoods/neo4j-java-driver that referenced this pull request May 26, 2021
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.
gjmwoods added a commit that referenced this pull request May 27, 2021
* Transaction Rollback (#887)

* Implement backend driver feature list (#889)

* 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.

Co-authored-by: Dmitriy Tverdiakov <11927660+injectives@users.noreply.github.com>
Co-authored-by: Antonio Barcélos <antonio.barcelos@neo4j.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants