Skip to content

Merge V2.15-develop into main - #318

Merged
oxesoft merged 164 commits into
mainfrom
merge/v2.15-develop-into-main
May 25, 2026
Merged

Merge V2.15-develop into main#318
oxesoft merged 164 commits into
mainfrom
merge/v2.15-develop-into-main

Conversation

@oxesoft

@oxesoft oxesoft commented May 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Notes on conflicts

  • main HEAD is a squash merge of release content that already exists on v2.15-develop as individual commits, so the true merge base is far back at Merge V2.11+fall2024 into main #174 (V2.11). A direct v2.15-develop -> main PR produced spurious conflicts.
  • This branch was built from origin/main, then v2.15-develop was merged in. The merge commit's tree is identical to v2.15-develop's tree, with both main HEAD (54ea96f) and v2.15-develop HEAD (2ac562d) preserved as parents.

Test plan

  • CI passes on the PR
  • Smoke test the Test Harness against a representative DUT
  • Verify v2.15+spring2026 version is reported correctly

Replaces #317.

rquidute and others added 30 commits October 22, 2024 21:11
* Updated package-dependency-list.txt

* Updated missing dependencies
* Reverting the BR_VARIANT option to the otbr start script

* Adding TODO to the script

* Adding missing comment char

* Updating Script Comment Year

Co-authored-by: Romulo Quidute Filho <116586593+rquidute@users.noreply.github.com>

* Adding switch parameters options to the script

---------

Co-authored-by: Romulo Quidute Filho <116586593+rquidute@users.noreply.github.com>
* replace erro with warning

* Update test_collections/matter/sdk_tests/support/python_testing/models/python_test_parser.py

Co-authored-by: Carolina Lopes <116589288+ccruzagralopes@users.noreply.github.com>

* black error fix

* pr review

---------

Co-authored-by: Carolina Lopes <116589288+ccruzagralopes@users.noreply.github.com>
* Fixing intermittent problem with Python's Base Manager

Creating the manager now only once in the SDK Container Init

* Refactoring and updating the unit tests with related mocks
* Updated python_testing references after sdk refactoring

* Removed no wanted file changes

* Updated matter_testing references

* Fixed line too long

* Fixed code - black

* Updated SDK SHA

* Removed commented code
* Isolate SDK docker pull script

* Changes after code review

---------

Co-authored-by: Romulo Quidute Filho <rquidute@apple.com>
* Isolate SDK docker pull script

* Rebasing

* Apply suggestions from code review

Co-authored-by: Carolina Lopes <116589288+ccruzagralopes@users.noreply.github.com>

* Move call tp update-pull-sdk-docker-image.sh from setup.sh to update-sample-apps.sh

* Changes after code review

* Fixed CI issues

* Code review fixes

* Fixed unit tests

* Fixed mypy

---------

Co-authored-by: Romulo Quidute Filho <116586593+rquidute@users.noreply.github.com>
Co-authored-by: Carolina Lopes <116589288+ccruzagralopes@users.noreply.github.com>
Co-authored-by: Romulo Quidute Filho <rquidute@apple.com>
* Implemented changes for custom python test

* code review

* Code review

* Code reviewd

* Fixed code violations

* Chanegs after code review

* Remove unused import
* Fixed some python tests not listed

* code review
* Implement support to export project config

* Fix mypy

* Changed the returned schema and service endpoint name

* Added import project config

* Apply suggestions from code review

Co-authored-by: Carolina Lopes <116589288+ccruzagralopes@users.noreply.github.com>

* code review

---------

Co-authored-by: Carolina Lopes <116589288+ccruzagralopes@users.noreply.github.com>
* Added support to negated PICS

* Fixed flake

* Code review. No consider negatives PICS

* Update app/pics_applicable_test_cases.py

* Code review
* Remove unused file

* Updated cspell
* Fix CI issues from v2.12 branch

* Fixed merge

* Fix test case

* Updated SDK_DOCKER_IMAGE url

