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

[TC-DIAGSW-3.2] Command failing --ResetWatermarks #11578

Closed
ghost opened this issue Nov 9, 2021 · 1 comment · Fixed by #14806
Closed

[TC-DIAGSW-3.2] Command failing --ResetWatermarks #11578

ghost opened this issue Nov 9, 2021 · 1 comment · Fixed by #14806
Labels
cert spec Mismatch between spec and implementation V1.0

Comments

@ghost
Copy link

ghost commented Nov 9, 2021

Summary :
As per test plane the command ResetWatermarks is to be send from the TH and should return a successful status, however when we try to use this command in the YAML test scripts they seem to always fail, please refer attached screenshots and logs

Test-plan Screenshot:
image

Spec Screenshot:
image

Log file and Execution Screenshot:
DIASW_3_2.txt
image
@ghost ghost changed the title TE6 [TC-DIAGSW-3.2] Command failing --ResetWatermarks [TC-DIAGSW-3.2] Command failing --ResetWatermarks Nov 10, 2021
@tcarmelveilleux tcarmelveilleux added spec Mismatch between spec and implementation and removed v1_triage_split_3 labels Jan 31, 2022
@bzbarsky-apple
Copy link
Contributor

That's because the test is sending it to endpoint 1, not endpoint 0.

bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Feb 5, 2022
@bzbarsky-apple
Add correct checking for whether a command is supported.
d7717cc 
This also fixes the status reporting for unsupported endpoints and
clusters.  And fixes some tests to use the right endpoint, now that we
are checking that sort of thing.

Fixes project-chip#7612
Fixes project-chip#13920
Fixes project-chip#14022
Fixes project-chip#11578
Fixes project-chip#11185
@bzbarsky-apple bzbarsky-apple mentioned this issue Feb 5, 2022
Merged
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Feb 5, 2022
@bzbarsky-apple
Add correct checking for whether a command is supported.
df971f0 
This also fixes the status reporting for unsupported endpoints and
clusters.  And fixes some tests to use the right endpoint, now that we
are checking that sort of thing.

Fixes project-chip#7612
Fixes project-chip#13920
Fixes project-chip#14022
Fixes project-chip#11578
Fixes project-chip#11185
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Feb 5, 2022
@bzbarsky-apple
Add correct checking for whether a command is supported.
6eaf69a 
This also fixes the status reporting for unsupported endpoints and
clusters.  And fixes some tests to use the right endpoint, now that we
are checking that sort of thing.

Fixes project-chip#7612
Fixes project-chip#13920
Fixes project-chip#14022
Fixes project-chip#11578
Fixes project-chip#11185
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Feb 5, 2022
@bzbarsky-apple
Add correct checking for whether a command is supported.
606d9d2 
This also fixes the status reporting for unsupported endpoints and
clusters.  And fixes some tests to use the right endpoint, now that we
are checking that sort of thing.

Fixes project-chip#7612
Fixes project-chip#13920
Fixes project-chip#14022
Fixes project-chip#11578
Fixes project-chip#11185
bzbarsky-apple added a commit that referenced this issue Feb 7, 2022
@bzbarsky-apple
Add correct checking for whether a command is supported. (#14806)
26fa3fa 
This also fixes the status reporting for unsupported endpoints and
clusters.  And fixes some tests to use the right endpoint, now that we
are checking that sort of thing.

Fixes #7612
Fixes #13920
Fixes #14022
Fixes #11578
Fixes #11185
mkardous-silabs pushed a commit to mkardous-silabs/connectedhomeip that referenced this issue Feb 7, 2022
@bzbarsky-apple @mkardous-silabs
Add correct checking for whether a command is supported. (project-chi…
9684468 
…p#14806)

This also fixes the status reporting for unsupported endpoints and
clusters.  And fixes some tests to use the right endpoint, now that we
are checking that sort of thing.

Fixes project-chip#7612
Fixes project-chip#13920
Fixes project-chip#14022
Fixes project-chip#11578
Fixes project-chip#11185
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cert spec Mismatch between spec and implementation V1.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add correct checking for whether a command is supported. bzbarsky-apple/connectedhomeip
4 participants
@tcarmelveilleux @andy31415 @woody-apple @bzbarsky-apple