Feature/Support for parameter structured value as list. (#34)

Author: othercodes

connect/devops_testing/request.py

@@ -60,13 +60,17 @@
 
 def _param_members(
         param: dict,
-        value: Optional[Union[str, dict]] = None,
+        value: Optional[Union[str, dict, list]] = None,
         value_error: Optional[str] = None,
 ) -> dict:
     if isinstance(value, dict):
         key = 'structured_value'
         new_value = param.get(key, {})
         new_value.update(value)
+    elif isinstance(value, list):
+        key = 'structured_value'
+        new_value = param.get(key, [])
+        new_value.extend(value)
     else:
         key = 'value'
         new_value = value
@@ -167,7 +171,7 @@ def with_params(self, params: List[dict]) -> Builder:
     def with_param(
             self,
             param_id: str,
-            value: Optional[Union[str, dict]] = None,
+            value: Optional[Union[str, dict, list]] = None,
             value_error: Optional[str] = None,
             value_type: str = 'text',
     ) -> Builder:
@@ -293,7 +297,7 @@ def with_asset_params(self, params: List[dict]) -> Builder:
     def with_asset_param(
             self,
             param_id: str,
-            value: Optional[Union[str, dict]] = None,
+            value: Optional[Union[str, dict, list]] = None,
             value_error: Optional[str] = None,
             value_type: str = 'text',
     ) -> Builder:
@@ -392,7 +396,7 @@ def with_asset_configuration_params(self, params: List[dict]) -> Builder:
     def with_asset_configuration_param(
             self,
             param_id: str,
-            value: Optional[Union[str, dict]] = None,
+            value: Optional[Union[str, dict, list]] = None,
             value_error: Optional[str] = None,
             value_type: str = 'text',
     ) -> Builder:
@@ -500,7 +504,7 @@ def with_tier_configuration_params(self, params: List[dict]) -> Builder:
     def with_tier_configuration_param(
             self,
             param_id: str,
-            value: Optional[Union[str, dict]] = None,
+            value: Optional[Union[str, dict, list]] = None,
             value_error: Optional[str] = None,
             value_type: str = 'text',
     ) -> Builder:
@@ -523,7 +527,7 @@ def with_tier_configuration_param(
     def with_tier_configuration_configuration_param(
             self,
             param_id: str,
-            value: Optional[Union[str, dict]] = None,
+            value: Optional[Union[str, dict, list]] = None,
             value_error: Optional[str] = None,
             value_type: str = 'text',
     ) -> Builder:

tests/test_request.py

@@ -80,6 +80,8 @@ def test_request_builder_should_build_successfully_a_valid_asset_request():
     request.with_asset_configuration_params([
         {'param_id': 'AS_CFG_ID_001', 'value': 'Cfg value', 'value_error': 'Cfg error value'},
         {'param_id': 'AS_CFG_ID_001', 'value': 'Cfg value updated', 'value_error': 'Cfg error value updated'},
+        {'param_id': 'IS_A_LIST_001', 'value': [1, 2]},
+        {'param_id': 'IS_A_LIST_001', 'value': [3, 4]},
     ])
 
     assert request.is_asset_request()
@@ -131,6 +133,9 @@ def test_request_builder_should_build_successfully_a_valid_asset_request():
     assert request['asset']['configuration']['params'][0]['value'] == 'Cfg value updated'
     assert request['asset']['configuration']['params'][0]['value_error'] == 'Cfg error value updated'
 
+    assert request['asset']['configuration']['params'][1]['id'] == 'IS_A_LIST_001'
+    assert request['asset']['configuration']['params'][1]['structured_value'] == [1, 2, 3, 4]
+
 
 def test_request_builder_should_build_successfully_a_valid_tier_config_request():
     request = Builder()