Skip to content
Permalink

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
Choose a base ref
...
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
Choose a head ref
  • 3 commits
  • 70 files changed
  • 8 contributors

Commits on Jul 3, 2025

  1. [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
    andy31415 authored Jul 3, 2025
    Configuration menu
    Copy the full SHA
    98483e5 View commit details
    Browse the repository at this point in the history
  2. 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>
    rquidute and andy31415 authored Jul 3, 2025
    Configuration menu
    Copy the full SHA
    b89fa4c View commit details
    Browse the repository at this point in the history

Commits on Jul 8, 2025

  1. 1.4.2 cherry picks (#39842)

    * 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>
    6 people authored Jul 8, 2025
    Configuration menu
    Copy the full SHA
    f36e159 View commit details
    Browse the repository at this point in the history
Loading