SESPRINGPYTHONPY-134: Fixed bug where AopProxy was hard coded to only handle classic classes.

Greg Turnquist · Greg Turnquist · commit 256c91e8b3cb · 2010-05-20T09:58:21.000-05:00
diff --git a/src/springpython/aop/__init__.py b/src/springpython/aop/__init__.py
@@ -167,8 +167,6 @@ class AopProxy(object):
     lookups are not intercepted, but instead fetched from the actual target object."""
 
     def __init__(self, target, interceptors):
-        if type(target).__name__ != "instance":
-            raise Exception("Target attribute must be an instance.")
         self.target = target
         if type(interceptors) == list:
             self.interceptors = interceptors
diff --git a/test/springpythontest/aopTestCases.py b/test/springpythontest/aopTestCases.py
@@ -61,9 +61,9 @@ def testCreatingAopProxyFactoryAndAddingInterceptorToNewStyleClassProgammaticall
         factory.target = NewStyleSampleService()
         factory.interceptors.append(WrappingInterceptor())
         service = factory.getProxy()
-        self.assertEquals("<Wrapped>Alright!</Wrapped>", service.doSomething())
-        self.assertEquals("<Wrapped>You made it! => test</Wrapped>", service.method("test"))
-        self.assertEquals("sample", service.attribute)
+        self.assertEquals("<Wrapped>Even better!</Wrapped>", service.doSomething())
+        self.assertEquals("<Wrapped>You made it to a new style class! => test</Wrapped>", service.method("test"))
+        self.assertEquals("new_sample", service.attribute)
 
     def testCreatingAProxyFactoryAndAddingAnInterceptorIoC(self):
         factory = self.appContext.get_object("factory")
