Test fixes

Author: HardNorth

reportportal_client/steps/__init__.py

@@ -85,8 +85,13 @@ def __enter__(self):
         self.__item_id = rp_client.step_reporter \
             .start_nested_step(self.name, timestamp(), parameters=self.params)
         if self.params:
-            rp_client.log(timestamp(), "Parameters: " + str(self.params),
-                          level='INFO', item_id=self.__item_id)
+            param_list = [
+                str(key) + ": " + str(value)
+                for key, value in sorted(self.params.items())
+            ]
+            param_str = 'Parameters: ' + '; '.join(param_list)
+            rp_client.log(timestamp(), param_str, level='INFO',
+                          item_id=self.__item_id)
 
     def __exit__(self, exc_type, exc_val, exc_tb):
         # Cannot call _local.current() early since it will be initialized
@@ -112,6 +117,7 @@ def wrapper(*args, **kwargs):
                 params = get_function_params(func, args, kwargs)
             with Step(self.name, params, self.status, self.client):
                 return func(*args, **kwargs)
+
         return wrapper
 
 

tests/steps/test_steps.py

@@ -123,23 +123,23 @@ def test_verify_parameters_logging_default_value(rp_client):
     nested_step_params(1, 'two')
     assert len(rp_client._log_manager._logs_batch) == 1
     assert rp_client._log_manager._logs_batch[0].message \
-           == "Parameters: {'param1': 1, 'param2': 'two'}"
+           == "Parameters: param1: 1; param2: two"
 
 
 def test_verify_parameters_logging_no_default_value(rp_client):
     rp_client.step_reporter.set_parent('STEP', PARENT_STEP_ID)
     nested_step_params(1, 'two', 'three')
     assert len(rp_client._log_manager._logs_batch) == 1
     assert rp_client._log_manager._logs_batch[0].message \
-           == "Parameters: {'param1': 1, 'param2': 'two', 'param3': 'three'}"
+           == "Parameters: param1: 1; param2: two; param3: three"
 
 
 def test_verify_parameters_logging_named_value(rp_client):
     rp_client.step_reporter.set_parent('STEP', PARENT_STEP_ID)
     nested_step_params(1, 'two', param3='three')
     assert len(rp_client._log_manager._logs_batch) == 1
     assert rp_client._log_manager._logs_batch[0].message \
-           == "Parameters: {'param1': 1, 'param2': 'two', 'param3': 'three'}"
+           == "Parameters: param1: 1; param2: two; param3: three"
 
 
 def test_verify_parameters_inline_logging(rp_client):
@@ -148,4 +148,4 @@ def test_verify_parameters_inline_logging(rp_client):
         pass
     assert len(rp_client._log_manager._logs_batch) == 1
     assert rp_client._log_manager._logs_batch[0].message \
-           == "Parameters: {'param1': 1, 'param2': 'two'}"
+           == "Parameters: param1: 1; param2: two"