allow conversion Path -> str

news/934.bugfix

@@ -0,0 +1 @@
+Revert an accidental behavior change where implicit conversion from `Path` to `str` was disallowed.

omegaconf/nodes.py

@@ -189,7 +189,7 @@ def _validate_and_convert_impl(self, value: Any) -> str:
         if (
             OmegaConf.is_config(value)
             or is_primitive_container(value)
-            or isinstance(value, (bytes, Path))
+            or isinstance(value, bytes)
         ):
             raise ValidationError("Cannot convert '$VALUE_TYPE' to string: '$VALUE'")
         return str(value)

tests/structured_conf/test_structured_config.py

@@ -71,8 +71,15 @@ class IntegersConfigAssignments:
 
 
 class StringConfigAssignments:
-    legal = ["10", "-10", "foo", "", (Color.BLUE, "Color.BLUE")]
-    illegal = [b"binary", Path("hello.txt")]
+    legal = [
+        "10",
+        "-10",
+        "foo",
+        "",
+        (Color.BLUE, "Color.BLUE"),
+        (Path("hello.txt"), "hello.txt"),
+    ]
+    illegal = [b"binary"]
 
 
 class BytesConfigAssignments:
@@ -2259,3 +2266,9 @@ def test_support_pep_604(self, module: Any) -> None:
         assert _utils.get_type_hint(cfg, "uis_with_default") == Union[int, str]
         assert cfg.uisn is None
         assert cfg.uis_with_default == 123
+
+    def test_assign_path_to_string_typed_field(self, module: Any) -> None:
+        cfg = OmegaConf.create(module.StringConfig)
+        cfg.null_default = Path("hello.txt")
+        assert isinstance(cfg.null_default, str)
+        assert cfg.null_default == "hello.txt"

tests/test_nodes.py

@@ -596,6 +596,7 @@ def test_accepts_mandatory_missing(
             {
                 "PathNode": copy.copy,
                 "AnyNode": copy.copy,
+                "StringNode": str,
             },
             id="path-data",
         ),

tests/test_utils.py

@@ -146,7 +146,7 @@ def test_node_wrap(
         # str
         param(str, "foo", StringNode("foo"), id="str"),
         param(str, b"binary", ValidationError, id="str"),
-        param(str, Path("hello.txt"), ValidationError, id="str"),
+        param(str, Path("hello.txt"), StringNode("hello.txt"), id="str"),
         param(str, True, StringNode("True"), id="str"),
         param(str, 1, StringNode("1"), id="str"),
         param(str, 1.0, StringNode("1.0"), id="str"),