Support for default rule on mapping? about pykwalify HOT 6 CLOSED

@grovduck Can you provide an example data that you run against?

from pykwalify.

@Grokzen, sure, here's the schema file (strata_schema.yaml):

type: map
mapping:
  =:
    type: seq
    required: true
    sequence:
      - type: map
        mapping:
          'code':
            type: int
            required: true
            unique: true
          'key':
            type: str
            required: true
          'alias':
            type: str
            required: true

and here's a snippet of the data that causes a failure (strata_definitions.yaml):

OWNERSHIP:
  - code: 1
    key: BLM-BURNS
    alias: BLM-BURNS

WHT:
  - code: 1000
    key: LP
    alias: LP
  - code: 2000
    key: PPDF
    alias: PPDF

SC_10:
  - code: 0
    key: NO_SNAGS
    alias: NO_SNAGS

Here's the command and error message:

> pykwalify -d strata_definitions.yaml -s strata_schema.yaml

pykwalify.errors.SchemaError: <SchemaError: error code 2: Schema validation failed:
 - Cannot find required key '='. Path: ''.
 - Key 'WHT' was not defined. Path: ''.
 - Key 'SC_10' was not defined. Path: ''.
 - Key 'OWNERSHIP' was not defined. Path: ''.: Path: '/'>

I'm using pykwalify==1.5.1, PyYAML==3.11 on Windows 7 using Python 2.7 64-bit. Thanks for your help.

from pykwalify.

Aha, i see what the problem is, that features is just missing here. It should also be rather easy to implement and i can look at implementing it over the weekend.

from pykwalify.

Hi @grovduck. This feature is implemented in this commit 94f326c. I created a success and a fail test case to show that it should work and i did not have to change any existing tests to make it compatible and that is a good sign it will work well. Please play around with it and if you find any issues with the implementation please open a new issue.

Closing this issue as fixed/implemented. Thanks for pointing out this missing feature :)

from pykwalify.

Hi @Grokzen, sorry for the late response. Thanks so much for providing this feature. Tested and confirmed it's now validating correctly against my schema file

from pykwalify.

@grovduck thanks for confirming.

from pykwalify.