Skip to content

Immutable validators + automated schema testing#1

Merged
dkraczkowski merged 17 commits intomainfrom
improve-json-schema-support
Nov 10, 2021
Merged

Immutable validators + automated schema testing#1
dkraczkowski merged 17 commits intomainfrom
improve-json-schema-support

Conversation

@dkraczkowski
Copy link
Copy Markdown
Contributor

No description provided.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 10, 2021

Codecov Report

Merging #1 (0fdf562) into main (05ade19) will decrease coverage by 2.11%.
The diff coverage is 80.53%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main       #1      +/-   ##
==========================================
- Coverage   93.20%   91.09%   -2.12%     
==========================================
  Files          31       31              
  Lines        1885     1943      +58     
==========================================
+ Hits         1757     1770      +13     
- Misses        128      173      +45     
Impacted Files Coverage Δ
opyapi/__version__.py 0.00% <0.00%> (ø)
tests/fixtures/generated_dtos.py 100.00% <ø> (ø)
tests/test_suite.py 39.47% <39.47%> (ø)
opyapi/json_schema.py 84.93% <40.00%> (-1.91%) ⬇️
opyapi/validators/object_validators.py 92.85% <50.00%> (-1.59%) ⬇️
opyapi/validators/number_validators.py 93.75% <66.66%> (-1.91%) ⬇️
opyapi/validators/__init__.py 72.13% <69.04%> (-2.87%) ⬇️
opyapi/schema_validator.py 90.82% <86.88%> (-3.71%) ⬇️
opyapi/validators/array_validators.py 90.27% <88.23%> (-0.64%) ⬇️
opyapi/validators/format_validators.py 97.45% <97.67%> (+3.10%) ⬆️
... and 12 more

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 05ade19...0fdf562. Read the comment docs.

@dkraczkowski dkraczkowski merged commit 958e052 into main Nov 10, 2021
@dkraczkowski dkraczkowski deleted the improve-json-schema-support branch November 10, 2021 22:34
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.

2 participants