project-chip/connectedhomeip

Remove redundancy from CI execution bulrbs for python tests

開放

#38,665 建立於 2025年4月29日

 (0 則留言) (0 個反應) (1 位負責人)C++ (1,659 個分叉)batch import
CI/CD improvementsgood first issue

倉庫指標

星標
 (6,410 顆星)
PR 合併指標
 (PR 指標待抓取)

描述

CI test arguments are generally documented at https://project-chip.github.io/connectedhomeip-doc/testing/python.html#defining-the-ci-test-arguments

When looking through examples like https://github.com/project-chip/connectedhomeip/blob/master/src/python_testing/TC_CC_2_2.py#L23 , a lot of arguments are repeated throughout all tests. As test generally are copied & pasted, it makes the reviewr not focus on "what changes are relevant" and instead the app-specific parts are contained within the "noise" of other parameters (like discriminator, passcode, trace-to, kvs, commissioning-method, storage path are all the same almost always).

We should allow having only "important CI parts" in the test argument blurb and everything else could be a default (e.g. we already have a yaml file with all paths, we can add defaults in there).

There was some work started in #37494 however PR author in the mean time left the project. We need an updated implementation and have progress on this.

We can do this in steps:

  • support common arguments (including removal of defaults somehow
  • integrate this into the default environment
  • update all python tests to remove redundant bits

貢獻者指南