project-chip / project-chip/matter-test-scripts
[SU] Refactor TC-SU-2.2 to Handle Long Attribute/Event Reporting Intervals
@juandediosg is already working on this.
Since Dec 10, 2025.
- Dominant language
- Python
- Stars
- 19
- Forks
- 7
- Avg merge
- 3d 17h
- Merged PRs (30d)
- 3
Description
Summary
Following feedback from project-chip/connectedhomeip#40366, the logic in TC-SU-2.2 for tracking OTA attributes and events needs to be refactored to properly handle long min/max intervals for ICD devices.
Validation note: Also the feedback suggests waiting for the provider’s actual QueryImage command before validating subscriptions. If this logic is available now, it will be added to this task. If not, a follow-up task will track its implementation. Implementing this will allow tightening subscription timing and simplify debugging.
Feedback: https://github.com/project-chip/connectedhomeip/pull/40366#pullrequestreview-3562911530
Proposed follow-up:
- Refactor Steps 2, 3, and 4 in TC-SU-2.2 to support long reporting intervals using event-based validation on attributes.
- Refactor Steps 6 in TC-SU-2.2 to support long reporting intervals using event-based validation on events.
- Refactor step 7, the event subscription/wait logic to support long timeout intervals (e.g., 620 s) without losing intermediate events.
- Follow-up task: Implement waiting for provider’s actual QueryImage command before validating subscriptions.
Feedbacks
- Step 2:
- Keep
min_interval_sec=0.5instead of0. https://github.com/project-chip/connectedhomeip/pull/40366#discussion_r2607022733 - Keep
max_interval_sec=0.5instead of20. https://github.com/project-chip/connectedhomeip/pull/40366#discussion_r2607612019
- Keep
- Step 3:
- Keep
min_interval_sec=0.5instead of0. https://github.com/project-chip/connectedhomeip/pull/40366#discussion_r2607079266 - Keep
max_interval_sec=0.5instead of20. https://github.com/project-chip/connectedhomeip/pull/40366#discussion_r2607621729
- Keep
- Step 4:
- Keep
min_interval_sec=0.5instead of0. https://github.com/project-chip/connectedhomeip/pull/40366#discussion_r2607634576 - Keep
max_interval_sec=0.5instead of20. https://github.com/project-chip/connectedhomeip/pull/40366#discussion_r2608470439
- Keep
- Step 6 (events):
- Keep
min_interval_sec=1instead of0. https://github.com/project-chip/connectedhomeip/pull/40366#discussion_r2607173489 - Keep
max_interval_sec=1instead of20. https://github.com/project-chip/connectedhomeip/pull/40366#discussion_r2607174008
- Keep
NOTE: Using short intervals now avoids missing initial states or events. The follow-up refactor will safely support longer intervals for ICD devices.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.