fix(robot): handle OSErrors if creating/reading user robot.toml

should fix: #187

Author: d-biehl

packages/robot/src/robotcode/robot/config/loader.py

@@ -34,6 +34,7 @@ class ConfigType(str, Enum):
     ROBOT_TOML = "robot.toml (project file)"
     LOCAL_ROBOT_TOML = ".robot.toml (local file)"
     USER_DEFAULT_CONFIG_TOML = "robot.toml (user default config)"
+    DEFAULT_CONFIG_TOML = "(default config)"
     CUSTOM_TOML = ".toml (custom file)"
 
 
@@ -118,6 +119,12 @@ def load_config_from_path(
     result = config_type()
 
     for __path in __paths:
+        if isinstance(__path, tuple):
+            path, c_type = __path
+            if path.name == "__no_user_config__.toml" and c_type == ConfigType.DEFAULT_CONFIG_TOML:
+                result.add_options(get_default_config())
+                continue
+
         result.add_options(
             _load_config_data_from_path(
                 config_type,

packages/robot/src/robotcode/robot/config/utils.py

@@ -16,28 +16,29 @@ def get_user_config_file(
     create: bool = True,
     verbose_callback: Optional[Callable[[str], None]] = None,
 ) -> Optional[Path]:
-    result = Path(platformdirs.user_config_dir("robotcode", appauthor=False), "robot.toml")
-    if result.is_file():
-        if verbose_callback:
-            verbose_callback(f"Found user configuration file:\n    {result}")
-        return result
+    try:
+        result = Path(platformdirs.user_config_dir("robotcode", appauthor=False), "robot.toml")
+        if result.is_file():
+            if verbose_callback:
+                verbose_callback(f"Found user configuration file:\n    {result}")
+            return result
+
+        if not create:
+            if verbose_callback:
+                verbose_callback("User configuration file not found, but create is set to False.")
+            return None
 
-    if not create:
         if verbose_callback:
-            verbose_callback("User configuration file not found, but create is set to False.")
-        return None
+            verbose_callback(f"User configuration file not found, try to create it at:\n    {result}")
 
-    if verbose_callback:
-        verbose_callback(f"User configuration file not found, try to create it at:\n    {result}")
-    try:
         get_default_config().save(result)
+
+        return result
     except OSError as e:
         if verbose_callback:
             verbose_callback(f"Cannot create user configuration file `{result}`:\n    {e}")
         return None
 
-    return result
-
 
 def get_config_files(
     paths: Optional[Sequence[Union[str, Path]]] = None,
@@ -73,7 +74,11 @@ def get_config_files(
 
     return (
         [
-            *([(user_config, ConfigType.USER_DEFAULT_CONFIG_TOML)] if user_config else []),
+            *(
+                [(user_config, ConfigType.USER_DEFAULT_CONFIG_TOML)]
+                if user_config
+                else [(Path("__no_user_config__.toml"), ConfigType.DEFAULT_CONFIG_TOML)]
+            ),
             *result,
         ],
         root_folder,