Fix no-ctypes fallback on windows (#403)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: youknowone

src/platformdirs/windows.py

@@ -188,6 +188,9 @@ def get_win_folder_from_registry(csidl_name: str) -> str:
     for all CSIDL_* names.
 
     """
+    machine_names = {
+        "CSIDL_COMMON_APPDATA",
+    }
     shell_folder_name = {
         "CSIDL_APPDATA": "AppData",
         "CSIDL_COMMON_APPDATA": "Common AppData",
@@ -205,7 +208,10 @@ def get_win_folder_from_registry(csidl_name: str) -> str:
         raise NotImplementedError
     import winreg  # noqa: PLC0415
 
-    key = winreg.OpenKey(winreg.HKEY_CURRENT_USER, r"Software\Microsoft\Windows\CurrentVersion\Explorer\Shell Folders")
+    # Use HKEY_LOCAL_MACHINE for system-wide folders, HKEY_CURRENT_USER for user-specific folders
+    hkey = winreg.HKEY_LOCAL_MACHINE if csidl_name in machine_names else winreg.HKEY_CURRENT_USER
+
+    key = winreg.OpenKey(hkey, r"Software\Microsoft\Windows\CurrentVersion\Explorer\Shell Folders")
     directory, _ = winreg.QueryValueEx(key, shell_folder_name)
     return str(directory)
 

tests/test_api.py

@@ -99,15 +99,15 @@ def _fake_import(name: str, *args: Any, **kwargs: Any) -> ModuleType:  # noqa: A
     return builtin_import(name, *args, **kwargs)
 
 
-def mock_import(func: Callable[[], None]) -> Callable[[], None]:
+def mock_import(func: Callable[..., None]) -> Callable[..., None]:
     @functools.wraps(func)
-    def wrap() -> None:
+    def wrap(*args: Any, **kwargs: Any) -> None:  # noqa: ANN401
         platformdirs_module_items = [item for item in sys.modules.items() if item[0].startswith("platformdirs")]
         try:
             builtins.__import__ = _fake_import
             for name, _ in platformdirs_module_items:
                 del sys.modules[name]
-            return func()
+            return func(*args, **kwargs)
         finally:
             # restore original modules
             builtins.__import__ = builtin_import
@@ -118,11 +118,15 @@ def wrap() -> None:
 
 
 @mock_import
-def test_no_ctypes() -> None:
+def test_no_ctypes(func: str) -> None:
     import platformdirs  # noqa: PLC0415
 
     assert platformdirs
 
+    dirs = platformdirs.PlatformDirs("MyApp", "MyCompany", version="1.0")
+    result = getattr(dirs, func)
+    assert isinstance(result, str)
+
 
 def test_mypy_subclassing() -> None:
     # Ensure that PlatformDirs / AppDirs is seen as a valid superclass by mypy