Skip to content

Commit 356d704

Browse files
committed
test: Fix import errors and mypy exception
1 parent a2f2b18 commit 356d704

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

tests/table/test_replace.py

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,16 @@
1414
# KIND, either express or implied. See the License for the
1515
# specific language governing permissions and limitations
1616
# under the License.
17+
import pytest
1718
from pyiceberg.catalog import Catalog
18-
from pyiceberg.manifest import DataFile, DataFileContent, FileFormat, ManifestEntryStatus
19-
from pyiceberg.schema import Schema
19+
from pyiceberg.manifest import (
20+
DataFile,
21+
DataFileContent,
22+
FileFormat,
23+
ManifestContent,
24+
ManifestEntry,
25+
ManifestEntryStatus,
26+
)from pyiceberg.schema import Schema
2027
from pyiceberg.table.snapshots import Operation
2128
from pyiceberg.typedef import Record
2229

@@ -204,6 +211,7 @@ def test_replace_reuses_unaffected_manifests(catalog: Catalog) -> None:
204211
rewrite.append_data_file(file_c)
205212

206213
snapshot_after = table.current_snapshot()
214+
assert snapshot_after is not None
207215
manifests_after = snapshot_after.manifests(table.io)
208216

209217
# We expect 3 manifests:
@@ -434,6 +442,7 @@ def test_replace_passes_through_delete_manifests(catalog: Catalog) -> None:
434442

435443
# Verify the delete manifest was passed through unchanged
436444
snapshot_after = table.current_snapshot()
445+
assert snapshot_after is not None
437446
manifests_after = snapshot_after.manifests(table.io)
438447
manifest_paths_after = [m.manifest_path for m in manifests_after]
439448

0 commit comments

Comments
 (0)