Do not require .value for Node.

Author: tobiasraabe

src/_pytask/node_protocols.py

@@ -29,8 +29,6 @@ def state(self) -> str | None:
 class Node(MetaNode, Protocol):
     """Protocol for nodes."""
 
-    value: Any
-
     def load(self) -> Any:
         """Return the value of the node that will be injected into the task."""
         ...

src/_pytask/nodes.py

@@ -106,20 +106,16 @@ class PathNode(Node):
 
     name: str = ""
     """Name of the node which makes it identifiable in the DAG."""
-    value: Path | None = None
-    """Value passed to the decorator which can be requested inside the function."""
-
-    @property
-    def path(self) -> Path:
-        return self.value
+    path: Path | None = None
+    """The path to the file."""
 
     def from_annot(self, value: Path) -> None:
         """Set path and if other attributes are not set, set sensible defaults."""
         if not isinstance(value, Path):
             raise TypeError("'value' must be a 'pathlib.Path'.")
         if not self.name:
             self.name = value.as_posix()
-        self.value = value
+        self.path = value
 
     @classmethod
     @functools.lru_cache
@@ -131,7 +127,7 @@ def from_path(cls, path: Path) -> PathNode:
         """
         if not path.is_absolute():
             raise ValueError("Node must be instantiated from absolute path.")
-        return cls(name=path.as_posix(), value=path)
+        return cls(name=path.as_posix(), path=path)
 
     def state(self) -> str | None:
         """Calculate the state of the node.
@@ -145,7 +141,7 @@ def state(self) -> str | None:
 
     def load(self) -> Path:
         """Load the value."""
-        return self.value
+        return self.path
 
     def save(self, value: bytes | str) -> None:
         """Save strings or bytes to file."""

tests/test_collect_command.py

@@ -385,10 +385,8 @@ def test_print_collected_tasks_with_nodes(capsys):
                 base_name="function",
                 path=Path("task_path.py"),
                 function=function,
-                depends_on={
-                    "depends_on": PathNode(name="in.txt", value=Path("in.txt"))
-                },
-                produces={0: PathNode(name="out.txt", value=Path("out.txt"))},
+                depends_on={"depends_on": PathNode(name="in.txt", path=Path("in.txt"))},
+                produces={0: PathNode(name="out.txt", path=Path("out.txt"))},
             )
         ]
     }