sweep: DIRACGrid#7766 RSS fix: test edge case of no sites

DIRACGridBot · Aug 28, 2024 · 6abaf82 · 6abaf82
1 parent 2aa75fc
commit 6abaf82
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/DIRAC/ResourceStatusSystem/Client/SiteStatus.py b/src/DIRAC/ResourceStatusSystem/Client/SiteStatus.py
@@ -170,9 +170,9 @@ def getUsableSites(self, siteNames=None):
         siteStatusDictRes = self.getSiteStatuses(siteNames)
         if not siteStatusDictRes["OK"]:
             return siteStatusDictRes
-        siteStatusList = [x[0] for x in siteStatusDictRes["Value"].items() if x[1] in ["Active", "Degraded"]]
-
-        return S_OK(siteStatusList)
+        if not siteStatusDictRes["Value"]:
+            return S_OK([])
+        return S_OK([x[0] for x in siteStatusDictRes["Value"].items() if x[1] in ["Active", "Degraded"]])
 
     def getSites(self, siteState="Active"):
         """
@@ -203,6 +203,8 @@ def getSites(self, siteState="Active"):
         siteStatusDictRes = self.getSiteStatuses()
         if not siteStatusDictRes["OK"]:
             return siteStatusDictRes
+        if not siteStatusDictRes["Value"]:
+            return S_OK([])
 
         if siteState.capitalize() == "All":
             # if no siteState is set return everything