Skip to content

Fix bugs with generating arguments #319

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

Merged
merged 1 commit into from
Sep 5, 2019

Conversation

smyrick
Copy link
Contributor

@smyrick smyrick commented Sep 5, 2019

Fixes #249
Fixes #318

Resolve the two bugs and refactor the function to its own ArgumentBuilder so we can more easily test the bugs

Fixes ExpediaGroup#249
Fixes ExpediaGroup#318

Resolve the two bugs and refactor the function to its own ArgumentBuilder so we can more easily test the bugs
@smyrick smyrick added type: bug Something isn't working changes: patch Changes require a patch version labels Sep 5, 2019
@codecov
Copy link

codecov bot commented Sep 5, 2019

Codecov Report

Merging #319 into master will decrease coverage by 0.08%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #319      +/-   ##
============================================
- Coverage     93.59%   93.51%   -0.09%     
- Complexity      267      269       +2     
============================================
  Files            77       78       +1     
  Lines           968      971       +3     
  Branches        176      176              
============================================
+ Hits            906      908       +2     
  Misses           36       36              
- Partials         26       27       +1
Impacted Files Coverage Δ Complexity Δ
...raphql/generator/extensions/kPropertyExtensions.kt 100% <ø> (ø) 0 <0> (ø) ⬇️
...expedia/graphql/generator/types/FunctionBuilder.kt 100% <100%> (ø) 8 <0> (-2) ⬇️
...expedia/graphql/generator/types/ArgumentBuilder.kt 100% <100%> (ø) 3 <3> (?)
...aphql/generator/extensions/kParameterExtensions.kt 100% <100%> (ø) 0 <0> (ø) ⬇️
...n/com/expedia/graphql/generator/SchemaGenerator.kt 97.56% <100%> (+0.12%) 21 <1> (+1) ⬆️
...dia/graphql/federation/execution/EntityResolver.kt 80.48% <0%> (-2.44%) 6% <0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a13e883...b063f5c. Read the comment docs.

@smyrick smyrick merged commit 148c983 into ExpediaGroup:master Sep 5, 2019
@smyrick smyrick deleted the graphql-id-argument branch September 5, 2019 17:03
smyrick added a commit to smyrick/graphql-kotlin that referenced this pull request Sep 11, 2019
Fixes ExpediaGroup#249
Fixes ExpediaGroup#318

Resolve the two bugs and refactor the function to its own ArgumentBuilder so we can more easily test the bugs
dariuszkuc pushed a commit to dariuszkuc/graphql-kotlin that referenced this pull request Aug 5, 2022
Fixes ExpediaGroup#249
Fixes ExpediaGroup#318

Resolve the two bugs and refactor the function to its own ArgumentBuilder so we can more easily test the bugs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes: patch Changes require a patch version type: bug Something isn't working
Development

Successfully merging this pull request may close these issues.

@GraphQLName does not work on arguments Support GraphQL ID scalar as function arguments and return types
2 participants