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

in particular different reasons for ServerClusterCommandExists to test false should result in different errors here #13920

Closed
woody-apple opened this issue Jan 25, 2022 · 0 comments · Fixed by #14806

Comments

@woody-apple
Copy link
Contributor

false


// TODO:in particular different reasons for ServerClusterCommandExists to test false should result in different errors here
AddStatus(concretePath, Protocols::InteractionModel::Status::InvalidCommand);
}
// We have handled the error status above and put the error status in response, now return success status so we can process
// other commands in the invoke request.


This issue was generated by todo-issue based on a TODO comment in f0f5de0.
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Feb 4, 2022
This also fixes the status reporting for unsupported endpoints and clusters.

Fixes project-chip#7612
Fixes project-chip#13920
Fixes project-chip#14022
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Feb 4, 2022
This also fixes the status reporting for unsupported endpoints and clusters.

Fixes project-chip#7612
Fixes project-chip#13920
Fixes project-chip#14022
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Feb 4, 2022
This also fixes the status reporting for unsupported endpoints and clusters.

Fixes project-chip#7612
Fixes project-chip#13920
Fixes project-chip#14022
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Feb 5, 2022
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
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
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
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
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
…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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants