Skip to content

Commit 572ad26

Browse files
committed
[feature/PI-618-bulk_etl_e2e] add grace period to bulk test
1 parent ea7d5bb commit 572ad26

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

src/etl/sds/trigger/bulk/tests/test_bulk_trigger.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ def message(x):
3838
print(x) # noqa
3939

4040

41-
@pytest.mark.timeout(20)
41+
@pytest.mark.timeout(40)
4242
@pytest.mark.integration
4343
def test_bulk_trigger():
4444
# Prerequisites
@@ -126,5 +126,6 @@ def test_bulk_trigger():
126126
message("Load's input data is now in empty state")
127127

128128
while not was_state_lock_removed():
129-
message("State lock has been removed")
130129
time.sleep(1)
130+
message("State lock has been removed")
131+
time.sleep(10) # Grace period for Step Function to complete

src/etl/sds/trigger/manual/tests/test_manual_trigger.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,8 @@
2828
],
2929
)
3030
def test_manual_trigger(history_object):
31+
time.sleep(10) # Grace period for any previous Step Functions to complete
32+
3133
# Where the state is located
3234
etl_config = get_etl_config(f"{EXPECTED_CHANGELOG_NUMBER}.ldif")
3335
manual_trigger_arn = read_terraform_output("manual_trigger_arn.value")

0 commit comments

Comments
 (0)