Skip to content

Commit bd03933

Browse files
palangoistankovic
authored andcommitted
scenarios: Wait longer before checking reward claims
We sometimes see failures for this in CI, so let's give the MS a bit more time.
1 parent c8be4e0 commit bd03933

5 files changed

Lines changed: 10 additions & 10 deletions

File tree

raiden/tests/scenarios/bf1_basic_functionality.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -255,8 +255,8 @@ scenario:
255255
event_args: {closing_participant: 3}
256256

257257
## Monitored channel must be settled before the monitoring service can claim its reward.
258-
## To make sure the transactions gets mined in time, 5 additional blocks are added
258+
## To make sure the transactions gets mined in time, 20 additional blocks are added
259259
## Monitoring Service only reacts for the second closing because node4 settled to first
260260
## channel itself
261-
- wait_blocks: 5
261+
- wait_blocks: 20
262262
- assert_ms_claim: {channel_info_key: "MS Test Channel 3-4"}

raiden/tests/scenarios/ms1_simple_monitoring.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,6 @@ scenario:
6161
event_args: {closing_participant: 0}
6262

6363
## Monitored channel must be settled before the monitoring service can claim its reward
64-
## To make sure the transactions gets mined in time, 10 additional blocks are added
65-
- wait_blocks: 10
64+
## To make sure the transactions gets mined in time, 20 additional blocks are added
65+
- wait_blocks: 20
6666
- assert_ms_claim: {channel_info_key: "MS Test Channel"}

raiden/tests/scenarios/ms1_simple_monitoring_snapshot.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,6 @@ scenario:
6767
event_args: {closing_participant: 0}
6868

6969
## Monitored channel must be settled before the monitoring service can claim its reward
70-
## To make sure the transactions gets mined in time, 10 additional blocks are added
71-
- wait_blocks: 10
70+
## To make sure the transactions gets mined in time, 20 additional blocks are added
71+
- wait_blocks: 20
7272
- assert_ms_claim: {channel_info_key: "MS Test Channel"}

raiden/tests/scenarios/ms2_simple_monitoring.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,6 @@ scenario:
6565
- start_node: 1
6666

6767
## Monitored channel must be settled before the monitoring service can claim its reward
68-
## To make sure the transactions gets mined in time, 10 additional blocks are added
69-
- wait_blocks: 10
68+
## To make sure the transactions gets mined in time, 20 additional blocks are added
69+
- wait_blocks: 20
7070
- assert_ms_claim: {channel_info_key: "MS Test Channel"}

raiden/tests/scenarios/ms3_simple_monitoring.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ scenario:
7070

7171
## Monitored channel must be settled before the monitoring service can claim its reward
7272
## We already waited 22 blocks, so we can subtract that from the settlement_timeout_min.
73-
## Add 10 blocks as safety margin.
74-
- wait_blocks: {{ settlement_timeout_min - 22 + 10 }}
73+
## Add 20 blocks as safety margin.
74+
- wait_blocks: {{ settlement_timeout_min - 22 + 20 }}
7575
# will fail for now since channel was closed by node1.
7676
- assert_ms_claim: {channel_info_key: "MS Test Channel", must_claim: False}

0 commit comments

Comments
 (0)