Merge pull request #570 from Darsstar/bugfix/config_root

Fix keyring.util.platform_.config_root on Windows

CHANGES.rst

@@ -1,3 +1,8 @@
+v23.5.1
+-------
+
+* The correct config root is now used on Windows.
+
 v23.5.0
 -------
 

keyring/util/platform_.py

@@ -65,4 +65,4 @@ def _config_root_Linux():
 
 # by default, use Unix convention
 data_root = globals().get('_data_root_' + platform.system(), _data_root_Linux)
-config_root = globals().get('_config_root' + platform.system(), _config_root_Linux)
+config_root = globals().get('_config_root_' + platform.system(), _config_root_Linux)

tests/util/test_platform.py

@@ -0,0 +1,15 @@
+import pytest
+
+from keyring.util import platform_
+
+
+@pytest.mark.skipif('platform.system() != "Linux"')
+def test_platform_Linux():
+    assert platform_.config_root == platform_._config_root_Linux
+    assert platform_.data_root == platform_._data_root_Linux
+
+
+@pytest.mark.skipif('platform.system() != "Windows"')
+def test_platform_Windows():
+    assert platform_.config_root == platform_._config_root_Windows
+    assert platform_.data_root == platform_._data_root_Windows