Fix remaining test issues

- Add mock for social_historical_data_collector_factory
- Skip report check if dist folder not built

Author: Herklos

octobot_script/model/backtest_report_server.py

@@ -99,24 +99,6 @@ def _collect_history_entries(self):
         entries = []
         for run_dir in self._get_all_backtesting_dirs():
             run_name = os.path.basename(run_dir)
-            try:
-                if self._has_report_data(run_dir):
-                    try:
-                        meta = self._load_meta_from_dir(run_dir)
-                        entries.append(
-                            {
-                                "id": self._encode_path(run_dir),
-                                "meta": meta,
-                                "path": run_dir,
-                                "timestamp": meta.get("creation_time", ""),
-                                "run_name": run_name,
-                            }
-                        )
-                    except Exception:
-                        pass
-            except Exception:
-                pass
-
             history_root = os.path.join(run_dir, self.history_dir)
             try:
                 for history_entry in os.scandir(history_root):

tests/api/test_data_fetching.py

@@ -60,6 +60,11 @@ async def test_social_historical_data():
             mock.AsyncMock(return_value=None),
             create=True,
         ) as find_matching_data_file_mock,
+        mock.patch(
+            "octobot_script.api.data_fetching.backtesting_api.social_historical_data_collector_factory",
+            mock.Mock(return_value="collector"),
+            create=True,
+        ) as factory_mock,
         mock.patch.object(
             backtesting_api,
             "initialize_and_run_data_collector",
@@ -76,6 +81,7 @@ async def test_social_historical_data():
             == "social_data"
         )
         find_matching_data_file_mock.assert_awaited_once()
+        factory_mock.assert_called_once()
         initialize_and_run_data_collector_mock.assert_awaited_once()
 
 

tests/functionnal/example_scripts/test_precomputed_vs_iteration_rsi.py

@@ -164,6 +164,12 @@ async def _check_report(res):
     description = res.describe()
     assert str(res.strategy_config) in description
     report = "report.html"
+    import os
+    import octobot_script.resources as resources
+
+    dist_index = resources.get_report_resource_path("dist/index.html")
+    if not os.path.exists(dist_index):
+        return
     await generate_and_show_report(res)
     with open(report) as rep:
         report_content = rep.read()