fix: resolve path of InTemporaryDirectory name (#569)

Author: mayeut

src/auditwheel/tmpdirs.py

@@ -27,15 +27,17 @@ class InTemporaryDirectory:
 
     def __init__(self) -> None:
         self._tmpdir = TemporaryDirectory()
+        self._name = Path(self._tmpdir.name).resolve(strict=True)
 
     @property
     def name(self) -> Path:
-        return Path(self._tmpdir.name)
+        return self._name
 
     def __enter__(self) -> Path:
         self._pwd = Path.cwd()
-        os.chdir(self._tmpdir.name)
-        return Path(self._tmpdir.__enter__())
+        os.chdir(self._name)
+        self._tmpdir.__enter__()
+        return self._name
 
     def __exit__(
         self,
@@ -44,7 +46,7 @@ def __exit__(
         tb: TracebackType | None,
     ) -> None:
         os.chdir(self._pwd)
-        return self._tmpdir.__exit__(exc, value, tb)
+        self._tmpdir.__exit__(exc, value, tb)
 
 
 class InGivenDirectory:

tests/unit/test_wheeltools.py

@@ -1,18 +1,23 @@
 from __future__ import annotations
 
 import re
+import tempfile
+from pathlib import Path
 
 import pytest
 
 from auditwheel.architecture import Architecture
 from auditwheel.error import NonPlatformWheel
 from auditwheel.libc import Libc
 from auditwheel.wheeltools import (
+    InWheelCtx,
     WheelToolsError,
     get_wheel_architecture,
     get_wheel_libc,
 )
 
+HERE = Path(__file__).parent.resolve()
+
 
 @pytest.mark.parametrize(
     ("filename", "expected"),
@@ -76,3 +81,19 @@ def test_get_wheel_libc_multiple(filename: str) -> None:
     match = re.escape("multiple libc are not supported")
     with pytest.raises(WheelToolsError, match=match):
         get_wheel_libc(filename)
+
+
+def test_inwheel_tmpdir(tmp_path, monkeypatch):
+    wheel_path = (
+        HERE
+        / "../integration/arch-wheels/glibc/testsimple-0.0.1-cp313-cp313-linux_x86_64.whl"
+    )
+    tmp_path = tmp_path.resolve(strict=True)
+    tmpdir = tmp_path / "tmpdir"
+    tmpdir.mkdir()
+    tmpdir_symlink = tmp_path / "symlink"
+    tmpdir_symlink.symlink_to(str(tmpdir), target_is_directory=True)
+    monkeypatch.setattr(tempfile, "gettempdir", lambda: str(tmpdir_symlink))
+    with InWheelCtx(wheel_path, tmp_path / wheel_path.name) as context:
+        Path(context._tmpdir.name).relative_to(tmpdir_symlink)
+        context.name.relative_to(tmpdir)