-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Permalink
Choose a base ref
{{ refName }}
default
Choose a head ref
{{ refName }}
default
Comparing changes
Choose two branches to see what’s changed or to start a new pull request.
If you need to, you can also or
learn more about diff comparisons.
Open a pull request
Create a new pull request by comparing changes across two branches. If you need to, you can also .
Learn more about diff comparisons here.
base repository: project-chip/connectedhomeip
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
...
head repository: project-chip/connectedhomeip
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v1.4.2-branch
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
- 3 commits
- 70 files changed
- 8 contributors
Commits on Jul 3, 2025
-
[v1.4.2-branch] Update to sve tests (#39753)
* Update src/app/tests/suites and src/python_testing to SVE. This pulls the test scripts to match SVE rather than be based on master. * Remove files added since SVE test harness creation * These changes should NOT be pulled * These changes should NOT be pulled * These changes should NOT be pulled * These changes should NOT be pulled * These changes should NOT be pulled * These changes should NOT be pulled * These changes should NOT be pulled * These changes should NOT be pulled * These changes should NOT be pulled
Configuration menu - View commit details
-
Copy full SHA for 98483e5 - Browse repository at this point
Copy the full SHA 98483e5View commit details -
Cherrypick #39597 into v1.4.2-branch (#39766)
* Convert and log complex exceptions as simple Exception with stacktrace in on_fail (#39597) * Simplify exception on_fail * Improve code comment * restyled * Convert complex exceptions to simple Exception * Restyled * Added missing import traceback * Restyled * Undo changes in matter_testing.py and add __reduce__ in chip.exceptions at __init__.py --------- Co-authored-by: Andrei Litvin <andy314@gmail.com> * Code review --------- Co-authored-by: Andrei Litvin <andy314@gmail.com>
Configuration menu - View commit details
-
Copy full SHA for b89fa4c - Browse repository at this point
Copy the full SHA b89fa4cView commit details
Commits on Jul 8, 2025
-
* TC-CC-2.1: Fix checks for nullable attributes (#39566) * TC-CC-2.1: Fix nullables Fixes the null check for StartUpColorTemperatureMireds, add null checks for other nullable values. * Update src/python_testing/TC_CC_2_1.py Co-authored-by: Tennessee Carmel-Veilleux <tennessee.carmelveilleux@gmail.com> * number of primaries can also be null * minor fix to log --------- Co-authored-by: Tennessee Carmel-Veilleux <tennessee.carmelveilleux@gmail.com> * [Fix] Fix CADMIN_1_19 test step 3 fabric filtered (#39609) - Changed to make test step 3 unfiltered now as mentioned in issue #(636)[project-chip/certification-tool#636] * TC-SC-4.3: Fix test for no TXT record returned from DUT (#39623) * TC-SC-4.3: Fix test for no TXT record The spec is a BIT fuzzy in this area, but in my re-reading, a TXT record is not specifically mandated, only certain keys in certain situations (which would then require a TXT record). If there are no mandatory TXT keys (which can happen for operational) then the TXT record can be omitted and that's fine. * Apply suggestions from code review * linter aka here's a deeply unimportant thing I'm going to waste your time on * TC-IDM-4.2: indicate the DUT needs to be commissioned (#39638) Side note - yes, I know how terrible this is. It's now written into the TH and will take a while to fix. We're working on it. * Revert matter_testing back to tag * Move app pipe and mark_all_remaining_steps_skipped back to master * Revert "Adding helper function for commands running errors (#38799)" This reverts commit 1bf9a3e. * some mypy conformance was added after sve * move opstate common back to the app pipe style on master * Put back changes for TestClusters on master * Take TBRM tests from master see https://github.com/project-chip/connectedhomeip/pull/39555/files * Rename from skip_all_remaining_steps_skipped to mark_all_remaining_steps_skipped (#38729) * Updated name from skip_all_remaining_steps_skipped to mark_all_remaining_steps_skipped. Updated test cases using the old name to new one. * fix: missing method name replace. * fix: updated test cases * Update step start to skip at src/python_testing/TC_CNET_4_3.py Co-authored-by: Andrey Khodyrev <akhodyrev@griddynamics.com> * fix: updated docstring and method args type as the step method. * Added missing returns after marked as skip --------- Co-authored-by: Andrey Khodyrev <akhodyrev@griddynamics.com> * Move all camera tests back up to master These weren't run through SVE anyway, won't be selected by the 1.4.2 PICS and this spec is enough is flux that there were changes to the cluster definitions between SVE and this branch that are affecting the test. * Proposal to remove --app-pid and --app-pipe-prefix for out of band commands. (#38778) * Updated linux apps to read the FIFO without using the app_pid. Now uses a value from a env variable or static value. This will be usefull on tests to use only -app_pipe <app_prefix>+ instead of two paramters. * Update write_to_app_pipe to only use app_pipe argument which is the full path to pipe file <prefix>_<id>, raises FileNotFound error if the pipe file is not found from start. Removed arguments app_pid and app_pipe_prefix as they are not needed anymore. * Removed arguments --app-pid and --app-pipe-prefix * Updated app-pipe_prefix to app-pipe with APP_ID in test cases * Updated OPSTATE testcases and OVENOPSTATE test cases to use --app-pippe with id instead of PID of the app. * Fix not found when the parameter is not available * Removed unused ci actions * Added APP_IDs for workflows. Currently those are static values for CI. * Remove automatically appended --app-pid on execution. Now it dependes on the CI parameters provided by the user. * Fix typo and restyle * Restyled by whitespace * Restyled by clang-format * fix: Solved typo for TC_RVCRUM_2_1. Added quotes for APP_ID. * fix: Fixed id for bridge app. Added --app-pipe for BRBINFO. Added missing app-pipe for TC_SWITCH * Typo in variable name * Typo RCV_CHIP * Missing file exists check when the app_pipe is send from the test case. * fix: code review fix, optimize for string. * Removed vars from yaml file * Removed APP_ID from Test case CI arguments from --app-pipe * Removed env variable for custom app-id. Replaced for app argument --app-id * Restyled by whitespace * Restore change. * fix: Fixed from comments, app_id set to empty string instead of nullptr, this will avoid uneeded check on each of the apps. * fix: removed comma * Added app-pipe from scripts args to app-args if present. This is to avoid having the fifo path in scripts args and app-args and avoid posible run errors. * Added custom fifo path for each one of the test cases. * Removed prefix. Updated app-id to app-pipe. Now app pipe gets the full path of the fifo path. If not defined avoid sChipNamedPipeCommands.Start() * Restyled by clang-format * Updated empty string compare * Added --app-pipe into app-args. Removed auto append --app-pipe this will need repeated value but will not force same fifo and allow more scenarios. * Updated tc_soil_2_2 with new fifo format. --------- Co-authored-by: Restyled.io <commits@restyled.io> * add chime to do not run list Good grief why are you fighing me so hard, CI? Chime tests aren't even part of 1.4.2. If this continues, I'm going to delete the lot of you. --------- Co-authored-by: Tennessee Carmel-Veilleux <tennessee.carmelveilleux@gmail.com> Co-authored-by: Jake Ororke <jororke@csa-iot.org> Co-authored-by: jtrejoespinoza-grid <juanerasmo@google.com> Co-authored-by: Andrey Khodyrev <akhodyrev@griddynamics.com> Co-authored-by: Restyled.io <commits@restyled.io>
Configuration menu - View commit details
-
Copy full SHA for f36e159 - Browse repository at this point
Copy the full SHA f36e159View commit details
Loading
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff master...v1.4.2-branch