Skip to content

Commit 792895d

Browse files
authoredSep 24, 2024
TC-OPSTATE-2.5: Force expire session on reboot (project-chip#35712)
If the DUT has rebooted, the TH will need to establish the session again, which means expiring the current. In a real product, a fail on the case would trigger a retry, but it's not clear that we want that in the TH because it can paper over problems where the DUT is crashing. This is an intentional reboot, wihch is different. Probably not caught in CI / QA because the CI app doesn't actually reboot.
1 parent 222338e commit 792895d

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed
 

‎src/python_testing/TC_OpstateCommon.py

+2-1
Original file line numberDiff line numberDiff line change
@@ -1112,10 +1112,11 @@ async def TEST_TC_OPSTATE_BASE_2_5(self, endpoint=1):
11121112

11131113
# STEP 11: Restart DUT
11141114
self.step(11)
1115-
# In CI environment, the STOP coommand (step 8) already resets the variables. Only ask for
1115+
# In CI environment, the STOP command (step 8) already resets the variables. Only ask for
11161116
# reboot outside CI environment.
11171117
if not self.is_ci:
11181118
self.wait_for_user_input(prompt_msg="Restart DUT. Press Enter when ready.\n")
1119+
self.default_controller.ExpireSessions(self.dut_node_id)
11191120

11201121
# STEP 12: TH waits for {PIXIT.WAITTIME.REBOOT}
11211122
self.step(12)

0 commit comments

Comments
 (0)
Please sign in to comment.