-
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
[chiptest] Add support for door lock app tests #15412
Conversation
PR #15412: Size comparison from 593b427 to bb91bf4 Full report (34 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
bb91bf4
to
7f89f66
Compare
7f89f66
to
77442e7
Compare
PR #15412: Size comparison from 9fa5641 to 77442e7 Full report (34 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
77442e7
to
edb69fb
Compare
PR #15412: Size comparison from 9fa5641 to edb69fb Full report (34 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
5e64cf1
to
46fb28b
Compare
PR #15412: Size comparison from 031edd0 to 46fb28b Full report (32 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Fast tracking, given this has had enough time for review. |
/rebase |
46fb28b
to
a3ed150
Compare
PR #15412: Size comparison from 1ccc2af to a3ed150 Increases above 0.2%:
Increases (1 build for nrfconnect)
Full report (30 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
@woody-apple I think that this PR will have to wait a little bit more. After adding door lock app to the test suite, an error has emerged. There is a use after free error reported by clang thread analyses. Currently, I'm trying to resolve this use after free issue. Maybe I should convert this PR into a PR draft? |
PR #15412: Size comparison from c116381 to fa3f0a8 Full report (31 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
PR #15412: Size comparison from 8fce49d to 808e070 Full report (31 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
PR #15412: Size comparison from 999aeae to 0f33686 Full report (29 builds for cyw30739, efr32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
PR #15412: Size comparison from 65440ab to 245cd92 Full report (31 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
* [chiptest] Fix warnings reported by PEP8 checker * [chiptest] Add support for door lock app tests
Problem
Door lock tests are skipped when running tests with
./scripts/tests/run_test_suite.py
.Change overview
run
command (--door-lock-app
)Testing
Running test suite on Linux:
./scripts/tests/run_test_suite.py \ --target-glob "DL_*" \ --chip-tool out/examples/chip-tool/chip-tool \ run \ --all-clusters-app out/examples/all-clusters-app/chip-all-clusters-app \ --door-lock-app out/examples/door-lock-app/chip-door-lock-app \ --tv-app out/examples/tv-app/chip-tv-app