fix: fix test for SJA

Author: fstagni

src/DIRAC/WorkloadManagementSystem/Agent/JobCleaningAgent.py

@@ -47,7 +47,7 @@ class JobCleaningAgent(AgentModule):
 
     def __init__(self, *args, **kwargs):
         """c'tor"""
-        AgentModule.__init__(self, *args, **kwargs)
+	super().__init__(*args, **kwargs)
 
         # clients
         self.jobDB = None

src/DIRAC/WorkloadManagementSystem/Agent/PilotStatusAgent.py

@@ -32,7 +32,7 @@ class PilotStatusAgent(AgentModule):
 
     def __init__(self, *args, **kwargs):
         """c'tor"""
-        AgentModule.__init__(self, *args, **kwargs)
+	super().__init__(*args, **kwargs)
 
         self.jobDB = None
         self.pilotDB = None

src/DIRAC/WorkloadManagementSystem/Agent/TaskQueuesAgent.py

@@ -17,7 +17,7 @@ class TaskQueuesAgent(AgentModule):
 
     def __init__(self, *args, **kwargs):
         """c'tor"""
-        AgentModule.__init__(self, *args, **kwargs)
+	super().__init__(*args, **kwargs)
 
         # clients
         self.tqDB = None

src/DIRAC/WorkloadManagementSystem/Agent/test/Test_Agent_StalledJobAgent.py

@@ -23,9 +23,13 @@ def sja(mocker):
         side_effect=lambda x, y=None: y,
         create=True,
     )
-    mocker.patch("DIRAC.WorkloadManagementSystem.Agent.StalledJobAgent.JobDB.__init__", side_effect=mockNone)
-    mocker.patch("DIRAC.WorkloadManagementSystem.Agent.StalledJobAgent.JobLoggingDB.__init__", side_effect=mockNone)
+    mocker.patch("DIRAC.WorkloadManagementSystem.Agent.StalledJobAgent.JobDB")
+    mocker.patch("DIRAC.WorkloadManagementSystem.Agent.StalledJobAgent.JobLoggingDB")
     mocker.patch("DIRAC.WorkloadManagementSystem.Agent.StalledJobAgent.getSystemInstance", side_effect=mockNone)
+    mocker.patch("DIRAC.WorkloadManagementSystem.Agent.StalledJobAgent.JobMonitoringClient", return_value=MagicMock())
+    mocker.patch("DIRAC.WorkloadManagementSystem.Agent.StalledJobAgent.PilotManagerClient", return_value=MagicMock())
+    mocker.patch("DIRAC.WorkloadManagementSystem.Agent.StalledJobAgent.WMSClient", return_value=MagicMock())
+    mocker.patch("DIRAC.WorkloadManagementSystem.Agent.StalledJobAgent.getSystemInstance", return_value="/bof/bih")
 
     stalledJobAgent = StalledJobAgent()
     stalledJobAgent._AgentModule__configDefaults = mockAM
@@ -41,7 +45,7 @@ def sja(mocker):
 def test__sjaFunctions(sja):
     """Testing StalledJobAgent()"""
 
-    assert not sja._failSubmittingJobs()["OK"]
-    assert not sja._kickStuckJobs()["OK"]
-    assert not sja._failStalledJobs(0)["OK"]
+    assert sja._failSubmittingJobs()["OK"]
+    assert sja._kickStuckJobs()["OK"]
+    assert sja._failStalledJobs(0)["OK"]
     assert not sja._markStalledJobs(0)["OK"]

src/DIRAC/WorkloadManagementSystem/Client/WMSAdministratorClient.py

@@ -16,7 +16,7 @@ def __init__(self, url=None, **kwargs):
         :param kwargs: forwarded to the Base Client class
         """
 
-        super(WMSAdministratorClient, self).__init__(**kwargs)
+	super().__init__(**kwargs)
 
         if not url:
             self.serverURL = "WorkloadManagement/WMSAdministrator"