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

Update test_schema to mirror the new ValidationErrors in 3.0.0 #52588

Closed
wants to merge 33 commits into from

Conversation

Ch3LL
Copy link
Contributor

@Ch3LL Ch3LL commented Apr 17, 2019

What does this PR do?

Fixes the failing tests: unit.utils.test_schema

In jsonschema version 3.0.0 the validation errors were updated to grab the best match, instead of the default is not valid under any of the given schemas in this commit: python-jsonschema/jsonschema@17fb9cb

to fix this issue: python-jsonschema/jsonschema#498

This updates the tests to check for the error's with more detail when testing with >= 3.0.0

What issues does this PR fix or reference?

Fixes #52085

Tests written?

No- Fixes current tests failing on osx

Commits signed with GPG?

Yes

s0undt3ch and others added 30 commits March 6, 2019 06:57
Bring the Nox changes from 2017.7.9 to 2017.7
Conflicts:
* requirements/static/centos-6.txt
* requirements/static/centos-7.txt
* requirements/static/debian-8.txt
* requirements/static/debian-9.txt
* requirements/static/fedora-28.txt
* requirements/static/fedora-29.txt
* requirements/static/opensuse-42.txt
* requirements/static/opensuse-leap-15.txt
* requirements/static/ubuntu-14.04.txt
* requirements/static/ubuntu-16.04.txt
* requirements/static/ubuntu-18.04.txt
…ate-author

Update my contact details, as the previous email address is no longer in use
@Ch3LL Ch3LL changed the base branch from 2017.7 to 2017.7.9 April 18, 2019 21:08
@Ch3LL Ch3LL closed this Apr 19, 2019
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.

4 participants