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

Fix Not Allowing to Pass null as Variable Input Value #616

Merged

Conversation

DimuthuMadushan
Copy link
Contributor

Purpose

Fixes: #2696

Examples

N/A

Checklist

  • Linked to an issue
  • Updated the changelog
  • Added tests

@codecov
Copy link

codecov bot commented Feb 21, 2022

Codecov Report

Merging #616 (8de4b58) into master (834fee6) will increase coverage by 0.01%.
The diff coverage is 97.56%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #616      +/-   ##
============================================
+ Coverage     93.89%   93.91%   +0.01%     
  Complexity      347      347              
============================================
  Files            54       54              
  Lines          3671     3682      +11     
  Branches       1998     2006       +8     
============================================
+ Hits           3447     3458      +11     
  Misses          197      197              
  Partials         27       27              
Impacted Files Coverage Δ
ballerina/validator_visitor.bal 96.35% <95.00%> (+0.07%) ⬆️
ballerina/common_utils.bal 94.44% <100.00%> (+0.02%) ⬆️
ballerina/variable_validator.bal 98.25% <100.00%> (+<0.01%) ⬆️

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 834fee6...8de4b58. Read the comment docs.

ballerina/variable_validator.bal Outdated Show resolved Hide resolved
ballerina/validator_visitor.bal Outdated Show resolved Hide resolved
@DimuthuMadushan DimuthuMadushan merged commit a197fcf into ballerina-platform:master Feb 23, 2022
@DimuthuMadushan DimuthuMadushan deleted the fix-null-value branch February 23, 2022 05:28
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.

Cannot Pass null with GraphQL Variable Values
2 participants