Better discovery of macos system version to determine proper platform tag (#314)

Author: Czaki

tests/test_macosx_libfile.py

@@ -0,0 +1,114 @@
+import os
+import sys
+import distutils.util
+
+from wheel.macosx_libfile import extract_macosx_min_system_version
+from wheel.pep425tags import get_platform
+
+
+def test_read_from_dynlib():
+    dirname = os.path.dirname(__file__)
+    dylib_dir = os.path.join(dirname, "testdata",
+                             "macosx_minimal_system_version")
+    versions = [
+        ("test_lib_10_6_fat.dynlib", "10.6"),
+        ("test_lib_10_10_fat.dynlib", "10.10"),
+        ("test_lib_10_14_fat.dynlib", "10.14"),
+        ("test_lib_10_6.dynlib", "10.6"),
+        ("test_lib_10_10.dynlib", "10.10"),
+        ("test_lib_10_14.dynlib", "10.14"),
+        ("test_lib_10_6_386.dynlib", "10.6"),
+        ("test_lib_10_10_386.dynlib", "10.10"),
+        ("test_lib_10_14_386.dynlib", "10.14"),
+        ("test_lib_multiple_fat.dynlib", "10.14")
+    ]
+    for file_name, ver in versions:
+        extracted = extract_macosx_min_system_version(
+            os.path.join(dylib_dir, file_name)
+        )
+        str_ver = str(extracted[0]) + "." + str(extracted[1])
+        assert str_ver == ver
+        assert extract_macosx_min_system_version(
+            os.path.join(dylib_dir, "test_lib.c")
+        ) is None
+
+
+def return_factory(return_val):
+    def fun(*args, **kwargs):
+        return return_val
+
+    return fun
+
+
+class TestGetPlatformMacosx:
+    def test_simple(self, monkeypatch):
+        dirname = os.path.dirname(__file__)
+        dylib_dir = os.path.join(dirname, "testdata", "macosx_minimal_system_version")
+        monkeypatch.setattr(distutils.util, "get_platform", return_factory("macosx-10.14-x86_64"))
+        assert get_platform(dylib_dir) == "macosx_10_14_x86_64"
+
+    def test_version_bump(self, monkeypatch, capsys):
+        dirname = os.path.dirname(__file__)
+        dylib_dir = os.path.join(dirname, "testdata", "macosx_minimal_system_version")
+        monkeypatch.setattr(distutils.util, "get_platform", return_factory("macosx-10.9-x86_64"))
+        assert get_platform(dylib_dir) == "macosx_10_14_x86_64"
+        captured = capsys.readouterr()
+        assert "[WARNING] This wheel needs higher macosx version than" in captured.err
+
+    def test_information_about_problematic_files_python_version(self, monkeypatch, capsys):
+        dirname = os.path.dirname(__file__)
+        dylib_dir = os.path.join(dirname, "testdata", "macosx_minimal_system_version")
+        monkeypatch.setattr(distutils.util, "get_platform", return_factory("macosx-10.9-x86_64"))
+        monkeypatch.setattr(os, "walk", return_factory(
+            [(dylib_dir, [], ["test_lib_10_6.dynlib", "test_lib_10_10_fat.dynlib"])]
+        ))
+        assert get_platform(dylib_dir) == "macosx_10_10_x86_64"
+        captured = capsys.readouterr()
+        assert "[WARNING] This wheel needs higher macosx version than" in captured.err
+        assert "your python is compiled against." in captured.err
+        assert "test_lib_10_10_fat.dynlib" in captured.err
+
+    def test_information_about_problematic_files_env_variable(self, monkeypatch, capsys):
+        dirname = os.path.dirname(__file__)
+        dylib_dir = os.path.join(dirname, "testdata", "macosx_minimal_system_version")
+        monkeypatch.setattr(distutils.util, "get_platform", return_factory("macosx-10.9-x86_64"))
+        monkeypatch.setenv("MACOSX_DEPLOYMENT_TARGET", "10.8")
+        monkeypatch.setattr(os, "walk", return_factory(
+            [(dylib_dir, [], ["test_lib_10_6.dynlib", "test_lib_10_10_fat.dynlib"])]
+        ))
+        assert get_platform(dylib_dir) == "macosx_10_10_x86_64"
+        captured = capsys.readouterr()
+        assert "[WARNING] This wheel needs higher macosx version than" in captured.err
+        assert "is set in MACOSX_DEPLOYMENT_TARGET variable." in captured.err
+        assert "test_lib_10_10_fat.dynlib" in captured.err
+
+    def test_bump_platform_tag_by_env_variable(self, monkeypatch, capsys):
+        dirname = os.path.dirname(__file__)
+        dylib_dir = os.path.join(dirname, "testdata", "macosx_minimal_system_version")
+        monkeypatch.setattr(distutils.util, "get_platform", return_factory("macosx-10.9-x86_64"))
+        monkeypatch.setattr(os, "walk", return_factory(
+            [(dylib_dir, [], ["test_lib_10_6.dynlib", "test_lib_10_6_fat.dynlib"])]
+        ))
+        assert get_platform(dylib_dir) == "macosx_10_9_x86_64"
+        monkeypatch.setenv("MACOSX_DEPLOYMENT_TARGET", "10.10")
+        assert get_platform(dylib_dir) == "macosx_10_10_x86_64"
+        captured = capsys.readouterr()
+        assert captured.err == ""
+
+    def test_warning_on_to_low_env_variable(self, monkeypatch, capsys):
+        dirname = os.path.dirname(__file__)
+        dylib_dir = os.path.join(dirname, "testdata", "macosx_minimal_system_version")
+        monkeypatch.setattr(distutils.util, "get_platform", return_factory("macosx-10.9-x86_64"))
+        monkeypatch.setenv("MACOSX_DEPLOYMENT_TARGET", "10.8")
+        monkeypatch.setattr(os, "walk", return_factory(
+            [(dylib_dir, [], ["test_lib_10_6.dynlib", "test_lib_10_6_fat.dynlib"])]
+        ))
+        assert get_platform(dylib_dir) == "macosx_10_9_x86_64"
+        captured = capsys.readouterr()
+        assert "MACOSX_DEPLOYMENT_TARGET is set to lower value than your python" in captured.err
+
+
+def test_get_platform_linux(monkeypatch):
+    monkeypatch.setattr(distutils.util, "get_platform", return_factory("linux_x86_64"))
+    monkeypatch.setattr(sys, "maxsize", 2147483647)
+    assert get_platform(None) == "linux_i686"

tests/testdata/macosx_minimal_system_version/test_lib.c

@@ -0,0 +1,13 @@
+int num_of_letters(char* text){
+    int num = 0;
+    char * lett = text;
+    while (lett != 0){
+        if (*lett >= 'a' && *lett <= 'z'){
+            num += 1;
+        } else if (*lett >= 'A' && *lett <= 'Z'){
+            num += 1;
+        }
+        lett += 1;
+    }
+    return num;
+}