diff options
author | Jenkins <jenkins@review.openstack.org> | 2012-10-12 18:07:36 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2012-10-12 18:07:36 +0000 |
commit | f182936be1467355e11490726d1b78b928d8d2d0 (patch) | |
tree | afaf803581ec48271c4f5bb9573db7d3823d9f1e /tests | |
parent | fda069b557f6e432b62021db5244e8fdbda42551 (diff) | |
parent | 76751a6b4e096443f7dfea1e5dba595790cc6ec1 (diff) | |
download | oslo-f182936be1467355e11490726d1b78b928d8d2d0.tar.gz oslo-f182936be1467355e11490726d1b78b928d8d2d0.tar.xz oslo-f182936be1467355e11490726d1b78b928d8d2d0.zip |
Merge "Revert "Add support for finer-grained policy decisions""
Diffstat (limited to 'tests')
-rw-r--r-- | tests/unit/test_policy.py | 132 |
1 files changed, 5 insertions, 127 deletions
diff --git a/tests/unit/test_policy.py b/tests/unit/test_policy.py index 3499ea2..a0cae6f 100644 --- a/tests/unit/test_policy.py +++ b/tests/unit/test_policy.py @@ -291,78 +291,6 @@ class OrCheckTestCase(unittest.TestCase): rules[1].assert_called_once_with('target', 'cred') -class ResultCheckTestCase(unittest.TestCase): - def test_init(self): - check = policy.ResultCheck('rule', 'result') - - self.assertEqual(check.rule, 'rule') - self.assertEqual(check.result, 'result') - - def test_str(self): - check = policy.ResultCheck('rule', 'result') - - self.assertEqual(str(check), "'result'=rule") - - def test_call_true(self): - rule = mock.Mock(return_value=True) - check = policy.ResultCheck(rule, 'result') - - self.assertEqual(check('target', 'cred'), 'result') - rule.assert_called_once_with('target', 'cred') - - def test_call_false(self): - rule = mock.Mock(return_value=False) - check = policy.ResultCheck(rule, 'result') - - self.assertEqual(check('target', 'cred'), False) - rule.assert_called_once_with('target', 'cred') - - -class CaseCheckTestCase(unittest.TestCase): - def test_init(self): - check = policy.CaseCheck(['case1', 'case2']) - - self.assertEqual(check.cases, ['case1', 'case2']) - - def test_str(self): - check = policy.CaseCheck(["'case1'=check1", "'case2'=check2"]) - - self.assertEqual(str(check), "case { 'case1'=check1; 'case2'=check2 }") - - def test_call_first(self): - cases = [ - mock.Mock(return_value='case1'), - mock.Mock(return_value='case2'), - ] - check = policy.CaseCheck(cases) - - self.assertEqual(check('target', 'cred'), 'case1') - cases[0].assert_called_once_with('target', 'cred') - self.assertFalse(cases[1].called) - - def test_call_second(self): - cases = [ - mock.Mock(return_value=False), - mock.Mock(return_value='case2'), - ] - check = policy.CaseCheck(cases) - - self.assertEqual(check('target', 'cred'), 'case2') - cases[0].assert_called_once_with('target', 'cred') - cases[1].assert_called_once_with('target', 'cred') - - def test_call_none(self): - cases = [ - mock.Mock(return_value=False), - mock.Mock(return_value=False), - ] - check = policy.CaseCheck(cases) - - self.assertEqual(check('target', 'cred'), False) - cases[0].assert_called_once_with('target', 'cred') - cases[1].assert_called_once_with('target', 'cred') - - class ParseCheckTestCase(unittest.TestCase): def test_false(self): result = policy._parse_check('!') @@ -479,16 +407,15 @@ class ParseListRuleTestCase(unittest.TestCase): class ParseTokenizeTestCase(unittest.TestCase): @mock.patch.object(policy, '_parse_check', lambda x: x) def test_tokenize(self): - exemplar = ("(( ( (((caSe) And)) or ) (check:%(miss)s) not))" - "{}{}=;=;=;'a-string';\"another-string\"") + exemplar = ("(( ( ((() And)) or ) (check:%(miss)s) not)) " + "'a-string' \"another-string\"") expected = [ ('(', '('), ('(', '('), ('(', '('), ('(', '('), ('(', '('), - ('(', '('), ('case', 'caSe'), (')', ')'), ('and', 'And'), + ('(', '('), (')', ')'), ('and', 'And'), (')', ')'), (')', ')'), ('or', 'or'), (')', ')'), ('(', '('), ('check', 'check:%(miss)s'), (')', ')'), ('not', 'not'), - (')', ')'), (')', ')'), ('{', '{'), ('}', '}'), ('{', '{'), - ('}', '}'), ('=', '='), (';', ';'), ('=', '='), (';', ';'), - ('=', '='), (';', ';'), ('string', 'a-string'), (';', ';'), + (')', ')'), (')', ')'), + ('string', 'a-string'), ('string', 'another-string'), ] @@ -695,55 +622,6 @@ class ParseStateTestCase(unittest.TestCase): self.assertEqual(result, [('check', 'not check')]) - @mock.patch.object(policy, 'ResultCheck', lambda x, y: (x, y)) - def test_make_result(self): - state = policy.ParseState() - - result = state._make_result('result', ':', 'check', '/') - - self.assertEqual(result, [ - ('result_expr', ('check', 'result')), - ('/', '/'), - ]) - - def test_make_result_list(self): - state = policy.ParseState() - - result = state._make_result_list('expr1', ';', 'expr2', '/') - - self.assertEqual(result, [ - ('result_list', ['expr1', 'expr2']), - ('/', '/'), - ]) - - def test_extend_result_list(self): - state = policy.ParseState() - - result = state._extend_result_list(['expr1', 'expr2'], ';', 'expr3', - '/') - - self.assertEqual(result, [ - ('result_list', ['expr1', 'expr2', 'expr3']), - ('/', '/'), - ]) - - @mock.patch.object(policy, 'CaseCheck', lambda x: x) - def test_make_case_from_list(self): - state = policy.ParseState() - - result = state._make_case_from_list('case', '{', ['expr1', 'expr2'], - '}') - - self.assertEqual(result, [('case_expr', ['expr1', 'expr2'])]) - - @mock.patch.object(policy, 'CaseCheck', lambda x: x) - def test_make_case_from_expr(self): - state = policy.ParseState() - - result = state._make_case_from_expr('case', '{', 'expr', '}') - - self.assertEqual(result, [('case_expr', ['expr'])]) - class ParseTextRuleTestCase(unittest.TestCase): def test_empty(self): |