Add result_is_success() function

Author: yidas

README.md

@@ -255,7 +255,8 @@ result_callback_function (id: int, config, result, log: dict) -> Any
   ```python
   def get_tps(logs: dict=None, debug: bool=False, interval: float=0, reverse_interval: bool = False, display_intervals: bool = False) -> dict:
   ```
-  The log dict matches the format of the [get_logs()](#get_logs) and refers to it by default. Each task within a log will be validated for success according to the [callback_function](#task.result_callback) result rule.
+  The log dict matches the format of the [get_logs()](#get_logs) and refers to it by default. 
+  Each task within a log will be validated for success according to the [callback_function()](#task.result_callback) result rule.
 
 ### Scenarios
 

src/worker_dispatcher/worker_dispatcher.py

@@ -704,7 +704,7 @@ def _tps_calculate(started_at: float, ended_at: float, logs: list, display_valid
         exec_time_min = exec_time if not exec_time_min or exec_time < exec_time_min else exec_time_min
         # Success case check
         result = log['result']
-        if (isinstance(result, requests.Response) and result.status_code != 200) or not result:
+        if not result_is_success(result):
             continue
         # Rewrite success_id_set if in the argument
         if isinstance(success_id_set, set):
@@ -757,6 +757,12 @@ def _tps_calculate(started_at: float, ended_at: float, logs: list, display_valid
 def _validate_log_format(log) -> bool:
     return all(key in log for key in ('started_at', 'ended_at', 'result'))
 
+def result_is_success(result) -> bool:
+    if (isinstance(result, requests.Response) and result.status_code != 200) or not result:
+        return False
+    else:
+        return True
+
 def get_last_config() -> dict:
     return last_config