Skip to content

Added schema version detection with schema tag#306

Merged
stevehu merged 1 commit intonetworknt:masterfrom
Subhajitdas298:master
Jun 25, 2020
Merged

Added schema version detection with schema tag#306
stevehu merged 1 commit intonetworknt:masterfrom
Subhajitdas298:master

Conversation

@Subhajitdas298
Copy link
Copy Markdown
Contributor

For #305

Additional utility to auto detect version. Will make life easier switching between schema versions, by updating schema JSON only, without touching the java code.

Added test cases, as necessary.

Additionally, some small code clean up.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #306 into master will not change coverage.
The diff coverage is 76.08%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #306   +/-   ##
=========================================
  Coverage     70.96%   70.96%           
- Complexity      636      643    +7     
=========================================
  Files            73       74    +1     
  Lines          2445     2445           
  Branches        513      515    +2     
=========================================
  Hits           1735     1735           
  Misses          506      506           
  Partials        204      204           
Impacted Files Coverage Δ Complexity Δ
...n/java/com/networknt/schema/JsonSchemaFactory.java 75.92% <63.63%> (-1.35%) 29.00 <4.00> (ø)
...java/com/networknt/schema/JsonSchemaException.java 80.00% <80.00%> (ø) 4.00 <4.00> (ø)
...java/com/networknt/schema/SpecVersionDetector.java 92.85% <92.85%> (ø) 7.00 <7.00> (?)

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 bea2d41...2fa85e6. Read the comment docs.

@stevehu stevehu merged commit b5e962d into networknt:master Jun 25, 2020
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.

3 participants