search

alecthomas / voluptuous

CONTRIBUTIONS ONLY: Voluptuous, despite the name, is a Python data validation library.
https://pypi.org/project/voluptuous
BSD 3-Clause "New" or "Revised" License
1.82k stars 218 forks source link

`Required` flag not set properly after #379 fix #419

Closed EarQuack closed 4 years ago

EarQuack commented 4 years ago

Following test (this will be a new test, if accepted) fails:

def test_new_required_test():
    schema = Schema({
        'mykey': All(int, Range(1, 20)),
    }, required=True)
    assert_true(schema.required)

Suspect problem was introduced in #379

Will propose a solution and issue pull request to fix

EarQuack commented 4 years ago

@alecthomas Please assign me #419 and I will propose solution and passing test. Best regards!

tusharmakkar08 commented 4 years ago

Assigned @EarQuack