fix: resolve ruff import ordering and pagination cleanup bug

Merge the duplicate typedef import in test_rest.py that ruff's
import sorter flagged. Also fix clean_up() in conftest, which
iterated and mutated (drop_table/drop_namespace) over list_* results
in the same pass - now that those can be lazily-paginated, deleting
an item before the next page is fetched shifts the server-side
offset and skips entries. Materialize with list() first.

Author: stark256-spec

tests/catalog/test_rest.py

@@ -65,10 +65,9 @@
 from pyiceberg.table.metadata import TableMetadataV1
 from pyiceberg.table.sorting import SortField, SortOrder
 from pyiceberg.transforms import IdentityTransform, TruncateTransform
-from pyiceberg.typedef import RecursiveDict
+from pyiceberg.typedef import PaginationList, RecursiveDict
 from pyiceberg.types import StringType
 from pyiceberg.utils.config import Config
-from pyiceberg.typedef import PaginationList
 from pyiceberg.view import View
 from pyiceberg.view.metadata import ViewMetadata, ViewVersion
 

tests/conftest.py

@@ -2505,13 +2505,15 @@ def fixture_s3_client() -> boto3.client:
 
 def clean_up(test_catalog: Catalog) -> None:
     """Clean all databases and tables created during the integration test."""
-    for database_tuple in test_catalog.list_namespaces():
+    # Materialize before mutating: list_* may return a lazily-paginated list, and
+    # dropping entries while paging would shift the underlying page offsets.
+    for database_tuple in list(test_catalog.list_namespaces()):
         database_name = database_tuple[0]
         if "my_iceberg_database-" in database_name:
-            for identifier in test_catalog.list_tables(database_name):
+            for identifier in list(test_catalog.list_tables(database_name)):
                 test_catalog.drop_table(identifier)
             try:
-                for identifier in test_catalog.list_views(database_name):
+                for identifier in list(test_catalog.list_views(database_name)):
                     test_catalog.drop_view(identifier)
             except NotImplementedError:
                 pass