Fix [Tests] : Update tests and fix Girder cases

Author: Guillaume V.

src/vip_client/classes/VipGirder.py

@@ -738,7 +738,7 @@ def parse_value(input):
     # ------------------------------------------------
 
     # Get the input settings after files are parsed as PathLib objects
-    def _get_input_settings(self, location="girder") -> dict:
+    def _get_input_settings(self, location="girder") -> list[dict]:
         """
         Returns the input settings with filenames adapted to `location`.
         - if `location` = "girder", returns Girder paths string format.
@@ -768,10 +768,10 @@ def get_input(value, location) -> str:
         if location not in ("girder", "vip-girder"):
             return super()._get_input_settings(location)
         # Browse input settings
-        return {
-            key: get_input(value, location)
-            for key, value in self._input_settings.items()
-        }
+        return [
+            {key: get_input(value, location) if isinstance(value, list) else str(value) for key, value in input_dict.items()}
+            for input_dict in self._input_settings
+        ]
     # ------------------------------------------------
 
 ######################################################

tests/test_VipGirder.py

@@ -163,11 +163,14 @@ def test_properties_interface(mocker):
     # Backup the inputs
     backup = s.input_settings
     # Run a subtest for each property
-    for prop in s.input_settings:
-        setattr(s, prop, None) # Calls deleter
-        assert getattr(s, prop) is None # Public attribute must be None
-        assert not s._is_defined("_" + prop) # Private attribute must be unset
-        setattr(s, prop, backup[prop]) # Reset
+    for i, map in enumerate(s.input_settings):
+        for key, value in map.items():
+            setattr(s, key, None) # Calls deleter
+            assert getattr(s, key) is None # Public attribute must be None
+            assert not s._is_defined("_" + key) # Private attribute must be unset
+            setattr(s, key, backup[i][key]) # Reset
+
     # Test correct reset
-    for key, value in s.input_settings.items():
-        assert getattr(s, key) == value
+    for map in s.input_settings:
+        for key, value in map.items():
+            assert getattr(s, key) == value

tests/test_VipLauncher.py

@@ -202,11 +202,14 @@ def test_properties_interface(mocker):
     # Backup the inputs
     backup = s.input_settings
     # Run a subtest for each property
-    for prop in s.input_settings:
-        setattr(s, prop, None) # Calls deleter
-        assert getattr(s, prop) is None # Public attribute must be None
-        assert not s._is_defined("_" + prop) # Private attribute must be unset
-        setattr(s, prop, backup[prop]) # Reset
+    for i, map in enumerate(s.input_settings):
+        for key, value in map.items():
+            setattr(s, key, None) # Calls deleter
+            assert getattr(s, key) is None # Public attribute must be None
+            assert not s._is_defined("_" + key) # Private attribute must be unset
+            setattr(s, key, backup[i][key]) # Reset
+
     # Test correct reset
-    for key, value in s.input_settings.items():
-        assert getattr(s, key) == value
+    for map in s.input_settings:
+        for key, value in map.items():
+            assert getattr(s, key) == value

tests/test_VipSession.py

@@ -189,11 +189,14 @@ def test_properties_interface(mocker):
     # Backup the inputs
     backup = s.input_settings
     # Run a subtest for each property
-    for prop in s.input_settings:
-        setattr(s, prop, None) # Calls deleter
-        assert getattr(s, prop) is None # Public attribute must be None
-        assert not s._is_defined("_" + prop) # Private attribute must be unset
-        setattr(s, prop, backup[prop]) # Reset
+    for i, map in enumerate(s.input_settings):
+        for key, value in map.items():
+            setattr(s, key, None) # Calls deleter
+            assert getattr(s, key) is None # Public attribute must be None
+            assert not s._is_defined("_" + key) # Private attribute must be unset
+            setattr(s, key, backup[i][key]) # Reset
+
     # Test correct reset
-    for key, value in s.input_settings.items():
-        assert getattr(s, key) == value
+    for map in s.input_settings:
+        for key, value in map.items():
+            assert getattr(s, key) == value