-
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
[qpg] Added test event delegate for fault injection #22691
[qpg] Added test event delegate for fault injection #22691
Conversation
@nikita-s-wrk - need an issue on this one, linter will complain without it. |
Accepted for 1.0: mostly platform change. 2 files added to core, but they seem to be in a sane location and only added for qpg at this time (but wondering if others will start needing those too) |
PR #22691: Size comparison from a736e56 to 0d6cfb3 Increases above 0.2%:
Increases (6 builds for esp32, qpg, telink)
Decreases (3 builds for bl702, efr32, psoc6)
Full report (37 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
@nikita-s-wrk - this seems to overlap with #22695 which includes tests and fixes some spec compilance. |
src/app/clusters/general-diagnostics-server/GenericFaultTestEventTriggerDelegate.cpp
Show resolved
Hide resolved
src/app/clusters/general-diagnostics-server/GenericFaultTestEventTriggerDelegate.cpp
Outdated
Show resolved
Hide resolved
0d6cfb3
to
bcbd626
Compare
PR #22691: Size comparison from b44263f to bcbd626 Increases above 0.2%:
Increases (8 builds for bl602, bl702, psoc6, qpg, telink)
Full report (37 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
* [qpg]Added test event delegate for fault injection * Check for eventTrigger in GenericFault delegate
* [qpg]Added test event delegate for fault injection * Check for eventTrigger in GenericFault delegate
* [qpg]Added test event delegate for fault injection * Check for eventTrigger in GenericFault delegate
Change overview
This PR