Refactor retrieve_uris function to use a simple for loop instead of ThreadPoolExecutor

Author: gabino

cuenca/resources/resources.py

@@ -1,5 +1,4 @@
 import re
-from concurrent.futures import ThreadPoolExecutor
 from typing import Dict, List, cast
 
 from .base import Retrievable
@@ -17,5 +16,11 @@ def retrieve_uri(uri: str) -> Retrievable:
 
 
 def retrieve_uris(uris: List[str]) -> List[Retrievable]:
-    with ThreadPoolExecutor(max_workers=len(uris)) as executor:
-        return [obj for obj in executor.map(retrieve_uri, uris)]
+    # Changed the implementation to use a simple for loop instead of
+    # ThreadPoolExecutor. The list of URIs is small, so the performance
+    # difference is negligible. Additionally, using ThreadPoolExecutor
+    # caused issues with VCR tests, as the recordings were not retrieved
+    # in the correct order, leading to unexpected HTTP calls instead of
+    # using the mocked recordings.
+
+    return [retrieve_uri(uri) for uri in uris]