From 3d6ab69b46e5be32af94ecdfb5a696973eeaf7c4 Mon Sep 17 00:00:00 2001
From: Jason Gerard DeRose <jderose@redhat.com>
Date: Wed, 24 Sep 2008 22:19:43 +0000
Subject: [PATCH] 351: Removed Object.Method property and added in its place
 Object.methods instance attribute

---
 ipalib/frontend.py            | 9 +++------
 ipalib/tests/test_frontend.py | 9 ++++-----
 2 files changed, 7 insertions(+), 11 deletions(-)

diff --git a/ipalib/frontend.py b/ipalib/frontend.py
index 80579b7b3..c3b1707b2 100644
--- a/ipalib/frontend.py
+++ b/ipalib/frontend.py
@@ -511,11 +511,11 @@ class Command(plugable.Plugin):
 
 class Object(plugable.Plugin):
     __public__ = frozenset((
-        'Method',
+        'methods',
         'Property',
         'params'
     ))
-    __Method = None
+    methods = None
     __Property = None
     takes_params = tuple()
 
@@ -528,9 +528,6 @@ class Object(plugable.Plugin):
         for param in self.takes_params:
             yield create_param(param)
 
-    def __get_Method(self):
-        return self.__Method
-    Method = property(__get_Method)
 
     def __get_Property(self):
         return self.__Property
@@ -538,7 +535,7 @@ class Object(plugable.Plugin):
 
     def set_api(self, api):
         super(Object, self).set_api(api)
-        self.__Method = self.__create_namespace('Method')
+        self.methods = self.__create_namespace('Method')
         self.__Property = self.__create_namespace('Property')
 
     def __create_namespace(self, name):
diff --git a/ipalib/tests/test_frontend.py b/ipalib/tests/test_frontend.py
index 06d6b8cb1..f0ba524d5 100644
--- a/ipalib/tests/test_frontend.py
+++ b/ipalib/tests/test_frontend.py
@@ -760,7 +760,6 @@ class test_Object(ClassChecker):
 
     def test_class(self):
         assert self.cls.__bases__ == (plugable.Plugin,)
-        assert type(self.cls.Method) is property
         assert type(self.cls.Property) is property
 
     def test_init(self):
@@ -768,7 +767,7 @@ class test_Object(ClassChecker):
         Tests the `frontend.Object.__init__` method.
         """
         o = self.cls()
-        assert read_only(o, 'Method') is None
+        assert o.methods is None
         assert read_only(o, 'Property') is None
 
     def test_set_api(self):
@@ -798,13 +797,13 @@ class test_Object(ClassChecker):
 
         cnt = 10
         formats = dict(
-            Method='method_%d',
+            methods='method_%d',
             Property='property_%d',
         )
 
         class api(object):
             Method = plugable.NameSpace(
-                get_attributes(cnt, formats['Method'])
+                get_attributes(cnt, formats['methods'])
             )
             Property = plugable.NameSpace(
                 get_attributes(cnt, formats['Property'])
@@ -819,7 +818,7 @@ class test_Object(ClassChecker):
         o = user()
         o.set_api(api)
         assert read_only(o, 'api') is api
-        for name in ['Method', 'Property']:
+        for name in ['methods', 'Property']:
             namespace = getattr(o, name)
             assert isinstance(namespace, plugable.NameSpace)
             assert len(namespace) == cnt