Fix macos user_site_dir

CHANGES.rst

@@ -1,6 +1,11 @@
 platformdirs Changelog
 ======================
 
+platformdirs 3.0.0 (2023-02-06)
+-------------------------------
+- **BREAKING** Correct the config directory on OSX/macOS, again: "*/Library/Application Support" (#98)
+- macOS: remove erroneous trailing slash from user_config_dir and user_data_dir (#134)
+
 platformdirs 2.6.2 (2022-12-28)
 -------------------------------
 - Fix missing ``typing-extensions`` dependency.

src/platformdirs/macos.py

@@ -16,7 +16,7 @@ class MacOS(PlatformDirsABC):
     @property
     def user_data_dir(self) -> str:
         """:return: data directory tied to the user, e.g. ``~/Library/Application Support/$appname/$version``"""
-        return self._append_app_name_and_version(os.path.expanduser("~/Library/Application Support/"))
+        return self._append_app_name_and_version(os.path.expanduser("~/Library/Application Support"))
 
     @property
     def site_data_dir(self) -> str:
@@ -25,13 +25,13 @@ def site_data_dir(self) -> str:
 
     @property
     def user_config_dir(self) -> str:
-        """:return: config directory tied to the user, e.g. ``~/Library/Preferences/$appname/$version``"""
-        return self._append_app_name_and_version(os.path.expanduser("~/Library/Preferences/"))
+        """:return: config directory tied to the user, same as `user_data_dir`"""
+        return self.user_data_dir
 
     @property
     def site_config_dir(self) -> str:
-        """:return: config directory shared by the users, e.g. ``/Library/Preferences/$appname``"""
-        return self._append_app_name_and_version("/Library/Preferences")
+        """:return: config directory shared by the users, same as `site_data_dir`"""
+        return self.site_data_dir
 
     @property
     def user_cache_dir(self) -> str:

tests/test_comp_with_appdirs.py

@@ -57,8 +57,6 @@ def test_compatibility(params: dict[str, Any], func: str) -> None:
     if sys.platform == "darwin":
         msg = {  # pragma: no cover
             "user_log_dir": "without appname produces NoneType error",
-            "site_config_dir": "ignores the version argument",
-            "user_config_dir": "uses Library/Preferences instead Application Support",
         }
         if func in msg:  # pragma: no cover
             pytest.skip(f"`appdirs.{func}` {msg[func]} on macOS")  # pragma: no cover
@@ -72,4 +70,4 @@ def test_compatibility(params: dict[str, Any], func: str) -> None:
     new = getattr(platformdirs, func)(*params)
     old = getattr(appdirs, func)(*params)
 
-    assert new == old
+    assert new == old.rstrip("/")

tests/test_macos.py

@@ -0,0 +1,52 @@
+from __future__ import annotations
+
+import os
+from typing import Any
+
+import pytest
+
+from platformdirs.macos import MacOS
+
+
+@pytest.mark.parametrize(
+    "params",
+    [
+        {},
+        {"appname": "foo"},
+        {"appname": "foo", "version": "v1.0"},
+    ],
+    ids=[
+        "no_args",
+        "app_name",
+        "app_name_version",
+    ],
+)
+def test_macos(params: dict[str, Any], func: str) -> None:
+    result = getattr(MacOS(**params), func)
+
+    home = os.path.expanduser("~")
+    suffix_elements = []
+    if "appname" in params:
+        suffix_elements.append(params["appname"])
+    if "version" in params:
+        suffix_elements.append(params["version"])
+    if suffix_elements:
+        suffix_elements.insert(0, "")
+    suffix = "/".join(suffix_elements)
+
+    expected_map = {
+        "user_data_dir": f"{home}/Library/Application Support{suffix}",
+        "site_data_dir": f"/Library/Application Support{suffix}",
+        # Please note that the config dirs are NOT */Library/Preferences!
+        # For details see: https://github.com/platformdirs/platformdirs/issues/98
+        "user_config_dir": f"{home}/Library/Application Support{suffix}",
+        "site_config_dir": f"/Library/Application Support{suffix}",
+        "user_cache_dir": f"{home}/Library/Caches{suffix}",
+        "user_state_dir": f"{home}/Library/Application Support{suffix}",
+        "user_log_dir": f"{home}/Library/Logs{suffix}",
+        "user_documents_dir": f"{home}/Documents",
+        "user_runtime_dir": f"{home}/Library/Caches/TemporaryItems{suffix}",
+    }
+    expected = expected_map[func]
+
+    assert result == expected