-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Group] Add Group msg yaml test case #11808
[Group] Add Group msg yaml test case #11808
Conversation
PR #11808: Size comparison from 1903401 to c71b92e Increases above 0.2%:
Increases (3 builds for linux)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Add Group membership command when implemented Issue #11077connectedhomeip/src/app/tests/suites/TestGroupMessaging.yaml Lines 22 to 32 in 9317ec0
This comment was generated by todo based on a
|
"connectedhomeip/src/app/tests/suites/TestGroupMessaging.yaml Lines 24 to 34 in 9317ec0
This comment was generated by todo based on a
|
Update this function to work in all possible conditions Issue #11850connectedhomeip/src/controller/CHIPCluster.cpp Lines 47 to 57 in 9317ec0
This comment was generated by todo based on a
|
Issue #11604connectedhomeip/src/controller/WriteInteraction.h Lines 102 to 112 in 9317ec0
This comment was generated by todo based on a
|
* Add Group msg test case
* Add Group msg test case
Problem
Fix #11389
Also partial fix of #11078 since the
ClusterBase::Associate
method was changed for group instantiation.Change overview
Add the group communication yaml test case as discussed with @bzbarsky-apple and @vivien-apple
Still needs #3377 , #11597 #11604 before enabling the test case.
Testing
Tested with the test case added until we reached the coded failure point.
Then the test case was disabled to prevent CI failure.