mirror of
https://github.com/pre-commit/pre-commit.git
synced 2026-01-13 20:40:08 -06:00
Deepcopy defaults to prevent weird references in yaml.
This commit is contained in:
@@ -16,7 +16,9 @@ def extend_with_default(validator_class):
|
||||
|
||||
for property, subschema in properties.iteritems():
|
||||
if "default" in subschema:
|
||||
instance.setdefault(property, subschema["default"])
|
||||
instance.setdefault(
|
||||
property, copy.deepcopy(subschema["default"]),
|
||||
)
|
||||
|
||||
return jsonschema.validators.extend(
|
||||
validator_class, {"properties" : set_defaults},
|
||||
|
||||
@@ -49,3 +49,10 @@ def test_apply_defaults_deep():
|
||||
},
|
||||
)
|
||||
assert ret == {'foo': {'bar': {'baz': 'herp'}}}
|
||||
|
||||
|
||||
def test_apply_defaults_copies():
|
||||
schema = {'properties': {'foo': {'default': []}}}
|
||||
ret1 = apply_defaults({}, schema)
|
||||
ret2 = apply_defaults({}, schema)
|
||||
assert ret1['foo'] is not ret2['foo']
|
||||
|
||||
Reference in New Issue
Block a user