test #32
Closed
test #32
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
Contributor
mkardous-silabs
commented
Oct 3, 2024
mkardous-silabs
commented
… EWATERHTR test scripts (#35560) * Added missing "is_commissioning=True" to step 1 in all DEM, EEVSE and EWATERHTR test cases. * Added missing EEVSE_2.3
* Added new wifi BRD2605A board support in matter * Moved si91x support files to common place for brd4338a and brd2605a boards * Updating matter_support pointer * Updated matter_support pointer to latest main * Added the default setting chip_device_platform to SiWx917 into build_for_wifi_gnfile.gn * added brd2605a in efr32.py and targets.py
The "id" was a copy/paste error. For the public API we can't do much about that, but internally we can sanity-check and use the right type.
…542) * Separate out the cmake logic a bit. Still somewhat broken through as the codegen data model is not built * Fix build and remove INCLUDE_SERVER cmake argument
…(#35546). (#35548) * Formalize command line option and argument processing. * Add support '-h' for usage in addition to '--help'. * Change shebang from '/bin/bash' to '/usr/bin/env bash'. * Update the file copyright header.
…. (#35541) * Stop persistent operational browse when all controllers are suspended. * Address review comments.
* [nxp][examples][platform] Add cluster-handling functions & remove RestoreState * Add virtual cluster-handling functions that must be implemented per application based on cluster configuration. * Remove RestoreState as it may read attributes from cluster before the enpoints are initialized. Signed-off-by: Andrei Menzopol <[email protected]> * [nxp][examples][common] Implement cluster-handling functions Signed-off-by: Andrei Menzopol <[email protected]> * [nxp][platform][mcxw71_k32w1] Add ResetInIdle functions Signed-off-by: Andrei Menzopol <[email protected]> * [nxp][platform][common][legacy] Add SetRebootDelaySec function Signed-off-by: Andrei Menzopol <[email protected]> * [nxp][platform][mcxw71_k32w1] Platform changes * Add __wrap__calloc_r * Use ProcessSetStateClusterHandler in ButtonManager Signed-off-by: Andrei Menzopol <[email protected]> * [nxp] Add initial lock app common files Signed-off-by: marius-alex-tache <[email protected]> * [nxp][examples][common][lock-app] LockApp changes needed for KW/MCX * Add cluster-handling functions * Put OnSubscriptionRequested override under define * Call RestoreState in PostAttributeChangeCallback * Call RestoreState in DoorLockClusterInitCallback * Add FSL_OSA_MAIN_FUNC_ENABLE main_task * Update lock-app/nxp/zap/lock-app zap files * Include stdint Signed-off-by: Andrei Menzopol <[email protected]> * [nxp][examples][k32w1] Remove cluster attribute and endpoint defines Signed-off-by: Andrei Menzopol <[email protected]> * [nxp][examples][mcxw71] Remove cluster attribute and endpoint defines Signed-off-by: Andrei Menzopol <[email protected]> * [nxp][platform][common][icd] Do not change publisher-selected maximum interval Our code from OnSubscriptionRequested sets the publisher-selected maximum interval to kSubscriptionMaxIntervalPublisherLimit to 60 mins which is not spec compliant, it should be set to `Idle Mode Duration` as done in the default implementation from src/app/ReadHandler.cpp. Signed-off-by: Andrei Menzopol <[email protected]> * [nxp][examples][k32w1] Add LockApp examples Signed-off-by: Andrei Menzopol <[email protected]> * [nxp][examples][mcxw71] Add LockApp examples Signed-off-by: Andrei Menzopol <[email protected]> * [nxp] Add support for lock app Signed-off-by: marius-alex-tache <[email protected]> * Restyled by whitespace * Restyled by clang-format * Restyled by gn * Restyled by prettier-markdown * [nxp] Fix lock app mcxw71 readme files Signed-off-by: marius-alex-tache <[email protected]> * Restyled by prettier-markdown * [nxp] Fix a typo for lighting app readme Signed-off-by: marius-alex-tache <[email protected]> * [nxp] Add common lock app readme file Signed-off-by: marius-alex-tache <[email protected]> * Restyled by prettier-markdown * [nxp][mcxw71] Fix lock app naming Signed-off-by: marius-alex-tache <[email protected]> * [nxp] Fix lock app zap files Signed-off-by: marius-alex-tache <[email protected]> --------- Signed-off-by: Andrei Menzopol <[email protected]> Signed-off-by: marius-alex-tache <[email protected]> Co-authored-by: Andrei Menzopol <[email protected]> Co-authored-by: Restyled.io <[email protected]>
…#35509) Bumps [jinja2](https://github.com/pallets/jinja) from 3.1.3 to 3.1.4. - [Release notes](https://github.com/pallets/jinja/releases) - [Changelog](https://github.com/pallets/jinja/blob/main/CHANGES.rst) - [Commits](pallets/jinja@3.1.3...3.1.4) --- updated-dependencies: - dependency-name: jinja2 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…about a state change. (#35577)
…ature (#35579) * Update TC_TSTAT_4_2 to work with thermostats that do not have AUTO feature * Python formatting for restyled
…tforms (#35559) * Add data-mode-enabled/disabled variants to all CMAKE builds. Platforms changed: - esp32 - mbed - nrfconnect - telink - nxp I tested all these except NXP which I could not compile in TOT due to missing vscode configuration. * add options for data model to qpg builds as this can be used to check for size deltas * Fix unit test * Restyle * Fix nxp typo and strip * Update quoting logic * Update config/telink/chip-module/CMakeLists.txt Co-authored-by: Terence Hampson <[email protected]> * more update on the 'I' text * Fix nxp build placement of data model interface * Switch to zephyr KConfig for data model interface setting * Use bool for DM enabling in places * Apply review comment * Fix nxp typo --------- Co-authored-by: Andrei Litvin <[email protected]> Co-authored-by: Terence Hampson <[email protected]>
* Darwin: Add support for zippered builds * Restyled by shellharden * Restyled by shfmt --------- Co-authored-by: Restyled.io <[email protected]>
* Add a wildcard attribute read API on MTRDevice. * Adding new XPC protocol * Restyled by whitespace * Restyled by clang-format * Add unit test for NSCoding bits. * Fix up XPC protocol bits. --------- Co-authored-by: Justin Wood <[email protected]> Co-authored-by: Restyled.io <[email protected]>
…to `zap` (#35549). (#35582)
* scripts/tools/zap/run_zaptool.sh
* Display the value of 'ZAP_CMD' to standard output.
* Add support for the `--zap` command line option to specify the `zap` executable to run.
* Added usage documentation about `ZAP_DEVELOPMENT_PATH` and `ZAP_INSTALL_PATH`.
* docs/code_generation.md: Added documentation about the `--zap ZAP` option to `run_zaptool.sh`.
Commissioning window can be opened using timeout exceeding the maximum value of 900 s defined by the spec. This can happen if selected transport is IP, but the device uses BLE extended announcement feature. Added checking if device is commissioned to be able to determine what max timeout should be used for the particular scenario. Renamed CHIP_DEVICE_CONFIG_BLE_EXT_ADVERTISING to the CHIP_DEVICE_CONFIG_EXT_ADVERTISING, as config name sounds misleading and it seems it relates only to BLE. Fixes: #35505
* Remove illegal Identify clusters from examples Problem: - Many examples have the Identify cluster on endpoint 0 and this is not useful or legal. The Identify cluster has to be on a simple endpoint. This PR: - Removes the identify clusters in the wrong places (verified they are elsewhere where required) - Adds a lint rule that EP0 can't have Identify - Fixes YAML test that expected Identify on EP0 to use EP1 available in all examples. Testing done: - Integration tests still pass - New lint rule failed before PR, passes now * Fix CI * Update PairOnNetworkLongImInvokeCommand.kt * Update PairOnNetworkLongImInvokeCommand.kt * Update PairOnNetworkLongImInvokeCommand.kt * Update PairOnNetworkLongImInvokeCommand.kt * fix java/kotlin style --------- Co-authored-by: yunhanw-google <[email protected]>
…925) * Refactor Key Attributes classes Implement `keyAttributesBase` and add persistent lifetime functionality. * Implement `FindFreeKeySlotInRange` for PSA key slots * Fix ICD CIP related build dependencies * Remake `ICDMonitoringTable::Find()` not to overwrite all entry fields If entry is not found its key handle field must not be filled with last checked entry when using PSA as it will cause key slot to be cleared by accident. * Add ICD CIP and DAC key slots for PSA * Implement `PersistICDKey` API * Implement setting key persitence for ICD server * Address PR comments Modify implementation of `PersistICDKey` not to depend on type of ICD key handle (AES, HMAC)
* Update Tizen Docker image to 76 * Add user option to tizen qemu * Use console pool to print QEMU output in real-time --------- Co-authored-by: Jakub Latusek <[email protected]>
Signed-off-by: Axel Le Bourhis <[email protected]>
…pts (#35598) Signed-off-by: marius-alex-tache <[email protected]>
* Update matter_device_types.json Add Water Valve * Chef WaterValve * Remove Groups and Scenes management clusters * Update rootnode_watervalve_6bb39f1f67.matter * Manual correction of device type order due to ZAP tool bug. * Manual correction of device type order due to ZAP tool bug * Remove Groups cluster * Update rootnode_watervalve_6bb39f1f67.zap * Update rootnode_watervalve_6bb39f1f67.matter * Update rootnode_watervalve_6bb39f1f67.zap
…ipped (#35584) * Temporarily add clusters that should never have shipped to availability.yaml as deprecated. BarrierControl, BinaryInputBasic, ElectricalMeasurement, OnOffSwitchConfiguration were never certifiable and should not have shipped. Add them as deprecated with a dummy release that has some future-enough version numbers. * Regenerate generated code with deprecations. * Mark these clusters as removed and move them out of generated code and into manual shims. This should only have moved code, not new code. * Regenerate generated code with the clusters removed. This has no changes to API headers. * Regenerate darwin-framework-tool.
* Add Write Handler in Chef Fan Control AttributeAccessInterface * Restyled by clang-format * Update examples/chef/common/chef-fan-control-manager.cpp Co-authored-by: Andrei Litvin <[email protected]> --------- Co-authored-by: Restyled.io <[email protected]> Co-authored-by: Andrei Litvin <[email protected]>
… (#35602) * Prevent setting up a commissioning session on a suspended controller. Since commissioning session setup does not go through MTRDevice or MTRBaseDevice this was not blocked. * Update src/darwin/Framework/CHIP/MTRDeviceController_Concrete.mm Co-authored-by: Kiel Oleson <[email protected]> --------- Co-authored-by: Kiel Oleson <[email protected]>
* Updated PICS Generator to match 1.4 PICS * Add script to validate PICS and cluster macthing * Updated based on review feedback * Apply suggestions from code review Co-authored-by: C Freeman <[email protected]> Co-authored-by: Andrei Litvin <[email protected]> * Add XMLPICSValidator to wordlist * Restyled by prettier-markdown * Restyled by isort --------- Co-authored-by: C Freeman <[email protected]> Co-authored-by: Andrei Litvin <[email protected]> Co-authored-by: Restyled.io <[email protected]>
* Adding endpoint information to test steps methods and hooks along with logging. Also, the TC_ACE_1_3 and TC_BOOLCFG_2_1 tests were updated to demonstrate how to used the updated hooks to inform the endpoint associated with every step. * Rolling back the test stop unnecessary exception parameter * Updating TestDecoratorHooks with endpoint as well. * Minor grammar fix Co-authored-by: Boris Zbarsky <[email protected]> --------- Co-authored-by: Boris Zbarsky <[email protected]>
* Fix Android resolve exception issue * Restyled by google-java-format --------- Co-authored-by: Restyled.io <[email protected]>
* fan rocksetting; thermostat dry mode * revert change
* Update rootnode_waterleakdetector_0b067acfa3.zap Add Power Source cluster to EP1 to simulate a battery * Update rootnode_waterleakdetector_0b067acfa3.matter Add Power Source cluster to EP1 to Matter file * Update rootnode_waterleakdetector_0b067acfa3.matter * Update rootnode_waterleakdetector_0b067acfa3.zap * Update rootnode_waterleakdetector_0b067acfa3.matter Remove Scenes Management Cluster * Manual correction of device type order due to ZAP tool bug * Update rootnode_waterleakdetector_0b067acfa3.zap * Update rootnode_waterleakdetector_0b067acfa3.zap * Update rootnode_waterleakdetector_0b067acfa3.zap * Update rootnode_waterleakdetector_0b067acfa3.matter * Update rootnode_waterleakdetector_0b067acfa3.zap * Update rootnode_waterleakdetector_0b067acfa3.matter
* [feat] Add littlefs to access flash storage * fix restyle * update ci build job and remove regex in BLConfig_littlefs.cpp * fix restyle * fix compile error * fix restyle * fix compile error after restyle * fix restyle with include order
* Remove event list attribute support * Restyle * Restyled by clang-format * Remove matter event list from defaults and from cmake builds --------- Co-authored-by: Restyled.io <[email protected]>
…t is runned with atexit into its own autorelease pool (#35878)
…OL instead of DISPATCH_QUEUE_SERIAL (#35876)
* getter for air quality * Added Air Quality Sample app for silabs * Restyled by whitespace * Restyled by clang-format * Restyled by prettier-markdown * added build in CI * minor fix * Revert "added build in CI" This reverts commit 5a1fcda96096a172f1e7fb36fdbe8d269fce741d. * Revert "minor fix" This reverts commit 8fa8df289dcbfe6840e425d2330d3ee740577e6b. * added build for app * added app to target * resolved comments * Restyled by whitespace * Restyled by clang-format * using schedulework * using schedulework instead of chiplock * Restyled by clang-format * removing unneccessary loggs, resolved memory leak --------- Co-authored-by: Restyled.io <[email protected]>
…rsion checks (#35863) * Update repoting Engine to be able to use DataModel::Provider for version checks. * Restyle * Fix an include * Fix an include * Fix an include * Restyled by clang-format * Update src/app/reporting/Read-Ember.cpp Co-authored-by: Boris Zbarsky <[email protected]> --------- Co-authored-by: Andrei Litvin <[email protected]> Co-authored-by: Restyled.io <[email protected]> Co-authored-by: Boris Zbarsky <[email protected]>
…heckEventSupportStatus from ember-compatibility) (#35873) * Update event support checks * Fix an include * Restyled by clang-format * Fix typo * Restyled by clang-format * Added a EndpointExists call to the datamodel provider along with a default implementation * Remove missing config use after merge with master * Restyled by clang-format --------- Co-authored-by: Restyled.io <[email protected]>
* Reuse Subprocess in run_python_test.py script * Add --app-ready-pattern option to run_python_test.py * Replace script-start-delay with app-ready-pattern * Drop support for script-start-delay * Use rmtree() instead of explicit "rm -rf" calls * Update missed python.md section * Silence output from tests * Fix removing files * Restyled by prettier-markdown * Fix documentation builder warning * Remove unused import * Fix Metadata unit test --------- Co-authored-by: Restyled.io <[email protected]>
* [Chef] Update Refrigerator to TemperatureNumber * Restyled by clang-format * Restyled by gn * Fix comformance issues checked by python ./src/python_testing/TC_DeviceBasicComposition.py --storage-path admin_storage.json --manual-code 34970112332 --bool-arg ignore_in_progress:True allow_provisional:True --PICS src/app/tests/suites/certification/ci-pics-values python ./src/python_testing/TC_DeviceConformance.py --storage-path admin_storage.json --manual-code 34970112332 --bool-arg ignore_in_progress:True allow_provisional:True --PICS src/app/tests/suites/certification/ci-pics-values * Add taglist to Endpoints to fix conformance issue Caused by test_TC_DESC_2_2: For device types with more than one endpoint listed, ensure each of the listed endpoints has a tag attribute and the tag attributes are not the same * Restyled by clang-format * Use MATTER_DM_PLUGIN_REFRIGERATOR_ALARM_SERVER Use MATTER_DM_PLUGIN_REFRIGERATOR_ALARM_SERVER to enable TagList for refrigerator since MATTER_DM_PLUGIN_REFRIGERATOR_AND_TEMPERATURE_CONTROLLED_CABINET_MODE_SERVER could be used by oven * Use unique_ptr instead of new * Restyled by clang-format --------- Co-authored-by: Restyled.io <[email protected]>
We don't want the LoggerForModule() implementation to call retain/autorelease on the logger, because its unnecessary, and because LoggerForModule() can be called from a C++ context without an auto-release pool in place. To avoid this, we can have LoggerForModule() return a raw pointer, and handle the bridge cast to os_log_t on the caller side. This also avoids unnecessary code on the caller side that would otherwise attempt to rescue the presumed-autoreleased logger object from the ARP.
…ramework. (#35882)
…perature control cluster support (#35857) * Add chef laundry dryer sample app with operation state cluster support * Restyled by prettier-json * Use TL feature instead of TN for tempereature control cluster --------- Co-authored-by: Restyled.io <[email protected]>
…mmandHandler. (#35868) We want do to that before we actually fetch the credential we are trying to get the status of, since determining the next occupied credential index might stomp on data returned by emberAfPluginDoorLockGetCredential.
…p implementation (#35883)
* Replace PIXIT with PICS in log statements * Apply suggestions from code review Fixed step numbers being out and made PICS F00(EnergyManagement) / F01(TankPercentage) to be more in line with the test plan. --------- Co-authored-by: jamesharrow <[email protected]>
…35888) * [NXP][platform][common] Fix wifi onnetwork build issue Signed-off-by: Martin Girardot <[email protected]> * [NXP][platform][common] Add wlan status code Signed-off-by: Martin Girardot <[email protected]> * Restyled by whitespace * Restyled by clang-format --------- Signed-off-by: Martin Girardot <[email protected]> Co-authored-by: Restyled.io <[email protected]>
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.