Plugin.__init__() now checks that subclass hasn't defined attributes that conflict with the logger methods; added corresponding unit test

1 files changed, 8 insertions, 0 deletions

diff --git a/tests/test_ipalib/test_plugable.py b/tests/test_ipalib/test_plugable.py
index b381675b6..b05943235 100644
--- a/tests/test_ipalib/test_plugable.py
+++ b/tests/test_ipalib/test_plugable.py
@@ -337,6 +337,14 @@ class test_Plugin(ClassChecker):
         assert o.doc is None
         assert o.summary == '<%s>' % o.fullname
 
+        # Test that Plugin makes sure the subclass hasn't defined attributes
+        # whose names conflict with the logger methods set in Plugin.__init__():
+        class check(self.cls):
+            info = 'whatever'
+        e = raises(StandardError, check)
+        assert str(e) == \
+            "check.info attribute ('whatever') conflicts with Plugin logger"
+
     def test_implements(self):
         """
         Test the `ipalib.plugable.Plugin.implements` classmethod.