Fixed issue with 'required' not being set properly and added test

alecthomas · Mar 31, 2020 · 415042e · 415042e
1 parent 82de2ef
commit 415042e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/voluptuous/tests/tests.py b/voluptuous/tests/tests.py
@@ -16,6 +16,13 @@
 from voluptuous.util import u, Capitalize, Lower, Strip, Title, Upper
 
 
+def test_new_required_test():
+    schema = Schema({
+        'my_key': All(int, Range(1, 20)),
+    }, required=True)
+    assert_true(schema.required)
+
+
 def test_exact_sequence():
     schema = Schema(ExactSequence([int, int]))
     try:

diff --git a/voluptuous/validators.py b/voluptuous/validators.py
@@ -197,10 +197,12 @@ def __init__(self, *validators, **kwargs):
 
     def __voluptuous_compile__(self, schema):
         self._compiled = []
+        old_required = schema.required
         self.schema = schema
         for v in self.validators:
             schema.required = self.required
             self._compiled.append(schema._compile(v))
+        schema.required = old_required
         return self._run
 
     def _run(self, path, value):