Skip to content

Commit e5e11b9

Browse files
committed
test: fix invalid operation assertion and add license headers
1 parent 6800bb4 commit e5e11b9

File tree

2 files changed

+18
-2
lines changed

2 files changed

+18
-2
lines changed

tests/table/test_replace.py

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,19 @@
1+
# Licensed to the Apache Software Foundation (ASF) under one
2+
# or more contributor license agreements. See the NOTICE file
3+
# distributed with this work for additional information
4+
# regarding copyright ownership. The ASF licenses this file
5+
# to you under the Apache License, Version 2.0 (the
6+
# "License"); you may not use this file except in compliance
7+
# with the License. You may obtain a copy of the License at
8+
#
9+
# http://www.apache.org/licenses/LICENSE-2.0
10+
#
11+
# Unless required by applicable law or agreed to in writing,
12+
# software distributed under the License is distributed on an
13+
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
14+
# KIND, either express or implied. See the License for the
15+
# specific language governing permissions and limitations
16+
# under the License.
117
from pyiceberg.catalog import Catalog
218
from pyiceberg.manifest import DataFile, DataFileContent, FileFormat
319
from pyiceberg.schema import Schema

tests/table/test_snapshots.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -398,8 +398,8 @@ def test_merge_snapshot_summaries_overwrite_summary() -> None:
398398

399399
def test_invalid_operation() -> None:
400400
with pytest.raises(ValueError) as e:
401-
update_snapshot_summaries(summary=Summary(Operation.REPLACE))
402-
assert "Operation not implemented: Operation.REPLACE" in str(e.value)
401+
update_snapshot_summaries(summary=Summary(Operation("invalid"))) # type: ignore
402+
assert "Operation not implemented: Operation" in str(e.value)
403403

404404

405405
def test_invalid_type() -> None:

0 commit comments

Comments
 (0)