Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

YAML: Darwin issues w/ configurable argument on DelayCommand #15192

Closed
jmeg-sfy opened this issue Feb 15, 2022 · 0 comments · Fixed by #15199
Closed

YAML: Darwin issues w/ configurable argument on DelayCommand #15192

jmeg-sfy opened this issue Feb 15, 2022 · 0 comments · Fixed by #15199
Assignees
Labels
sdk requirements from test harness Test Harness issues related to the CSG Test Harness

Comments

@jmeg-sfy
Copy link
Contributor

Problem

YAML test Compilation error w/ darwin when arguments are used for configuring WaitForMS for example

`config:
nodeId: 0x12344321
cluster: "Window Covering"
endpoint: 1
fastMotionDuration:
type: INT16U
defaultValue: 3000

  cluster: "DelayCommands"
  command: "WaitForMs"
  arguments:
      values:
          - name: "ms"
            value: fastMotionDuration

`

--> give Error below on Darwin only (it is working fine on Linux)
/Users/runner/work/connectedhomeip/connectedhomeip/src/darwin/Framework/CHIPTests/CHIPClustersTests.m:27523:35: error: use of undeclared identifier 'fastMotionDuration'
WaitForMs(expectation, queue, fastMotionDuration);

@jmeg-sfy jmeg-sfy added sdk requirements from test harness Test Harness issues related to the CSG Test Harness labels Feb 15, 2022
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Feb 15, 2022
Still does not allow specifying an override value, but should not at
least use the config defaults properly.

Fixes project-chip#15192
woody-apple pushed a commit that referenced this issue Feb 15, 2022
Still does not allow specifying an override value, but should not at
least use the config defaults properly.

Fixes #15192
jmeg-sfy added a commit to jmeg-sfy/connectedhomeip that referenced this issue Feb 16, 2022
andy31415 pushed a commit that referenced this issue Feb 17, 2022
* Test: Start adding tests backport

* TEST: Yaml add WNCV 3.4 3.5 4.3 4.4

* DEV: Remove 2.3 need another PR for it

* DEV: Add wait for commissionee

* DEV: Remove Darwin tests 3.5 3.4 due to compile issues

- Add comment about todos + issues

* DEV: Zap all-cluster-app activate missing functions for CI

* Restyled by whitespace

* Restyled by prettier-yaml

* DEV: Activate 3.4 3.5 on Darwin

- Issue #15192 is resolved

Co-authored-by: Restyled.io <[email protected]>
jamesluo11 pushed a commit to jamesluo11/connectedhomeip that referenced this issue Apr 26, 2022
…t-chip#15199)

Still does not allow specifying an override value, but should not at
least use the config defaults properly.

Fixes project-chip#15192
jamesluo11 pushed a commit to jamesluo11/connectedhomeip that referenced this issue Apr 26, 2022
* Test: Start adding tests backport

* TEST: Yaml add WNCV 3.4 3.5 4.3 4.4

* DEV: Remove 2.3 need another PR for it

* DEV: Add wait for commissionee

* DEV: Remove Darwin tests 3.5 3.4 due to compile issues

- Add comment about todos + issues

* DEV: Zap all-cluster-app activate missing functions for CI

* Restyled by whitespace

* Restyled by prettier-yaml

* DEV: Activate 3.4 3.5 on Darwin

- Issue project-chip#15192 is resolved

Co-authored-by: Restyled.io <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sdk requirements from test harness Test Harness issues related to the CSG Test Harness
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants