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.

Sign up for GitHub

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

Jump to bottom

Need YAML support for config variable names for a struct field response check. #23508

Closed
manjunath-grl opened this issue Nov 4, 2022 · 0 comments · Fixed by #23574
Closed

Need YAML support for config variable names for a struct field response check. #23508

manjunath-grl opened this issue Nov 4, 2022 · 0 comments · Fixed by #23574

Comments

@manjunath-grl
Copy link
Contributor

Feature description

Tried to read Extension attribute struct response check using the config variable name eg: D_OK_EMPTY, but got response as " Test Failure: extension[0].data value mismatch, expecting D_OK_EMPTY " even though the response data matching but getting this above failure.

Platform

all

Platform Version(s)

No response

Anything else?

Diff link: Diff link: master...manjunath-grl:connectedhomeip:ACL_Extension_Attribute_Read_Error
image
image
ACL logs:
ACL_Attribute_ReadError_2_3.zip
@manjunath-grl manjunath-grl mentioned this issue Nov 4, 2022
Closed
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Nov 10, 2022
@bzbarsky-apple
Fix use of string-typed config values in YAML expected value checks.
6a0363f 
We were never looking at the config if the type was string.

Fixes project-chip#23508
@bzbarsky-apple bzbarsky-apple mentioned this issue Nov 10, 2022
Merged
yufengwangca pushed a commit that referenced this issue Nov 14, 2022
@bzbarsky-apple
Fix use of string-typed config values in YAML expected value checks. (#…
8898fd8 
…23574)

We were never looking at the config if the type was string.

Fixes #23508
adbridge pushed a commit to ARM-software/connectedhomeip that referenced this issue Nov 18, 2022
@bzbarsky-apple @adbridge
Fix use of string-typed config values in YAML expected value checks. (p…
07a021f 
…roject-chip#23574)

We were never looking at the config if the type was string.

Fixes project-chip#23508
adbridge pushed a commit to ARM-software/connectedhomeip that referenced this issue Nov 18, 2022
@bzbarsky-apple @adbridge
Fix use of string-typed config values in YAML expected value checks. (p…
0591a6c 
…roject-chip#23574)

We were never looking at the config if the type was string.

Fixes project-chip#23508
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix use of string-typed config values in YAML expected value checks. bzbarsky-apple/connectedhomeip
1 participant
@manjunath-grl