added caching back in to see if it runs on github actions tests

Author: sfc-gh-sshetkar

src/snowflake/connector/platform_detection.py

@@ -4,16 +4,14 @@
 import re
 from concurrent.futures.thread import ThreadPoolExecutor
 from enum import Enum
+from functools import cache
 
 import boto3
 from botocore.config import Config
 from botocore.utils import IMDSFetcher
 
 from .vendored import requests
 
-# TODO: add this back
-# from functools import cache
-
 
 class _DetectionState(Enum):
     """Internal enum to represent the detection state of a platform."""
@@ -361,8 +359,7 @@ def is_github_action():
     )
 
 
-# TODO: add cache back in once I verify this is the root cause issue
-# @cache
+@cache
 def detect_platforms(timeout_seconds: float | None) -> list[str]:
     """
     Detect all potential platforms that the current environment may be running on.

test/unit/test_detect_platforms.py

@@ -21,9 +21,7 @@ class TestDetectPlatforms:
     def teardown(self):
         with patch.dict(os.environ, clear=True):
             yield
-
-    #     TODO: add this back probably once I verify caching is causing the test failures
-    #     detect_platforms.cache_clear()  # clear cache after each test
+            detect_platforms.cache_clear()  # clear cache after each test
 
     def test_no_platforms_detected(self, broken_metadata_service):
         result = detect_platforms(timeout_seconds=None)