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

common: Add test to check that a required field that is allowed to be null passes validation #621

Closed
dilyand opened this issue Jun 9, 2022 · 0 comments
Milestone

Comments

@dilyand
Copy link
Contributor

dilyand commented Jun 9, 2022

If we have a schema like:

{
  (...),
  "properties": {
    "a": { "type": ["boolean", "null"] }
  },
  "additionalProperties": false,
  "required": [ "a" ]
}

we should expect a payload like:

{
  "a": null
}

to pass validation.

This test demonstrates the behaviour with the client_session context, which has a property like that and is one of our OOB contexts (ie, users have no control over it).

@dilyand dilyand added this to the 3.2.0 milestone Jun 9, 2022
dilyand added a commit that referenced this issue Jun 9, 2022
@istreeter istreeter changed the title Add test to check that a required field that is allowed to be null passes validation common: Add test to check that a required field that is allowed to be null passes validation Jun 10, 2022
istreeter pushed a commit that referenced this issue Jun 10, 2022
dilyand added a commit that referenced this issue Jun 23, 2022
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

No branches or pull requests

1 participant