Fix bug on null response

AppsFuel · Feb 15, 2013 · 8cf3ea6 · 8cf3ea6
1 parent 57c1cfb
commit 8cf3ea6
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/resuds/api.py b/resuds/api.py
@@ -88,6 +88,8 @@ def __repr__(self):
 class ElementFactory(object):
     @classmethod
     def rebuild(cls, obj):
+        if not obj:
+            return ''
         if isinstance(obj, (int, float, str)):
             return str(obj)
         if cls.is_list(obj):

diff --git a/resuds/tests/test_api.py b/resuds/tests/test_api.py
@@ -119,6 +119,11 @@ def testCreateAdspace(self):
         res = self.client.CreateAdspace(name='', pwd='', operatorId='', Adspace=adspace)
         self.assertTrue(res.isdigit())
 
+    @httprettified
+    def testAANoneTypeObjectInRebuild(self):
+        self.setupHttpPretty('deleteAdspace')
+        self.client.DeleteAdSpace(name='', pwd='', operatorId='', adSpaceId=123456)
+
     @httprettified
     def testComplicateCase(self):
         with open(os.path.join(LOCALDIR, 'getReport.wsdl')) as fp:

diff --git a/resuds/tests/wsdl/deleteAdspace.wsdl b/resuds/tests/wsdl/deleteAdspace.wsdl
@@ -0,0 +1 @@
+<soap:Envelope xmlns:soap="http://schemas.xmlsoap.org/soap/envelope/"><soap:Body><ns2:EmptyResponse xmlns:ns3="http://Amobee.com/" xmlns:ns2="http://Amobee.com/Common/Schema"/></soap:Body></soap:Envelope>