From 25156926acc95e810e69846a014597009f74d85d Mon Sep 17 00:00:00 2001 From: Raul Marquez <130402456+raul-marquez-csa@users.noreply.github.com> Date: Thu, 15 Feb 2024 21:36:54 -0800 Subject: [PATCH] Modified tests jan 23 new (#32151) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Modifed YAML tests: TC_DISHM_1_2 TC_DISHM_2_1 TC_DISHM_3_1 TC_DISHM_3_3 TC_LWM_1_1 TC_LWM_1_2 TC_OTCCM_1_1 TC_OTCCM_1_2 TC_OTCCM_2_1 TC_OTCCM_3_1 TC_OTCCM_3_3 TC_TCCM_1_1 TC_TCCM_1_2 * Auto generated files * Restyled by whitespace * Fix the CI failure for LWM-1.1 & TCCM-1.1 test * Fix CI failure OTCCM-1.1 test * Revert tests TC-LWM-1.1 TC_TCCM-1.1 TC-OTCCM-1.1 * Fixes issue: 109, 110, 111, 112, 113, 114, 115, 116 TC-CNET-4.1 TC-CONTENTLAUNCHER-10.4 TC-DRYERCTRL-2.1 TC-MEDIAPLAYBACK-6.8 TC-MEDIAPLAYBACK-6.9 TC-MEDIAPLAYBACK-6.10 TC-OTCCM-1.2 * Restyled by whitespace * Modifed YAML tests: TC_DISHM_1_2 TC_DISHM_2_1 TC_DISHM_3_1 TC_DISHM_3_3 TC_LWM_1_1 TC_LWM_1_2 TC_OTCCM_1_1 TC_OTCCM_1_2 TC_OTCCM_2_1 TC_OTCCM_3_1 TC_OTCCM_3_3 TC_TCCM_1_1 TC_TCCM_1_2 * Auto generated files * Restyled by whitespace * Fix the CI failure for LWM-1.1 & TCCM-1.1 test * Fix CI failure OTCCM-1.1 test * Revert tests TC-LWM-1.1 TC_TCCM-1.1 TC-OTCCM-1.1 * Fixes issue: 109, 110, 111, 112, 113, 114, 115, 116 TC-CNET-4.1 TC-CONTENTLAUNCHER-10.4 TC-DRYERCTRL-2.1 TC-MEDIAPLAYBACK-6.8 TC-MEDIAPLAYBACK-6.9 TC-MEDIAPLAYBACK-6.10 TC-OTCCM-1.2 * Restyled by whitespace * Fix conflict * Fix merge * Fix merge * Modifed YAML tests: TC_DISHM_1_2 TC_DISHM_2_1 TC_DISHM_3_1 TC_DISHM_3_3 TC_LWM_1_1 TC_LWM_1_2 TC_OTCCM_1_1 TC_OTCCM_1_2 TC_OTCCM_2_1 TC_OTCCM_3_1 TC_OTCCM_3_3 TC_TCCM_1_1 TC_TCCM_1_2 * Auto generated files * Restyled by whitespace * Fix the CI failure for LWM-1.1 & TCCM-1.1 test * Fix CI failure OTCCM-1.1 test * Revert tests TC-LWM-1.1 TC_TCCM-1.1 TC-OTCCM-1.1 * Fixes issue: 109, 110, 111, 112, 113, 114, 115, 116 TC-CNET-4.1 TC-CONTENTLAUNCHER-10.4 TC-DRYERCTRL-2.1 TC-MEDIAPLAYBACK-6.8 TC-MEDIAPLAYBACK-6.9 TC-MEDIAPLAYBACK-6.10 TC-OTCCM-1.2 * Restyled by whitespace * Fix merge * Fix merge * remove dead java encode_value.zapt (#31896) * Move getting the reboot cause to platform abstraction. Set a getter for the reboot cause so different stacks can obtain it (#31891) * Bump third_party/imgui/repo from `96839b4` to `f50ddc4` (#31917) Bumps [third_party/imgui/repo](https://github.com/ocornut/imgui) from `96839b4` to `f50ddc4`. - [Release notes](https://github.com/ocornut/imgui/releases) - [Commits](https://github.com/ocornut/imgui/compare/96839b445e32e46d87a44fd43a9cdd60c806f7e1...f50ddc431e3b8840036e88abc4c3cf74500aa12b) --- updated-dependencies: - dependency-name: third_party/imgui/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump third_party/nanopb/repo from `f8724ee` to `1f0c2e1` (#31915) Bumps [third_party/nanopb/repo](https://github.com/nanopb/nanopb) from `f8724ee` to `1f0c2e1`. - [Commits](https://github.com/nanopb/nanopb/compare/f8724ee54b3733aebfc25ac0ed5e8b142569c59e...1f0c2e19c661f18dd88428858b8e965a26589e03) --- updated-dependencies: - dependency-name: third_party/nanopb/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump third_party/openthread/repo from `bf587dd` to `cf357d7` (#31912) Bumps [third_party/openthread/repo](https://github.com/openthread/openthread) from `bf587dd` to `cf357d7`. - [Release notes](https://github.com/openthread/openthread/releases) - [Commits](https://github.com/openthread/openthread/compare/bf587dd09005abc49a5595f8691afb9512fac9c1...cf357d70be94e6f3f21d07b3c16e29bb7ec5cc7e) --- updated-dependencies: - dependency-name: third_party/openthread/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump github/issue-labeler from 3.3 to 3.4 (#31918) Bumps [github/issue-labeler](https://github.com/github/issue-labeler) from 3.3 to 3.4. - [Release notes](https://github.com/github/issue-labeler/releases) - [Commits](https://github.com/github/issue-labeler/compare/v3.3...v3.4) --- updated-dependencies: - dependency-name: github/issue-labeler dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump natiginfo/action-detekt-all from 1.23.4 to 1.23.5 (#31919) Bumps [natiginfo/action-detekt-all](https://github.com/natiginfo/action-detekt-all) from 1.23.4 to 1.23.5. - [Release notes](https://github.com/natiginfo/action-detekt-all/releases) - [Commits](https://github.com/natiginfo/action-detekt-all/compare/1.23.4...1.23.5) --- updated-dependencies: - dependency-name: natiginfo/action-detekt-all dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Add "orphan" files from bdx to gn (#31874) * Add "orphan" files from bdx to gn This should help layer enforcement. * Restyle * Increase number of retries on websocket in YAML test harness. (#31893) We seem to be hitting the 4-retry cap a lot. The question is whether waiting an extra few seconds would help or not. Co-authored-by: Andrei Litvin * Add missing function to set delegate in Door Lock Server (#31846) * Add missing function to set delegate in Door Lock Server * Restyled by clang-format (#31847) Co-authored-by: Restyled.io --------- Co-authored-by: restyled-io[bot] <32688539+restyled-io[bot]@users.noreply.github.com> Co-authored-by: Restyled.io * Bump third_party/mbedtls/repo from `1b9cea3` to `6b36a5d` (#31913) Bumps [third_party/mbedtls/repo](https://github.com/ARMmbed/mbedtls) from `1b9cea3` to `6b36a5d`. - [Release notes](https://github.com/ARMmbed/mbedtls/releases) - [Commits](https://github.com/ARMmbed/mbedtls/compare/1b9cea30b1d59a42df7d0f98aed97462985d42d5...6b36a5db07ae0e299c9e67f526c28db93e586e25) --- updated-dependencies: - dependency-name: third_party/mbedtls/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [Scenes] Test Update (#31614) * Updated PICS and server yamls Test_TC_... to fit test plan * Apply suggestions convert hex to dec base Co-authored-by: Junior Martinez <67972863+jmartinez-silabs@users.noreply.github.com> * Updated remaining transition times and added delay for Darwin CI test * Added verification for steps 4b and 4c in TC_S_2_5 * Restyled by whitespace * Restyled by prettier-yaml * Removed Un-necessary PICS, added verification where it was missing in TC_S_2_5 and TC_S_2_6 --------- Co-authored-by: Junior Martinez <67972863+jmartinez-silabs@users.noreply.github.com> Co-authored-by: Restyled.io * Bump third_party/ot-br-posix/repo from `13d583e` to `9326cc3` (#31914) Bumps [third_party/ot-br-posix/repo](https://github.com/openthread/ot-br-posix) from `13d583e` to `9326cc3`. - [Release notes](https://github.com/openthread/ot-br-posix/releases) - [Commits](https://github.com/openthread/ot-br-posix/compare/13d583e361c7038b967b601d5e5f6739b0bcf736...9326cc38b6b6ce81958a96d7821c4cb21976d636) --- updated-dependencies: - dependency-name: third_party/ot-br-posix/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [Tizen] Use spec complaint BLE advertising intervals (#31790) * [Tizen] Use spec complaint BLE advertising intervals * Transition from fast adv to slow after timeout * Refactor Tizen specific BLE initialization function * Use ScheduleLambda instead of ScheduleWork * Wrap C callbacks with lambda to make methods non-static * Define BLE_CONNECTION_OBJECT for Tizen platform * Get rid of another static method * Update name to align with convention * Do not use sInstance directly if possible * Added the --app-pid flag to the python matter testing support (#31866) * Added the --app-pid flag to the python matter testing support to provide a way for test scripts to get the PID of the app they are running against. * fixed typo. Co-authored-by: Petru Lauric <81822411+plauric@users.noreply.github.com> --------- Co-authored-by: Petru Lauric <81822411+plauric@users.noreply.github.com> * TC-ACE-2.1/2: Update default timeout (#31643) * TC-ACE-2.1/2: Update default timeout * Fix default timeout Hey, you know what helps? Committing changes in the correct branch before uploading. * python testing: Add PICS parser for xml (#31493) * python testing: Add PICS parser for xml Code blatently stolen from Jon Rhees. * Restyled by isort * remove some debug --------- Co-authored-by: Restyled.io * [Linux] Do not reuse cancellable object (#31828) * [Linux] Do not reuse cancellable object Per documentation for g_cancellable_reset(): > Note that it is generally not a good idea to reuse an existing > cancellable for more operations after it has been cancelled once, as > this function might tempt you to do. The recommended practice is to > drop the reference to a cancellable after cancelling it, and let it > die with the outstanding async operations. * Update cancellable in ChipDeviceScanner * [ICD] Refactor ICDManager to use System::Clock types (#31881) * Refactor time members * add using namespace * Fix ActiveModeThreshold type and cast * [MatterYamlTests][darwin-framework-tool] Use a list of tests to run f… (#31792) * Properly handle Aliro credential validation in door lock server (#31932) * Handle Aliro credential in door lock server * Restyled by clang-format (#31933) Co-authored-by: Restyled.io * Update src/app/clusters/door-lock-server/door-lock-server.cpp Co-authored-by: Boris Zbarsky * Restyled by clang-format (#31935) Co-authored-by: Restyled.io * Update constant name from git chage * Update to use SupportsAliroProvisioning API --------- Co-authored-by: restyled-io[bot] <32688539+restyled-io[bot]@users.noreply.github.com> Co-authored-by: Restyled.io Co-authored-by: Boris Zbarsky * Implement support for more configurable server endpoints in Matter.framework. (#31814) * Implement support for more configurable server endpoints in Matter.framework. * Public APIs on MTRDeviceController to add/remove an endpoint. * Internal APIs on MTRDeviceController to query the access grants for a cluster path and the declared "minimum privilege needed" to read a given attribute. * Changes to the controller factory to stop using app/dynamic_server and instead use the new infrastructure to expose OTA Provider on endpoint 0. * Internal APIs on the controller factory to query access grants and declared privileges. * An implemenation of AccessControl::Delegate to do ACL checks using the above-mentioned APIs. * A fix to MTRServerAttribute's setValue for arrays: it was not expecting the correct data-value structure for an array. This requires fixing some tests too, to provide the right structures. * Changes to the MTRServer* data structures to allow passing nil to associateWithController, to support the OTA endpoint which is not associated with any controller. * Changes to MTRServerCluster to create an AttributeAccessInterface, the set of EmberAfAttributeMetadata needed to represent its attributes, and various other things needed to register a cluster with the "ember" bits. * Changes to MTRServerEndpoint to create an EmberAfEndpointType, a list of EmberAfCluster, and various other things needed to register an endpoint with the "ember" bits. * (Re-)addition of MTRIMDispatch to handle command dispatch for OTA and host a few other functions the "ember" bits expect to exist. * Addition of some headers that the "ember" bits expect to exist at specific paths and with some specific content: "app/PluginApplicationCallbacks.h" and "zap-generated/endpoint_config.h". Importantly, the latter sets FIXED_ENDPOINT_COUNT to 0. * Addition of unit tests that exercise the non-OTA bits of the above (OTA is covered by existing tests), including the ACL checks and so on. * Including a bunch of src/app and src/app/util files needed for the "ember" stuff to work in the framework. * Turning off the chip_build_controller_dynamic_server bit that we are no longer using (and which conflicts with the above bits). * Configure Darwin to support 254 dynamic endpoints (the maximum that makes sense) by default. * Adjusting include paths for the Xcode darwin-framework-tool project, so that it sees the new headers that were added. * Address review comments. * Fix test timeout due to resolving IPv4 non-locahost addresses. * Remove stale comment. * EFR32 and SiWx917 can now use the same ConfigurationManagerImpl.cpp implementation (#31939) * Remove vestige of the initial ICD event base management (#31940) * [darwin-framework-tool] Remove codegen tests (#31942) * Add strong guarantee on monotonicity of system clock (#31885) * Add strong guarantee on monotonicity of system clock - It was observed some platforms have decrementing values from `SystemClock::GetMonotonicMilliseconds64()` implementation which can violate the required invariant that they NEVER go back, since multiple deadlines are computed from this clock, which could then never hit. This was observed to break in the field in subtle ways after ~48 days (2^32 milliseconds). - This change introduces a VerifyOrDie to allow crash/restart if the invariant is broken, since that leaves the stack in a possibly wedged state. - The new invariant is constructed to be reentrant-safe to maintain the invariant, in a way that it will eventually be caught, even if there are many threads/cores calling `GetMonotonicTimestamp()`. This was done in a lock-free manner, and was added since the public API of SystemLayer is not guaranteed to only be called from Matter thread context. Issue #31875 Testing done: - All unit tests still pass. - All integration tests still pass. - When `mockClock.AdvanceMonotonic(200_ms);` is replaced with `mockClock.AdvanceMonotonic(0xffff'ffff'ffff'ffff_ms);` in `TestSystemTimer.cpp::CheckOrder()`, the test crashes as expected due to detection of wraparound. * Fix review comments and move to intrinsics * Restyled by clang-format * Fix build and add warning * Fix build * Restyled by clang-format * Address review comment, add new define instead of linux && darwin * Restyled by clang-format --------- Co-authored-by: tennessee.carmelveilleux@gmail.com Co-authored-by: Restyled.io * Initial XML for Power Topology cluster (#31689) * Initial XML for Power Topology cluster * Regen * Restyled really wants newlines at the end of every JSON file * Rename LeafTopology to TreeTopology * [Linux] Add support for extended advertisement (#31668) * Use ScheduleLambda() instead of ScheduleWork * Set min/max advertisement interval * Matter BLE advertisement type shall be peripheral * Allow intervals to be set after initialization * Keep BlueZ default for adv slicing duration * Allow to update intervals on the fly - fast/slow * Transition to slow advertising after timeout * Fix advertisement typos in all places in the codebase * Do not specify advertisement timeout in BlueZ * Add support for extended advertisement * Post review changes - add manual how to enable experimental features in BlueZ * Restyled by prettier-markdown * Assert that extended adv interval is >= fast adv interval * Cancel HandleAdvertisingTimer timer in case of error --------- Co-authored-by: Restyled.io * ESP32: Add a menuconfig option to enable/disable persist subscription (#30889) * ESP32: Add a menuconfig option to enable/disable persist subscription * enable persistent subscription by default * Regenerate ZAP after merge conflict (#31956) * Make the help text for write-by-id say the right thing about the value list. (#31884) https://github.com/project-chip/connectedhomeip/pull/20413 changed the value separator from comma to semicolon, but did not change the help text. Fixes https://github.com/project-chip/connectedhomeip/issues/31854 * Improve the logic around schedule application in LockEndpoint. (#31946) The old logic had several bugs: * For kYearDayScheduleUser users, there was no schedule enforcement at all, since the outer "if" never tested true. * For kWeekDayScheduleUser users, there was no schedule enforcement at all, since the inner "if" never tested true. * For kScheduleRestrictedUser users, access was allowed if there was any schedule, weekday or yearday, that granted access. But the intent is that access should be disallowed if there are year day schedules and none of them grants access, no matter what's going on with weekday. And vice versa. * Make it clearer that cluster-enums-check is returning kUnknownEnumValue. (#31951) Instead of hardcoding the returns via the ZAP template, just use the symbolic name. * Spell out inet and lwip sources in GN (#31931) * Spell out inet and lwip sources Prepare to have some scripts that force all headers/sources to be known to GN. * Fix some copy and paste issues * Another copy and paste issue fixed * Restyle * Board-specific LWIP options for silabs boards (rs911, wf200, thread) were never used (#31929) * Add silabs-specific lwip options to GN * Make the lwipopts conditional * Revert "Make the lwipopts conditional" This reverts commit ff91e81193fad3b2b4941048122a1b9353cb6b3f. * Add another dependency for lwip * Remove TAP support from lwip (#31886) * Update SupportsAnyCredential function to include Aliro (#31949) * Update SupportsAnyCredential function to include Aliro * Restyled by clang-format (#31950) Co-authored-by: Restyled.io --------- Co-authored-by: restyled-io[bot] <32688539+restyled-io[bot]@users.noreply.github.com> Co-authored-by: Restyled.io * Implement PayloadTestRequest command in DGGEN (#31907) * Add DataModelTest feature to general diagnostics * Regen ZAP * Fix TC-DGGEN-1.1 * Restyled by whitespace * Restyled by clang-format * Restyled by prettier-json * Restyled by isort * Fix integration tests * Fix Java codegen * Address review comments * Fix Java codegen * Fix Java codegen * Fix Java codegen golden data * Fix TC-DGGEN-1.1 * Fix removal of Commands.h * Fix YAML test --------- Co-authored-by: tennessee.carmelveilleux@gmail.com Co-authored-by: Restyled.io * Add orphan header files from `src/controller` to gn (#31882) * Add files from src/controller to gn * Add more orphaned files from controller itself * Restyle * Move more files into sources since CHIPCluster actually includes them. Seems VERY odd to have this header used but not actually have any implementation. This is a bug! * Attempt to fix some of the include oddities. Controller is VERY broken and feels like a god object definition * Move more headers ... CHIPDeviceController strongly pulls the entire world. * Fix typo * One more header * Fix chef * Add more includes that seem needed * Add another dependency * Add includes config to controller as it needs relative includes * Restyle * Fix clang-tidy logic * Apply some clang-tidy fixes since I looked at controller. Minor nullptr usage and remove else after return * Ensure no errors about unused assignments * Yet another fix for logic to make sure all defined lists are used * Include only headers as the special targets to force include errors * Fix typo * Fix typo --------- Co-authored-by: Andrei Litvin * Add discover port parameter (#31948) * Adds MdnsDiscovery class (#31645) * Adds MdnsDiscovery class * Fix restyle/lint * Fix restyle * Adds zeroconf dependency in tests.yaml * Relocates zeroconf dependency from tests.yaml to requirements.txt * Addresses latest review comments * Fixes typo * Updates instantiation method and initial discovery * Fix restyle/lint * Addresses latest review comments * Addresses latest review comments * Fix restlye/lint * Addresses review comments * restore enum * Refactor progress * Major refactor to discover() and get_operational_service_info, pending other get methods * Fix restyle * Fix restyle/lint * Updates descriptions and variables * Major refactor #2 * Fix restyle * Updates method descriptions * Fix restyle * Addresses review comments * Fix restyle * Fix lint * Default parameters adjustment --------- Co-authored-by: C Freeman * Allow chip-repl to know how many InvokeResponseMessages were recieved (#31781) * Allow chip-repl to know how many InvokeResponseMessages were recieved * Restyled by autopep8 * Clean up callback to be test only specifically * Restyled by clang-format * Address PR comments * Small edit to add line that shouldn't be removed in this PR * Address PR comment --------- Co-authored-by: Restyled.io * [ICD] Refactor ICDCounter logic (#31957) * Refactor ICD Check-In counter Fix initial Check-In counter value * Add basic integration tests to validate behavior * Restyled by whitespace * Restyled by prettier-yaml * rename source_set * Rename constant name * Apply suggestions from code review Co-authored-by: Boris Zbarsky * addres review comments * Add init/shutdown unit-test for the ICDManager * Fix Build.gn - multiple inclusion of CheckInMessage.cpp --------- Co-authored-by: Restyled.io Co-authored-by: Boris Zbarsky * Add assertChipStackLockedByCurrentThread for LogEvent (#31895) * Updated test comments to display the proper keys values (#31968) * Move messagedef to its own buildfile (#31972) Co-authored-by: Andrei Litvin * Upload .ips bits in Xcode derived data on failure in Darwin tests as an artifact. (#31964) If there are crashes in the test, the crash logs should be in there, and we can try to debug those. * [Telink] Add B95 (TLSR9258A) skeleton & Update builds to docker version 35 (#31762) * add the board of tlsr9253b and tlsr9253b_retention * riscv: telink: update board setting - update the tlsr9253b setting . Signed-off-by: Haiwen Xia * telink: b93: update the build sample and b93 platform - update the light-switch and lighting-app . - set the tlsr9253b and tlsr9253b_retention for the ci environment . Signed-off-by: Haiwen Xia * telink: b93: fix the build environment - set the tlsr9253b and tlsr9253b_retention for the ci environment . Signed-off-by: Haiwen Xia * telink: b93: loopback to remote version - change the github setting to telink . Signed-off-by: Haiwen Xia * [Telink] Add B93 to Readme files * [Telink] Fix CI builds * [Telink] Reduce number of builds * [Telink] Disable update Zephyr to specific revision * Update builds to docker version 34 * [Telink] remove duplicated code * revert k32w image update * [Telink] rename B93 into B95 * Update builds to docker version 35 * fix doc build --------- Signed-off-by: Haiwen Xia Co-authored-by: haiwen.xia * Enforce that files in the `src` directory are referenced from BUILD.gn (#31960) * Start creating a script * Have much more functionality * Restyle * Add some doc comments ... this starts being usable * Add workflow to validate that gn knows about files * Remove controller from known exceptions: we fixed that one * Fix flake8 * Add more known failures * Better error reporting for gn reachability * Remove the platform specific orphan file listing * Force the "not failures anymore" to be fatal --------- Co-authored-by: Andrei Litvin * Fix linter error: messagedef files are now covered by gn (#31995) * Rvc op state delegate dummy start stop handlers (#31764) * Added dummy implementations of the start and stop handler methods to the RvcOperationalState::Delegate class. * Removed the start and stop handler methods defined in the all-clusters-app RvcOperationalStateDelegate. * Removed the start and stop handler methods defined in the rvc-app RvcOperationalStateDelegate. * Restyled by clang-format * Apply typo fixes from code review Co-authored-by: Petru Lauric <81822411+plauric@users.noreply.github.com> --------- Co-authored-by: Restyled.io Co-authored-by: Petru Lauric <81822411+plauric@users.noreply.github.com> * Rename EMBER_AF to MATTER_DM (#31971) * Mark ElectricalPowerMeasurement and PowerTopology provisional on Darwin. (#31959) * ubuntu:35 is not a valid runner (#32007) * Rearranged python test scripts order into an alphabetical format (#31930) * Rearranged order of the pytest scripts alphbetically accordingly to the arguments * Fixed 2 scripts alphabetically were not in line * This changes reflect the sorting of the testing python scripts via machine runned .py script. --------- Co-authored-by: Andrei Litvin * Fix global reference leak across the android in matter (#31890) * [ICD] Trigger Check-In messages at boot (#31993) * Send Check-In message at boot * Update reboot test * update comment * restyle * refactor function so that it can be used outside of the reboot * Remove unused define function --------- Co-authored-by: Andrei Litvin * TV Matter Media: Resolve cluster revisions (#31863) * Resolve the cluster revision issues * Restyled by whitespace * Restyled by clang-format * Update per comments * Restyled by whitespace * Restyled by clang-format * Update Content Launcher * Restyled by whitespace * Restyled by clang-format --------- Co-authored-by: Restyled.io * Linux: Implement PDC support in WiFiDriver / ConnectivityManagerImpl (#31983) * Use DecodeChipCert instead of LoadCert in chip-cert print-cert LoadCert performs additional checks (only relevant to operational certs) that prevent PDC identities from being printed. * Simplify CHIPCert test vector code * Add ConvertECDSAKeypairRawToDER to CHIPCert * Linux: Make WiFiNetwork struct private, add helpers - Make WiFiNetwork struct private - Move static_asserts to where the fields are defined - Add Empty() and Match() helpers to WiFiNetwork - Validate index parameter in ReorderNetwork() - Use CHIP_ERROR_FORMAT * Linux: Implement PDC support in WiFiDriver / ConnectivityManagerImpl * Address review comments * Clear handled err before the next call * [crypto] Extend the OperationalKeystore API to allow migration (#31279) - Extended the OperationalKeystore API by mechanism for migration of operational keys stored in one Operational Keystore implementation to another. - Extended the OperationalKeystore API by exporting keypair for Fabric. - Added Unit tests to PersistentStorageOpKeyStore and PSAOpKeystore regarding the new OperationalKeystore for migration and exporting OpKeys. Added first unit tests, created export method * Remove `CHIPClusters.h` and corresponding `.zapt` file from the build (#32004) * Completely remove all references to CHIPClusters.h * Restyle * Remove references to CHIPClusters.zapt as well * Use BaseCluster instead of specific clusters * Fix names and includes * add java and python files from `src/controller` to gn (#31977) * Add 2 java files to be known to gn * Add python files * Fix commas and format * Mark a set of files handled --------- Co-authored-by: Andrei Litvin * Add files from `src/lib/dnssd` to be tracked by BUILD.gn files (#31976) * Make sure dnssd constants and browsedelegate is tracked by gn * Add more minmdns sources to be known to gn * Move more things into Types to make sure that platform Dnssd knows about them * Fix system layer dependency in linux dnssd impl * Remove dnssd files from lint * Add Delegate to Types as well ... it seems required for platform * Remove duplicate source * Restyle --------- Co-authored-by: Andrei Litvin * Log when MTRServerAttribute not bound to a controller has its value updated. (#31973) This should enable API consumers to more easily figure out why their attribute updates are not being published. * Quick cleanup of unused type (#32009) * TV Matter Media: Add logic to message cluster (#31943) * Adding basic logic * Updates * Update message cluster * Update copyright * Restyled by whitespace * Restyled by clang-format --------- Co-authored-by: Restyled.io * Refactor CommandSender newly added APIs to be more extendable (#31837) * Update rvc app (#31827) * Replaced the vendor specific mode tag for the mapping mode with the mapping mode tag. * Updated the rvc-app's RvcOperationalStateDelegate to allow setting of a callback furtion for when the GoHome commond in received. * Implemented the GoHome handler for rvc-app. * Updated the PICS file for the rvc-app. * Fixed the Admin commissioning cluter's zap config for the rvc-app * Renamed the PICS values file to be similar to the ci file. Updated the script that runs all the RVC yaml tests. * Updated the rvc-app state machine diagram. * Updated theh rvc-app README. * Restyled by clang-format * Restyled by prettier-markdown * Restyled by shfmt * Updated the rvc-app .matter file. * Reduced the text of some errors as it was longer that allowed. * Added an out-of-band message to reset the rvc-app to its start-up state. * Updated a returend error following the spec change. * Excluded the RVC state diagram png from the lint check for %zu. * Updated the rvc-app readme. * Removed PIXIT settings from the rvc-app-pics values. * Excluded all binary files from the lint check for %zu. * Re-removed the OpenBasicCommissioningWindow command of the Administrator Commissioning cluster in the rvc-app due to security concerns. * Updated the rvc-app .matter file. --------- Co-authored-by: Restyled.io * TC-ACE-2.1/2: Add commissioning step (#31715) This is apparently what the TH is using to determine if a test requires commissioning, so we best add it. * Tweaks to evse test plans based on review(issue #31460) (#31901) * Added support for test event triggers and handling of reading events into matter_testing_support. * Made TC_EEVSE_Utils.py use the matter_testing_support instead of its own local copy. * Restyled by isort * Added TC_EEVSE_2_2, 2_4, 2_5 to tests.yaml. Fixed compile warning treated as error due to sign conversion. * Updated default min_charge and max_charge in TC_EEVSE_Utils send_enable_charge_command to have sensible default values if not specified. * Fixed test app name * Moved test runs later in test yaml * Fixed discriminator used in script to match that used in the app. * Added --endpoint 1 so it tests the correct endpoint * Code review comment fixes. * Fixed trailing whitepace * Merged TC_EEVSE tests back in --------- Co-authored-by: Restyled.io * Cleanup a few more files that were not recognized by gn (#31999) * Cleanup and fix some tests dependencies * Fix test compilation. Header was forcing C-linkage of the name * restyle * Fix code lint * Fix one more * Fix dependencies for more common integ files * Remove assert because this file is exercised in dependencies * Add Mock events handling * Remove TestOperationalDeviceProxy - this proxy does not seem to exist anymore and this test is never run * One more file update * Update linter * Restyle --------- Co-authored-by: Andrei Litvin * [Darwin] MTRDevice cache make use of controller storage for persistent cache (#31952) * [Darwin] MTRDevice cache make use of controller storage for persistent cache --------- Co-authored-by: Boris Zbarsky * Make MTRServerAttribute threadsafe. (#31970) * Make MTRServerAttribute threadsafe. If two API clients are both touching the same instance of MTRServerAttribute on different threads, we should handle that correctly. * Address review comments. * [Python] CommissonWithCode support discoveryType (#31904) * [Linux] fix memory leak * [Python] call StopDiscovery after DiscoveryNodes * [Python] CommissionWithCode support DiscoveryType * fix param error * add e2e test * automatically run in CI * Test different modes using different devices * fix error manual code * Microwave oven: Fix conformance (#32029) * [IM] Remove the EMBER_ZCL_STATUS_DUPLICATE_EXISTS error code (#32017) * Remove the EMBER_ZCL_STATUS_DUPLICATE_EXISTS error code * fix comments * [crypto] Add HKDF key handle and use it during PASE (#31311) * [crypto] Add HKDF key handle and use it during PASE Current SPAKE2+ interface assumes that raw shared secret is extracted and used by the application to derive session keys. This prevents using secure crypto APIs, such as PSA, to perform SPAKE2+ and do the key derivation in a secure environment, and isolate the application from key material. 1. Add Hkdf128KeyHandle type and add methods for deriving session keys from an HKDF key. 2. Change SPAKE2+ interface to return HKDF key handle instead of raw key secret. A similar approach can be taken to improve CASE security in the future though we would need 256-bit HKDF key support in such a case. * Change HKDF key handle to hold key of any length * Code review * Enable rvc python tests to run in ci (#31872) * Replaced the vendor specific mode tag for the mapping mode with the mapping mode tag. * Updated the rvc-app's RvcOperationalStateDelegate to allow setting of a callback furtion for when the GoHome commond in received. * Implemented the GoHome handler for rvc-app. * Updated the PICS file for the rvc-app. * Fixed the Admin commissioning cluter's zap config for the rvc-app * Renamed the PICS values file to be similar to the ci file. Updated the script that runs all the RVC yaml tests. * Updated the rvc-app state machine diagram. * Updated theh rvc-app README. * Restyled by clang-format * Restyled by prettier-markdown * Restyled by shfmt * Updated the rvc-app .matter file. * Reduced the text of some errors as it was longer that allowed. * Added an out-of-band message to reset the rvc-app to its start-up state. * Updated a returend error following the spec change. * Excluded the RVC state diagram png from the lint check for %zu. * Added the PICS_SDK_CI_ONLY PICS to the rvc-app-pics-values to allow it to run tests in CI. * Improved the TC_RVCCLEANM_2_1 test to allow the CI to run the tests against the rvc-app. * Improved the TC_RVCRUNM_2_1 test to allow the CI to run the tests against the rvc-app. * Improved the TC_RVCCLEAN_2_2 test to allow the CI to run the tests against the rvc-app. * Improved the TC_RVCRUNN_2_2 test to allow the CI to run the tests against the rvc-app. * Improved the TC_RVCOPSTATE_2_1 test to allow the CI to run the tests against the rvc-app. * Improved the TC_RVCOPSTATE_2_3 test to allow the CI to run the tests against the rvc-app. * Restyled by autopep8 * Restyled by isort * Updated the rvc-app readme. * Removed PIXIT settings from the rvc-app-pics values. * Excluded all binary files from the lint check for %zu. * Fixed typos from code review Co-authored-by: Petru Lauric <81822411+plauric@users.noreply.github.com> * Re-removed the OpenBasicCommissioningWindow command of the Administrator Commissioning cluster in the rvc-app due to security concerns. * Updated the rvc-app .matter file. * Removed sleeps between messages to the DUT as they are not needed. * Restyled by isort * Fixed extra imports in python tests. --------- Co-authored-by: Restyled.io Co-authored-by: Petru Lauric <81822411+plauric@users.noreply.github.com> * Delete ContentLaunch::SupportedProtocolsBitmap::kWebRTC (#32042) Co-authored-by: Andrei Litvin * TC-SM-1.2: Use correct data type (#31644) * TC-SM-1.2: Use correct data type Credit to Manjunath for finding this. Fixes: https://github.com/project-chip/matter-test-scripts/issues/105 * one more fix * one more * Pull this out and add a test * linter * add docstring * Stop including af-enums.h in Darwin unit tests. (#32044) These are Objective-C files, and should not be including potentially-C++ headers. A bunch of "test that the code is 0" checks were replaced with "error is nil" checks (and some were removed, if "error is nil" was already being tested). The macro definition reformatting in MTRBackwardsCompatTests.m is just from updates to our clang-format version, looks like. * Add some more progress-style logging to Darwin server endpoints. (#32047) Lets us track when endpoints/clusters/attributes are set up and attribute values changed. * Remove include InteractionModelEngine.h in reporting/Engine.h (#32027) * [ICD] Add OperatingMode attribute to the LIT-ICD-app (#31997) * [Interaction Model]Sync AmberAfStatus to InteractionModel::Status (#32016) * Sync AmberAfStatus to InteractionModel::Status * fix comments * fix darwin build * add TODO for enum class * fix Darwin * tracing: Added a macro to report the metric values from the sdk to the tracing framework. (#32032) * Added metric support * Addressed the review comments * [CI] When the CI is starting the server (chip-tool or darwin-framework-tool) wait to see for the websocket message ready before trying to connect (#32006) * [darwin-framework-tool] Do not use platform::LogV since this is a no-op now * [CI] When the CI is starting the server (chip-tool or darwin-framework-tool) wait to see for the websocket message ready before trying to connect * [TC-MWOxxx] Add MWO tests (#30856) This PR adds test for the MWO clusters. - Test for Microwave Oven Mode cluster - Test for Microwave Oven Control cluster Note: tests reflect spec [PR 8588](https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/8588) and [PR 8715](https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/8715) This PR also: - Adds a CI build target for the microwave oven example app - Adds the python tests to CI - Uses the example app for tests instead of all-clusters - Fixes an SDK build problem for the microwave control cluster * Bump third_party/ot-br-posix/repo from `9326cc3` to `c5a7a35` (#32069) Bumps [third_party/ot-br-posix/repo](https://github.com/openthread/ot-br-posix) from `9326cc3` to `c5a7a35`. - [Release notes](https://github.com/openthread/ot-br-posix/releases) - [Commits](https://github.com/openthread/ot-br-posix/compare/9326cc38b6b6ce81958a96d7821c4cb21976d636...c5a7a35e3bd2f3da8cce1e2e2a3bbe5cdeedb729) --- updated-dependencies: - dependency-name: third_party/ot-br-posix/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump Wandalen/wretry.action from 1.3.0 to 1.4.4 (#32063) Bumps [Wandalen/wretry.action](https://github.com/wandalen/wretry.action) from 1.3.0 to 1.4.4. - [Release notes](https://github.com/wandalen/wretry.action/releases) - [Commits](https://github.com/wandalen/wretry.action/compare/v1.3.0...v1.4.4) --- updated-dependencies: - dependency-name: Wandalen/wretry.action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [placeholder] Remove tests generated with zap (#32038) * [placeholder] Remove examples/placeholder tests codegen * Remove src/app/tests/suites/pics * Remove src/app/tests/suites/include * Remove src/app/tests/suites/templates * Remove src/app/tests/suites/commands/[delay|discovery|log|system] * Remove examples/darwin-framework-tool/commands/tests/TestCommandBridge.h * [TI] CC2674 migration OTA support (#32026) * ota support for cc2674 * Restyled by clang-format * Restyled by prettier-markdown --------- Co-authored-by: Restyled.io * Update src/app/util/ emberafstatus (#32054) * 31599 DEMM & EVSEM FeatureMap list has to be updated as per test plan (should be 0) (#32057) * Fixes: #31599 - Changed to remove kOnOff support in EEVSEM and DEMM clusters. * Added a kNoFeatures enum to eevse and demm xml, and then regen_all * Revert "Fixes: #31599 - Changed to remove kOnOff support in EEVSEM and DEMM clusters." This reverts commit b79eec63fb35508772f852e5e402147de17df5fb. * Fix threadsafety issue in MTRServerAttribute. (#32084) An attempt to get the description could race with updates of _parentCluster. * Add Electrical power measurement to energy management app (#31616) * Hoist shared enums and bitmaps into detail namespace * Regenerate * Add electrical measurement clusters * Add NumberOfMeasurements attribute * Bump to latest spec * Bump ZAP version * Remove Electrical Measurement cluster * Add initial Electrical Power Measurement cluster implementation * Revert "Remove Electrical Measurement cluster" This reverts commit 47f5298ad6d1ed5f18b3ae3768960adbf604965c. * Fix incorrect min/max values on energy attributes * Formatting electrical-power-measurement-server * Regen after restoring deprecated electrical measurement cluster * Re-add inexplicably important blank line to zap_execution.py * De-alphabetize list of files to avoid breaking GH action * Semi-realphabetize? * Added EPM cluster to Energy Management App * Restore strangely dropped events * Better BitMask handling * Change min/max on electrical measurements to be decimal instead of hex * Rename meas-and-sense to measurement-and-sensing.xml * Remove seemingly superfluous attribute requirements on Descriptor cluster on Electrical Measurement * Updates to electrical-power-measurement-server based on comments * Take all-clusters-zap from upstream-master. * Added side=server into electrical-power and electrical-energy clusters XML so events can be enabled in ZAP * Updated all-clusters.zap and reran autogen. Added EPM cluster to energy-management-app.zap * Added initial version of EPM delegate and instance to energy-management-app * Enabled Electrical Energy Measurement in example-energy-management-app ZAP and regen all. * Added ElectricalEnergyMeasurement to chip-repl __init__.py * Restyled by isort * Changed feature map to indicate CUME and IMP support only * Adding into __all__ some missing clusters which are causing flake8 issues. * Removed out of date comment * Added a test event trigger to fake energy reporting readings into the EEM cluster (hooks ready for EPM cluster) * Added new Energy Reporting TE Trigger delegate * Updated random calculation and scaled into mWh (divide by 3600) * Restyled by whitespace * Restyled by gn * Removed accidental launch.json inclusion * Fix for ARM linux etc builds due to gn dependency check error * Added code review suggestions. * More code review suggestions fixed. * Hoist shared enums and bitmaps into detail namespace * Regenerate * Add electrical measurement clusters * Add NumberOfMeasurements attribute * Bump to latest spec * Bump ZAP version * Remove Electrical Measurement cluster * Add initial Electrical Power Measurement cluster implementation * Revert "Remove Electrical Measurement cluster" This reverts commit 47f5298ad6d1ed5f18b3ae3768960adbf604965c. * Fix incorrect min/max values on energy attributes * Formatting electrical-power-measurement-server * Regen after restoring deprecated electrical measurement cluster * Re-add inexplicably important blank line to zap_execution.py * De-alphabetize list of files to avoid breaking GH action * Semi-realphabetize? * Restore strangely dropped events * Better BitMask handling * Change min/max on electrical measurements to be decimal instead of hex * Rename meas-and-sense to measurement-and-sensing.xml * Remove seemingly superfluous attribute requirements on Descriptor cluster on Electrical Measurement * Updates to electrical-power-measurement-server based on comments * Remove defaults from MeasurementAccuracyRangeStruct to match spec update * Restore side="server" to events * Move common enums and bitmaps to detail:: instead of detail::Enums and detail::Bitmaps; remove superfluous using statement * Assign ID to Electrical Sensor device type * Removed EPM and EEM from Root Node Device * Restyled formatting is different than clang-format * Re-add FeatureMap to attributeAccessInterfaceAttributes for EEM and EPM * Regen after merge * Added electrical-energy-measurement-server to CMakelist to fix linker issue. * Lock client on Electrical Sensor device type * Remove unneeded using statement now that Enums are in detail:: * Check for null iterators and error * Switch to ResourceExhausted from CHIP_ERROR_INTERNAL * Re-enabled EEM in energy management app and regen all after previous merge * Some refactoring to add EPM Instance into the EVSEManufacturer class to clean up containment. Added ability to fake voltage, power and current to the TE triggers. * Missed one file. * Fixed crash due to unassigned dg pointer. Power/Voltage/Current faking working too. * Touch file since restyled crashed * Restyled by gn * Restyled by isort * Add stub for EPM cluster * Reverted whitespace change * Did regen_all after merge from master to resolve conflicts. * Put back line of clusters which somehow got deleted accidentally. * Remerged ZAP file and regen all after resolving conflicts. * Fixes for Python tests * Correct name for Ember init callback * Formatting * Sync optional attributes list with .zap file for EPM * Add missing features to EPM stub * Revert FeatureMap in attributeAccessInterfaceAttributes * Allow FeatureMap in EEM constructor; add all-clusters-app EEM stub * Forgot zcl-with-test-extensions * Unregister EEM attribute access in destructor * Remove redundant returns to keep clang-tidy happy * Fix for issue mentioned in code review on EEM cluster limiting the number of endpoints it allows. * Refactoring to have a common EvseMain across all platforms to avoid making changes in multiple places * Added electrical-power-measurement-server to ESP32 CMakeLists.txt * Updated Matter device types to add EVSE * Open and saved energy-management-app.zap and regen_all * Removed duplicate ElectricalEnergyMeasurment class which was accidentally merged in. Fixed issue raised about ElectricalEnergyMeasurement array size not working on bridges. * Added support for test event triggers and handling of reading events into matter_testing_support. * Made TC_EEVSE_Utils.py use the matter_testing_support instead of its own local copy. * Restyled by isort * Cherry pick from Tweaks to EVSE Test plans (Issue #31460) * Changed the random value generation to make the values +/- and handle sign conversion to avoid compile warnings * Enabled cumulativeEnergyExported in Energy-management-app. * Added initial electrical power measurement 2.2 test case * Changed copyright date * Code review comment fixes. * Changed to c++ style cast * Fixed trailing whitepace * Added support for testing read of EEM attributes and change of values * Corrected EPM references in TC_EEM_2_2. Added TC_EEM_2_3 * Added periodic energy reporting, and new cumulativeEnergyReset attribute into energy-management-app.zap * Added periodic energy reading support and TC_EEM_2_3 to 2_5. * Python removed unused logging and EventChangeCallback * Updates to align to test plan PR #3949 * Added initial EEM_2_1 test script. * Added example of setting EEM Accuracy and EEM CumulativeEnergyReset structure - TC_EEM_2_1 now passes * Restyled by whitespace * Restyled * Removed extra spaces in TC_EEM_2_1.py * Removed unused EnergyManagementManager.cpp/.h * Fixed PowerMode = kAc * Initial TC_EPM_2_1.py script * Restyled by isort * Merged TC_EEVSE tests back in * Initialized NumberOfMeasurementTypes * Added EEM 2.1,2.2,2.3,2.4,2.5 and EPM 2.1,2.2 into CI workflow tests.yaml * Interim state - partially refactored how Measurement Structs are encoded similar to how ModeBase clusters are implemented. Needs tidy up. Will break all-clusters for now * Removed SetNumberOfMeasurementTypes since this can be derived from the ArraySize(kMeasurementAccuracies). Added more stringent checking in test script of measurementTypes and ranges. * Completed TC_EPM_2_1.py script * Corrected test plan spec reference. * Test EPM_2_1 now runs and passes. Allows checking that attributes are supported, and skips test if not. Validation of values ignores Nulls (which are allowed). Turned on Ranges attribute. * Revert unintended change to tests.yaml * Python test case code-review updates * Removed old range iterator. * Fixed lint issues and adjusted timings to match the test plan pr. * Fixed all-clusters electrical-power-measurement cluster by using the energy-management-app/common Delegate * Implemented HarmonicCurrents and HarmonicStructs (to return empty list for now) * Changed the API to ensure that a delegate doesn't change the data mid-way through a read, so the cluster server signals with a StartxxxRead and EndxxxRead call * Applying suggested changes * Incorporated feedback from review to simplify the code * Removed comment per code review --------- Co-authored-by: Hasty Granbery Co-authored-by: Restyled.io * Darwin: Simplify test setup and teardown (#32061) * Darwin: Simplify test setup and teardown Avoid relying on dummy tests that are ordered first / last. * fix unused variables * Bump third_party/openthread/repo from `cf357d7` to `b212a0a` (#32067) Bumps [third_party/openthread/repo](https://github.com/openthread/openthread) from `cf357d7` to `b212a0a`. - [Release notes](https://github.com/openthread/openthread/releases) - [Commits](https://github.com/openthread/openthread/compare/cf357d70be94e6f3f21d07b3c16e29bb7ec5cc7e...b212a0a748070ccbda765c3ebed2aab8b6b08fce) --- updated-dependencies: - dependency-name: third_party/openthread/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump third_party/mbedtls/repo from `6b36a5d` to `ede909f` (#32066) Bumps [third_party/mbedtls/repo](https://github.com/ARMmbed/mbedtls) from `6b36a5d` to `ede909f`. - [Release notes](https://github.com/ARMmbed/mbedtls/releases) - [Commits](https://github.com/ARMmbed/mbedtls/compare/6b36a5db07ae0e299c9e67f526c28db93e586e25...ede909f99ab6e6a958a41e365251c2a1d2c4ed4d) --- updated-dependencies: - dependency-name: third_party/mbedtls/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump third_party/imgui/repo from `f50ddc4` to `5360903` (#32068) Bumps [third_party/imgui/repo](https://github.com/ocornut/imgui) from `f50ddc4` to `5360903`. - [Release notes](https://github.com/ocornut/imgui/releases) - [Commits](https://github.com/ocornut/imgui/compare/f50ddc431e3b8840036e88abc4c3cf74500aa12b...536090303a8fca7d896f77d6d63dc59249bc87f4) --- updated-dependencies: - dependency-name: third_party/imgui/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Update chef.py (#32060) * DNS-SD: Remove calls to set T flag (#32083) * DNS-SD: Remove calls to set T flag Nothing currently supports TCP, therefore nothing should be setting this flag. Per the new spec text, it is now forbidden to have the lowest bit marked. This will bring the SDK examples into compliance with 1.3. Note that this PR does not fix the advertising parameters API. We will need a new API for this flag when TCP support lands. Leaving this for a follow up PR as changing the API should happen separately from bringing the examples into conformance as it may affect the platforms, who have implementations outside of the SDK. Test: tested with all-clusters and avahi. Also fixed tests to omit this flag and ensure it does not appear in the TXT record. Automated cert test for operational records is in progress and requires this PR to pass. * Restyled by clang-format --------- Co-authored-by: Restyled.io * [nrfconnect] Fix memory leak in Window Covering sample. (#32041) We were leaking 8 bytes each time the new attribute change was reported by the WindowCovering Server. It was resulting in running out of heap when stressing the device (by sending multiple up-or-open/down-or-close commands). Signed-off-by: Marcin Kajor * [Silabs] Update Silabs ICD status LED functionality description within README.md's (#31888) * Updated Silabs ICD status LED functionality description * Restyled by prettier-markdown * Updated style and spelling * Updated style and spelling * Updated to include ICD vs. non-ICD LED behavior description * Removed unnecessary underscores * Correct description/documentation to refactored location of Silabs lighting example * Restyled by prettier-markdown * Restyled by shfmt --------- Co-authored-by: Restyled.io * Enabling wpa3 mode by default. (#31858) * Mark PayloadTestRequest/Response provisional on Darwin for now. (#32085) * [Examples] Remove EmberAfStatus from example files (#32056) * Replace EmberAfStatus in examples files * Fix Tv-app * fix ci * remove using in headers * [Android] Add Status handling for onReport code path (#32082) * [ICD]Convert the ICD DNS advertiser variable from optional bool to an enum class (#32080) * Convert the ICD DNS advertiser variable from optional bool to an enum class * Apply suggestions from code review Co-authored-by: mkardous-silabs <84793247+mkardous-silabs@users.noreply.github.com> * default mICDModeAdvertise to kNone --------- Co-authored-by: mkardous-silabs <84793247+mkardous-silabs@users.noreply.github.com> * Update DGGEN_1_1 and DGGEN_2_3 to match latest testplan spec (#31998) * Update DGGEN_1_1 and DGGEN_2_3 to match latest testplan spec * Fix CI * Fix CI * Restyled by whitespace * Restyled by prettier-yaml * Fix CI * Fix CI * Quick fix * Update based on PR comments * Restyled by whitespace --------- Co-authored-by: Restyled.io * Add test steps to IDM_1_4 to validate multiple InvokeResponseMessages (#31979) * Add test steps to IDM_1_4 to validate multiple InvokeResponseMessages * Restyled by autopep8 * Address PR comments and CI issues * rename variable * Fix CI * Fix incorrect merge attempt * Fix CI --------- Co-authored-by: Restyled.io * [Tizen] Shink scope of `capi-system-peripheral-io` dep (#32048) * [Diagnostic Logs] Followup for #31638 (#31833) * [CI] Make it possible to specify the timeout when waiting for a specific string in scripts/tests/chiptest/test_definition.py since it may be longer than 10 seconds * [CI] Add a second instance of the configured applications such that YAML can start the app multiple times if needed * [BDX:DiagnosticLog] Update the server such that it supports multiple downloads in parallel * [darwin-framework-tool] Address post-landing comments of #31638 [CI] Add some tests for bdx download with darwin-framework-tool * Use `include-what-you-use` on lib/core (#32046) * A first pass at running IWYU with some manual fixes * Refert platform file changes * Move one header up due to self-review * Move self-header to top of file * Another pass this time with -Xiwyu flags * Start adding a IWYU mapping so we need less manual fixes * Fix command line * Update iwyu.imp * Remove forward decl * Fix for compile * Fixes to compile locally ... several things could be cleaned * One more cleanup, linux all-clusters compiles at least --------- Co-authored-by: Andrei Litvin * DM XMLs: 1.3 0.7 recirculation ballot (#31980) * DM XML: Scrape from 1.3 0.7 recrirculation ballot * Manual fixes for aliased clusters (pending 1.2.4 fix in scraper) * Manual fix for base device type This broke again when I re-levelled the section headers. Will fix in spec. * Update API maturity where missing on 1.3 clusters (#32101) - API maturity was not set properly on some clusters - API maturity is being updated to provisional for new clusters Fixes #32099 Testing done: - Non functional change * [IM] Remove EmberAfStatus from src/app/clusters (#32055) * Remove EmberAfStatus from src/app/clusters * fix ci * fix specs compliance * remove using Status from headers * fix comments * Remove using Status from headers * fix comments * fix ci * fix door lock * Skip binary files when checking for code patterns in the lint test. (#32102) * [NXP][K32W] k32w1 sdk 2.12.6 updates & k32w0 small fixes (#31924) * [common] Remove k32w1 device_platform, use nxp instead Signed-off-by: Andrei Menzopol Signed-off-by: Marius Tache * [K32W1] Build, OTA, Diagnostic changes Signed-off-by: Doru Gucea Signed-off-by: Marius Tache Signed-off-by: Andrei Menzopol * [K32W1] Rename SNT files to ELEMU * [K32W] Add OPENTHREAD_PLATFORM_CORE_CONFIG_FILE define Signed-off-by: Marius Tache * [K32W1] Fix MBEDTLS build flags conflict between Matter vs. OpenThread Matter uses a multithread flavor of MBEDTLS while OpenThread uses a single thread flavor. This triggers a dangerous situation where Matter, OpenThread and MBEDTLS don't agree on the size of various data structures exchanged through the interface which later leads to memory access violations. This update fixes the build configuration so that OpenThread also uses the multithread flavor of MBEDTLS. Signed-off-by: Marian Chereji * [NXP] Update ot-nxp commit-id Signed-off-by: Andrei Menzopol * [K32W0] Add necessary check for env variable When computing the path for the signing script, the existence of env variable NXP_K32W0_SDK_ROOT should be checked first to avoid issues. Signed-off-by: marius-alex-tache * [K32W1] Remove ResetWatermarks empty implementation Signed-off-by: Marius Tache * [K32W1] Set rotating device id unique id length to max by default Signed-off-by: Marius Tache * [K32W1] Fix an issue with __wrap_realloc Current memory manager is MML, which has realloc support. Removed old function that was relevant only in FreeRTOS memory manager (heap4/5) context. Signed-off-by: Marius Tache * [K32W1] Accomodate low power API update PWR_AllowDeviceToSleep/PWR_DisallowDeviceToSleep API reintroduced in SDK. power.c removed from ot-nxp. Signed-off-by: Marius Tache * [K32W1] Clear keys from counter area when factory resetting Signed-off-by: Marius Tache * [K32W1] Use MML API for heap memory consumption statistics WTRMRK feature of the Software Diagnostic Cluster will be implemented once the memory manager will implement the needed API. Signed-off-by: Marius Tache * [K32W1] Remove duplicated code in crypto PAL P256Keypair members were moved into protected scope, so derived classes (P256KeypairSSS) can access these members. Signed-off-by: Marius Tache * [K32W1] Call PLATFORM_InitTimerManager before PLATFORM_InitBLE since this function won't be initializing the timer manager anymore * [K32W1] Post events from ISR correctly Signed-off-by: Marius Tache * [K32W1] MML realloc should be protected from task switching Signed-off-by: Marius Tache * [K32W1] Add initial FactoryDataProviderImpl Signed-off-by: Marius Tache * [K32W1] Update certification declaration in apps Signed-off-by: Marius Tache * [K32W1] Update README files in apps Signed-off-by: Marius Tache * [K32W1] Add DAC private key convert declaration TBD Signed-off-by: Marius Tache * [k32w1] Fix usage of deprecated factory data provider fields Signed-off-by: marius-alex-tache * [COMMON] Add dac_key_use_sss_blob option dac_key_use_sss_blob can be used to select which data the private key area stores: plaintext/encrypted private key or an SSS exported blob. The option is False by default. Signed-off-by: Marius Tache * [K32W1] Add SSS API usage Signed-off-by: Marius Tache * [K32W1] Add chip_convert_dac_private_key option chip_convert_dac_private_key can be used to enable one-time-only conversion of the DAC private key to an SSS encrypted blob. Signed-off-by: Marius Tache * [K32W1] Add jlink conversion script example Signed-off-by: Marius Tache * [K32W1] Update readme to point to docs guide Signed-off-by: Marius Tache * [K32W1] Remove unused mbedtls modules Signed-off-by: Marius Tache * [K32W1] Factory data section should only be reserved in factory data context Signed-off-by: Marius Tache * [K32W1] Add destructor to free SSS object context Signed-off-by: Marius Tache * [K32W1] Add verbose logging for conversion steps Signed-off-by: Marius Tache * [K32W1] Restore NVM sector number The underlying issue, for which the increase in NVM sectors was a workaround, was solved. Signed-off-by: Andrei Menzopol * [K32W1] Add BLE ResetController function Signed-off-by: Andrei Menzopol * [K32W1] Allocate OT buffers from SMU2 When using use_smu2_dynamic=true, OpenThread buffers will be allocated from a 13KB SMU2 region after a successful commissioning process until a factory reset is done. * Add SMU2 namespace in SMU2Manager.h/.cpp * Add build changes * Add allocator changes * Add necessary OpenThread config defines * Add gn parameter/defines for each SMU2 usecase Signed-off-by: Andrei Menzopol * [K32W1] Offload SHA256 operations to S200 * Offload simple sha256, simple hmac sha256, update/finish sha256, update/finish hmac sha256 from Matter and OT. * Implement solution for long sha256 that uses context save/export/restore mechanism. This solution saves the intermediate data on heap and computes the hash on demand (by calling GetDigest). * Put changes under define/gn parameter use_hw_sha256 as they increase timings (due to hardware accelerator initialization time) and use a significant amount of heap. Signed-off-by: Andrei Menzopol * [K32W1] Simplify HW DRBG/TRNG usage S200 TRNG is already used though otPlatEntropyGet from OT. Simplify Matter and OT DRBG usage of HW TRNG by not using mbedtls entropy. Signed-off-by: Andrei Menzopol * [K32W1] Offload AES operations to S200 * Offload AES-CCM from Matter. * Offload AES-ECB from OT. * Use software AES-CCM when Additional authentical data len is zero. * Put changes under define/gn parameter use_hw_aes as they increase timings (due to hardware accelerator initialization time). Signed-off-by: Andrei Menzopol * [K32W1] Fix duplicated flag Signed-off-by: Marius Tache * [COMMON] Add pw RPC support The pw RPC integration should be usable by all NXP platforms that support SerialManager, since the pw_sys_io backend uses streamer_nxp.cpp for serial communication. A scope is defined in pw_rpc_server.gni to be used by all apps by forwarding the variables in the application executable scope: forward_variables_from(pw_rpc_server, "*") Logging and RPC communication share the same serial port. Logging should be inspected in chip-console, which manages the HDLC channels. Signed-off-by: Marius Tache * [COMMON] Update streamer to read RX data ASAP Moved reading the RX FIFO inside the actual interrupt handler, to avoid time penalty when notifying the consumer task (RPC/App CLI). The data is instead stored in a cache buffer and read once the task is unblocked through notifying by the ISR. Signed-off-by: Marius Tache * [K32W1] Enable pw RPC on lighting app To build with RPC enabled, the user should generate the files: gn gen out/debug --args='import("//with_pw_rpc.gni") treat_warnings_as_errors=false' pw RPC requires c++ 17. Some example commands are written in the README, under section `Running RPC console`. Signed-off-by: Marius Tache * [COMMON] Revert streamer implementation and add UART DMA support pw RPC requires UART DMA to avoid missing commands and errors related to frame integrity checks. Applications that enable RPC support should use UART DMA (if possible). Signed-off-by: Marius Tache * [K32W1] Use UART DMA for pw RPC Signed-off-by: Marius Tache * k32w1: enable sit icd Signed-off-by: Doru Gucea * [K32W1] Implement watermark feature using MML API Signed-off-by: Marius Tache * [K32W1] Update free SMU2 region in linker script The SMU2 free region shrank due to addition of lowpower flag in NBU. This overlap might affect Matter instances placed in SMU2 region. Signed-off-by: Andrei Menzopol * [K32W1] Fix TotalOperatingHours attribute reset The UpTime attribute of the GeneralDiagnostics cluster was being reset periodically due to the limited range of the LPTMR timestamp (32 bits). To fix it, support for a 64bit timestamp was added to the NXP OpenThread library and now Matter can use the "otPlatTimeGet()" API to acquire the timestamp in microseconds. Signed-off-by: Marian Chereji * [K32W1] Fix LP current draw for Contact Sensor App after framework changes After framework changes with reintroducing PWR_AllowDeviceToSleep & PWR_DisallowDeviceToSleep functions, Contact Sensor application does not enter into low power anymore after BLE connection event. Adding PWR_DisallowDeviceToSleep call at connection event before PWR_AllowDeviceToSleep as framework request. Signed-off-by: Marius Vilvoi * [K32W] Send a report before resetting the device during OTA State-transition event from Downloading to Applying was not successfully sent to a subscriber during OTA because the device would reset before actually sending the ReportData message. Added an explicit call to sync send a report if any events were logged. This is managed by the ReportingEnginge. Signed-off-by: marius-alex-tache * [K32W] Create dedicated zap file for contact sensor Signed-off-by: Andrei Menzopol * [K32W1] Switch to dedicated zap file The common zap file configuration has many optional cluster enabled that are not needed in our contact-sensor reference app Signed-off-by: Andrei Menzopol * [K32W] Change lighting-app root node revision to 2 Signed-off-by: Andrei Menzopol * [K32W1] Adjust rambuffer size and KVS max number of keys Signed-off-by: Andrei Menzopol * [K32W1] Add CleanReset API Clean reset ensures: * Matter event loop is stopped. * Shutdown is called. * Pending operations are completed in NVM. * MCU is reset. Signed-off-by: marius-alex-tache * [K32W1] Change long press SW3 action from OTA query to clean soft reset When long pressing SW3, the reference applications will call the platform manager clean soft reset API. This will reset the MCU after the Matter shutdown procedure is called and all NVM pending operations are finished. Signed-off-by: marius-alex-tache * [K32W] define CHIP_ADDRESS_RESOLVE_IMPL_INCLUDE_HEADER Signed-off-by: Andrei Menzopol * [K32W] Update docker image version Signed-off-by: Andrei Menzopol * Restyled by whitespace * Restyled by clang-format * Restyled by prettier-markdown * Fix spelling Signed-off-by: Andrei Menzopol * [K32W] Update contact-sensor-app zap files Signed-off-by: Andrei Menzopol * [K32W] Remove deprecated code Signed-off-by: marius-alex-tache * [K32W1] Fix gn check errors Signed-off-by: marius-alex-tache * [K32W1] Fix build Signed-off-by: Andrei Menzopol * Restyled by gn * [K32W1] Fix lints Signed-off-by: Andrei Menzopol * [NXP] Update ot-nxp commit-id Signed-off-by: Andrei Menzopol * [NXP] Update matter zap file Signed-off-by: Andrei Menzopol --------- Signed-off-by: Andrei Menzopol Signed-off-by: Marius Tache Signed-off-by: Doru Gucea Signed-off-by: Marian Chereji Signed-off-by: marius-alex-tache Signed-off-by: Marius Vilvoi Co-authored-by: Yanis Viola Co-authored-by: Marius Tache Co-authored-by: Marian Chereji Co-authored-by: Yassine El-aissaoui Co-authored-by: Doru Gucea Co-authored-by: Marius Vilvoi Co-authored-by: Restyled.io * Update TI GN build to build dependent TI SDK libraries (#32052) * Update TI GN build to build depdent TI SDK libraries Co-authored-by: Henry Isobe * Update 32XX ISA * Restyled by gn * Restyled by autopep8 * Update NV pages to 5 --------- Co-authored-by: Restyled.io * [Linux] Use sync version of BLE connect function (#32013) * Remove unused BluezEndpoint member * [Linux] Use sync version of BLE connect function This way we can report the connection error to the caller. The connection happens on the GLib thread anyway. * Remove unused ConnectParams * Add device connection logging * Fix build error in case of disabled logging * [Python] Allow early logging initialization (#31945) * [Python] Allow early logging initialization Allow to initialize logging before initializing the stack. Similar to tracing, logging doesn't need the stack to be setup. This allows to redirect logging to Python as early as possible. * Log using chip.native module for clarity Use chip.native module for all redirected logs. This separates it from logs written by the Python bindings themselfs. * Updated the TC_RVCRUNM_2_2 python test script following changes in th… (#32071) * Updated the TC_RVCRUNM_2_2 python test script following changes in the test plan. * Restyled by autopep8 * Update src/python_testing/TC_RVCRUNM_2_2.py Co-authored-by: Petru Lauric <81822411+plauric@users.noreply.github.com> --------- Co-authored-by: Restyled.io Co-authored-by: Petru Lauric <81822411+plauric@users.noreply.github.com> * Update TC_RVCOPSTATE_2_4 following test plan changes (#32072) * Updated the TC_RVCOPSTATE_2_4 python test script following changes in the test plan. * Enabled TC_RVCOPSTATE_2_4 to run in CI agianst the rvc-app. * Future proofed the enum to text functions. * Restyled by autopep8 * Update src/python_testing/TC_RVCOPSTATE_2_4.py Co-authored-by: Petru Lauric <81822411+plauric@users.noreply.github.com> --------- Co-authored-by: Restyled.io Co-authored-by: Petru Lauric <81822411+plauric@users.noreply.github.com> * Removed direct inclusion of Electrical Power Measurement and Electrical Energy Measurement clusters from EVSE device type. Per spec PR #8834 (#32111) * Issue #31002 Enable RPC for all-clusters-app (#31006) * Issue #31002 Enable RPC for all-clusters-app * Restyled by gn --------- Co-authored-by: Restyled.io * Finish remaning ICD configs from Interval to Duration (#32109) * [SDK] Rename EMBER_XXX to MATTER_XXX (#32078) * Rename EMBER_ to MATTER_ * Fix last comments * Update some structure memory layour based on pahole feedback, see if we gain RAM (#32106) Co-authored-by: Andrei Litvin * [App] Removing af-enums.h (#32110) * Purge EMBER_ZCL_STATUS * Add to_underlying in examples * fix CI * Fix CI and comments * Darwin changes for logging (#32053) * [icd] Introduced gn flags to allow conditional code compilation (#31795) The ICD implementation is compiled with a full set of features, even if the device uses SIT configuration. It results in a big flash memory waste on a functionalities that cannot be used by certain configuration. Introduced new gn flags corresponding to the ICD features - LIT, CIP and UAT. Modified the ICD Manager implementation and icd-management-server code to use created defines and cut off unused code by the preprocessor. * [NXP][zephyr] Updates to NXP Zephyr platform enablement (#31823) * Fix duplicate definition of recvmsg on latest zephyr tree * Remove redundant config flags * Configure event queue size to 64 * Remove recovery mechanism in wifi implementation * Always enable CHIP_FACTORY_RESET_ERASE_NVS * Update net configuration for wifi Signed-off-by: Axel Le Bourhis * [Python] Make Logging Filtering global (#31944) Log filtering is a global affair, so move the filtering functions to a global context too. This allows to setup logging before creating any device controller. It aligns with how the SDK treats logging. Also enable log filtering for the Linux platform to make the functionality useful on Linux. * Remove some unneeded dependencies from generic-callbacks.h (#32051) Co-authored-by: Andrei Litvin * QA team requested that all attributes in EPM cluster be turned on in energy-management-app (#32095) * Fixes: #32089 In all-clusters-app, Set the EPM PowerMode attribute to AC. In energy-management-app turned on all features and attributes for testing. Fixed issue in TC_EPM_2_1.py when valid attribute is returned. Improved type checking for HarmonicCurrents and HarmonicPhases * Turned on additional attributes in EPM cluster on energy-management-app. Now runs tests without skipping unsupported attributes. * Turned on additional attributes in EPM cluster on energy-management-app. Now runs tests without skipping unsupported attributes. * Fixes #31925 - QA team expect HarmonicCurrents and HarmonicPhases to have 1 entry in the list for testing * Turned off 3 provisional features which fail the TC_DeviceConformance.py test * Removed EVSE attributes which are provisional in 1.3 (SOC, PNC, V2X) * [TCTL] Change name from Mode to State (#31978) * Change name * Restyled by prettier-yaml --------- Co-authored-by: Restyled.io * Add oven opstate yaml (#31557) * Adding OvenOpState test scripts and ciTests.json file * Restyled by whitespace * Restyled by prettier-json * Restyled by prettier-yaml * Restyled by autopep8 * added OvenOpstate2_4 and updated other files * Delete src/app/tests/suites/certification/Test_TC_OVENOPSTATE_1_1.yml * Delete src/app/tests/suites/certification/Test_TC_OVENOPSTATE_2_2.yml * Restyled by whitespace * Restyled by prettier-json * Restyled by prettier-yaml * updated test scripts * updated Test_TC_OVENOPSTATE_2_2.yaml * Restyled by prettier-yaml * updated yamls and ran zap regen * Restyled by isort * Restyled by isort * updated __init__.py * updated citests and mannual tests * updated python scripts * Restyled by autopep8 * updated step3 to automated test in 2_2 * updated citests.json * updated Tests and ciTest file * Revert "updated Tests and ciTest file" This reverts commit 163c8105fc8e96c1be42e01af5f43cd628697dad. * updated according review * zap regen * Revert "zap regen" This reverts commit b057c11a05a2990bf13f2247c0b1b544cf8baed1. * removed zap gen * revert zap gen * Fix opstate pics * Updated commands * Fix white space * fix for PR * Update commands * fixes PR * fix * fix test cases * Restyled by prettier-yaml * fixed place for test case * fixed test json * zap regen * added Ovenopstate2_2 and 2_5 yaml and python files. Also some associated changed files * Removed YAML test * added python tests * Updated zap all cluster app * improvements opstte 1_1 * improvement base code * update example app * improvements on ovenopstate * ci pics update * ci pics updates * fix matter testing support * improvements on test cases * Adding commads for example app * added TC 2.2 to the base test * enabled all tests for oven cavity opstate * added app id to the CI parameters * Blocking start when in fault * added tests for CI * Fix lints * Added test 2.3 * Updated business rules on application * Added app pid parameter to mobile test script * fix pipe * fix lint * Added test 2.4 * fixed parameter from run python script * Fixed Lint * Deleted OVENOPSTATE YAMLs * Fix run python script * Fix lint * Added opstate test cases * Fixed opstate app * removed opstate from YAML * fixes according test plan update * Fixed missing test errors * added countdown behavior to opstate * clean up command line linux * Added TC 2.5 automated * Added OPSTATE and OVENOPSTATE to CI * fixed according review * Restyled by whitespace * Restyled by clang-format * Restyled by autopep8 * Restyled by isort * Updates according review * Restyled by autopep8 * Restyled by isort --------- Co-authored-by: Prakash Ravi Co-authored-by: Restyled.io Co-authored-by: Prakash Ravi <68609205+prakashece@users.noreply.github.com> * Update boolcfg-3-1 to match latest test plan (#32075) * [VALCC] Update test scripts to match latest test plan (#32074) * Update test scripts to match latest test plan * Update src/python_testing/TC_VALCC_4_5.py Co-authored-by: C Freeman * Added missing skipped logging * Updated copyright year --------- Co-authored-by: C Freeman * Removing file (#30933) Co-authored-by: abeck-riis <98488327+abeck-riis@users.noreply.github.com> * Fix MRP SessionIdleInterval per spec to 500ms (#32124) * Fix MRP SessionIdleInterval per spec to 500ms Spec has the default SESSION_IDLE_INTERVAL defined as 500ms https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/1.2/src/secure_channel/Message_Reliability_MRP.adoc#8-parameters-and-constants Originally defined as 300ms, this change was introduced as part of spec PR https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/7186 * Update Sleepy -> Session in context of active/idle intervals Reflecting the spec update https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/7186 * Initial pass for Issue 30906 "[Feature] Integrate tv-app and tv-casting-app into our CI system" (#32012) * Create examples-tvapp.yaml * Delete .github/workflows/examples-tvapp.yaml * Create examples-tv-app.yaml Using nrfconnect as a template * Update examples-tv-app.yaml * [Scenes] Yaml steps discrepancies (#32003) * Updated the steps to match the comments in the chip test plan * Updated label for LasConfiguredBy attribute test * Applied comments about verification steps in S_2_5 and S_2_6 * Non concurrent mode now sends connect network response as per spec (#31739) * Update Non-concurrent mode to latest spec (#31660) * Handle unused variable (#31660) * Restyled by clang-format * Trigger from BTP, use regular member (#31660) * Fix override (#31660) * Restyled by whitespace * Restyled by clang-format * Use generic OperationalNetworkStarted (#31660) * Replace variable with mState (#31660) Use mState instead of mTerminateOnPacketTxComplete * Sorted namespace out (#31660) * Restyled by clang-format * Address review comments (#31660)ffffffffffffffffffff Close GATT so that BlueZ will disconnect. Add kOperationalNetworkStarted event type * Restyled by clang-format * Access BleLayer mState through function (#31660) * Restyled by clang-format --------- Co-authored-by: Restyled.io * [Scenes] MaxCapacity fix (#31981) * Added refresh on read to the FabricSceneInfo capacity and patched Resource Exhausted for Copy Scene and Store Scene along with tests * Applied suggested fix on ReturnOnFailure and /* = name*/ * Remove PoolCommon::ResetObject (#32120) * Remove PoolCommon: bad name, non-member method, very very limited usage * Restyle * Added comment about returning null on failure * Fix compile on fixed size pools * Restyle * Switch to not use a cast * Fix types ... use auto because types are messy, unsure why * Rename for smaller diff * Rename for smaller diff * Update src/transport/UnauthenticatedSessionTable.h Co-authored-by: Boris Zbarsky --------- Co-authored-by: Boris Zbarsky * Decouple InteractionModelEngine from TimedHandler (#32076) * Decouple TimedHandler.h/cpp from InteractionModelEngine * Restyle * Use override instead of virtual * Update comment - re-add a variant of the previous comment explaining why mTimeLimit is last * Pull in the IM pointers support, to make less RAM/BSS usage for embedded * Fix typo and kick restyler * Restyle * Previous align was better * fix name for handler --------- Co-authored-by: Andrei Litvin * [Scenes] Precondition TC_S_3_1 (#32005) * Added a precondition step to TC_S_3_1 * Split preconditionning into separate steps * Applied nit * Python testing: Fix skip_ to use test plan numbers (#32130) * Python testing: Fix skip_ to use test plan numbers * Update src/python_testing/matter_testing_support.py Co-authored-by: Terence Hampson * address review comments * Update src/python_testing/matter_testing_support.py Co-authored-by: Terence Hampson --------- Co-authored-by: Terence Hampson * Fix restyle --------- Signed-off-by: dependabot[bot] Signed-off-by: Haiwen Xia Signed-off-by: Marcin Kajor Signed-off-by: Andrei Menzopol Signed-off-by: Marius Tache Signed-off-by: Doru Gucea Signed-off-by: Marian Chereji Signed-off-by: marius-alex-tache Signed-off-by: Marius Vilvoi Signed-off-by: Axel Le Bourhis Co-authored-by: manjunath-grl Co-authored-by: Restyled.io Co-authored-by: yunhanw-google Co-authored-by: Junior Martinez <67972863+jmartinez-silabs@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrei Litvin Co-authored-by: Boris Zbarsky Co-authored-by: Joshua Villasenor Co-authored-by: restyled-io[bot] <32688539+restyled-io[bot]@users.noreply.github.com> Co-authored-by: lpbeliveau-silabs <112982107+lpbeliveau-silabs@users.noreply.github.com> Co-authored-by: Arkadiusz Bokowy Co-authored-by: William Co-authored-by: Petru Lauric <81822411+plauric@users.noreply.github.com> Co-authored-by: C Freeman Co-authored-by: mkardous-silabs <84793247+mkardous-silabs@users.noreply.github.com> Co-authored-by: Vivien Nicolas Co-authored-by: Tennessee Carmel-Veilleux Co-authored-by: tennessee.carmelveilleux@gmail.com Co-authored-by: Hasty Granbery Co-authored-by: Wang Qixiang <43193572+wqx6@users.noreply.github.com> Co-authored-by: Andrei Litvin Co-authored-by: joonhaengHeo <85541460+joonhaengHeo@users.noreply.github.com> Co-authored-by: Terence Hampson Co-authored-by: Alex Tsitsiura Co-authored-by: haiwen.xia Co-authored-by: Jean-Francois Penven <67962328+jepenven-silabs@users.noreply.github.com> Co-authored-by: Vatsal Ghelani <152916324+vatsalghelani-csa@users.noreply.github.com> Co-authored-by: Lazar Kovacic Co-authored-by: Karsten Sperling <113487422+ksperling-apple@users.noreply.github.com> Co-authored-by: Arkadiusz Bałys Co-authored-by: jamesharrow <93921463+jamesharrow@users.noreply.github.com> Co-authored-by: Jeff Tung <100387939+jtung-apple@users.noreply.github.com> Co-authored-by: tianfeng.yang <130436698+tianfeng-yang@users.noreply.github.com> Co-authored-by: Damian Królik <66667989+Damian-Nordic@users.noreply.github.com> Co-authored-by: Yuanyao Zhong <82843247+yyzhong-g@users.noreply.github.com> Co-authored-by: shripad621git <79364691+shripad621git@users.noreply.github.com> Co-authored-by: Rob Bultman Co-authored-by: Anu Biradar <104591549+abiradarti@users.noreply.github.com> Co-authored-by: Ludovic BOUÉ Co-authored-by: Marcin Kajor <98948394+markaj-nordic@users.noreply.github.com> Co-authored-by: CuRahman <111998767+CuRahman@users.noreply.github.com> Co-authored-by: arun-silabs <141724790+arun-silabs@users.noreply.github.com> Co-authored-by: Gene Harvey Co-authored-by: andrei-menzopol <96489227+andrei-menzopol@users.noreply.github.com> Co-authored-by: Yanis Viola Co-authored-by: Marius Tache Co-authored-by: Marian Chereji Co-authored-by: Yassine El-aissaoui Co-authored-by: Doru Gucea Co-authored-by: Marius Vilvoi Co-authored-by: adabreuti <76965454+adabreuti@users.noreply.github.com> Co-authored-by: Stefan Agner Co-authored-by: Adis Mulic <82869083+adis-ikea@users.noreply.github.com> Co-authored-by: Justin Wood Co-authored-by: Kamil Kasperczyk <66371704+kkasperczyk-no@users.noreply.github.com> Co-authored-by: Axel Le Bourhis <45206070+axelnxp@users.noreply.github.com> Co-authored-by: beckerzito Co-authored-by: Prakash Ravi Co-authored-by: Prakash Ravi <68609205+prakashece@users.noreply.github.com> Co-authored-by: René Josefsen <69624991+ReneJosefsen@users.noreply.github.com> Co-authored-by: abeck-whirlpool <129295708+abeck-whirlpool@users.noreply.github.com> Co-authored-by: abeck-riis <98488327+abeck-riis@users.noreply.github.com> Co-authored-by: Abdul Samad Co-authored-by: Kai Liao <140431279+kliao-csa@users.noreply.github.com> Co-authored-by: simonhmorris1 <112178216+simonhmorris1@users.noreply.github.com> --- src/app/tests/suites/certification/PICS.yaml | 3 + .../certification/Test_TC_CNET_4_1.yaml | 3 +- .../Test_TC_CONTENTLAUNCHER_10_4.yaml | 94 +++++---- .../certification/Test_TC_DISHM_1_2.yaml | 22 --- .../certification/Test_TC_DISHM_2_1.yaml | 5 +- .../certification/Test_TC_DISHM_3_1.yaml | 16 +- .../certification/Test_TC_DISHM_3_3.yaml | 4 +- .../certification/Test_TC_DRYERCTRL_2_1.yaml | 4 +- .../suites/certification/Test_TC_LWM_1_1.yaml | 2 + .../suites/certification/Test_TC_LWM_1_2.yaml | 22 --- .../Test_TC_MEDIAPLAYBACK_6_10.yaml | 182 +++++++++--------- .../Test_TC_MEDIAPLAYBACK_6_8.yaml | 60 ++++++ .../certification/Test_TC_OTCCM_1_1.yaml | 1 - .../certification/Test_TC_OTCCM_1_2.yaml | 119 +++++++++--- .../certification/Test_TC_OTCCM_2_1.yaml | 10 +- .../certification/Test_TC_OTCCM_3_1.yaml | 19 +- .../certification/Test_TC_OTCCM_3_3.yaml | 4 +- .../certification/Test_TC_TCCM_1_1.yaml | 2 + .../certification/Test_TC_TCCM_1_2.yaml | 22 --- .../tests/suites/certification/ci-pics-values | 1 + 20 files changed, 349 insertions(+), 246 deletions(-) diff --git a/src/app/tests/suites/certification/PICS.yaml b/src/app/tests/suites/certification/PICS.yaml index 5096e6e544022b..068db7f0dfe525 100644 --- a/src/app/tests/suites/certification/PICS.yaml +++ b/src/app/tests/suites/certification/PICS.yaml @@ -9177,6 +9177,9 @@ PICS: - label: "Does the DUT support testing the failed ChangeToMode command?" id: OTCCM.S.M.CAN_TEST_MODE_FAILURE + - label: "Can the mode change be manually controlled?" + id: OTCCM.S.M.CAN_MANUALLY_CONTROLLED + - label: "Id of mode the device will fail to transition to, given its current state" diff --git a/src/app/tests/suites/certification/Test_TC_CNET_4_1.yaml b/src/app/tests/suites/certification/Test_TC_CNET_4_1.yaml index fda0797ddbd6e0..54f3f8560b0c5c 100644 --- a/src/app/tests/suites/certification/Test_TC_CNET_4_1.yaml +++ b/src/app/tests/suites/certification/Test_TC_CNET_4_1.yaml @@ -193,5 +193,6 @@ tests: attribute: "SupportedWiFiBands" response: constraints: - type: enum8 + type: list minLength: 1 + maxLength: 6 diff --git a/src/app/tests/suites/certification/Test_TC_CONTENTLAUNCHER_10_4.yaml b/src/app/tests/suites/certification/Test_TC_CONTENTLAUNCHER_10_4.yaml index f192bc0d11dd7f..1fbbdab81d8a26 100644 --- a/src/app/tests/suites/certification/Test_TC_CONTENTLAUNCHER_10_4.yaml +++ b/src/app/tests/suites/certification/Test_TC_CONTENTLAUNCHER_10_4.yaml @@ -36,40 +36,66 @@ tests: verification: | Product maker needs to provide instructions for how to trigger the command on the DUT. For comparison, the DUT behavior for this test step can be simulated using chip-tool (when DUT is a commissioner) or tv-casting-app (when DUT is a commissionee): - ./chip-tool contentlauncher launch-content '{"parameterList": []}' 1 1 1 - ./chip-tv-casting-app contentlauncher launch-content '{"parameterList": []}' 1 1 1 + ./chip-tool contentlauncher launch-content '{"parameterList": [ { "type": 0, "value": "exampleString", "externalIDList": [ { "name": "exampleName", "value": "exampleValue"} ] } ] }' 0 '{"playbackPosition": 0, "textTrack": {"languageCode": "", "audioOutputIndex": 0}}' 1 1 - On TH (tv-app), Verify the launch content command is received successfully. - [1658531449325] [21924:325733] CHIP: [DMG] InvokeRequestMessage = - [1658531449325] [21924:325733] CHIP: [DMG] { - [1658531449325] [21924:325733] CHIP: [DMG] suppressResponse = false, - [1658531449325] [21924:325733] CHIP: [DMG] timedRequest = false, - [1658531449325] [21924:325733] CHIP: [DMG] InvokeRequests = - [1658531449325] [21924:325733] CHIP: [DMG] [ - [1658531449325] [21924:325733] CHIP: [DMG] CommandDataIB = - [1658531449325] [21924:325733] CHIP: [DMG] { - [1658531449325] [21924:325733] CHIP: [DMG] CommandPathIB = - [1658531449325] [21924:325733] CHIP: [DMG] { - [1658531449325] [21924:325733] CHIP: [DMG] EndpointId = 0x1, - [1658531449325] [21924:325733] CHIP: [DMG] ClusterId = 0x50a, - [1658531449325] [21924:325733] CHIP: [DMG] CommandId = 0x0, - [1658531449325] [21924:325733] CHIP: [DMG] }, - [1658531449325] [21924:325733] CHIP: [DMG] - [1658531449325] [21924:325733] CHIP: [DMG] CommandFields = - [1658531449325] [21924:325733] CHIP: [DMG] { - [1658531449325] [21924:325733] CHIP: [DMG] 0x0 = - [1658531449325] [21924:325733] CHIP: [DMG] { - [1658531449325] [21924:325733] CHIP: [DMG] 0x0 = [ - [1658531449325] [21924:325733] CHIP: [DMG] - [1658531449326] [21924:325733] CHIP: [DMG] ], - [1658531449326] [21924:325733] CHIP: [DMG] }, - [1658531449326] [21924:325733] CHIP: [DMG] 0x1 = true, - [1658531449326] [21924:325733] CHIP: [DMG] }, - [1658531449326] [21924:325733] CHIP: [DMG] }, - [1658531449326] [21924:325733] CHIP: [DMG] - [1658531449326] [21924:325733] CHIP: [DMG] ], - [1658531449326] [21924:325733] CHIP: [DMG] - [1658531449326] [21924:325733] CHIP: [DMG] InteractionModelRevision = 1 - [1658531449326] [21924:325733] CHIP: [DMG] }, + ./chip-tv-casting-app contentlauncher launch-content '{"parameterList": [ { "type": 0, "value": "exampleString", "externalIDList": [ { "name": "exampleName", "value": "exampleValue"} ] } ] }' 0 '{"playbackPosition": 0, "textTrack": {"languageCode": "", "audioOutputIndex": 0}}' 1 1 + + + On TH (tv-app), Verify the launch content command is received successfully. + [1706167504.647497][12245:12245] CHIP:EM: >>> [E:12537r S:1570 M:15519558] (S) Msg RX from 2:000000000001B669 [9F65] --- Type 0001:08 (IM:InvokeCommandRequest) + [1706167504.647555][12245:12245] CHIP:EM: Handling via exchange: 12537r, Delegate: 0xaaaac9aaa7c8 + [1706167504.647660][12245:12245] CHIP:DMG: InvokeRequestMessage = + [1706167504.647694][12245:12245] CHIP:DMG: { + [1706167504.647719][12245:12245] CHIP:DMG: suppressResponse = false, + [1706167504.647748][12245:12245] CHIP:DMG: timedRequest = false, + [1706167504.647774][12245:12245] CHIP:DMG: InvokeRequests = + [1706167504.647822][12245:12245] CHIP:DMG: [ + [1706167504.647853][12245:12245] CHIP:DMG: CommandDataIB = + [1706167504.647887][12245:12245] CHIP:DMG: { + [1706167504.647917][12245:12245] CHIP:DMG: CommandPathIB = + [1706167504.647953][12245:12245] CHIP:DMG: { + [1706167504.647987][12245:12245] CHIP:DMG: EndpointId = 0x1, + [1706167504.648023][12245:12245] CHIP:DMG: ClusterId = 0x50a, + [1706167504.648058][12245:12245] CHIP:DMG: CommandId = 0x0, + [1706167504.648090][12245:12245] CHIP:DMG: }, + [1706167504.648124][12245:12245] CHIP:DMG: + [1706167504.648154][12245:12245] CHIP:DMG: CommandFields = + [1706167504.648186][12245:12245] CHIP:DMG: { + [1706167504.648217][12245:12245] CHIP:DMG: 0x0 = + [1706167504.648251][12245:12245] CHIP:DMG: { + [1706167504.648285][12245:12245] CHIP:DMG: 0x0 = [ + [1706167504.648322][12245:12245] CHIP:DMG: + [1706167504.648362][12245:12245] CHIP:DMG: { + [1706167504.648404][12245:12245] CHIP:DMG: 0x0 = 0, + [1706167504.648449][12245:12245] CHIP:DMG: 0x1 = "exampleString" (13 chars), + [1706167504.648491][12245:12245] CHIP:DMG: 0x2 = [ + [1706167504.648533][12245:12245] CHIP:DMG: + [1706167504.648583][12245:12245] CHIP:DMG: { + [1706167504.648630][12245:12245] CHIP:DMG: 0x0 = "exampleName" (11 chars), + [1706167504.648684][12245:12245] CHIP:DMG: 0x1 = "exampleValue" (12 chars), + [1706167504.648733][12245:12245] CHIP:DMG: }, + [1706167504.648777][12245:12245] CHIP:DMG: ], + [1706167504.648819][12245:12245] CHIP:DMG: }, + [1706167504.648859][12245:12245] CHIP:DMG: ], + [1706167504.648895][12245:12245] CHIP:DMG: }, + [1706167504.648930][12245:12245] CHIP:DMG: 0x1 = false, + [1706167504.648963][12245:12245] CHIP:DMG: 0x3 = + [1706167504.648997][12245:12245] CHIP:DMG: { + [1706167504.649033][12245:12245] CHIP:DMG: 0x0 = 0, + [1706167504.649069][12245:12245] CHIP:DMG: 0x1 = + [1706167504.649107][12245:12245] CHIP:DMG: { + [1706167504.649147][12245:12245] CHIP:DMG: 0x0 = "" (0 chars), + [1706167504.649191][12245:12245] CHIP:DMG: 0x2 = 0, + [1706167504.649233][12245:12245] CHIP:DMG: }, + [1706167504.649271][12245:12245] CHIP:DMG: }, + [1706167504.649307][12245:12245] CHIP:DMG: }, + [1706167504.649339][12245:12245] CHIP:DMG: }, + [1706167504.649385][12245:12245] CHIP:DMG: + [1706167504.649412][12245:12245] CHIP:DMG: ], + [1706167504.649457][12245:12245] CHIP:DMG: + [1706167504.649484][12245:12245] CHIP:DMG: InteractionModelRevision = 11 + [1706167504.649511][12245:12245] CHIP:DMG: }, + [1706167504.649703][12245:12245] CHIP:DMG: AccessControl: checking f=2 a=c s=0x000000000001B669 t= c=0x0000_050A e=1 p=o + [1706167504.649746][12245:12245] CHIP:DMG: AccessControl: allowed disabled: true diff --git a/src/app/tests/suites/certification/Test_TC_DISHM_1_2.yaml b/src/app/tests/suites/certification/Test_TC_DISHM_1_2.yaml index 17ee54b3df5fc7..0bb760685297ef 100644 --- a/src/app/tests/suites/certification/Test_TC_DISHM_1_2.yaml +++ b/src/app/tests/suites/certification/Test_TC_DISHM_1_2.yaml @@ -91,25 +91,3 @@ tests: [1689997642.999367][360160:360162] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0059 Attribute 0x0000_0001 DataVersion: 1994417976 [1689997642.999395][360160:360162] CHIP:TOO: CurrentMode: 0 disabled: true - - - label: "Step 4: TH reads from the DUT the OnMode attribute." - PICS: DISHM.S.A0003 - verification: | - ./chip-tool dishwashermode read on-mode 1 1 - - Verify on TH(chip-tool) logs, OnMode attribute value is an integer from supported_modes_dut or null, below is the sample log provided for the raspi platform, Here OnMode attribute value is Null - - [1649676072.465968][10754:10759] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0059 Attribute 0x0000_0003 DataVersion: 4277065072 - [1649676072.466094][10754:10759] CHIP:TOO: OnMode: null - disabled: true - - - label: "Step 5: TH reads from the DUT the StartUpMode attribute." - PICS: DISHM.S.A0002 - verification: | - ./chip-tool dishwashermode read start-up-mode 1 1 - - Verify on TH(chip-tool) logs, StartUpMode attribute value is an integer from supported_modes_dut or null, below is the sample log provided for the raspi platform, Here StartUpMode attribute value is Null - - [1689997956.636092][360258:360260] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0059 Attribute 0x0000_0002 DataVersion: 1994417976 - [1689997956.636125][360258:360260] CHIP:TOO: StartUpMode: null - disabled: true diff --git a/src/app/tests/suites/certification/Test_TC_DISHM_2_1.yaml b/src/app/tests/suites/certification/Test_TC_DISHM_2_1.yaml index 5741789c5dd3f9..2d7ad1388505e5 100644 --- a/src/app/tests/suites/certification/Test_TC_DISHM_2_1.yaml +++ b/src/app/tests/suites/certification/Test_TC_DISHM_2_1.yaml @@ -13,7 +13,7 @@ # limitations under the License. # Auto-generated scripts for harness use only, please review before automation. The endpoints and cluster names are currently set to default -name: 263.3.1. [TC-DISHM-2.1] Change to Mode functionality with DUT as Server +name: 263.3.1. [TC-DISHM-2.1] Change to Mode functionality with DUT as Server PICS: - DISHM.S @@ -135,7 +135,8 @@ tests: ./chip-tool dishwashermode change-to-mode 2 1 1 - Verify on TH(chip-tool) log, DUT responds contains a ChangeToModeResponse command with a GenericFailure(0x02) status response and below is the sample log provided for the raspi platform: + Verify on TH(chip-tool) log, DUT responds contains a ChangeToModeResponse command with + Status field is set to GenericFailure(0x02), InvalidInMode(0x03) , or in the MfgCodes (0x80 to 0xBF) range and StatusText field has a length between 1 and 64, below is the sample log provided for the raspi platform: [1690264785.719168][29854:29856] CHIP:DMG: Received Command Response Data, Endpoint=1 Cluster=0x0000_0059 Command=0x0000_0001 [1690264785.719200][29854:29856] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0059 Command 0x0000_0001 diff --git a/src/app/tests/suites/certification/Test_TC_DISHM_3_1.yaml b/src/app/tests/suites/certification/Test_TC_DISHM_3_1.yaml index 7d00383866d22b..5b3cbf40b3b5d5 100644 --- a/src/app/tests/suites/certification/Test_TC_DISHM_3_1.yaml +++ b/src/app/tests/suites/certification/Test_TC_DISHM_3_1.yaml @@ -17,7 +17,7 @@ name: 263.4.1. [TC-DISHM-3.1] On Mode functionality with DUT as Server PICS: - DISHM.S.A0003 - - DISHM.S.F00 + - MOD.S.F00 - OO.S.C00.Rsp - OO.S.C01.Rsp @@ -95,7 +95,7 @@ tests: disabled: true - label: "Step 2: TH reads from the DUT the OnMode attribute." - PICS: DISHM.S.A0003 && DISHM.S.F00 + PICS: DISHM.S.A0003 && MOD.S.F00 verification: | ./chip-tool dishwashermode read on-mode 1 1 @@ -107,7 +107,7 @@ tests: disabled: true - label: "Step 3: TH reads from the DUT the CurrentMode attribute." - PICS: DISHM.S.A0001 && DISHM.S.F00 + PICS: DISHM.S.A0001 && MOD.S.F00 verification: | ./chip-tool dishwashermode read current-mode 1 1 @@ -121,7 +121,7 @@ tests: disabled: true - label: "Step 4: TH reads from the DUT the SupportedModes attribute." - PICS: DISHM.S.A0000 && DISHM.S.F00 + PICS: DISHM.S.A0000 && MOD.S.F00 verification: | ./chip-tool dishwashermode read supported-modes 1 1 @@ -171,7 +171,7 @@ tests: - label: "Step 5: TH sends a ChangeToMode command to the DUT with NewMode set to new_mode_th" - PICS: DISHM.S.C00.Rsp && DISHM.S.F00 + PICS: DISHM.S.C00.Rsp && MOD.S.F00 verification: | ./chip-tool dishwashermode change-to-mode 2 1 1 @@ -184,7 +184,7 @@ tests: disabled: true - label: "Step 6: TH sends a Off command to the DUT" - PICS: OO.S.C00.Rsp && DISHM.S.F00 + PICS: OO.S.C00.Rsp && MOD.S.F00 verification: | ./chip-tool onoff off 1 1 @@ -196,7 +196,7 @@ tests: disabled: true - label: "Step 7: TH sends a On command to the DUT" - PICS: OO.S.C01.Rsp && DISHM.S.F00 + PICS: OO.S.C01.Rsp && MOD.S.F00 verification: | ./chip-tool onoff on 1 1 @@ -208,7 +208,7 @@ tests: disabled: true - label: "Step 8: TH reads from the DUT the CurrentMode attribute." - PICS: DISHM.S.A0001 && DISHM.S.F00 + PICS: DISHM.S.A0001 && MOD.S.F00 verification: | ./chip-tool dishwashermode read current-mode 1 1 diff --git a/src/app/tests/suites/certification/Test_TC_DISHM_3_3.yaml b/src/app/tests/suites/certification/Test_TC_DISHM_3_3.yaml index 6a34bb32cd7cb2..c9e788d45a6eef 100644 --- a/src/app/tests/suites/certification/Test_TC_DISHM_3_3.yaml +++ b/src/app/tests/suites/certification/Test_TC_DISHM_3_3.yaml @@ -20,7 +20,7 @@ name: PICS: - DISHM.S.A0002 - DISHM.S.A0003 - - DISHM.S.F00 + - MOD.S.F00 - OO.S.A4003 config: @@ -221,7 +221,7 @@ tests: disabled: true - label: "Step 8: TH reads from the DUT the OnMode attribute." - PICS: DISHM.S.A0003 && DISHM.S.F00 + PICS: DISHM.S.A0003 && MOD.S.F00 verification: | ./chip-tool dishwashermode read on-mode 1 1 diff --git a/src/app/tests/suites/certification/Test_TC_DRYERCTRL_2_1.yaml b/src/app/tests/suites/certification/Test_TC_DRYERCTRL_2_1.yaml index e0a490cd60ec1c..865f86283a2830 100644 --- a/src/app/tests/suites/certification/Test_TC_DRYERCTRL_2_1.yaml +++ b/src/app/tests/suites/certification/Test_TC_DRYERCTRL_2_1.yaml @@ -48,7 +48,7 @@ tests: response: constraints: type: list - maxLength: 16 + maxLength: 4 - label: "Step 3: TH reads from the DUT the mandatory attribute: @@ -60,7 +60,7 @@ tests: constraints: type: enum8 minValue: 0 - maxValue: 15 + maxValue: 3 - label: "Step 4:TH writes a supported SelectedDrynessLevel attribute that is diff --git a/src/app/tests/suites/certification/Test_TC_LWM_1_1.yaml b/src/app/tests/suites/certification/Test_TC_LWM_1_1.yaml index 7c7caff3a7ce71..c10fb1c2f7602f 100644 --- a/src/app/tests/suites/certification/Test_TC_LWM_1_1.yaml +++ b/src/app/tests/suites/certification/Test_TC_LWM_1_1.yaml @@ -41,6 +41,7 @@ tests: constraints: type: int16u + #Issue https://github.com/project-chip/connectedhomeip/issues/31551 - label: "Step 3:TH reads from the DUT the FeatureMap attribute., bit 0: SHALL be 1 if and only if LWM.S.F00(DEPONOFF)" @@ -90,6 +91,7 @@ tests: type: list contains: [2] + #Issue https://github.com/project-chip/connectedhomeip/issues/31551 - label: "Step 4c: TH reads the Feature dependent attribute(DEPONOFF) in AttributeList" diff --git a/src/app/tests/suites/certification/Test_TC_LWM_1_2.yaml b/src/app/tests/suites/certification/Test_TC_LWM_1_2.yaml index 39c4a04ceb7163..fc9a0211fb7caf 100644 --- a/src/app/tests/suites/certification/Test_TC_LWM_1_2.yaml +++ b/src/app/tests/suites/certification/Test_TC_LWM_1_2.yaml @@ -99,25 +99,3 @@ tests: [1690184642.211028][15428:15430] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0051 Attribute 0x0000_0001 DataVersion: 3075628268 [1690184642.211042][15428:15430] CHIP:TOO: CurrentMode: 0 disabled: true - - - label: "Step 4: TH reads from the DUT the OnMode attribute." - PICS: LWM.S.A0003 - verification: | - ./chip-tool laundrywashermode read on-mode 1 1 - - Verify on TH(chip-tool) logs, OnMode attribute value is an integer value from supported_modes_dut or null, below is the sample log provided for the raspi platform, Here OnMode attribute value is Null - - [1690184668.519026][15432:15434] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0051 Attribute 0x0000_0003 DataVersion: 3075628268 - [1690184668.519041][15432:15434] CHIP:TOO: OnMode: null - disabled: true - - - label: "Step 5: TH reads from the DUT the StartUpMode attribute." - PICS: LWM.S.A0002 - verification: | - ./chip-tool laundrywashermode read start-up-mode 1 1 - - Verify on TH(chip-tool) logs, StartUpMode attribute value is an integer value from supported_modes_dut or null, below is the sample log provided for the raspi platform, Here StartUpMode attribute value is Null - - [1690184690.235037][15436:15438] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0051 Attribute 0x0000_0002 DataVersion: 3075628268 - [1690184690.235048][15436:15438] CHIP:TOO: StartUpMode: null - disabled: true diff --git a/src/app/tests/suites/certification/Test_TC_MEDIAPLAYBACK_6_10.yaml b/src/app/tests/suites/certification/Test_TC_MEDIAPLAYBACK_6_10.yaml index fce8304cea1c83..6e2fbf37ac8cad 100644 --- a/src/app/tests/suites/certification/Test_TC_MEDIAPLAYBACK_6_10.yaml +++ b/src/app/tests/suites/certification/Test_TC_MEDIAPLAYBACK_6_10.yaml @@ -30,39 +30,40 @@ tests: verification: | Product maker needs to provide instructions for how to trigger the command on the DUT. For comparison, the DUT behavior for this test step can be simulated using chip-tool (when DUT is a commissioner) or tv-casting-app (when DUT is a commissionee): - ./chip-tool mediaplayback activate-audio-track 1 2 1 1 + ./chip-tool mediaplayback activate-audio-track activeAudioTrackId_1 0 1 3 On TH (tv-app), Verify the ActivateAudioTrack command is received successfully. - - [1705561368.453512][4597:4597] CHIP:EM: Handling via exchange: 56290r, Delegate: 0xaaaad1391778 - [1705561368.453660][4597:4597] CHIP:DMG: InvokeRequestMessage = - [1705561368.453721][4597:4597] CHIP:DMG: { - [1705561368.453772][4597:4597] CHIP:DMG: suppressResponse = false, - [1705561368.453833][4597:4597] CHIP:DMG: timedRequest = false, - [1705561368.453890][4597:4597] CHIP:DMG: InvokeRequests = - [1705561368.453995][4597:4597] CHIP:DMG: [ - [1705561368.454052][4597:4597] CHIP:DMG: CommandDataIB = - [1705561368.454139][4597:4597] CHIP:DMG: { - [1705561368.454200][4597:4597] CHIP:DMG: CommandPathIB = - [1705561368.454273][4597:4597] CHIP:DMG: { - [1705561368.454367][4597:4597] CHIP:DMG: EndpointId = 0x1, - [1705561368.454442][4597:4597] CHIP:DMG: ClusterId = 0x506, - [1705561368.454536][4597:4597] CHIP:DMG: CommandId = 0xc, - [1705561368.454610][4597:4597] CHIP:DMG: }, - [1705561368.454721][4597:4597] CHIP:DMG: - [1705561368.454807][4597:4597] CHIP:DMG: CommandFields = - [1705561368.454879][4597:4597] CHIP:DMG: { - [1705561368.454976][4597:4597] CHIP:DMG: 0x0 = "1" (1 chars), - [1705561368.455055][4597:4597] CHIP:DMG: 0x1 = 2, - [1705561368.455130][4597:4597] CHIP:DMG: }, - [1705561368.455195][4597:4597] CHIP:DMG: }, - [1705561368.455297][4597:4597] CHIP:DMG: - [1705561368.455354][4597:4597] CHIP:DMG: ], - [1705561368.455447][4597:4597] CHIP:DMG: - [1705561368.455504][4597:4597] CHIP:DMG: InteractionModelRevision = 11 - [1705561368.455559][4597:4597] CHIP:DMG: }, - [1705561368.455805][4597:4597] CHIP:DMG: AccessControl: checking f=2 a=c s=0x000000000001B669 t= c=0x0000_0506 e=1 p=o - [1705561368.455903][4597:4597] CHIP:DMG: AccessControl: allowed + [1706092346.813652][4555:4555] CHIP:EM: Handling via exchange: 39139r, Delegate: 0xaaaaabf7a7c8 + [1706092346.813745][4555:4555] CHIP:DMG: InvokeRequestMessage = + [1706092346.813775][4555:4555] CHIP:DMG: { + [1706092346.813800][4555:4555] CHIP:DMG: suppressResponse = false, + [1706092346.813829][4555:4555] CHIP:DMG: timedRequest = false, + [1706092346.813857][4555:4555] CHIP:DMG: InvokeRequests = + [1706092346.813892][4555:4555] CHIP:DMG: [ + [1706092346.813919][4555:4555] CHIP:DMG: CommandDataIB = + [1706092346.813949][4555:4555] CHIP:DMG: { + [1706092346.813975][4555:4555] CHIP:DMG: CommandPathIB = + [1706092346.814004][4555:4555] CHIP:DMG: { + [1706092346.814041][4555:4555] CHIP:DMG: EndpointId = 0x3, + [1706092346.814077][4555:4555] CHIP:DMG: ClusterId = 0x506, + [1706092346.814111][4555:4555] CHIP:DMG: CommandId = 0xc, + [1706092346.814145][4555:4555] CHIP:DMG: }, + [1706092346.814179][4555:4555] CHIP:DMG: + [1706092346.814209][4555:4555] CHIP:DMG: CommandFields = + [1706092346.814242][4555:4555] CHIP:DMG: { + [1706092346.814278][4555:4555] CHIP:DMG: 0x0 = "activeAudioTrackId_1" (20 chars), + [1706092346.814316][4555:4555] CHIP:DMG: 0x1 = 0, + [1706092346.814351][4555:4555] CHIP:DMG: }, + [1706092346.814382][4555:4555] CHIP:DMG: }, + [1706092346.814417][4555:4555] CHIP:DMG: + [1706092346.814443][4555:4555] CHIP:DMG: ], + [1706092346.814476][4555:4555] CHIP:DMG: + [1706092346.814502][4555:4555] CHIP:DMG: InteractionModelRevision = 11 + [1706092346.814527][4555:4555] CHIP:DMG: }, + [1706092346.814651][4555:4555] CHIP:DMG: AccessControl: checking f=2 a=c s=0x000000000001B669 t= c=0x0000_0506 e=3 p=o + [1706092346.814692][4555:4555] CHIP:DMG: AccessControl: allowed + [1706092346.814776][4555:4555] CHIP:DMG: Received command for Endpoint=3 Cluster=0x0000_0506 Command=0x0000_000C + [1706092346.814824][4555:4555] CHIP:DL: GetContentAppByEndpointId() - endpoint 3 not found disabled: true - label: "Step 2: DUT sends ActivateTextTrack command to TH" @@ -70,38 +71,42 @@ tests: verification: | Product maker needs to provide instructions for how to trigger the command on the DUT. For comparison, the DUT behavior for this test step can be simulated using chip-tool (when DUT is a commissioner) or tv-casting-app (when DUT is a commissionee): - ./chip-tool mediaplayback activate-text-track 1 1 1 + ./chip-tool mediaplayback activate-text-track activeTextTrackId_1 1 3 On TH (tv-app), Verify the ActivateTextTrack command is received successfully. - [1705561407.549846][4597:4597] CHIP:EM: Handling via exchange: 12272r, Delegate: 0xaaaad1391778 - [1705561407.549918][4597:4597] CHIP:DMG: InvokeRequestMessage = - [1705561407.549942][4597:4597] CHIP:DMG: { - [1705561407.549961][4597:4597] CHIP:DMG: suppressResponse = false, - [1705561407.549999][4597:4597] CHIP:DMG: timedRequest = false, - [1705561407.550020][4597:4597] CHIP:DMG: InvokeRequests = - [1705561407.550048][4597:4597] CHIP:DMG: [ - [1705561407.550069][4597:4597] CHIP:DMG: CommandDataIB = - [1705561407.550103][4597:4597] CHIP:DMG: { - [1705561407.550125][4597:4597] CHIP:DMG: CommandPathIB = - [1705561407.550162][4597:4597] CHIP:DMG: { - [1705561407.550190][4597:4597] CHIP:DMG: EndpointId = 0x1, - [1705561407.550227][4597:4597] CHIP:DMG: ClusterId = 0x506, - [1705561407.550255][4597:4597] CHIP:DMG: CommandId = 0xd, - [1705561407.550290][4597:4597] CHIP:DMG: }, - [1705561407.550319][4597:4597] CHIP:DMG: - [1705561407.550342][4597:4597] CHIP:DMG: CommandFields = - [1705561407.550379][4597:4597] CHIP:DMG: { - [1705561407.550409][4597:4597] CHIP:DMG: 0x0 = "1" (1 chars), - [1705561407.550446][4597:4597] CHIP:DMG: }, - [1705561407.550470][4597:4597] CHIP:DMG: }, - [1705561407.550504][4597:4597] CHIP:DMG: - [1705561407.550526][4597:4597] CHIP:DMG: ], - [1705561407.550553][4597:4597] CHIP:DMG: - [1705561407.550584][4597:4597] CHIP:DMG: InteractionModelRevision = 11 - [1705561407.550605][4597:4597] CHIP:DMG: }, - [1705561407.550765][4597:4597] CHIP:DMG: AccessControl: checking f=2 a=c s=0x000000000001B669 t= c=0x0000_0506 e=1 p=o - [1705561407.550811][4597:4597] CHIP:DMG: AccessControl: allowed + [1706092322.174823][4555:4555] CHIP:EM: Handling via exchange: 27897r, Delegate: 0xaaaaabf7a7c8 + [1706092322.174908][4555:4555] CHIP:DMG: InvokeRequestMessage = + [1706092322.174936][4555:4555] CHIP:DMG: { + [1706092322.174956][4555:4555] CHIP:DMG: suppressResponse = false, + [1706092322.174981][4555:4555] CHIP:DMG: timedRequest = false, + [1706092322.175002][4555:4555] CHIP:DMG: InvokeRequests = + [1706092322.175030][4555:4555] CHIP:DMG: [ + [1706092322.175051][4555:4555] CHIP:DMG: CommandDataIB = + [1706092322.175077][4555:4555] CHIP:DMG: { + [1706092322.175101][4555:4555] CHIP:DMG: CommandPathIB = + [1706092322.175128][4555:4555] CHIP:DMG: { + [1706092322.175156][4555:4555] CHIP:DMG: EndpointId = 0x3, + [1706092322.175184][4555:4555] CHIP:DMG: ClusterId = 0x506, + [1706092322.175212][4555:4555] CHIP:DMG: CommandId = 0xd, + [1706092322.175239][4555:4555] CHIP:DMG: }, + [1706092322.175267][4555:4555] CHIP:DMG: + [1706092322.175291][4555:4555] CHIP:DMG: CommandFields = + [1706092322.175317][4555:4555] CHIP:DMG: { + [1706092322.175347][4555:4555] CHIP:DMG: 0x0 = "activeTextTrackId_1" (19 chars), + [1706092322.175376][4555:4555] CHIP:DMG: }, + [1706092322.175401][4555:4555] CHIP:DMG: }, + [1706092322.175428][4555:4555] CHIP:DMG: + [1706092322.175448][4555:4555] CHIP:DMG: ], + [1706092322.175475][4555:4555] CHIP:DMG: + [1706092322.175496][4555:4555] CHIP:DMG: InteractionModelRevision = 11 + [1706092322.175517][4555:4555] CHIP:DMG: }, + [1706092322.175618][4555:4555] CHIP:DMG: AccessControl: checking f=2 a=c s=0x000000000001B669 t= c=0x0000_0506 e=3 p=o + [1706092322.175653][4555:4555] CHIP:DMG: AccessControl: allowed + [1706092322.175678][4555:4555] CHIP:DMG: Received command for Endpoint=3 Cluster=0x0000_0506 Command=0x0000_000D + [1706092322.175713][4555:4555] CHIP:DL: GetContentAppByEndpointId() - endpoint 3 not found + [1706092322.175733][4555:4555] CHIP:ZCL: MediaPlayback NOT returning ContentApp delegate for endpoint:3 + [1706092322.175784][4555:4555] CHIP:DMG: Command handler moving to [NewRespons] disabled: true - label: "Step 1: DUT sends DeactivateTextTrack command to TH" @@ -109,35 +114,34 @@ tests: verification: | Product maker needs to provide instructions for how to trigger the command on the DUT. For comparison, the DUT behavior for this test step can be simulated using chip-tool (when DUT is a commissioner) or tv-casting-app (when DUT is a commissionee): - ./chip-tool mediaplayback deactivate-text-track 1 1 + ./chip-tool mediaplayback deactivate-text-track 1 3 On TH (tv-app), Verify the DeactivateTextTrack command is received successfully. - - [1705561431.594454][4597:4597] CHIP:EM: Handling via exchange: 47051r, Delegate: 0xaaaad1391778 - [1705561431.594534][4597:4597] CHIP:DMG: InvokeRequestMessage = - [1705561431.594564][4597:4597] CHIP:DMG: { - [1705561431.594588][4597:4597] CHIP:DMG: suppressResponse = false, - [1705561431.594689][4597:4597] CHIP:DMG: timedRequest = false, - [1705561431.594718][4597:4597] CHIP:DMG: InvokeRequests = - [1705561431.594764][4597:4597] CHIP:DMG: [ - [1705561431.594791][4597:4597] CHIP:DMG: CommandDataIB = - [1705561431.594821][4597:4597] CHIP:DMG: { - [1705561431.594848][4597:4597] CHIP:DMG: CommandPathIB = - [1705561431.594891][4597:4597] CHIP:DMG: { - [1705561431.594931][4597:4597] CHIP:DMG: EndpointId = 0x1, - [1705561431.594978][4597:4597] CHIP:DMG: ClusterId = 0x506, - [1705561431.595013][4597:4597] CHIP:DMG: CommandId = 0xe, - [1705561431.595056][4597:4597] CHIP:DMG: }, - [1705561431.595092][4597:4597] CHIP:DMG: - [1705561431.595130][4597:4597] CHIP:DMG: CommandFields = - [1705561431.595164][4597:4597] CHIP:DMG: { - [1705561431.595196][4597:4597] CHIP:DMG: }, - [1705561431.595235][4597:4597] CHIP:DMG: }, - [1705561431.595271][4597:4597] CHIP:DMG: - [1705561431.595297][4597:4597] CHIP:DMG: ], - [1705561431.595341][4597:4597] CHIP:DMG: - [1705561431.595367][4597:4597] CHIP:DMG: InteractionModelRevision = 11 - [1705561431.595403][4597:4597] CHIP:DMG: }, - [1705561431.595522][4597:4597] CHIP:DMG: AccessControl: checking f=2 a=c s=0x000000000001B669 t= c=0x0000_0506 e=1 p=o - [1705561431.595573][4597:4597] CHIP:DMG: AccessControl: allowed + [1706092292.772177][4555:4555] CHIP:DMG: InvokeRequestMessage = + [1706092292.772202][4555:4555] CHIP:DMG: { + [1706092292.772221][4555:4555] CHIP:DMG: suppressResponse = false, + [1706092292.772242][4555:4555] CHIP:DMG: timedRequest = false, + [1706092292.772262][4555:4555] CHIP:DMG: InvokeRequests = + [1706092292.772287][4555:4555] CHIP:DMG: [ + [1706092292.772306][4555:4555] CHIP:DMG: CommandDataIB = + [1706092292.772330][4555:4555] CHIP:DMG: { + [1706092292.772351][4555:4555] CHIP:DMG: CommandPathIB = + [1706092292.772376][4555:4555] CHIP:DMG: { + [1706092292.772401][4555:4555] CHIP:DMG: EndpointId = 0x3, + [1706092292.772428][4555:4555] CHIP:DMG: ClusterId = 0x506, + [1706092292.772453][4555:4555] CHIP:DMG: CommandId = 0xe, + [1706092292.772477][4555:4555] CHIP:DMG: }, + [1706092292.772503][4555:4555] CHIP:DMG: + [1706092292.772524][4555:4555] CHIP:DMG: CommandFields = + [1706092292.772547][4555:4555] CHIP:DMG: { + [1706092292.772571][4555:4555] CHIP:DMG: }, + [1706092292.772592][4555:4555] CHIP:DMG: }, + [1706092292.772615][4555:4555] CHIP:DMG: + [1706092292.772634][4555:4555] CHIP:DMG: ], + [1706092292.772657][4555:4555] CHIP:DMG: + [1706092292.772675][4555:4555] CHIP:DMG: InteractionModelRevision = 11 + [1706092292.772694][4555:4555] CHIP:DMG: }, + [1706092292.772782][4555:4555] CHIP:DMG: AccessControl: checking f=2 a=c s=0x000000000001B669 t= c=0x0000_0506 e=3 p=o + [1706092292.772814][4555:4555] CHIP:DMG: AccessControl: allowed + [1706092292.772837][4555:4555] CHIP:DMG: Received command for Endpoint=3 Cluster=0x0000_0506 Command=0x0000_000E disabled: true diff --git a/src/app/tests/suites/certification/Test_TC_MEDIAPLAYBACK_6_8.yaml b/src/app/tests/suites/certification/Test_TC_MEDIAPLAYBACK_6_8.yaml index 1d840bfe1b8da2..a4fed0de0f90a3 100644 --- a/src/app/tests/suites/certification/Test_TC_MEDIAPLAYBACK_6_8.yaml +++ b/src/app/tests/suites/certification/Test_TC_MEDIAPLAYBACK_6_8.yaml @@ -56,6 +56,66 @@ tests: "Step 3: TH sends a ActivateTextTrack command to DUT specifying the ID value from step 2" PICS: MEDIAPLAYBACK.S.C0d.Rsp + verification: | + ./chip-tool mediaplayback activate-text-track activeTextTrackId_1 1 3 + + On TH(chip-tool), Verify response is a successful (value 0) status response + + [1705920343.780331][14340:14342] CHIP:DMG: InteractionModelRevision = 11 + [1705920343.780356][14340:14342] CHIP:DMG: }, + [1705920343.780439][14340:14342] CHIP:DMG: Received Command Response Status for Endpoint=3 Cluster=0x0000_0506 Command=0x0000_000D Status=0x0 + [1705920343.780508][14340:14342] CHIP:DMG: ICR moving to [AwaitingDe] + [1705920343.780756][14340:14342] CHIP:EM: <<< [E:28106i S:13054 M:170208295 (Ack:222967153)] (S) Msg TX to 1:0000000000000001 [D27C] [UDP:[fe80::e65f:1ff:fe0f:19ff%wlan0]:5640] --- Type 0000:10 (SecureChannel:StandaloneAck) + disabled: true + + - label: "Step 4: TH reads the ActiveTextTrack attribute from the DUT" + PICS: MEDIAPLAYBACK.S.A0009 + verification: | + ./chip-tool mediaplayback read active-text-track 1 3 + + On TH(chip-tool), Verify that the attribute returned has a value equal to the ID value from step 2. + + [1705920389.490076][14344:14346] CHIP:DMG: InteractionModelRevision = 11 + [1705920389.490106][14344:14346] CHIP:DMG: } + [1705920389.490347][14344:14346] CHIP:TOO: Endpoint: 3 Cluster: 0x0000_0506 Attribute 0x0000_0009 DataVersion: 402582000 + [1705920389.490552][14344:14346] CHIP:TOO: ActiveTextTrack: { + [1705920389.490588][14344:14346] CHIP:TOO: Id: activeTextTrackId_1 + [1705920389.490634][14344:14346] CHIP:TOO: TrackAttributes: { + [1705920389.490697][14344:14346] CHIP:TOO: LanguageCode: languageCode2 + [1705920389.490731][14344:14346] CHIP:TOO: DisplayName: displayName2 + [1705920389.490761][14344:14346] CHIP:TOO: } + [1705920389.490790][14344:14346] CHIP:TOO: } + [1705920389.491114][14344:14346] CHIP:EM: <<< [E:21221i S:59617 M:230711552 (Ack:159745975)] (S) Msg TX to 1:0000000000000001 [D27C] [UDP:[fe80::e65f:1ff:fe0f:19ff%wlan0]:5640] --- Type 0000:10 (SecureChannel:StandaloneAck) + disabled: true + + - label: "Step 5: TH sends a DeactivateTextTrack command to DUT" + PICS: MEDIAPLAYBACK.S.C0e.Rsp + verification: | + ./chip-tool mediaplayback deactivate-text-track 1 3 + + On TH(chip-tool), Verify response is a successful (value 0) status response + [1705920432.359994][14349:14351] CHIP:DMG: }, + [1705920432.360088][14349:14351] CHIP:DMG: Received Command Response Status for Endpoint=3 Cluster=0x0000_0506 Command=0x0000_000E Status=0x0 + [1705920432.360161][14349:14351] CHIP:DMG: ICR moving to [AwaitingDe] + [1705920432.360460][14349:14351] CHIP:EM: <<< [E:34606i S:60686 M:245440743 (Ack:79584870)] (S) Msg TX to 1:0000000000000001 [D27C] [UDP:[fe80::e65f:1ff:fe0f:19ff%wlan0]:5640] --- Type 0000:10 (SecureChannel:StandaloneAck) + [1705920432.360651][14349:14351] CHIP:EM: Flushed pending ack for MessageCounter:79584870 on exchange 34606i + disabled: true + + - label: "Step 6: TH reads the ActiveTextTrack attribute from the DUT" + PICS: MEDIAPLAYBACK.S.A0009 + verification: | + ./chip-tool mediaplayback read active-text-track 1 3 + + On TH(chip-tool), Verify that the attribute returned has a null value. + [1705920471.994181][14352:14354] CHIP:DMG: } + [1705920471.994472][14352:14354] CHIP:TOO: Endpoint: 3 Cluster: 0x0000_0506 Attribute 0x0000_0009 DataVersion: 402582000 + [1705920471.994741][14352:14354] CHIP:TOO: ActiveTextTrack: { + [1705920471.994789][14352:14354] CHIP:TOO: Id: + [1705920471.994845][14352:14354] CHIP:TOO: TrackAttributes: null + [1705920471.994886][14352:14354] CHIP:TOO: } + [1705920471.995254][14352:14354] CHIP:EM: <<< [E:930i S:17726 M:66812520 (Ack:87654763)] (S) Msg TX to 1:0000000000000001 [D27C] [UDP:[fe80::e65f:1ff:fe0f:19ff%wlan0]:5640] --- Type 0000:10 (SecureChannel:StandaloneAck) + [1705920471.995502][14352:14354] CHIP:EM: Flushed pending ack for MessageCounter:87654763 on exchange 930i + disabled: true command: "ActivateTextTrack" arguments: values: diff --git a/src/app/tests/suites/certification/Test_TC_OTCCM_1_1.yaml b/src/app/tests/suites/certification/Test_TC_OTCCM_1_1.yaml index 180490a567c742..560eb4cac417ef 100644 --- a/src/app/tests/suites/certification/Test_TC_OTCCM_1_1.yaml +++ b/src/app/tests/suites/certification/Test_TC_OTCCM_1_1.yaml @@ -11,7 +11,6 @@ # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. # See the License for the specific language governing permissions and # limitations under the License. -# Auto-generated scripts for harness use only, please review before automation. The endpoints and cluster names are currently set to default name: 260.1.1. [TC-OTCCM-1.1] Global attributes with DUT as Server diff --git a/src/app/tests/suites/certification/Test_TC_OTCCM_1_2.yaml b/src/app/tests/suites/certification/Test_TC_OTCCM_1_2.yaml index 7b793c9694fe10..ce90c982b8bcf9 100644 --- a/src/app/tests/suites/certification/Test_TC_OTCCM_1_2.yaml +++ b/src/app/tests/suites/certification/Test_TC_OTCCM_1_2.yaml @@ -34,38 +34,105 @@ tests: - label: "Step 2: TH reads from the DUT the SupportedModes attribute." PICS: OTCCM.S.A0000 verification: | - ./chip-tool Ovenmode read supported-modes 1 1 + ./chip-tool ovenmode read supported-modes 1 1 - Verify that the DUT response contains list of ModeOptionsStruct entries. - - Verify that the list has at least 2 and at most 255 entries - - Verify that each ModeOptionsStruct entry has a unique Mode field value and Label field value - - If ModeOptionsStruct entry’s ModeTags field is not empty, then Verify the values of the Value fields that are not larger than 16 bits, for each Value field: Is the mode tag value a defined common tag value ( Auto(0x0000), Quick(0x0001), Quiet(0x0002), LowNoise(0x0003), LowEnergy(0x0004), Vacation(0x0005), Min(0x0006), Max(0x0007), Night(0x0008), Day(0x0009)) or a defined cluster derived tag value ( Normal, Delicate, Heavy, Whites) or in the MfgTags (0x8000 to 0xBFFF) range - - If the Value field is in the MfgTags (0x8000 to 0xBFFF) range, the TagName field is a string with a length between 1 and 64 - - Verify that at least one ModeOptionsStruct entry includes the Normal mode tag - - Save the Mode field values as supported_modes_dut on the TH (Chip-tool) - disabled: true - - - label: "Step 3: TH reads from the DUT the CurrentMode attribute." - PICS: OTCCM.S.A0001 - verification: | - ./chip-tool Ovenmode read current-mode 1 1 + - Verify that the DUT response contains a list of ModeOptionsStruct entries + -Verify that the list has at least 2 and at most 255 entries + - Verify that each ModeOptionsStruct entry has a unique Mode field value + - Verify that each ModeOptionsStruct entry has a unique Label field value + - Verify that each ModeOptionsStruct entry’s ModeTags field has: + at least one entry the values of the Value fields that are not larger than 16 bits + - for each Value field: Is the mode tag value a defined common tag value (Auto(0x0000), Quick(0x0001), Quiet(0x0002), LowNoise(0x0003), LowEnergy(0x0004), Vacation(0x0005), Min(0x0006), Max(0x0007), Night(0x0008), Day(0x0009)) or a defined derived cluster tag value (Bake(0x4000), Convection(0x4001), Grill(0x4002), Roast(0x4003), Clean(0x4004), Convection Bake(0x4005), Convection Roast(0x4006), Warming(0x4007), Proofing(0x4008)) or in the MfgTags (0x8000 to 0xBFFF) range + - for at least one Value field: Is the mode tag value a defined common tag value (Auto(0x0000), Quick(0x0001), Quiet(0x0002), LowNoise(0x0003), LowEnergy(0x0004), Vacation(0x0005), Min(0x0006), Max(0x0007), Night(0x0008), Day(0x0009)) or a derived cluster value (Normal, Defrost) + - if the Value field is in the MfgTags (0x8000 to 0xBFFF) range, the TagName field is a string with a length between 1 and 64 + - Verify that at least one ModeOptionsStruct entry includes the Bake(0x4000) mode tag in the ModeTags field + - Save the Mode field values as supported_modes_dut, On the TH (Chip-tool) and below is the sample log provided for the raspi platform: - Verify on TH(chip-tool) logs, CurrentMode attribute value is an integer value from supported_modes_dut + [1705647707.587205][79487:79492] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0049 Attribute 0x0000_0000 DataVersion: 3240184500 + [1705647707.587250][79487:79492] CHIP:TOO: SupportedModes: 9 entries + [1705647707.587265][79487:79492] CHIP:TOO: [1]: { + [1705647707.587268][79487:79492] CHIP:TOO: Label: Bake + [1705647707.587275][79487:79492] CHIP:TOO: Mode: 0 + [1705647707.587282][79487:79492] CHIP:TOO: ModeTags: 1 entries + [1705647707.587287][79487:79492] CHIP:TOO: [1]: { + [1705647707.587290][79487:79492] CHIP:TOO: Value: 16384 + [1705647707.587294][79487:79492] CHIP:TOO: } + [1705647707.587297][79487:79492] CHIP:TOO: } + [1705647707.587303][79487:79492] CHIP:TOO: [2]: { + [1705647707.587305][79487:79492] CHIP:TOO: Label: Convection + [1705647707.587308][79487:79492] CHIP:TOO: Mode: 1 + [1705647707.587311][79487:79492] CHIP:TOO: ModeTags: 1 entries + [1705647707.587316][79487:79492] CHIP:TOO: [1]: { + [1705647707.587318][79487:79492] CHIP:TOO: Value: 16385 + [1705647707.587321][79487:79492] CHIP:TOO: } + [1705647707.587324][79487:79492] CHIP:TOO: } + [1705647707.587330][79487:79492] CHIP:TOO: [3]: { + [1705647707.587333][79487:79492] CHIP:TOO: Label: Grill + [1705647707.587335][79487:79492] CHIP:TOO: Mode: 2 + [1705647707.587338][79487:79492] CHIP:TOO: ModeTags: 1 entries + [1705647707.587342][79487:79492] CHIP:TOO: [1]: { + [1705647707.587345][79487:79492] CHIP:TOO: Value: 16386 + [1705647707.587348][79487:79492] CHIP:TOO: } + [1705647707.587351][79487:79492] CHIP:TOO: } + [1705647707.587356][79487:79492] CHIP:TOO: [4]: { + [1705647707.587359][79487:79492] CHIP:TOO: Label: Roast + [1705647707.587361][79487:79492] CHIP:TOO: Mode: 3 + [1705647707.587364][79487:79492] CHIP:TOO: ModeTags: 1 entries + [1705647707.587369][79487:79492] CHIP:TOO: [1]: { + [1705647707.587371][79487:79492] CHIP:TOO: Value: 16387 + [1705647707.587374][79487:79492] CHIP:TOO: } + [1705647707.587376][79487:79492] CHIP:TOO: } + [1705647707.587382][79487:79492] CHIP:TOO: [5]: { + [1705647707.587384][79487:79492] CHIP:TOO: Label: Clean + [1705647707.587387][79487:79492] CHIP:TOO: Mode: 4 + [1705647707.587390][79487:79492] CHIP:TOO: ModeTags: 1 entries + [1705647707.587394][79487:79492] CHIP:TOO: [1]: { + [1705647707.587397][79487:79492] CHIP:TOO: Value: 16388 + [1705647707.587400][79487:79492] CHIP:TOO: } + [1705647707.587402][79487:79492] CHIP:TOO: } + [1705647707.587408][79487:79492] CHIP:TOO: [6]: { + [1705647707.587410][79487:79492] CHIP:TOO: Label: Convection Bake + [1705647707.587413][79487:79492] CHIP:TOO: Mode: 5 + [1705647707.587416][79487:79492] CHIP:TOO: ModeTags: 1 entries + [1705647707.587420][79487:79492] CHIP:TOO: [1]: { + [1705647707.587423][79487:79492] CHIP:TOO: Value: 16389 + [1705647707.587425][79487:79492] CHIP:TOO: } + [1705647707.587428][79487:79492] CHIP:TOO: } + [1705647707.587434][79487:79492] CHIP:TOO: [7]: { + [1705647707.587436][79487:79492] CHIP:TOO: Label: Convection Roast + [1705647707.587439][79487:79492] CHIP:TOO: Mode: 6 + [1705647707.587442][79487:79492] CHIP:TOO: ModeTags: 1 entries + [1705647707.587446][79487:79492] CHIP:TOO: [1]: { + [1705647707.587449][79487:79492] CHIP:TOO: Value: 16390 + [1705647707.587451][79487:79492] CHIP:TOO: } + [1705647707.587454][79487:79492] CHIP:TOO: } + [1705647707.587460][79487:79492] CHIP:TOO: [8]: { + [1705647707.587462][79487:79492] CHIP:TOO: Label: Warming + [1705647707.587465][79487:79492] CHIP:TOO: Mode: 7 + [1705647707.587468][79487:79492] CHIP:TOO: ModeTags: 1 entries + [1705647707.587472][79487:79492] CHIP:TOO: [1]: { + [1705647707.587475][79487:79492] CHIP:TOO: Value: 16391 + [1705647707.587477][79487:79492] CHIP:TOO: } + [1705647707.587480][79487:79492] CHIP:TOO: } + [1705647707.587485][79487:79492] CHIP:TOO: [9]: { + [1705647707.587488][79487:79492] CHIP:TOO: Label: Proofing + [1705647707.587490][79487:79492] CHIP:TOO: Mode: 8 + [1705647707.587493][79487:79492] CHIP:TOO: ModeTags: 1 entries + [1705647707.587497][79487:79492] CHIP:TOO: [1]: { + [1705647707.587500][79487:79492] CHIP:TOO: Value: 16392 + [1705647707.587502][79487:79492] CHIP:TOO: } + [1705647707.587505][79487:79492] CHIP:TOO: } disabled: true - - label: "Step 4: TH reads from the DUT the OnMode attribute." - PICS: OTCCM.S.A0003 + - label: "Step 3: TH reads from the DUT the CurrentMode attribute." + PICS: OTCCM.S.A0001 verification: | - ./chip-tool Ovenmode read on-mode 1 1 + ./chip-tool ovenmode read current-mode 1 1 - Verify on TH(chip-tool) logs, OnMode attribute value is an integer value from supported_modes_dut or null - disabled: true - - - label: "Step 5: TH reads from the DUT the StartUpMode attribute." - PICS: OTCCM.S.A0002 - verification: | - ./chip-tool Ovenmode read start-up-mode 1 1 + Verify on TH(chip-tool) logs, CurrentMode attribute value is an integer value from supported_modes_dut, below is the sample log provided for the raspi platform, Here CurrentMode attribute value is 0 - Verify on TH(chip-tool) logs, StartUpMode attribute value is an integer value from supported_modes_dut or null + [1705647748.316225][81944:81949] CHIP:DMG: } + [1705647748.316354][81944:81949] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0049 Attribute 0x0000_0001 DataVersion: 3240184500 + [1705647748.316395][81944:81949] CHIP:TOO: CurrentMode: 0 disabled: true diff --git a/src/app/tests/suites/certification/Test_TC_OTCCM_2_1.yaml b/src/app/tests/suites/certification/Test_TC_OTCCM_2_1.yaml index 1e6d17ff71679f..00f08a50b89afa 100644 --- a/src/app/tests/suites/certification/Test_TC_OTCCM_2_1.yaml +++ b/src/app/tests/suites/certification/Test_TC_OTCCM_2_1.yaml @@ -83,7 +83,9 @@ tests: - label: "Step 5: Manually put the device in a state from which it will FAIL to transition to PIXIT.OTCCM.MODE_CHANGE_FAIL" - PICS: OTCCM.S.M.CAN_TEST_MODE_FAILURE && PICS_SKIP_SAMPLE_APP + PICS: + OTCCM.S.M.CAN_TEST_MODE_FAILURE && PICS_SKIP_SAMPLE_APP && + OTCCM.S.M.CAN_MANUALLY_CONTROLLED verification: | Manual operation required cluster: "LogCommands" @@ -96,7 +98,7 @@ tests: value: "y" - label: "Step 6: TH reads from the DUT the CurrentMode attribute." - PICS: OTCCM.S.A0001 + PICS: OTCCM.S.A0001 && OTCCM.S.M.CAN_MANUALLY_CONTROLLED command: "readAttribute" attribute: "CurrentMode" response: @@ -146,7 +148,7 @@ tests: - label: "Step 9: Manually put the device in a state from which it will SUCCESSFULLY transition to PIXIT.OTCCM.MODE_CHANGE_OK" - PICS: PICS_SKIP_SAMPLE_APP + PICS: PICS_SKIP_SAMPLE_APP && OTCCM.S.M.CAN_MANUALLY_CONTROLLED verification: | Manual operation required cluster: "LogCommands" @@ -159,7 +161,7 @@ tests: value: "y" - label: "Step 10: TH reads from the DUT the CurrentMode attribute." - PICS: OTCCM.S.A0001 + PICS: OTCCM.S.A0001 && OTCCM.S.M.CAN_MANUALLY_CONTROLLED command: "readAttribute" attribute: "CurrentMode" response: diff --git a/src/app/tests/suites/certification/Test_TC_OTCCM_3_1.yaml b/src/app/tests/suites/certification/Test_TC_OTCCM_3_1.yaml index 9f805fb0dd8101..3d886d15ab6253 100644 --- a/src/app/tests/suites/certification/Test_TC_OTCCM_3_1.yaml +++ b/src/app/tests/suites/certification/Test_TC_OTCCM_3_1.yaml @@ -16,7 +16,7 @@ name: 3.4 [TC-OTCCM-3.1] On Mode functionality with DUT as Server PICS: - OTCCM.S.A0003 - - OTCCM.S.F00 + - MOD.S.F00 - OO.S.C00.Rsp - OO.S.C01.Rsp @@ -47,14 +47,14 @@ tests: "Precondition: TH writes from the DUT the OnMode attribute.NOTE: To execute this test case set onmode to any integer value because as default it value has null." - PICS: OTCCM.S.A0003 && OTCCM.S.F00 + PICS: OTCCM.S.A0003 && MOD.S.F00 command: "writeAttribute" attribute: "OnMode" arguments: value: ConfigureOnMode - label: "Step 2: TH reads from the DUT the OnMode attribute." - PICS: OTCCM.S.A0003 && OTCCM.S.F00 + PICS: OTCCM.S.A0003 && MOD.S.F00 command: "readAttribute" attribute: "OnMode" response: @@ -63,7 +63,7 @@ tests: type: int8u - label: "Step 3: TH reads from the DUT the CurrentMode attribute." - PICS: OTCCM.S.A0001 && OTCCM.S.F00 + PICS: OTCCM.S.A0001 && MOD.S.F00 command: "readAttribute" attribute: "CurrentMode" response: @@ -74,6 +74,7 @@ tests: - label: "If on_mode_dut is equal to old_current_mode_dut proceed to step 4. Else proceed to step 6." + PICS: OTCCM.S.A0001 && MOD.S.F00 && OTCCM.S.A0003 cluster: "EqualityCommands" command: "UnsignedNumberEquals" arguments: @@ -89,7 +90,7 @@ tests: - label: "Step 4: TH reads from the DUT the SupportedModes attribute." runIf: IsExpectedValue - PICS: OTCCM.S.A0000 && OTCCM.S.F00 + PICS: OTCCM.S.A0000 && MOD.S.F00 command: "readAttribute" attribute: "SupportedModes" response: @@ -101,7 +102,7 @@ tests: "Step 5: TH sends a ChangeToMode command to the DUT with NewMode set to new_mode_th" runIf: IsExpectedValue - PICS: OTCCM.S.C00.Rsp && OTCCM.S.F00 + PICS: OTCCM.S.C00.Rsp && MOD.S.F00 command: "ChangeToMode" arguments: values: @@ -113,17 +114,17 @@ tests: value: 0x00 - label: "Step 6:TH sends a Off command to the DUT" - PICS: OO.S.C00.Rsp && OTCCM.S.F00 + PICS: OO.S.C00.Rsp && MOD.S.F00 cluster: "On/Off" command: "Off" - label: "Step 7:TH sends a On command to the DUT" - PICS: OO.S.C01.Rsp && OTCCM.S.F00 + PICS: OO.S.C01.Rsp && MOD.S.F00 cluster: "On/Off" command: "On" - label: "Step 8: TH reads from the DUT the CurrentMode attribute." - PICS: OTCCM.S.A0001 && OTCCM.S.F00 + PICS: OTCCM.S.A0001 && MOD.S.F00 command: "readAttribute" attribute: "CurrentMode" response: diff --git a/src/app/tests/suites/certification/Test_TC_OTCCM_3_3.yaml b/src/app/tests/suites/certification/Test_TC_OTCCM_3_3.yaml index bb1e8edf932f16..27184569f51b5a 100644 --- a/src/app/tests/suites/certification/Test_TC_OTCCM_3_3.yaml +++ b/src/app/tests/suites/certification/Test_TC_OTCCM_3_3.yaml @@ -18,7 +18,7 @@ name: PICS: - OTCCM.S.A0002 - OTCCM.S.A0003 - - OTCCM.S.F00 + - MOD.S.F00 - OO.S.A4003 config: @@ -135,7 +135,7 @@ tests: value: new_mode_th - label: "Step 8: TH reads from the DUT the OnMode attribute." - PICS: OTCCM.S.A0003 && OTCCM.S.F00 + PICS: OTCCM.S.A0003 && MOD.S.F00 command: "readAttribute" attribute: "OnMode" response: diff --git a/src/app/tests/suites/certification/Test_TC_TCCM_1_1.yaml b/src/app/tests/suites/certification/Test_TC_TCCM_1_1.yaml index c3cd45da41d4db..4a7a44a2bfc6e8 100644 --- a/src/app/tests/suites/certification/Test_TC_TCCM_1_1.yaml +++ b/src/app/tests/suites/certification/Test_TC_TCCM_1_1.yaml @@ -41,6 +41,7 @@ tests: constraints: type: int16u + #Issue https://github.com/project-chip/connectedhomeip/issues/31551 - label: "Step 3: TH reads from the DUT the FeatureMap attribute." command: "readAttribute" attribute: "FeatureMap" @@ -90,6 +91,7 @@ tests: type: list contains: [2] + #Issue https://github.com/project-chip/connectedhomeip/issues/31551 - label: "Step 4c: TH reads from the DUT the AttributeList attribute. 0x0003 SHALL be included if and only if TCCM.S.F00(DEPONOFF)" diff --git a/src/app/tests/suites/certification/Test_TC_TCCM_1_2.yaml b/src/app/tests/suites/certification/Test_TC_TCCM_1_2.yaml index ebeb15ee878c7c..8e3d3190ba249c 100644 --- a/src/app/tests/suites/certification/Test_TC_TCCM_1_2.yaml +++ b/src/app/tests/suites/certification/Test_TC_TCCM_1_2.yaml @@ -82,25 +82,3 @@ tests: [1690189682.782276][16162:16164] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0052 Attribute 0x0000_0001 DataVersion: 473867547 [1690189682.782314][16162:16164] CHIP:TOO: CurrentMode: 0 disabled: true - - - label: "Step 4: TH reads from the DUT the OnMode attribute." - PICS: TCCM.S.A0003 - verification: | - ./chip-tool refrigeratorandtemperaturecontrolledcabinetmode read on-mode 1 1 - - Verify on TH(chip-tool) logs, OnMode attribute value is an integer from supported_modes_dut or null, below is the sample log provided for the raspi platform, Here OnMode attribute value is Null - - [1690189699.711642][16169:16171] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0052 Attribute 0x0000_0003 DataVersion: 473867547 - [1690189699.711687][16169:16171] CHIP:TOO: OnMode: null - disabled: true - - - label: "Step 5: TH reads from the DUT the StartUpMode attribute." - PICS: TCCM.S.A0002 - verification: | - ./chip-tool refrigeratorandtemperaturecontrolledcabinetmode read start-up-mode 1 1 - - Verify on TH(chip-tool) logs, StartUpMode attribute value is an integer from supported_modes_dut or null, below is the sample log provided for the raspi platform, Here StartUpMode attribute value is Null - - [1690189713.973520][16173:16175] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0052 Attribute 0x0000_0002 DataVersion: 473867547 - [1690189713.973576][16173:16175] CHIP:TOO: StartUpMode: null - disabled: true diff --git a/src/app/tests/suites/certification/ci-pics-values b/src/app/tests/suites/certification/ci-pics-values index 3e8f5ed69b37ae..c710f846f0a13a 100644 --- a/src/app/tests/suites/certification/ci-pics-values +++ b/src/app/tests/suites/certification/ci-pics-values @@ -2842,6 +2842,7 @@ OTCCM.S.C01.Tx=0 #Manual controllable OTCCM.S.M.CAN_TEST_MODE_FAILURE=0 OTCCM.S.M.CAN_MANUALLY_CONTROLLED=0 +OTCCM.S.M.CAN_MANUALLY_CONTROLLED=0 #Energy EVSE Cluster EEVSE.S=0