* Updated SDK_DOCKER_IMAGE url
…ing (#173)

* Add support for configuring terms and conditions in device commissioning

This commit introduces a new `enhanced_setup_flow` configuration
property to the test harness backend, allowing device manufacturers to
define the terms and conditions (T&C) that must be acknowledged and set
during device commissioning. These values are critical for supporting
the Terms and Conditions Certification Test Suite.

Key changes include:
- Added the `EnhancedSetupFlowConfig` class to `DutConfig`, which allows
  specifying the T&C version and user response required during
  commissioning.
- Updated `generate_command_arguments` in `utils.py` to append the
  necessary arguments for T&C validation and the device's commissioning
  method.
- Adjusted model validation to ensure the `enhanced_setup_flow` field is
  processed correctly within `DutConfig`.

These changes ensure the test harness can enforce manufacturer-specific
T&C values during the commissioning process, aligning with the broader
requirements of T&C validation.

* * Ran ./scripts/format.sh

* Update test_collections/matter/test_environment_config.py

Co-authored-by: Romulo Quidute Filho <116586593+rquidute@users.noreply.github.com>

* Update app/user_prompt_support/uploaded_file_support.py

Co-authored-by: Romulo Quidute Filho <116586593+rquidute@users.noreply.github.com>

* Update test_collections/matter/sdk_tests/support/performance_tests/utils.py

Co-authored-by: Romulo Quidute Filho <116586593+rquidute@users.noreply.github.com>

* Update test_collections/matter/sdk_tests/support/python_testing/models/utils.py

Co-authored-by: Romulo Quidute Filho <116586593+rquidute@users.noreply.github.com>

* Update test_collections/matter/sdk_tests/support/python_testing/models/utils.py

Co-authored-by: Romulo Quidute Filho <116586593+rquidute@users.noreply.github.com>

* Update test_collections/matter/sdk_tests/support/python_testing/__init__.py

Co-authored-by: Romulo Quidute Filho <116586593+rquidute@users.noreply.github.com>

* Revert changes to test_collections/matter/sdk_tests/support/performance_tests/utils.py

* Restyle test_environment_config.py

* test: Add in-test-commissioning-method parameter for manual commissioning validation

Add new parameter to control commissioning flow during tests, allowing manual
validation of the commissioning process instead of auto-commissioning. This
enables test cases to verify terms and conditions acknowledgements during the
commissioning flow.

- Add in-test-commissioning-method parameter to test flags
- Support manual commissioning flow validation
- Enable testing of T&C acknowledgement scenarios

* Add new field "enhanced_setup_flow" to exception message

* Cherry-pick test_python_parser.py from origin/main

* Cherry-pick python_test_parser.py from origin/main

* Revert "Cherry-pick python_test_parser.py from origin/main"

This reverts commit 5d65ce5.

* Revert "Cherry-pick test_python_parser.py from origin/main"

This reverts commit 2cd99b7.

* Removed broken test `test_python_parser.py`

* Revert "Removed broken test `test_python_parser.py`"

This reverts commit 48c0e32.

---------

Co-authored-by: Romulo Quidute Filho <116586593+rquidute@users.noreply.github.com>
Updated Node.js installation source from setup_18.x to setup_20.x to
align with Ubuntu 22.04 base image requirements
* First implementation

* Implement prompt and copy files from/to contanier

* Unit tests

* Updated SKA SHA

* Unit test typo fixes

* Mypy flake8 fixes

* Mypy fixes

* Update test_collections/matter/sdk_tests/support/python_testing/models/utils.py

Co-authored-by: Carolina Lopes <116589288+ccruzagralopes@users.noreply.github.com>

* Update test_collections/matter/sdk_tests/support/utils.py

Co-authored-by: Carolina Lopes <116589288+ccruzagralopes@users.noreply.github.com>

* Code review

* Review fixes

* Remove unused import

---------

Co-authored-by: Carolina Lopes <116589288+ccruzagralopes@users.noreply.github.com>
* Add logs for project config

* Updated cspell.json

* Update app/test_engine/test_runner.py

Co-authored-by: hiltonlima <116589806+hiltonlima@users.noreply.github.com>

* Fixed unit test

* Unit test

* Unit test

---------

Co-authored-by: hiltonlima <116589806+hiltonlima@users.noreply.github.com>
rquidute and others added 16 commits February 24, 2026 11:19
…sses imported from additional Matter SDK modules. (#300)

* Fix python script parser

* Code review - gemini

* Code review - adjust endswith for TestBase in python script parser

* Code review - mypy fix
* Improved logic to load python test scripts

* Added test_python_test_initialization.py unit test file

* Update copyright year

* Code review - mypy/flake8

* Code review - mypy/flake8

* Code review - mypy/flake8

* Code review - flake8

* Update test_collections/matter/sdk_tests/support/python_testing/list_python_tests_classes.py

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* Update test_collections/matter/sdk_tests/support/python_testing/list_python_tests_classes.py

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* Code review - gemini code assist

* Code review

---------

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
* Fix json-arg not working

* Code review- Gemini Code assist

* Code review- Gemini Code assist

* Code review- Gemini Code assist

* Added more unit test in order to covert more scenarios
* Updating infrastructure to handle temporary PICS for Test Run Executions

* Fixing test_run_executions.py docstring

* Improving return expression in the test_run.py file

* Fixing the related unit tests

* Removing unnecessary import
…n their base class is defined in support_modules (#306)

* Consider using extra dir(SDK files) when discovering custom test cases

* Code review - Google code assist

* Fix black
* Remove commissioning prompt from no commissioning suite

* Fix mypy

* Code review
)

* Fix TC_DD_3_24 hang on user prompt timeout in _show_prompt_request.  When wait_for_user_input times out, response_str is None so nothing was sent to the test socket, leaving the SDK process blocked on input() forever.

* Update test_collections/matter/sdk_tests/support/python_testing/models/test_case.py

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* Code review

---------

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
CommissionDeviceTest has been moved from matter.testing.commissioning
to matter.testing.CommissioningPreTest in the Matter SDK. The old path
no longer exports this symbol, which causes --get-test-info to fail and
the Python test list to be empty in the UI.

Fixes #981
# Conflicts:
#	test_collections/matter/sdk_tests/support/performance_tests/models/test_suite.py

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces several enhancements and optimizations to the test harness, including performance test support with summary report generation, optimized Python test initialization, and real-time logging capabilities. The database schema and API were updated to handle execution-specific configurations and PICS overrides, while a new THREAD_MESHCOP pairing mode was added. Feedback from the review identifies several opportunities to improve code robustness, such as adding timeouts to HTTP requests, using value equality for status codes, URL-encoding parameters, and ensuring safe attribute access and explicit file encoding.

Comment thread app/api/api_v1/endpoints/test_run_executions.py
Comment thread app/api/api_v1/endpoints/test_run_executions.py
Comment thread app/api/api_v1/endpoints/test_run_executions.py
Comment thread app/crud/crud_test_run_execution.py
@oxesoft
oxesoft merged commit 5c1b9ac into main May 25, 2026
7 of 8 checks passed
@oxesoft
oxesoft deleted the merge/v2.15-develop-into-main branch May 25, 2026 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants