-
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
Integrate IM event with server #12641
Integrate IM event with server #12641
Conversation
PR #12641: Size comparison from d5e9b66 to c47e159 Increases above 0.2%:
Increases (11 builds for linux, p6, telink)
Decreases (5 builds for linux, p6, telink)
Full report (12 builds for linux, p6, telink)
|
c47e159
to
01e19b0
Compare
PR #12641: Size comparison from db6d572 to 58b4fb0 Increases above 0.2%:
Increases (12 builds for linux, p6, telink)
Decreases (3 builds for p6)
Full report (12 builds for linux, p6, telink)
|
3e2b634
to
925e63c
Compare
PR #12641: Size comparison from a1cb341 to 925e63c Increases above 0.2%:
Increases (20 builds for esp32, linux, mbed, nrfconnect, telink)
Decreases (3 builds for mbed)
Full report (26 builds for esp32, linux, mbed, nrfconnect, telink)
|
925e63c
to
d66dc74
Compare
PR #12641: Size comparison from e98e43f to f8b3150 Increases above 0.2%:
Increases (6 builds for p6, qpg, telink)
Decreases (3 builds for p6)
Full report (7 builds for p6, qpg, telink)
|
f8b3150
to
4f82bac
Compare
Add code to pull arg4, arg5 and arg6 from the arguments of the commandconnectedhomeip/src/app/clusters/test-cluster-server/test-cluster-server.cpp Lines 457 to 467 in 4f82bac
This comment was generated by todo based on a
|
36cc5c0
to
d6bdf19
Compare
PR #12641: Size comparison from 051685b to d6bdf19 Increases above 0.2%:
Increases (3 builds for qpg, telink)
Full report (4 builds for qpg, telink)
|
d6bdf19
to
dfa0cbc
Compare
PR #12641: Size comparison from 051685b to dfa0cbc Increases above 0.2%:
Increases (22 builds for efr32, esp32, linux, mbed, p6, qpg, telink)
Decreases (6 builds for mbed, p6)
Full report (26 builds for efr32, esp32, linux, mbed, p6, qpg, telink)
|
dfa0cbc
to
598e05f
Compare
598e05f
to
b8e6280
Compare
PR #12641: Size comparison from 051685b to b8e6280 Increases above 0.2%:
|
PR #12641: Size comparison from 051685b to d4b241f Increases above 0.2%:
Increases (7 builds for efr32, qpg, telink)
Full report (8 builds for efr32, qpg, telink)
|
4e42805
to
d44e49a
Compare
PR #12641: Size comparison from 2ece989 to d44e49a Increases above 0.2%:
Increases (17 builds for efr32, linux, p6, qpg, telink)
Decreases (3 builds for p6)
Full report (19 builds for efr32, linux, p6, qpg, telink)
|
d44e49a
to
ea5e2d4
Compare
PR #12641: Size comparison from 9e0baa5 to ea5e2d4 Increases above 0.2%:
Increases (31 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (6 builds for mbed, p6)
Full report (39 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Problem
No support for IM Event Server Integration
Change overview
-- Port change from #12059
-- Hook LogEvent with EventManagement, and make necessary change in EventManagement and EventLogger class.
-- Add TestCluster Event Integration.
Testing
Update python test to issue IM command which can instruct server to generate event, then issue read request to retrieve event from server. Then issue another command to generate another event, and check again via read interaction