fix: restore PathData.get_value to return Value object

- Return Value object instead of resolved string value
- Remove resolve parameter as it's now handled by Value.get()
- Update tests to verify Value object return
- Maintain proper logging

This provides better control over value resolution and access
to Value object features, especially for sensitive values.

Author: ritwik-g

helm_values_manager/models/path_data.py

@@ -72,24 +72,21 @@ def set_value(self, environment: str, value: Value) -> None:
 
         self._values[environment] = value
 
-    def get_value(self, environment: str, resolve: bool = False) -> Optional[str]:
+    def get_value(self, environment: str) -> Optional[Value]:
         """
-        Get the value for a specific environment.
+        Get the Value object for a specific environment.
 
         Args:
-            environment: The environment to get the value for
-            resolve: Whether to resolve the value (for sensitive values)
+            environment (str): The environment to get the value for
 
         Returns:
-            Optional[str]: The value if found, None otherwise
+            Optional[Value]: The Value object if it exists, None otherwise
         """
         logger.debug("Getting value for path %s in environment %s", self.path, environment)
         value = self._values.get(environment)
         if value is None:
             logger.debug("No value found for path %s in environment %s", self.path, environment)
-            return None
-
-        return value.get(resolve=resolve)
+        return value
 
     def get_environments(self) -> Iterator:
         """

tests/unit/models/test_path_data.py

@@ -50,9 +50,8 @@ def test_get_value_nonexistent_environment(path_data):
 
 def test_get_value(path_data, mock_value):
     """Test getting a value."""
-    mock_value.get.return_value = "test_value"
     path_data.set_value("test", mock_value)
-    assert path_data.get_value("test") == "test_value"
+    assert path_data.get_value("test") == mock_value
 
 
 def test_get_environments(path_data, mock_value):