Sapcontrol local socket testing (#2)

* Fix local socket not handling do_open correctly

Signed-off-by: Nicolas Bettembourg <[email protected]>

Author: nbttmbrg

plugins/modules/sap_control_exec.py

@@ -286,33 +286,16 @@ def connect(self):
         self.sock = socket.socket(socket.AF_UNIX, socket.SOCK_STREAM)
         self.sock.connect(self.socketpath)
 
-
 class LocalSocketHandler(HTTPHandler):
     """HTTP handler for Unix domain sockets."""
     def __init__(self, debuglevel=0, socketpath=None):
-        HTTPHandler.__init__(self, debuglevel)
+        self._debuglevel = debuglevel
         self._socketpath = socketpath
 
     def http_open(self, req):
         return self.do_open(LocalSocketHttpConnection, req, socketpath=self._socketpath)
 
-    def do_open(self, http_class, req, **http_conn_args):
-        """Override to handle Unix socket connections."""
-        host = req.host
-        if not host:
-            raise ValueError('no host given')
-
-        http_conn = http_class(host, **http_conn_args)
-        http_conn.set_debuglevel(self._debuglevel)
-
-        try:
-            http_conn.connect()
-            r = http_conn.getresponse()
-        except Exception as err:
-            http_conn.close()
-            raise err
 
-        return r
 def choices():
     retlist = ["Start", "Stop", "Shutdown", "InstanceStart", "InstanceStop", "Bootstrap", "ParameterValue", "GetProcessList",
                "GetProcessList2", "GetStartProfile", "GetTraceFile", "GetAlertTree", "GetAlerts", "RestartService",