Fixed some more tests

Author: vinjai

pyiceberg/table/update/__init__.py

@@ -623,8 +623,6 @@ def validate(self, base_metadata: Optional[TableMetadata]) -> None:
             if self.snapshot_id is None:
                 raise CommitFailedException(f"Requirement failed: {ref_type} {self.ref} was created concurrently")
             elif self.snapshot_id != snapshot_ref.snapshot_id:
-                if ref_type == SnapshotRefType.TAG:
-                    raise CommitFailedException(f"Requirement failed: TAG {self.ref} can't be updated once created")
                 raise CommitFailedException(
                     f"Requirement failed: {ref_type} {self.ref} has changed: expected id {self.snapshot_id}, found {snapshot_ref.snapshot_id}"
                 )

tests/table/test_init.py

@@ -1015,7 +1015,7 @@ def test_assert_ref_snapshot_id(table_v2: Table) -> None:
 
     with pytest.raises(
         CommitFailedException,
-        match="Requirement failed: TAG test can't be updated once created",
+        match="Requirement failed: tag test has changed: expected id 3055729675574597004, found 3051729675574597004",
     ):
         AssertRefSnapshotId(ref="test", snapshot_id=3055729675574597004).validate(base_metadata)