-
Notifications
You must be signed in to change notification settings - Fork 4
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: project-chip/connectedhomeip#40366 (review)
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. TC-SU-2.2 python test connectedhomeip#40366 (comment) - Keep
max_interval_sec=0.5instead of20. TC-SU-2.2 python test connectedhomeip#40366 (comment)
- Keep
- Step 3:
- Keep
min_interval_sec=0.5instead of0. TC-SU-2.2 python test connectedhomeip#40366 (comment) - Keep
max_interval_sec=0.5instead of20. TC-SU-2.2 python test connectedhomeip#40366 (comment)
- Keep
- Step 4:
- Keep
min_interval_sec=0.5instead of0. TC-SU-2.2 python test connectedhomeip#40366 (comment) - Keep
max_interval_sec=0.5instead of20. TC-SU-2.2 python test connectedhomeip#40366 (comment)
- Keep
- Step 6 (events):
- Keep
min_interval_sec=1instead of0. TC-SU-2.2 python test connectedhomeip#40366 (comment) - Keep
max_interval_sec=1instead of20. TC-SU-2.2 python test connectedhomeip#40366 (comment)
- Keep
NOTE: Using short intervals now avoids missing initial states or events. The follow-up refactor will safely support longer intervals for ICD devices.
Metadata
Metadata
Assignees
Labels
Type
Projects
Status