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

grpc_json_transcoder: Switch tests to compare JSON objects #16085

Merged
merged 1 commit into from
Apr 20, 2021

Conversation

nareddyt
Copy link
Contributor

Commit Message:
grpc_json_transcoder: Switch tests to compare JSON objects

Additional Description:

  • We should not using string comparisons for JSON equality. Switch to use the pre-existing JSON comparison function instead.
  • Add a hacky way to support parsing root-level JSON arrays. This is needed to compare some streaming JSON responses in the transcoder.

Risk Level: None
Testing: Unit tests for new utility function.
Docs Changes: None
Release Notes: None
Platform Specific Features: None

Signed-off-by: Teju Nareddy nareddyt@google.com

Signed-off-by: Teju Nareddy <nareddyt@google.com>
@nareddyt nareddyt requested a review from lizan as a code owner April 20, 2021 17:52
@yanavlasov yanavlasov merged commit cc98169 into envoyproxy:main Apr 20, 2021
gokulnair pushed a commit to gokulnair/envoy that referenced this pull request May 6, 2021
…y#16085)

Signed-off-by: Teju Nareddy <nareddyt@google.com>
Signed-off-by: Gokul Nair <gnair@twitter.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.

3 participants