comments for not supported api method setTestMode #14

Author: Luiko Czub

src/testlink/testlinkapigeneric.py

@@ -228,9 +228,24 @@ def reportTCResult(self):
 #    * @param struct $args
 #    * @return boolean
 #    * @access protected
-#    */  
+#    */  testmode
 #   public function setTestMode($args)
 
+
+    #FIXME: LC 20140202 makes setTestMode sense in python client?
+    # xmlrpc calls are isolated transactions. 
+    # the second call works with another TL api instance than the frist one
+    # so the second call works with default TestMode settings (false), even if
+    # the first call has active the TestMode.
+    # So using setTestMode make only sense, if the php api instance is directly
+    # used
+#     @decoMakerApiCallWithArgs(['testmode'])
+#     def setTestMode(self):
+#         """ turn on/off testMode 
+#         This method is meant primarily for testing and debugging during development
+#         """
+
+
     @decoMakerApiCallReplaceTLResponseError()          
     @decoApiCallAddDevKey               
     @decoMakerApiCallWithArgs(['testplanid'], 
@@ -1000,7 +1015,7 @@ def _checkResponse(self, response, methodNameAPI, argsOptional):
                     raise testlinkerrors.TLResponseError(
                                     methodNameAPI, argsOptional,
                                     response[0]['message'], response[0]['code'])
-            except (TypeError, KeyError), msg:
+            except (TypeError, KeyError):
                 # if the reponse has not a [{..}] structure, the check
                 #    'code' in response[0]
                 # raise an error. Following causes are ok

test/utest-online/testlinkapi_online_test.py

@@ -259,6 +259,13 @@ def test_testLinkVersion(self):
     def test_getUserByLogin_unknownKey(self):
         with self.assertRaisesRegexp(TLResponseError, '10000.*User Login'):
             self.client.getUserByLogin(user='unknownUser')
+            
+#     def test_setTestMode(self):
+#         response = self.client.setTestMode(True)
+#         self.assertTrue(response)
+#         response = self.client.setTestMode(False)
+#         self.assertTrue(response)
+            
 
 if __name__ == "__main__":
     #import sys;sys.argv = ['', 'Test.testName']

test/utest-online/testlinkapigeneric_online_test.py

@@ -277,6 +277,11 @@ def test_getUserByID_unknownKey(self):
          with self.assertRaisesRegexp(TLResponseError, 'NO_USER_BY_ID_LOGIN.*User with DB ID'):
             self.client.getUserByID(4711)
 
+#     def test_setTestMode(self):
+#         response = self.client.setTestMode(True)
+#         self.assertTrue(response)
+#         response = self.client.setTestMode(False)
+#         self.assertTrue(response)
 
 
 if __name__ == "__main__":