diff options
| author | melissaml <ma.lei@99cloud.net> | 2016-10-14 10:17:18 +0800 |
|---|---|---|
| committer | melissaml <ma.lei@99cloud.net> | 2016-10-14 10:18:35 +0800 |
| commit | 573c52bb1c4f67e49bcacd1c1ce5c2af672e0f76 (patch) | |
| tree | e6427cca374690b2911a472eeaa9eaa9932813a1 | |
| parent | 890e2bfa90854b8535b8c9aecf6c4e0377b4156e (diff) | |
| download | oslo-policy-573c52bb1c4f67e49bcacd1c1ce5c2af672e0f76.tar.gz | |
Change assertTrue(isinstance()) by optimal assert1.16.0
Some of tests use different method of
assertTrue(isinstance(A, B)) or assertEqual(type(A), B).
The correct way is to use assertIsInstance(A, B) provided
by testtools
Change-Id: Ia5156aef65da766520485bbc9495b61918407f22
| -rw-r--r-- | oslo_policy/tests/test_policy.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/oslo_policy/tests/test_policy.py b/oslo_policy/tests/test_policy.py index dc3ede7..6b077bf 100644 --- a/oslo_policy/tests/test_policy.py +++ b/oslo_policy/tests/test_policy.py @@ -772,7 +772,7 @@ class BaseCheckTypesTestCase(base.PolicyBaseTestCase): class RuleDefaultTestCase(base.PolicyBaseTestCase): def test_rule_is_parsed(self): opt = policy.RuleDefault(name='foo', check_str='rule:foo') - self.assertTrue(isinstance(opt.check, _checks.BaseCheck)) + self.assertIsInstance(opt.check, _checks.BaseCheck) self.assertEqual('rule:foo', str(opt.check)) def test_str(self): |