Bug: Or on deeper structures breaks

Hi, i found a bug when trying to “or” deeper nested structures:

from strictyaml import load, Map, MapPattern, Str, Int

yaml="""
configuration: 
  test1:
    helptext: "some text"
  test2:
    helptext: "some other text."
    min: 0
"""

schema = Map(
    {
        "configuration": MapPattern(
            Str(), 
            Map({
                "helptext": Str(), 
            }) | Map({
                "helptext": Str(), 
                "min": Int(), 
            }),
            minimum_keys=1
        ),
    }
)

data = load(yaml, schema=schema)
print(data.data)

which results in

strictyaml.exceptions.YAMLValidationError: when expecting an integer
found arbitrary text
  in "<unicode string>", line 5, column 1:
        helptext: some other text.
    ^ (line: 5)

the strange thing is, it “works” when themin validator is a Str() so i guess the validators are applied to the wrong content?

[crdoconnor]

Thanks. And thanks once again for pointing it out.

I’m working on preventing the above bug from occurring by preventing the combination of Maps / Seqs using the Or validator. I’ll put a message hinting that the end user should use revalidation for their purposes so future users won’t fall down the same rabbit hole.

[kumekay]

I have a small question, Though it’s not stated that project version is following semantic versioning, my general feeling says that patch version update shouldn’t introduce breaking changes. Do you have some policy about versioning for this project?