Skip to content

Commit

Permalink
Fix Race condition in CI (#10990)
Browse files Browse the repository at this point in the history
* Fix Race condition in CI

Increase Delay from 100 to 120 mS to address race condition causing CI to fail

* Increase delay time

* Increased Level change time windows to avoid race conditions

* ZAP Regen
  • Loading branch information
chrisbeach59 authored Oct 26, 2021
1 parent 551e53f commit 7b14930
Show file tree
Hide file tree
Showing 4 changed files with 49 additions and 49 deletions.
16 changes: 8 additions & 8 deletions src/app/tests/suites/certification/Test_TC_LVL_2_1.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,13 +38,13 @@ tests:
- name: "optionOverride"
value: 1

- label: "Wait 10ms"
- label: "Wait 100ms"
cluster: "DelayCommands"
command: "WaitForMs"
arguments:
values:
- name: "ms"
value: 10
value: 100

- label: "reads current Level attribute from DUT"
command: "readAttribute"
Expand All @@ -65,13 +65,13 @@ tests:
- name: "optionOverride"
value: 1

- label: "Wait 100ms"
- label: "Wait 200ms"
cluster: "DelayCommands"
command: "WaitForMs"
arguments:
values:
- name: "ms"
value: 100
value: 200

- label: "reads current Level attribute from DUT"
command: "readAttribute"
Expand All @@ -98,13 +98,13 @@ tests:
- name: "optionOverride"
value: 1

- label: "Wait 1ms"
- label: "Wait 10ms"
cluster: "DelayCommands"
command: "WaitForMs"
arguments:
values:
- name: "ms"
value: 1
value: 10

- label: "reads current Level attribute from DUT"
command: "readAttribute"
Expand All @@ -125,10 +125,10 @@ tests:
- name: "optionOverride"
value: 1

- label: "Wait 10ms"
- label: "Wait 100ms"
cluster: "DelayCommands"
command: "WaitForMs"
arguments:
values:
- name: "ms"
value: 10
value: 100
12 changes: 6 additions & 6 deletions src/app/tests/suites/certification/Test_TC_LVL_3_1.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,13 +44,13 @@ tests:
- name: "optionOverride"
value: 1

- label: "Wait 2500ms"
- label: "Wait 3000ms"
cluster: "DelayCommands"
command: "WaitForMs"
arguments:
values:
- name: "ms"
value: 2500
value: 3000

- label: "reads current level attribute from DUT"
command: "readAttribute"
Expand All @@ -77,13 +77,13 @@ tests:
- name: "optionOverride"
value: 1

- label: "Wait 2500ms"
- label: "Wait 3000ms"
cluster: "DelayCommands"
command: "WaitForMs"
arguments:
values:
- name: "ms"
value: 2500
value: 3000

- label: "reads current level attribute from DUT"
command: "readAttribute"
Expand Down Expand Up @@ -118,13 +118,13 @@ tests:
- name: "optionOverride"
value: 1

- label: "Wait 10ms"
- label: "Wait 100ms"
cluster: "DelayCommands"
command: "WaitForMs"
arguments:
values:
- name: "ms"
value: 10
value: 100

- label: "reads current level attribute from DUT"
command: "readAttribute"
Expand Down
28 changes: 14 additions & 14 deletions src/darwin/Framework/CHIPTests/CHIPClustersTests.m

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

42 changes: 21 additions & 21 deletions zzz_generated/chip-tool/zap-generated/test/Commands.h

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 7b14930

Please sign in to comment.