Skip to content

Add parameterized test for GraphQL Optional field decoding across codecs#3275

Merged
velo merged 3 commits intomasterfrom
test/graphql-decoder-optional-field-codecs
Mar 18, 2026
Merged

Add parameterized test for GraphQL Optional field decoding across codecs#3275
velo merged 3 commits intomasterfrom
test/graphql-decoder-optional-field-codecs

Conversation

@velo
Copy link
Copy Markdown
Member

@velo velo commented Mar 17, 2026

Summary

  • Adds a parameterized test (GraphqlDecoderOptionalFieldCodecTest) that verifies Optional<T> record fields are correctly decoded by the GraphQL decoder using MockClient
  • Tests 3 payload scenarios: explicit null ("step": null), missing field, and present value — across Jackson and Jackson3 codecs
  • Adds feign-mock and feign-jackson3 as test dependencies to the graphql module

Test plan

  • All 6 parameterized test cases pass (3 payloads × 2 codecs)
  • Full graphql module test suite passes (51 tests)

Gson, Moshi, and Fastjson2 were evaluated but excluded — they don't support Optional<T> fields in records.

🤖 Generated with Claude Code

velo added 3 commits March 17, 2026 12:36
Signed-off-by: Marvin Froeder <velo.br@gmail.com>
Signed-off-by: Marvin Froeder <velo.br@gmail.com>
Signed-off-by: Marvin Froeder <velo.br@gmail.com>
@velo velo merged commit 668a796 into master Mar 18, 2026
4 checks passed
@velo velo deleted the test/graphql-decoder-optional-field-codecs branch March 18, 2026 09:50
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.

1 participant