Remove unused code from a test class

Unused methods and attributes of
`astroquery.utils.tap.conn.tests.DummyConnHandler.DummyConnHandler` have
been removed. `get_last_request()` was used, but trivial to replace with
an attribute lookup.

Author: eerovaher

astroquery/utils/tap/conn/tests/DummyConnHandler.py

@@ -27,8 +27,6 @@ def __init__(self):
         self.fileExt = ".ext"
         self.defaultResponse = None
         self.responses = {}
-        self.errorFileOutput = None
-        self.errorReceivedResponse = None
         self.contentType = None
         self.verbose = None
         self.query = None
@@ -40,21 +38,6 @@ def set_default_response(self, defaultResponse):
     def get_default_response(self):
         return self.defaultResponse
 
-    def get_last_request(self):
-        return self.request
-
-    def get_last_data(self):
-        return self.data
-
-    def get_last_query(self):
-        return self.query
-
-    def get_error_file_output(self):
-        return self.errorFileOutput
-
-    def get_error_received_response(self):
-        return self.errorReceivedResponse
-
     def set_response(self, request, response):
         self.responses[str(request)] = response
 
@@ -100,8 +83,6 @@ def __execute_post(self, subcontext=None, data=None,
         return self.__get_response(self.request)
 
     def dump_to_file(self, fileOutput, response):
-        self.errorFileOutput = fileOutput
-        self.errorReceivedResponse = response
         print(f"DummyConnHandler - dump to file: file: '{fileOutput}', \
         response status: {response.status}, response msg: {response.reason}")
 
@@ -148,9 +129,6 @@ def check_launch_response_status(self, response, debug,
     def get_suitable_extension(self, headers):
         return self.fileExt
 
-    def set_suitable_extension(self, ext):
-        self.fileExt = ext
-
     def get_suitable_extension_by_format(self, output_format):
         return self.fileExt
 

astroquery/utils/tap/tests/test_tap.py

@@ -78,35 +78,30 @@ def test_load_tables_parameters():
 
     # empty request
     tap.load_tables()
-    request = connHandler.get_last_request()
-    assert request == tableRequest
+    assert connHandler.request == tableRequest
 
     # flag only_names=false & share_accessible=false: equals to
     # empty request
     tap.load_tables(only_names=False, include_shared_tables=False)
-    request = connHandler.get_last_request()
-    assert request == tableRequest
+    assert connHandler.request == tableRequest
 
     # flag only_names
     tableRequest = "tables?only_tables=true"
     connHandler.set_response(tableRequest, responseLoadTable)
     tap.load_tables(only_names=True)
-    request = connHandler.get_last_request()
-    assert request == tableRequest
+    assert connHandler.request == tableRequest
 
     # flag share_accessible=true
     tableRequest = "tables?share_accessible=true"
     connHandler.set_response(tableRequest, responseLoadTable)
     tap.load_tables(include_shared_tables=True)
-    request = connHandler.get_last_request()
-    assert request == tableRequest
+    assert connHandler.request == tableRequest
 
     # flag only_names=true & share_accessible=true
     tableRequest = "tables?only_tables=true&share_accessible=true"
     connHandler.set_response(tableRequest, responseLoadTable)
     tap.load_tables(only_names=True, include_shared_tables=True)
-    request = connHandler.get_last_request()
-    assert request == tableRequest
+    assert connHandler.request == tableRequest
 
 
 def test_load_table():