[pull] master from project-chip:master #592
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Modified test Test_TC_CC_4_2 Test_TC_CC_7_2 Test_TC_PWRTL_1_1 Test_TC_SWTCH_3_2 Test_TC_OO_2_6 * Fix TC-OO-2.6 and SWTCH_3_2 tests * Restyled by whitespace * Restyled by prettier-json --------- Co-authored-by: Restyled.io <commits@restyled.io>
ZAP has this information. We should just use that instead of duplicating it. Fixes #35147
* Updated yaml script as per the Issues raised by SQA * Restyled by whitespace * Updated the default value of unsupportedNumberOfRinsesValue * Update src/app/tests/suites/certification/Test_TC_DD_2_1.yaml Co-authored-by: Leo Rozendaal <leo.rozendaal@signify.com> * Update src/app/tests/suites/certification/Test_TC_DD_2_1.yaml Co-authored-by: Leo Rozendaal <leo.rozendaal@signify.com> * Update src/app/tests/suites/certification/Test_TC_DESC_2_1.yaml Co-authored-by: Leo Rozendaal <leo.rozendaal@signify.com> * Update src/app/tests/suites/certification/Test_TC_DESC_2_1.yaml Co-authored-by: Leo Rozendaal <leo.rozendaal@signify.com> * Update src/app/tests/suites/certification/Test_TC_DESC_2_1.yaml Co-authored-by: Leo Rozendaal <leo.rozendaal@signify.com> * Update src/app/tests/suites/certification/Test_TC_DESC_2_1.yaml Co-authored-by: Leo Rozendaal <leo.rozendaal@signify.com> * Update src/app/tests/suites/certification/Test_TC_DESC_2_1.yaml Co-authored-by: Leo Rozendaal <leo.rozendaal@signify.com> * Update src/app/tests/suites/certification/Test_TC_DESC_2_1.yaml Co-authored-by: Leo Rozendaal <leo.rozendaal@signify.com> * Update src/app/tests/suites/certification/Test_TC_DD_2_1.yaml Co-authored-by: Leo Rozendaal <leo.rozendaal@signify.com> * Update src/app/tests/suites/certification/Test_TC_DD_2_1.yaml Co-authored-by: Leo Rozendaal <leo.rozendaal@signify.com> * Restyled by prettier-yaml --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: cjandhyala <68604034+cjandhyala@users.noreply.github.com> Co-authored-by: Justin Wood <woody@apple.com> Co-authored-by: Leo Rozendaal <leo.rozendaal@signify.com>
* Fixed default SceneTableSize value in waterleakdetector and improved the ifdef detection of the define * Removed Scenes Management from waterleak detector and template
* The flag of CONFIG_BUILD_FOR_HOST_UNIT_TEST is not actually tied to unit testing. Implement a separate flag to control if we crash on errors for IM/DM checks or not. * Update src/app/common_flags.gni Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * make the flag name singular * Add unit test cluster members * Modified zap and made these attributes optional * Zap regen * Attributes MUST be up to 0x4FFF so needed to switch codes * Update ids enabling * Move around things and add unit test * Update to run unit tests with enforced DM checking * Comment describing the updated options * Fix unit test * Restyle * Kotlin format since it seems different eforcement is going on here * Update src/app/zap-templates/zcl/data-model/chip/test-cluster.xml Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Update examples/all-clusters-app/linux/main-common.cpp Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Remove redundant using * Simplify/shorten code * Move base to 0x3000 for the new attributes for test cluster testing * Restyle * Fix python unit test * Fix all clusters app * Fix unused import in python * Zap regen * Fix cirque * Fix hardcoded paths in TestTimeSyncTrustedtimeSourceRunner.py * Typo fix * Remove extra spacing * Update text * Fix app path --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com> Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
* Updated DoorLock Python scripts and DRLK_2_1 yaml. * TC_DRLK_2_X.py - updated as per latest test plan * TC_BOOLCFG_3_1.py - Test description is changed. * TC_PWRTL_2_1.py - Added missing step for subset check as per latest test plan. * Test_TC_DRLK_2_1.yaml - Script is updated with latest changes from test plan. * Restyled by whitespace * Restyled by prettier-yaml * Restyled by autopep8 * Lint errors are fixed * ednpoint will be taken from command line args * Teardown function added for all scripts --------- Co-authored-by: Restyled.io <commits@restyled.io>
--------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: saurabhst <s.kumar9@samsung.com>
Without this, the TH is going to select this test for every device and every device except the MACL devices will fail.
* Basic comp tests: add fix for pre-commissioned devices * add check to CI
* Minor adjustments from TP feedback * Update src/python_testing/TC_LVL_2_3.py
* Improve the stability of test TC_MCORE_FS_1_4 * Address the review comments * Make wait_for_server_initialization global * Restyled by autopep8 --------- Co-authored-by: Restyled.io <commits@restyled.io>
* Fix ACL cluster ZAP XML and regen ZAP - Remove dead event AccessRestrictionEntryChanged - Set response command for ReviewFabricRestrictions - Set correct optionality for FabricRestrictionReviewUpdate * Fix access control server after XML change * Remove unsupportable commnad * Rename RedirectURL field to ARLRequestFlowUrl * Kick CI * Kick CI * Remove non-longer-applicable files * Increase size of event buffer pool in NIM sample * Restyled by clang-format * Fix format * Remove subscription for event that no longer exists --------- Co-authored-by: Restyled.io <commits@restyled.io>
…ulated as required by the test (#35413) * Add check for unpopulated ARLs * linter * Add check to TC-ACL-2.11 for populated ARLs
We are running the test against Matter applications that are starting up, and during startup we do some silly unregistering/re-registering of DNS-SD advertisements. The test was counting all the registrations, not subtracting the un-registrations, and asserting that the resulting count was not too high. But sometimes it is, if we get a "unregister/re-register" pair in there. The fix: 1) Fixes the test to better track un-registration bits. 2) Fixes the OnBleScanRemove code to call the right delegate method. 3) Fixes OnBrowseRemove to not call the delegate if we have not notified about the add for the relevant result. We could have just worked around this in the test, but notifying "removed" about an object with all fields set to nil just doesn't make much sense.
…ioning SecondaryNetworkInterface device (#35255)
* ESP32: Support Thread and Wi-Fi network revert configuration * Update src/platform/ESP32/NetworkCommissioningDriver.cpp Co-authored-by: Wang Qixiang <43193572+wqx6@users.noreply.github.com> * Update KeyValueStoreManagerImpl.cpp --------- Co-authored-by: Wang Qixiang <43193572+wqx6@users.noreply.github.com>
* Override timeout for FabricSync tests * Bump up timeouts just a little more * Remove TODOs that actually won't change since test will need to wait upto time anyways * Add unsaved files
* [ESP32] menuconfig options for groups cluster * all-clusters-app has 3 endpoints with group cluster
* Add MinValue/MaxValue helpers on our NumericAttributeTraits. Uses the new helpers in SceneHandlerImpl. Fixes #35328 * Address review comments.
… TV (#35331) * tv-casting-app cancel connection upon CancelPasscode CDC message from TV * Addressed comments by sharadb-amazon * Fix iOS app crash due to null cpp cluster
* Added a base38 rountrip Fuzzer to fuzz the encoding and decoding in base38 Added a base38 decoder Fuzzer to fuzz the function decoding base38 strings Added a dependancy in TLV-Reader config to correct the build error * adding all clusters * Restyled by gn * Moving this back * Restyled by gn * Update src/setup_payload/tests/FuzzBase38.cpp Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Update src/setup_payload/tests/FuzzBase38.cpp Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Update src/setup_payload/tests/FuzzBase38.cpp Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Removing this * Updating * Update src/setup_payload/tests/FuzzBase38Decode.cpp Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * restrict activation of enable_fuzz_test_targets --------- Co-authored-by: Justin Wood <woody@apple.com> Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )