python: Changed chart tests to print running the simulations and always do the chart exporting.

levy · levy · commit 59bbb86357ea · 2023-03-24T12:01:10.000+01:00
Silently running a lot of simulations is confusing for the user.
Silently not running the chart tests if one of the simulations fail is also very confusing.
diff --git a/python/inet/test/chart.py b/python/inet/test/chart.py
@@ -62,11 +62,8 @@ def __init__(self, multiple_simulation_tasks=None, name="chart test", multiple_t
         self.multiple_simulation_tasks = multiple_simulation_tasks
 
     def run_protected(self, **kwargs):
-        multiple_simulation_task_results = self.multiple_simulation_tasks.run_protected(output_stream=io.StringIO(), **kwargs)
-        if multiple_simulation_task_results.result != "DONE":
-            return self.multiple_task_results_class(self, result=multiple_simulation_task_results.result)
-        else:
-            return super().run_protected(**kwargs)
+        multiple_simulation_task_results = self.multiple_simulation_tasks.run_protected(**kwargs)
+        return super().run_protected(**kwargs)
 
 def get_chart_test_tasks(simulation_project=default_project, run_simulations=True, filter=None, working_directory_filter=None, pool_class=multiprocessing.Pool, **kwargs):
     test_tasks = []
