Skip to content

Conversation

@silabs-matter-ci-bot
Copy link

@silabs-matter-ci-bot silabs-matter-ci-bot bot commented Nov 7, 2025

This PR syncs the csa branch with the main branch.

PR MUST BE MERGED WITH MERGE COMMIT - ADMIN MUST ENABLE THE OPTION


Note

Cursor Bugbot is generating a summary for commit bab3aa0. Configure here.

chrisdecenzo and others added 17 commits November 6, 2025 00:26
* callback on passcode error

* comments

* comments

* restyle
* Add units to constant name

* Fix use of constant
…_1_3 and TC_CADMIN_1_4 for CI test runs (#41807)

* [Update] Add default times for TC_CADMIN_1_3 and TC_CADMIN_1_4 for CI run times
- Updated test_metadata.yaml slow_tests to include run times for TC_CADMIN_1_3 and TC_CADMIN_1_4
- Resorted tests in test_metadata.yaml slow_tests

* Update test_metadata.yaml

Updated as tests CADMIN_1_3 and CADMIN_1_4 are both contained in the same test module TC_CADMIN_1_3_4.py instead of seperate files. Was not originally sure if the name was based off the tests or the test file name. Appears according to the other item names listed its based off the test file names.
… [[nodiscard]] (#41796)

* Additional helper methods to help with transition to making ChipError [[nodiscard]]

* Apply review suggestions

* Apply review suggestions

* Apply suggestions from code review

Co-authored-by: Boris Zbarsky <[email protected]>

---------

Co-authored-by: Boris Zbarsky <[email protected]>
…_power_lit.conf (#41794)

* [nxp][cmake] Add CHIP_ICD_DSLS_SUPPORT cmake config

 * Add Intermittently Connected Device Dynamic SIT LIT support config
 * Link cmake config with gn argument

Signed-off-by: Andrei Menzopol <[email protected]>

* [nxp][examples][config] Add prj_thread_mtd_low_power_lit.conf

 * Add ICD LIT config file

Signed-off-by: Andrei Menzopol <[email protected]>

---------

Signed-off-by: Andrei Menzopol <[email protected]>
* Added TC_DD_3_23.py Script

* Update the script with latest Test plan, added extra argument checker for nfc reader index

* FIx for None type iterable bug for int-arg check

* FIx for None type iterable bug for int-arg check

* FIx for None type iterable bug for int-arg check

* implemented new logic for nfc and int-arg check

* fixed exit logic for nfc_reader_index

* fixed syntax erros suggested by co-pilot

* removed duplicate code block in runner.py

* log the data from nfc tag instead of matter_test

* Restyled by autopep8

* Restyled by isort

* fixed CI job of linux code lint and disabled the nfc build for python env by default

* added missing libs for nfc interaction

* changed the commissioning args in CI test

* updated runner.py to be in sync with master and updated ci run args

* added setuppincode for CI execution argumennts

* Updated the code and comments for NFC code as per comments

* added some checks for nfc reader index and max ndef data length

* Restyled by autopep8

* TC_DD_3_24.py will be skipped and documentation is improved

* Added check for qr_code presence in command line args

* Added data types to function arguments

* Restyled by prettier-yaml

* Restyled by autopep8

* Updated the code changes to ignore the identifier code in nfc-tag data

* resolved comments from co-pilot

* fixed mypy error on CI

* Restyled by autopep8

* fixed suggestions by co-pilot

* Updated code to be more readable

---------

Co-authored-by: Restyled.io <[email protected]>
* Adding matter closure specific attribute callback when markdirty is called

* Restyled by whitespace

* Adding attribute update callback for closure dimension

* Restyled by whitespace

* add mock in unit tests

* Restyled by clang-format

---------

Co-authored-by: Restyled.io <[email protected]>
* Check for common logging errors in Python code

* Enforce comprehension lists simplifications

* Restyled by autopep8

* Use legacy typing for tuple

* Add exception for LOG015

---------

Co-authored-by: Restyled.io <[email protected]>
…41702)

* Logic <-> Cluster -> merged into cluster for
NetworkCommissioningCluster.

* removing Logic.{h,cpp} files for Network Commissioning.

* Update src/app/clusters/network-commissioning/NetworkCommissioningCluster.cpp

Co-authored-by: Copilot <[email protected]>

* restyled.

* reference to general-commissioning-cluster removed.

---------

Co-authored-by: Andrei Litvin <[email protected]>
Co-authored-by: Copilot <[email protected]>
…uster commands (#41719)

* Change arm failsafe to be run on ep0

* Commissioning complete too

* fix breadcrumb
…ra-app within chip-cert-bins (#41785)

* Add libgstreamer plugin packages required for starting video pipeline

Signed-off-by: Charles Kim <[email protected]>

* Restyle

---------

Signed-off-by: Charles Kim <[email protected]>
Co-authored-by: Andrei Litvin <[email protected]>
* Closure button implementation

* Restyled by whitespace

* Restyled by clang-format

* Addressing PR comments

* addressing PR comments

* PR comments

* Restyled by whitespace

* Restyled by clang-format

* Updated logic to schedulelambda from apptask and remove unnessary chiplocks

* cleanup unnessary changes

* Restyled by whitespace

* Restyled by clang-format

---------

Co-authored-by: Restyled.io <[email protected]>
… (#41770)

* Add debug logs and fix

* remove debug logs

* PR comments

* Restyled by whitespace

* Restyled by clang-format

* review comments

* changes as per new analysis

---------

Co-authored-by: Restyled.io <[email protected]>
… subscription reporting. (#41806)

* Fix VideoStreamModify and SnapshotStreamModify to invoke callback for subscription reporting.

* Add test steps to scripts TC_AVSM_2_3.py and TC_AVSM_2_8.py to subscribe
to attributes and verify on report generation.

* Apply suggestions from code review

Co-authored-by: Copilot <[email protected]>

* Fix python linter errors.

* Fix wasModified logic check for optional OSD and WMark.

* Apply suggestions from code review

Co-authored-by: Abdul Samad <[email protected]>

---------

Co-authored-by: Copilot <[email protected]>
Co-authored-by: Abdul Samad <[email protected]>
@silabs-matter-ci-bot silabs-matter-ci-bot bot requested a review from a team as a code owner November 7, 2025 00:26
@silabs-matter-ci-bot silabs-matter-ci-bot bot added changing-git-submodules-on-purpose sl-require-admin-action This TAG indicates that an admin must take action. labels Nov 7, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

The check for sl-require-admin-action label CI failure for this job is normal. An admin must do the merge.

yufengwangca and others added 9 commits November 7, 2025 01:58
…729)

* [TC-AVSM-2.10] Fix Camera default resultion for CaptureSnapshot

* Update comments
* Add `CommandTesting` infrastructure

Add new testing utilities in src/app/clusters/testing/:
- CommandTesting.h: Header with MockCommandHandler and InvokeOperation classes.
- CommandTesting.cpp: Implementation of non-template methods for
  MockCommandHandler.

Refactoring:
- Refactored TestDiagnosticLogsCluster to use Testing::MockCommandHandler directly
- Updated TestPushAVStreamTransportCluster to inherit from Testing::MockCommandHandler while preserving specialized features

* Restyled by whitespace

* Add status storage to `MockCommandHandler` for testing

- Refactor MockCommandHandler: integrate status handling into FallibleAddStatus.
- Update TestPushAVStreamTransportCluster.

* Restyled by clang-format

* Refactor `TestPushAVStreamTransportCluster`

- Remove unused overrides

* Replace single response with response array in `MockCommandHandler`

- Replace single ResponseRecord with std::vector<ResponseRecord> for multiple responses
- Add methods to work with response arrays and specific responses by index
- Update existing methods to work with first response for backward compatibility
- Simplify test classes by removing duplicate functionality
- Add explicit response count assertions in tests

* Update src/app/clusters/testing/CommandTesting.h

Co-authored-by: Copilot <[email protected]>

* Update src/app/clusters/testing/CommandTesting.h

Co-authored-by: Copilot <[email protected]>

* Restyled by clang-format

* Restyled by gn

* Remove unused header file TestReadWriteAttribute.h from testing source set in BUILD.gn

* Update src/app/clusters/testing/CommandTesting.cpp

Co-authored-by: Copilot <[email protected]>

* Remove unnecessary comments.

* Refactor `InvokeOperation::Invoke` to use generated request structs

- Remove variadic template and conditional encoding branches.
- Update Invoke to accept a single RequestType object for encoding.
- Maintain existing TLV buffer and reader setup for invocation.

* Restyled by clang-format

* Refactor `InvokeOperation` class in `CommandTesting.h`

* Update src/app/clusters/diagnostic-logs-server/tests/TestDiagnosticLogsCluster.cpp

Co-authored-by: Copilot <[email protected]>

* Update src/app/clusters/diagnostic-logs-server/tests/TestDiagnosticLogsCluster.cpp

Co-authored-by: Copilot <[email protected]>

* Update src/app/clusters/diagnostic-logs-server/tests/TestDiagnosticLogsCluster.cpp

Co-authored-by: Copilot <[email protected]>

* Update src/app/clusters/diagnostic-logs-server/tests/TestDiagnosticLogsCluster.cpp

Co-authored-by: Copilot <[email protected]>

* Update src/app/clusters/diagnostic-logs-server/tests/TestDiagnosticLogsCluster.cpp

Co-authored-by: Copilot <[email protected]>

* Update src/app/clusters/diagnostic-logs-server/tests/TestDiagnosticLogsCluster.cpp

Co-authored-by: Copilot <[email protected]>

* Refactor `CommandTesting.h`

* Update src/app/clusters/testing/CommandTesting.h

Co-authored-by: Copilot <[email protected]>

* Refactor `ClusterTester` class to enhance command invocation capabilities and improve attribute read/write operations.
Removed the `InvokeOperation` class from `CommandTesting.h` as its functionality has been integrated into `ClusterTester`. Added new methods for command invocation and response handling.

* Restyled by clang-format

* Return helper function `InvokeCommand` to `ClusterTester`

* Restyled by clang-format

* Update src/app/clusters/testing/CommandTesting.h

Co-authored-by: Copilot <[email protected]>

* Refactoring of the `InvokeCommand` helper method:

- Replaced `InvokeCommand` with an overloaded `Invoke` method to simplify the code and enable invocation for basic tests without passing a `RequestType`.
- Refactored existing tests that use `InvokeCommand`.

* Restyled by clang-format

* Refactor `TestGroupcastCluster` to use `MockCommandHandler` from `CommandTesting`.
- Removed legacy `MockCommandHandler` implementation and adjusted test cases accordingly.

* Updated comments in `CommandTesting.h` and removed `InvokeCommand` method

* Restyled by clang-format

* Refactor `ClusterTester` class to enhance command invocation.

* Update `ClusterTester` to use member buffer for TLV operations.

* Restyled by clang-format

* Updated comments in `ClusterTester.h`

* Enhance `ClusterTester` constructor to initialize `mHandler` and clear responses/statuses in `InvokeOperation` methods.

* Remove unused `mSubjectDescriptor` member from `TestPushAVStreamTransportCluster`.

* Restyled by clang-format

* Enhance `Invoke` methods in `ClusterTester` to include `[[nodiscard]]` attribute for better compiler warnings on unused return values.

* Restyled by clang-format

* Make `mHandler` a value in `ClusterTester` remove `unique_ptr` overloads

* Update `PacketBufferHandle` initialization to use maximum size constant

* Refactor `Invoke` method in `ClusterTester` to use `ReturnErrorOnFailure` for error handling

* Restyled by clang-format

* Simplify `ClusterTester` `InvokeOperation` API and clarify intent
- Remove InvokeOperation(EndpointId, ClusterId, CommandId) overload
- Keep single overload taking ConcreteCommandPath
- Update comment to state purpose

* Restyled by clang-format

* Update `MockCommandHandler::AddResponseData` to use a fixed buffer size of 1024 for packet handling

* Update `MockCommandHandler::AddResponseData` to use `kMaxAppMessageLen` for packet buffer size

* Refactor `ClusterTester` to simplify command invocation methods and update `TestJoinGroupCommand` to use the new API.

* Restyled by clang-format

* Remove unused `ClusterTester` methods `GetHandler()` and `DecodeResponse`

* Refactor  to remove  namespace for types

* Restyled by clang-format

* Remove unused `GetRequest()` method from `ClusterTester`

* Restyled by clang-format

* Remove `Invoke` useless methods and Refactor `JoinGroup` command test in `TestGroupcastCluster`

* Restyled by clang-format

* Change `ReturnErrorOnFailure` to `VerifyOrReturnValue`

* Refactor `ClusterTester` to simplify command invocation and update `TestJoinGroupCommand` to use the new invocation method.

* Restyled by clang-format

* Update src/app/clusters/testing/CommandTesting.cpp

Co-authored-by: Andrei Litvin <[email protected]>

* Remove unused member variables from `ClusterTester.h`

* Replace `push_back` to `emplace_back`

* Rename `CommandTesting` to `MockCommandHandler`

* Update `BUILD.gn`

* Restyled by clang-format

* Update src/app/clusters/testing/ClusterTester.h

Co-authored-by: Copilot <[email protected]>

* Update src/app/clusters/diagnostic-logs-server/tests/TestDiagnosticLogsCluster.cpp

Co-authored-by: Copilot <[email protected]>

* Update src/app/clusters/groupcast/tests/TestGroupcastCluster.cpp

Co-authored-by: Copilot <[email protected]>

* Update src/app/clusters/push-av-stream-transport-server/tests/TestPushAVStreamTransportCluster.cpp

Co-authored-by: Copilot <[email protected]>

* Update src/app/clusters/testing/MockCommandHandler.cpp

Co-authored-by: Copilot <[email protected]>

* Refactor `MockCommandHandler`

* Fix android compile error (#41761)

* Refactor `MockCommandHandler` - updating fabric index initialization for test usage.

* Refactoring of the code

- Removal of method GetLastStatus
- Add comment to clarify commandId usage in ResponseRecord struct
- Fix JoinGroupCommand test to use value() for status retrieval

* Enhance `ClusterTester` and `JoinGroupCommand` test

- Introduced a MockCommandHandler in the JoinGroupCommand test to facilitate command invocation.
- Updated the Invoke method in ClusterTester to support command execution with a handler, improving test flexibility.
- Adjusted the JoinGroupCommand test to utilize the new Invoke method, ensuring proper status retrieval and response handling.

* Restyled by clang-format

* Restyled by clang-format

* Enhance documentation in `ClusterTester.h` to clarify usage of Invoke methods.

* Update src/app/clusters/testing/ClusterTester.h

Co-authored-by: Elen777300 <[email protected]>

* Restyled by clang-format

* Handle decode failure in `ClusterTester` Invoke by setting status and logging

* Refactor `ClusterTester` to improve command invocation and response handling

- Updated `Invoke` method in `ClusterTester` to handle response decoding more robustly, including support for `NullObjectType`.
- Enhanced test for `JoinGroupCommand` to utilize the new invocation method.
- Added necessary includes for `NullObjectType` in relevant files.

* Restyled by clang-format

* Fix `IsSuccess()` to handle `NullObjectType` response correctly

* Restyled by whitespace

* Restyled by clang-format

---------

Co-authored-by: Restyled.io <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Andrei Litvin <[email protected]>
Co-authored-by: Yufeng Wang <[email protected]>
Co-authored-by: Elen777300 <[email protected]>
Removed step to add jni entry for new cluster. No need to add entries for new cluster BUILD.gn
* Relocate TimerDelegate and TimerDelegateTest

* typo fix

* Restyled by clang-format

* not needed dependency for now

* move TimerDelegate base and TimerDelegateMock to lib/support/

* cleanup

* update includes

* Restyled by clang-format

* update namespace and header

* Restyled by clang-format

* update header include in examples

* update header include in examples

* File rename and source_set split

* Restyled by clang-format

* Update src/lib/support/TimerDelegate.h

Co-authored-by: Sergio Soares <[email protected]>

---------

Co-authored-by: Restyled.io <[email protected]>
Co-authored-by: Sergio Soares <[email protected]>
…dMessageHandler (#41536)

* Save the SendInit handler, re-register it during shutdown, make it so Shutdown is actually called.

* Moved implementation to Shutdown, simplifying

* Restyled fixes

* Init saves the handler so factory is not needed during shutdown

* Style and comment fixes

* Comment change
* closure LCD UI

* Add attribute reporting updates

* PR comments

* PR comments

* Restyled by whitespace

* Restyled by clang-format

* PR Comments

* Addressing PR comments and reducing code bloats

* remove non platfrom changes

* Restyled by whitespace

* Restyled by clang-format

* PR comments

* PR comments

* PR comments

* Restyled by clang-format

* PR comments

---------

Co-authored-by: Restyled.io <[email protected]>
* RVCRunMode RVCCleanMode align XMLs with spec

* XML regen
…ds change attributes (#41837)

* [PushAV] Add call to report generation on SetTransportStatus and
ModifyPushTransport.

* [TLSCert and TLSClient Management] Add attribute change reporting when
commands update the corresponding list attributes.
… (#41928)

* remove the coupling of InteractionModelEngine in DefaultTimerDelegate

* update dependency

* Restyled by clang-format

* add missing header

* add timer-delegate-default to app dependency

* relocate DefaultTimerDelegate to platform

* cleanup

* Restyled by clang-format

* update based on master change

---------

Co-authored-by: Restyled.io <[email protected]>
wqx6 and others added 17 commits November 12, 2025 17:27
* esp32: remove idf requirements from requirements.all.txt

* remove esp-idf-kconfig installing from chef workflow
* Batch #13 nodiscard errors: setup_payload, tools

* Update src/tools/spake2p/spake2p.cpp

Co-authored-by: Arkadiusz Bokowy <[email protected]>

---------

Co-authored-by: Arkadiusz Bokowy <[email protected]>
* Batch #6 nodiscard errors: controller code

* Restyle

* Fix CI

* Fix CI: unused var because ChipLogProgress is no-op

* Update src/controller/webrtc/WebRTCTransportProviderClient.cpp

Co-authored-by: Boris Zbarsky <[email protected]>

* Apply review suggestions

* Restyle

---------

Co-authored-by: Boris Zbarsky <[email protected]>
* Batch #7 nodiscard errors: access, ble, credentials

* Clang tidy
* Batch #12 nodiscard errors: protocols

* Review suggestions & fix CI

* Address review suggestions

* Fix clang tidy

* Revert conversion from macro -> template as pw_unit_test does not support GoogleTest << and does not output template param
…failures (#41924)

* Add consistent behavior on register failures in server cluster interface
registry.

* Fix typo and add comment.

* Update src/app/server-cluster/tests/TestServerClusterInterfaceRegistry.cpp

Co-authored-by: Boris Zbarsky <[email protected]>

* Add some documentation on error codes from SetContext.

* Update src/app/server-cluster/tests/TestServerClusterInterfaceRegistry.cpp

Co-authored-by: Copilot <[email protected]>

* Update src/app/server-cluster/ServerClusterInterfaceRegistry.cpp

Co-authored-by: Copilot <[email protected]>

* Update src/app/server-cluster/tests/TestServerClusterInterfaceRegistry.cpp

Co-authored-by: Copilot <[email protected]>

* Update src/app/server-cluster/ServerClusterInterfaceRegistry.cpp

Co-authored-by: Copilot <[email protected]>

---------

Co-authored-by: Boris Zbarsky <[email protected]>
Co-authored-by: Copilot <[email protected]>
* Update to use namespace.so for linux REPL runs.

* Update .github/workflows/tests.yaml

Co-authored-by: Copilot <[email protected]>

---------

Co-authored-by: Copilot <[email protected]>
* Decouple time synchronization cluster part 2

* Restyled by clang-format

* Restyled by gn

* Remove unnecessary comments in CodegenIntegration

* Code update

* Restyled by clang-format

* Decouple time synchronization cluster part 2

* Restyled by clang-format

* Restyled by gn

* Remove unnecessary comments in CodegenIntegration

* Code update

* Restyled by clang-format

* Address review comments

* Restyled by clang-format

* Code update

* Restyled by clang-format

* Code update

* Restyled by clang-format

* Code update

* Restyled by clang-format

* Make some functions inline

* Restyled by clang-format

* Address review comments

* Restyled by clang-format

* Code update

* Restyled by clang-format

* Code update

* Restyled by clang-format

* Address review comments

* Restyled by clang-format

* Restyled by gn

* Move the global delegate to CodegenIntegration

* Restyled by clang-format

* Code update

* Restyled by clang-format

* Address review comments

* Restyled by clang-format

---------

Co-authored-by: Restyled.io <[email protected]>
* [Telink] Increase stack size.

* [Telink] Increase IDLE stack size

---------

Co-authored-by: Alex Tsitsiura <[email protected]>
…mware processor (#41944)

Using function local static state variables such as "ulCrtAddr" and
"ulEraseLen" in the OTAFirmwareProcessor::ProcessInternal method causes
issues when OTA transfers are interrupted and then resumed because the
function local variables cannot be accessed (and reset) by other class
methods (such as OTAFirmwareProcessor::Clear or
OTAFirmwareProcessor::AbortAction).

The use of function local state variables was dropped and the SDK
wireless framework OTA support layer is now in charge of managing block
addresses and erases.


Reviewed-by: Dina Benamar <[email protected]>

Signed-off-by: Marian Chereji <[email protected]>
* Cleanup rs911x macros and file names

* Addressed review comments

* Renamed ble files folder to siwx917_ble

* Restyled by prettier-markdown

* Added matter_support pointer

* Removed rs911x references and moved blefiles

* Resolved BUILD.gn issues

* Update src/platform/silabs/SiWx/ble/sl_si91x_ble_init.h

Co-authored-by: Copilot <[email protected]>

* Remove wiseconnect-wifi-bt-sdk submodule

* Cleaned 917 related variable names in gni

* Renamed rs9117.gni to args.gni

* Restyled by whitespace

* Restyled by gn

* Restyled by prettier-markdown

* Fixed restyling error

* Restyled by clang-format

* Added RSI_BLE_MAX_NBR_PERIPHERALS same as wifi sdk

* Created build.gn for SiWx NCP files

* Deleted args.gni and resolved review comments

* Restyled by gn

* Update src/platform/silabs/SiWx/ble/BUILD.gn

Co-authored-by: Copilot <[email protected]>

* Resolved review comments

* Restyled by gn

* Resolved review comments

* Restyled by gn

* Added SiWx BLE defines in public_configs and consumed in efr_sdk and siwx917_sdk gni files

* Restyled by gn

* Added chip_enable_ble_siwx as true for SiWx917 SoC

* Restyled by gn

* Resolved review comments

* Added matter_support pointer

* Restyled by gn

---------

Co-authored-by: Restyled.io <[email protected]>
Co-authored-by: Copilot <[email protected]>
…er` (#41291)

* Add unit tests for `TimeSnapshot` command in `GeneralDiagnosticsCluster`

- Introduced a mock CommandHandler and a NullProvider for testing.
- Implemented tests for TimeSnapshot command, including scenarios with and without POSIX time support.
- Verified response values and ensured correct handling of command invocations.
- Added helper methods for setting up TLV readers and buffers for command testing.

* Refactor `TestGeneralDiagnosticsCluster` to improve command invocation testing

- Moved command invocation logic into a helper method  for better code reuse and clarity.

* Handle status responses in `ClusterTester` when `InvokeCommand` returns `nullopt`

When InvokeCommand returns nullopt, check for both data responses and
status responses in the mock handler. Previously only data responses
were checked. Now if a status response exists, use the last status
from the handler.

* Handle status responses and unexpected states in ClusterTester

When InvokeCommand returns nullopt, check for both data responses
and status responses. If a status response exists, use the last
status from the handler. Also handle the unexpected case where
neither response nor status is provided by setting an error status
and logging a warning.

* Refactor `TestGeneralDiagnosticsCluster` to improve command invocation and response handling

- Moved the `InvokeTimeSnapshotAndGetResponse` method outside the test struct for better organization and clarity.
- Simplified the command invocation process by ensuring `ClusterTester` is instantiated within each test case.
- Enhanced readability by removing redundant code and ensuring consistent error handling during command execution.

* Refactor `InvokeTimeSnapshotAndGetResponse` to use `ClusterTester` directly in tests

- Updated the `InvokeTimeSnapshotAndGetResponse` function to accept `ClusterTester` instead of `GeneralDiagnosticsCluster` for improved clarity and consistency in test cases.
- Adjusted all relevant test cases to reflect this change, ensuring proper command invocation and response handling.

* Enhance error handling in `InvokeTimeSnapshotAndGetResponse` for improved robustness

- Updated the InvokeTimeSnapshotAndGetResponse function to return appropriate error codes for various failure scenarios, ensuring better error handling during command execution.
- Adjusted test cases to assert the return value of the function, enhancing the reliability of the tests and ensuring that errors are properly captured and reported.

* Fix test helper to properly propagate errors

Changed InvokeTimeSnapshotAndGetResponse() to return CHIP_ERROR instead
of void. ASSERT macros inside void helper functions only return from the
helper, not from the test itself, which can lead to tests continuing with
uninitialized data and undefined behavior.
harimau-qirex and others added 9 commits November 14, 2025 03:30
This commit simplifying the wake event implementation so that it doesn't
depend on the socket related APIs provided by the select loop. This
allows disabling sockets API on platforms that supports select() but no
POSIX socket APIs.
* [WebRTCR] Enable controller to manage multiple cameras

* Address review comments

* Explicitly initialize testSession.fabricIndex
* Test cross-workflow sharing of REPL build logic.

* Update logic: fix typo in artifact name.

* Switch run test instance to namespace.so

* Fix up the paths that are being uploaded.

* Artifacts do not need a unique name: they are for sharing across jobs in
the same workflow.

* Fix profile name

* Add temporary debug capability.

* Add andy31415 as an authorized user for breakpoints.

* Try to fix up artifacts: smaller and proper permissions.

* Fix the strip path

* Update/fix strip command.

* Clean up tar creation (less verbose).
Things seem to work now MUCH better.

* Avoid extra work for jupyter notebooks.

* Fix jupyter folder path.

* Make TSAN suppression file absolute

* use GITHUB_WORKSPACE since github.workspace seems to potentially point
to other places.

* use GITHUB_WORKSPACE since that is what works in containers according to .github/workflows/java-tests.yaml

* remove objdir-clone logic generally

* Try to fix up TSAN and CCACHE options

* Remove a tsan option that will be overriden

* Fixes

* Fix path for suppressions again (trailing quote)

* Update .github/workflows/tests.yaml

Co-authored-by: Copilot <[email protected]>

* Try to fix ASR running out of space.

* Update .github/workflows/tests.yaml

Co-authored-by: Copilot <[email protected]>

* Fix title typo

* Try to avoid "artifact already exists" error.

* Speed up archiving, add some debug option for run (figure out why we
fail on suppression file)

* Look to undo the suppression path error.

* Remove breakpoint

---------

Co-authored-by: Copilot <[email protected]>
Co-authored-by: Andrei Litvin <[email protected]>
…n between (#41981)

* Update app builds for linux tests:

- split builds into 2 app builds and one python build
- remove out directory between app builds
- remove the "retry without cache" on master: we do not have space to do
  a full rebuild

* Fix a bit of id duplication and continue logic.

---------

Co-authored-by: Andrei Litvin <[email protected]>
* Do RM-rf out for every build in linux standalone builds.

* Remove extra clean that is redundant.
* [Telink] Add device firmware update over BLE

* [Telink] Make DFU over BLE SMP independent from Matter OTA build

* [Telink] Add dfu-usb target build for CI usage

* [Telink] Remove redundant code for DFU over BLE SMP

* [Telink] Fix lock with dfu build and
add draft implementation of header processing
with DFU over BLE SMP

* [Telink] Add footer validation for DFU over SMP

* [Telink] Fix DFU over SMP for zephyr v3.3

* [Telink] Add image version verification for DFU over SMP

* [Telink] Add image generation for DFU over BLE SMP

* [Telink] Fix build target

---------

Co-authored-by: Nazarii Bodnar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changing-git-submodules-on-purpose sl-require-admin-action This TAG indicates that an admin must take action.

Projects

None yet

Development

Successfully merging this pull request may close these issues.