resolve review comments

Author: zhjwpku

pyiceberg/transforms.py

@@ -719,6 +719,7 @@ def transform(self, source: IcebergType) -> Callable[[S | None], S | None]:
         return lambda v: v
 
     def can_transform(self, source: IcebergType) -> bool:
+        # TODO: disallow VariantType when PyIceberg supports it.
         return source.is_primitive and not isinstance(source, (GeographyType, GeometryType))
 
     def result_type(self, source: IcebergType) -> IcebergType:

tests/test_transforms.py

@@ -258,7 +258,7 @@ def test_identity_transform_unknown_type() -> None:
 
 
 @pytest.mark.parametrize("type_var", [GeometryType(), GeographyType()])
-def test_identity_transform_unsupported_type(type_var: PrimitiveType) -> None:
+def test_identity_can_transform_unsupported_type(type_var: PrimitiveType) -> None:
     assert not IdentityTransform().can_transform(type_var)