Finish up this coverage stuff.

1 files changed, 29 insertions, 33 deletions

diff --git a/tests/acl_test.py b/tests/acl_test.py
index 1e9dcbf..a1d827e 100755
--- a/tests/acl_test.py
+++ b/tests/acl_test.py
@@ -4,17 +4,16 @@ import re
 import unittest
 
 import acl
-import unittest_utils    # Needed for --unittests_ignore_perf flag
+import unittest_utils  # Needed for --unittests_ignore_perf flag
 
 
 class TestSimpleACL(unittest.TestCase):
-
     def test_set_based_acl(self):
         even = acl.SetBasedACL(
-            allow_set = set([2, 4, 6, 8, 10]),
-            deny_set = set([1, 3, 5, 7, 9]),
-            order_to_check_allow_deny = acl.Order.ALLOW_DENY,
-            default_answer = False
+            allow_set=set([2, 4, 6, 8, 10]),
+            deny_set=set([1, 3, 5, 7, 9]),
+            order_to_check_allow_deny=acl.Order.ALLOW_DENY,
+            default_answer=False,
         )
         self.assertTrue(even(2))
         self.assertFalse(even(3))
@@ -22,9 +21,9 @@ class TestSimpleACL(unittest.TestCase):
 
     def test_wildcard_based_acl(self):
         a_or_b = acl.StringWildcardBasedACL(
-            allowed_patterns = ['a*', 'b*'],
-            order_to_check_allow_deny = acl.Order.ALLOW_DENY,
-            default_answer = False
+            allowed_patterns=['a*', 'b*'],
+            order_to_check_allow_deny=acl.Order.ALLOW_DENY,
+            default_answer=False,
         )
         self.assertTrue(a_or_b('aardvark'))
         self.assertTrue(a_or_b('baboon'))
@@ -32,12 +31,9 @@ class TestSimpleACL(unittest.TestCase):
 
     def test_re_based_acl(self):
         weird = acl.StringREBasedACL(
-            denied_regexs = [
-                re.compile('^a.*a$'),
-                re.compile('^b.*b$')
-            ],
-            order_to_check_allow_deny = acl.Order.DENY_ALLOW,
-            default_answer = True
+            denied_regexs=[re.compile('^a.*a$'), re.compile('^b.*b$')],
+            order_to_check_allow_deny=acl.Order.DENY_ALLOW,
+            default_answer=True,
         )
         self.assertTrue(weird('aardvark'))
         self.assertFalse(weird('anaconda'))
@@ -46,19 +42,19 @@ class TestSimpleACL(unittest.TestCase):
 
     def test_compound_acls_disjunction(self):
         a_b_c = acl.StringWildcardBasedACL(
-            allowed_patterns = ['a*', 'b*', 'c*'],
-            order_to_check_allow_deny = acl.Order.ALLOW_DENY,
-            default_answer = False
+            allowed_patterns=['a*', 'b*', 'c*'],
+            order_to_check_allow_deny=acl.Order.ALLOW_DENY,
+            default_answer=False,
         )
         c_d_e = acl.StringWildcardBasedACL(
-            allowed_patterns = ['c*', 'd*', 'e*'],
-            order_to_check_allow_deny = acl.Order.ALLOW_DENY,
-            default_answer = False
+            allowed_patterns=['c*', 'd*', 'e*'],
+            order_to_check_allow_deny=acl.Order.ALLOW_DENY,
+            default_answer=False,
         )
         disjunction = acl.AnyCompoundACL(
-            subacls = [a_b_c, c_d_e],
-            order_to_check_allow_deny = acl.Order.ALLOW_DENY,
-            default_answer = False,
+            subacls=[a_b_c, c_d_e],
+            order_to_check_allow_deny=acl.Order.ALLOW_DENY,
+            default_answer=False,
         )
         self.assertTrue(disjunction('aardvark'))
         self.assertTrue(disjunction('caribou'))
@@ -67,19 +63,19 @@ class TestSimpleACL(unittest.TestCase):
 
     def test_compound_acls_conjunction(self):
         a_b_c = acl.StringWildcardBasedACL(
-            allowed_patterns = ['a*', 'b*', 'c*'],
-            order_to_check_allow_deny = acl.Order.ALLOW_DENY,
-            default_answer = False
+            allowed_patterns=['a*', 'b*', 'c*'],
+            order_to_check_allow_deny=acl.Order.ALLOW_DENY,
+            default_answer=False,
         )
         c_d_e = acl.StringWildcardBasedACL(
-            allowed_patterns = ['c*', 'd*', 'e*'],
-            order_to_check_allow_deny = acl.Order.ALLOW_DENY,
-            default_answer = False
+            allowed_patterns=['c*', 'd*', 'e*'],
+            order_to_check_allow_deny=acl.Order.ALLOW_DENY,
+            default_answer=False,
         )
         conjunction = acl.AllCompoundACL(
-            subacls = [a_b_c, c_d_e],
-            order_to_check_allow_deny = acl.Order.ALLOW_DENY,
-            default_answer = False,
+            subacls=[a_b_c, c_d_e],
+            order_to_check_allow_deny=acl.Order.ALLOW_DENY,
+            default_answer=False,
         )
         self.assertFalse(conjunction('aardvark'))
         self.assertTrue(conjunction('caribou'))