property fields

Author: gabeiglio

pyiceberg/view/__init__.py

@@ -54,6 +54,7 @@ def current_version(self) -> ViewVersion:
         """Get the version of this view."""
         return next(version for version in self.metadata.versions if version.version_id == self.metadata.current_version_id)
 
+    @property
     def versions(self) -> list[ViewVersion]:
         """Get the versions of this view."""
         return self.metadata.versions
@@ -66,6 +67,7 @@ def history(self) -> list[ViewHistoryEntry]:
         """Get the version of this history view."""
         return self.metadata.version_log
 
+    @property
     def properties(self) -> dict[str, str]:
         """Return a map of string properties for this view."""
         return self.metadata.properties

tests/test_view.py

@@ -56,7 +56,7 @@ def test_view_current_version(view: View) -> None:
 
 
 def test_view_versions(view: View) -> None:
-    versions = view.versions()
+    versions = view.versions
     assert len(versions) == 1
     assert isinstance(versions[0], ViewVersion)
     assert versions[0].version_id == 1
@@ -78,7 +78,7 @@ def test_view_history(view: View) -> None:
 
 
 def test_view_properties(view: View) -> None:
-    assert view.properties() == {"comment": "this is a test view"}
+    assert view.properties == {"comment": "this is a test view"}
 
 
 def test_view_location(view: View) -> None:
@@ -107,7 +107,7 @@ def test_view_schemas_multiple(example_view_metadata_v1_multiple_versions: dict[
 
 def test_view_versions_multiple(example_view_metadata_v1_multiple_versions: dict[str, Any]) -> None:
     view = View(("default", "test_view"), ViewMetadata.model_validate(example_view_metadata_v1_multiple_versions))
-    assert len(view.versions()) == 2
+    assert len(view.versions) == 2
     assert view.current_version().version_id == 2