Get rid of get_tzinfo_options() altogether

botocore/compat.py

@@ -28,7 +28,6 @@
 
 from botocore.vendored import six
 from botocore.exceptions import MD5UnavailableError
-from dateutil.tz import tzlocal
 from urllib3 import exceptions
 
 logger = logging.getLogger(__name__)
@@ -276,20 +275,6 @@ def _windows_shell_split(s):
     return components
 
 
-def get_tzinfo_options():
-    # This function is not used internally anymore.
-
-    # Due to dateutil/dateutil#197, Windows may fail to parse times in the past
-    # with the system clock. We can alternatively fallback to tzwininfo when
-    # this happens, which will get time info from the Windows registry.
-    if sys.platform == 'win32':
-        from dateutil.tz import tzwinlocal
-
-        return (tzlocal, tzwinlocal)
-    else:
-        return (tzlocal,)
-
-
 # Detect if CRT is available for use
 try:
     import awscrt.auth

tests/unit/test_compat.py

@@ -19,7 +19,6 @@
     compat_shell_split,
     ensure_bytes,
     get_md5,
-    get_tzinfo_options,
     total_seconds,
     unquote_str,
 )
@@ -208,15 +207,6 @@ def assert_raises(self, s, exception_cls, platform):
             compat_shell_split(s, platform)
 
 
-class TestTimezoneOperations(unittest.TestCase):
-    def test_get_tzinfo_options(self):
-        options = get_tzinfo_options()
-        self.assertTrue(len(options) > 0)
-
-        for tzinfo in options:
-            self.assertIsInstance(tzinfo(), datetime.tzinfo)
-
-
 class TestCRTIntegration(unittest.TestCase):
     def test_has_crt_global(self):
         try: