Permalink
Browse files

The wild default has appeared! Use errors instead of warnings for unk…

…nwon permissions.
  • Loading branch information...
1 parent ef4e297 commit c0cbb126c14f6f28b49d721242bf48e05f9f4397 @mattbasta committed Dec 28, 2012
Showing with 2 additions and 1 deletion.
  1. +1 −0 appvalidator/specs/webapps.py
  2. +1 −1 tests/test_webapp.py
@@ -125,6 +125,7 @@ class WebappSpec(Spec):
"permissions": {
"allowed_nodes": PERMISSIONS,
"expected_type": dict,
+ "unknown_node_level": "error",
"child_nodes": {
"*": {
"expected_type": dict,
View
@@ -664,7 +664,7 @@ def test_permissions_extra_invalid(self):
self.set_permissions()
self.data["permissions"]["foo"] = {"description": "lol"}
self.analyze()
- self.assert_failed(with_warnings=True)
+ self.assert_failed(with_errors=True)
def test_permissions_missing_desc(self):
self.set_permissions()

0 comments on commit c0cbb12

Please sign in to comment.