diff options
author | Jason Gerard DeRose <jderose@redhat.com> | 2008-09-21 21:50:56 +0000 |
---|---|---|
committer | Jason Gerard DeRose <jderose@redhat.com> | 2008-09-21 21:50:56 +0000 |
commit | f73d976bdacae37557f0b2ccfa6da01ea58c685d (patch) | |
tree | 78f98c60cc3f268b3354ae6fda69d296ff330ae1 /ipalib/tests/test_plugable.py | |
parent | 5872221bd49dda962391ddfb88f22e86bf72afec (diff) | |
download | freeipa.git-f73d976bdacae37557f0b2ccfa6da01ea58c685d.tar.gz freeipa.git-f73d976bdacae37557f0b2ccfa6da01ea58c685d.tar.xz freeipa.git-f73d976bdacae37557f0b2ccfa6da01ea58c685d.zip |
307: Split Plugin.finalize() into two steps 1) Plugin.set_api() and 2) Plugin.finalize(); updated unit tests
Diffstat (limited to 'ipalib/tests/test_plugable.py')
-rw-r--r-- | ipalib/tests/test_plugable.py | 22 |
1 files changed, 3 insertions, 19 deletions
diff --git a/ipalib/tests/test_plugable.py b/ipalib/tests/test_plugable.py index 3972cfa9..6c796f9e 100644 --- a/ipalib/tests/test_plugable.py +++ b/ipalib/tests/test_plugable.py @@ -405,26 +405,10 @@ class test_Plugin(ClassChecker): """ Tests the `plugable.Plugin.finalize` method. """ - api = 'the api instance' o = self.cls() - assert read_only(o, 'name') == 'Plugin' - assert repr(o) == '%s.Plugin()' % plugable.__name__ - assert read_only(o, 'api') is None - raises(AssertionError, o.finalize, None) - o.finalize(api) - assert read_only(o, 'api') is api - raises(AssertionError, o.finalize, api) - - class some_plugin(self.cls): - pass - sub = some_plugin() - assert read_only(sub, 'name') == 'some_plugin' - assert repr(sub) == '%s.some_plugin()' % __name__ - assert read_only(sub, 'api') is None - raises(AssertionError, sub.finalize, None) - sub.finalize(api) - assert read_only(sub, 'api') is api - raises(AssertionError, sub.finalize, api) + assert not o.__islocked__() + o.finalize() + assert o.__islocked__() class test_PluginProxy(ClassChecker): |