Codecov Report

Merging #73 into master will increase coverage by 0.02%.
The diff coverage is 95%.

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
+ Coverage   98.22%   98.24%   +0.02%     
==========================================
  Files          54       54              
  Lines        1238     1253      +15     
==========================================
+ Hits         1216     1231      +15     
  Misses         22       22

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 4731504...0cbbdb0. Read the comment docs.

Looks good. Should allOf/oneOf have types? https://github.com/p1c2u/openapi-core/pull/73/files#diff-431dcafe8bb0f93322e443187632f52fR198

Looking at examples this is not the case https://swagger.io/docs/specification/data-models/oneof-anyof-allof-not/.

For #65 I tested using type: string like:

utctime:
  type: string
  oneOf:
    - type: string
      enum: [always, now]
    - type: string
      format: date-time

This passed validation in openapi-core but plugging it into swagger-ui yeilds a convoluted data structure.

That's why I didn't merge it still. I'm not 100% sure how it should work.

I need to add one change to any type unmarshaling.

This PR won't resolve #65 . It needs separate PR.