Fixed localtime._unix._get_localzone() (python-babel#990)

In certain OS installations, the target of /etc/localtime contains
double slashes. This is a valid path, but not a valid zoneinfo key.
This fix replaces `os.readlink` with `os.path.realpath`, which is
guaranteed to return a normalized path.

Fixes python-babel#990

babel/localtime/_unix.py

@@ -39,7 +39,10 @@ def _get_localzone(_root: str = '/') -> datetime.tzinfo:
     # zone on operating systems like OS X.  On OS X especially this is the
     # only one that actually works.
     try:
-        link_dst = os.readlink('/etc/localtime')
+        # Make sure the path is normalized so that it will be parsed correctly
+        # below, e.g. 'Etc//UTC' is a valid path but not a valid zoneinfo key.
+        # <https://github.com/python-babel/babel/issues/990>
+        link_dst = os.path.realpath('/etc/localtime')
     except OSError:
         pass
     else:

setup.py

@@ -71,6 +71,7 @@ def run(self):
         'dev': [
             'pytest>=6.0',
             'pytest-cov',
+            'pyfakefs~=5.0',
             'freezegun~=1.0',
         ],
     },

tests/test_localtime.py

@@ -5,13 +5,13 @@
 
 """
 import datetime
-import pytest
-import sys
 import os.path
+import sys
 from pathlib import Path
 
-from babel.localtime import get_localzone, LOCALTZ
+import pytest
 
+from babel.localtime import LOCALTZ, get_localzone
 
 _timezones = {
     "Etc/UTC": "UTC",
@@ -60,9 +60,16 @@ def timezone(self, zoneinfo, request) -> str:
         # standard-ish Unix implementation where `/etc/localtime` can be used
         # to set the time zone.
         # <https://www.unix.com/man-page/linux/4/zoneinfo>
+        #
+        # Note the double slash in the symlink source. Because os.symlink()
+        # does not normalize paths, this will appear on the file system as
+        # e.g. `zoneinfo//Etc/UTC`. Although irregular, this is a valid link
+        # and has been observed in certain Ubuntu installations, so make sure
+        # 'localtime' can handle them.
+        # <https://github.com/python-babel/babel/issues/990>.
         key, name = request.param
         os.remove("/etc/localtime")
-        os.symlink(f"{zoneinfo}/{key}", "/etc/localtime")
+        os.symlink(f"{zoneinfo}//{key}", "/etc/localtime")  # double slash OK
         return name
 
     def test_get_localzone(self, zoneinfo, timezone):
@@ -76,7 +83,7 @@ def test_localtz(self, zoneinfo):
         """ Test the LOCALTZ module attribute.
 
         """
-        assert LOCALTZ == get_localzone()
+        assert get_localzone() == LOCALTZ
 
 
 @pytest.mark.skipif(sys.platform != "win32", reason="Win32 tests")
@@ -90,4 +97,4 @@ def test_localtz(self):
         """ Test the LOCALTZ module attribute.
 
         """
-        assert LOCALTZ == get_localzone()
+        assert get_localzone() == LOCALTZ