Fix Test Cases.

Author: khushboovashi

web/pgadmin/tools/sqleditor/__init__.py

@@ -1895,11 +1895,11 @@ def set_pref_options(trans_id, operation):
 
         if operation == 'auto_commit':
             # Call the set_auto_commit method of transaction object
-            trans_obj.set_auto_commit(_data['auto_commit'])
+            trans_obj.set_auto_commit(_data)
         elif operation == 'auto_rollback':
-            trans_obj.set_auto_rollback(_data['auto_rollback'])
+            trans_obj.set_auto_rollback(_data)
         elif operation == 'server_cursor':
-            trans_obj.set_server_cursor(_data['server_cursor'])
+            trans_obj.set_server_cursor(_data)
 
         # As we changed the transaction object we need to
         # restore it and update the session variable.

web/pgadmin/tools/sqleditor/tests/test_server_cursor.py

@@ -64,10 +64,7 @@ def initiate_sql_query_tool(self, trans_id, sql_query):
 
     def set_server_cursor(self, server_cursor):
         _url = '/sqleditor/server_cursor/{0}'.format(self.trans_id)
-        res = self.tester.post(_url, data=json.dumps({
-            "server_cursor": server_cursor
-        }))
-
+        res = self.tester.post(_url, data=json.dumps(server_cursor))
         self.assertEqual(res.status_code, 200)
 
     def runTest(self):
@@ -93,8 +90,8 @@ def runTest(self):
         response = self.initiate_sql_query_tool(self.trans_id, self.sql)
 
         self.assertEqual(response.status_code, 200)
-        _resp = response.data.decode()
-        self.assertTrue(_resp.server_cursor)
+        _resp =json.loads(response.data.decode())
+        self.assertTrue(_resp['data']['server_cursor'])
 
         self.set_server_cursor(False)