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

Use mapping to directly validate mapped schema #338

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Commits on Apr 21, 2021

  1. Avoid failfast and return proper error when mapping is available

    (cherry picked from commit d743e52)
    (cherry picked from commit 818b0df)
    (cherry picked from commit 7b15e5e)
    riccardomanfrin committed Apr 21, 2021
    Configuration menu
    Copy the full SHA
    45d4919 View commit details
    Browse the repository at this point in the history
  2. Revert "Adds oneOf/discriminator/mapping management (getkin#321)"

    This reverts commit 1286d06.
    
    # Conflicts:
    #	openapi3filter/validation_discriminator_test.go
    
    (cherry picked from commit a6bc0e1)
    (cherry picked from commit 011aa82)
    riccardomanfrin committed Apr 21, 2021
    Configuration menu
    Copy the full SHA
    be8af26 View commit details
    Browse the repository at this point in the history
  3. Fixup missing Test prefix

    (cherry picked from commit 56c30a7)
    (cherry picked from commit 393fdec)
    riccardomanfrin committed Apr 21, 2021
    Configuration menu
    Copy the full SHA
    9aa5c5a View commit details
    Browse the repository at this point in the history
  4. Adds implicit mapping case

    (cherry picked from commit a3ea3ba)
    (cherry picked from commit dfd1108)
    riccardomanfrin committed Apr 21, 2021
    Configuration menu
    Copy the full SHA
    146d4b9 View commit details
    Browse the repository at this point in the history
  5. Cleaner [I'll squash this later]

    (cherry picked from commit 1b40d29)
    (cherry picked from commit f4f135f)
    riccardomanfrin committed Apr 21, 2021
    Configuration menu
    Copy the full SHA
    b343c1d View commit details
    Browse the repository at this point in the history

Commits on Apr 23, 2021

  1. Compliance

    riccardomanfrin committed Apr 23, 2021
    Configuration menu
    Copy the full SHA
    f937147 View commit details
    Browse the repository at this point in the history
  2. Revert "Fixup missing Test prefix"

    This reverts commit 9aa5c5a.
    riccardomanfrin committed Apr 23, 2021
    Configuration menu
    Copy the full SHA
    7800000 View commit details
    Browse the repository at this point in the history

Commits on Apr 26, 2021

  1. Now test passes

    This is what schema.go:856 else is for
    
    Spec quote:
    
        ``It is implied, that the property to which discriminator refers, contains the
        name of the target schema. In the example above, the objectType property should
        contain either simpleObject, or complexObject string.''*/
    riccardomanfrin committed Apr 26, 2021
    Configuration menu
    Copy the full SHA
    781e68f View commit details
    Browse the repository at this point in the history