Skip to content

Commit 192d7d7

Browse files
committed
Cover concurrent import table ownership
1 parent 970486d commit 192d7d7

1 file changed

Lines changed: 14 additions & 0 deletions

File tree

‎tests/test_atomic_import.py‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -359,3 +359,17 @@ def fail_first_normative_finish(*args, **kwargs):
359359
assert connection.execute(
360360
"select code, dataset_id from fidelity_event_catalog"
361361
).fetchall() == [("import_failed", None)]
362+
363+
def test_failed_table_create_never_claims_cleanup_ownership() -> None:
364+
class ConcurrentTable:
365+
def create(self, _connection):
366+
raise RuntimeError("table already created by concurrent import")
367+
368+
state = {"data_table_created": False}
369+
370+
with pytest.raises(RuntimeError, match="concurrent import"):
371+
wide._create_operation_owned_data_table(
372+
object(), ConcurrentTable(), state,
373+
)
374+
375+
assert state["data_table_created"] is False

0 commit comments

Comments
 (0)