fix: address linting and mypy type errors in maintenance tests

Author: qzyu999

tests/table/test_maintenance.py

@@ -15,28 +15,27 @@
 # specific language governing permissions and limitations
 # under the License.
 import random
+
 import pyarrow as pa
-import pytest
 
 from pyiceberg.catalog import Catalog
+from pyiceberg.exceptions import NoSuchNamespaceError
+from pyiceberg.partitioning import PartitionField, PartitionSpec
 from pyiceberg.schema import Schema
-from pyiceberg.partitioning import PartitionSpec, PartitionField
 from pyiceberg.transforms import IdentityTransform
-from pyiceberg.exceptions import NoSuchNamespaceError
 
 
 def test_maintenance_compact(catalog: Catalog) -> None:
     # Setup Schema and specs
-    from pyiceberg.types import NestedField, StringType, LongType
+    from pyiceberg.types import LongType, NestedField, StringType
+
     schema = Schema(
         NestedField(1, "id", LongType()),
         NestedField(2, "category", StringType()),
         NestedField(3, "value", LongType()),
     )
-    spec = PartitionSpec(
-        PartitionField(source_id=2, field_id=1000, transform=IdentityTransform(), name="category")
-    )
-    
+    spec = PartitionSpec(PartitionField(source_id=2, field_id=1000, transform=IdentityTransform(), name="category"))
+
     # Create the namespace and table
     try:
         catalog.create_namespace("default")
@@ -51,11 +50,15 @@ def test_maintenance_compact(catalog: Catalog) -> None:
     # Append many small data files
     categories = ["cat1", "cat2", "cat3"]
     for i in range(12):
-        table.append(pa.table({
-            "id": list(range(i * 10, (i + 1) * 10)),
-            "category": [categories[i % 3]] * 10,
-            "value": [random.randint(1, 100) for _ in range(10)],
-        }))
+        table.append(
+            pa.table(
+                {
+                    "id": list(range(i * 10, (i + 1) * 10)),
+                    "category": [categories[i % 3]] * 10,
+                    "value": [random.randint(1, 100) for _ in range(10)],
+                }
+            )
+        )
 
     # Verify state before compaction
     before_files = list(table.scan().plan_files())
@@ -74,28 +77,30 @@ def test_maintenance_compact(catalog: Catalog) -> None:
     # Ensure snapshot properties specify the replace-operation
     new_snapshot = table.current_snapshot()
     assert new_snapshot is not None
+    assert new_snapshot.summary is not None
     assert new_snapshot.summary.get("snapshot-type") == "replace"
     assert new_snapshot.summary.get("replace-operation") == "compaction"
 
 
 def test_maintenance_compact_empty_table(catalog: Catalog) -> None:
-    from pyiceberg.types import NestedField, StringType, LongType
+    from pyiceberg.types import LongType, NestedField, StringType
+
     schema = Schema(
         NestedField(1, "id", LongType()),
         NestedField(2, "category", StringType()),
     )
-    
+
     try:
         catalog.create_namespace("default")
     except NoSuchNamespaceError:
         pass
-    
+
     table = catalog.create_table("default.test_compaction_empty", schema=schema)
     before_snapshots = len(table.history())
-    
+
     # Should safely return doing nothing
     table.maintenance.compact()
-    
+
     table.refresh()
     after_snapshots = len(table.history())
     assert before_snapshots == after_snapshots  # No new snapshot should be made