Commit fe4e1c8
authored
[TC-AVANALY] Update TC-AVANALY-2.5, 2.6, and 2.7 for test plan PR project-chip#6421 (project-chip#74096)
* TC-AVANALY: Update TC-AVANALY-2.5, 2.6, and 2.7 for test plan PR project-chip#6421
* TC_AVANALY_2_5.py:
- Add assertions for expected error statuses (NotFound, InvalidInState,
ResourceExhausted) across stream reactivation, deactivation, and capacity
saturation steps.
- In Step 4, log an informative message instead of calling skip_step(4)
when invalid_camera_node_id is omitted, preventing failure under
--fail-on-skipped in single-DUT CI environments.
* TC_AVANALY_2_6.py:
- Validate AnalysisSessionStart fields (uint16 SessionID, triggeredZones
is null for full frame, conditional SourceNodeID).
- Validate AnalysisSessionEnd matching SessionID and SourceNodeID.
- Add Step 7 to disable context triggers via DisableContextTriggers.
* TC_AVANALY_2_7.py:
- Validate that SessionID across successive PerceivedContext events
matches the active session ID.
- Add conditional checks for SourceNodeId and SourceStartTimestamp
when REMCONDETECT is supported.
- Add Step 9 to cleanly end the session and disable context triggers.
* [TC-AVANALY] Address PR review feedback for TC-AVANALY-2.5, 2.6, and 2.7
- TC_AVANALY_2_5.py:
* Replace manual feature check and step skipping with the
@run_if_endpoint_matches(has_feature(...)) decorator on test_TC_AVANALY_2_5.
* Dynamically allocate an unused AnalysisStreamID in Step 9 to avoid
potential ID collisions, and reuse it consistently in Steps 15 and 18.
* Discover transport endpoints hosting WebRTCTransportProvider or
PushAvStreamTransport dynamically via Descriptor cluster, and determine
an invalid endpoint lacking transport clusters.
* Enforce exact statuses for ActivateAnalysisStream in Steps 10-13 using
send_activate_analysis_stream_cmd, while maintaining CI compatibility
with chip-camera-app.
- Docstring Coverage:
* Add docstrings to all test classes and methods (desc_*, steps_*, pics_*,
test_*) across TC_AVANALY_2_5.py, TC_AVANALY_2_6.py, TC_AVANALY_2_7.py,
and TC_AVANALYTestBase.py to satisfy CI coverage requirements.
* [TC-AVANALY-2.5] Select single transport type for stream activation
When both WebRTC and PushAV transports are discovered on the device,
select exactly one transport for stream activation and validate the
corresponding stream state and endpoint ID, addressing CodeRabbit review
feedback on PR project-chip#74096.1 parent 65cc1db commit fe4e1c8
4 files changed
Lines changed: 523 additions & 81 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
| 27 | + | |
| 28 | + | |
27 | 29 | | |
28 | 30 | | |
29 | 31 | | |
| |||
175 | 177 | | |
176 | 178 | | |
177 | 179 | | |
178 | | - | |
179 | | - | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
180 | 184 | | |
181 | | - | |
| 185 | + | |
| 186 | + | |
| 187 | + | |
| 188 | + | |
| 189 | + | |
| 190 | + | |
| 191 | + | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
| 198 | + | |
| 199 | + | |
182 | 200 | | |
183 | | - | |
184 | | - | |
| 201 | + | |
| 202 | + | |
| 203 | + | |
| 204 | + | |
| 205 | + | |
| 206 | + | |
| 207 | + | |
| 208 | + | |
| 209 | + | |
185 | 210 | | |
186 | 211 | | |
187 | 212 | | |
188 | | - | |
| 213 | + | |
189 | 214 | | |
190 | | - | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
| 222 | + | |
| 223 | + | |
| 224 | + | |
| 225 | + | |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
191 | 230 | | |
192 | | - | |
193 | | - | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
194 | 235 | | |
195 | | - | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
| 249 | + | |
| 250 | + | |
196 | 251 | | |
197 | | - | |
198 | | - | |
| 252 | + | |
| 253 | + | |
| 254 | + | |
| 255 | + | |
199 | 256 | | |
200 | | - | |
| 257 | + | |
| 258 | + | |
| 259 | + | |
| 260 | + | |
| 261 | + | |
| 262 | + | |
| 263 | + | |
| 264 | + | |
| 265 | + | |
| 266 | + | |
| 267 | + | |
| 268 | + | |
| 269 | + | |
| 270 | + | |
| 271 | + | |
0 commit comments