Scripts for feature "SDL behavior in case of LOW_VOLTAGE event" #1953
Conversation
GetmanetsIrina
force-pushed
the
feature/low_voltage
branch
from
c1c203b to
c317f3b
Compare
dboltovskyi
force-pushed
the
feature/low_voltage
branch
3 times, most recently
from
5bc9206 to
19f52dd
Compare
dboltovskyi
force-pushed
the
feature/low_voltage
branch
from
19f52dd to
c801cea
Compare
| -- 3) All apps have some data that can be resumed | ||
| -- 4) SDL get LOW_VOLTAGE signal | ||
| -- 5) And then SDL get WAKE_UP signal | ||
| -- 6) All apps are registered with the same hashID |
There was a problem hiding this comment.
does All apps are registered with the same hashID mean there is only one hashid and all the apps use the same id? probably not.
I think each app shall have its own hashid, when apps get disconnected and reconnect, each app uses the same id of their own
There was a problem hiding this comment.
@yang1070 Correct, each application has it's own hashId
There was a problem hiding this comment.
for clarification, reword to All apps are registered with their corresponding hashIDs
| -- 4) SDL get LOW_VOLTAGE signal | ||
| -- 5) And then SDL get IGNITION_OFF signal | ||
| -- 6) And then SDL is started | ||
| -- 7) All apps are registered with the same hashID |
There was a problem hiding this comment.
same question regarding All apps are registered with the same hashID
There was a problem hiding this comment.
@yang1070 Please see above
| -- 2. App has some persistent data that can be resumed | ||
| -- 2. Mobile app is disconnected during the time frame of 30 sec (inclusive) before "SDL_LOW_VOLTAGE" unix signal from HMI | ||
| -- 3. Mobile app registers with the same hashID during 30 sec. after "WAKE_UP" unix signal in the same ignition cycle | ||
| -- 4. there is application currently in LIMITED |
There was a problem hiding this comment.
with mobile projection, there can be multiple apps in LIMITED
There was a problem hiding this comment.
@yang1070 This was an original requirement for resumption functionality that is transferred to Low Voltage
| -- 2. App has some persistent data that can be resumed | ||
| -- 3. Mobile app is disconnected during the time frame of 30 sec (inclusive) before "SDL_LOW_VOLTAGE" unix signal from HMI | ||
| -- 4. "IGNITION_OFF" unix signal from HMI | ||
| -- 5. IGNITION_ON is performed |
There was a problem hiding this comment.
@yang1070 IGNITION_ONmeans new Ignition cycle is started
|
@theresalech Ford has reviewed, tested and approved this PR. It is ready for Livio to review. |
| -- 3) All apps have some data that can be resumed | ||
| -- 4) SDL get LOW_VOLTAGE signal | ||
| -- 5) And then SDL get WAKE_UP signal | ||
| -- 6) All apps are registered with the same hashID |
There was a problem hiding this comment.
for clarification, reword to All apps are registered with their corresponding hashIDs
| -- 4) And then SDL get WAKE_UP signal | ||
| -- SDL does: | ||
| -- 1) Resume it’s work successfully (as for Resumption) | ||
| -- 2) Discard processing of RPC |
There was a problem hiding this comment.
Based on this description, you should check that if the HMI sends a response to UI.Show after WAKE_UP, that it is ignored and a response is not sent to the mobile app.
| end | ||
|
|
||
| local function checkResumptionHMILevel() | ||
| common.getMobileSession(1):ExpectNotification("OnHMIStatus", |
|
@jacobkeeler , please find update in 820e2ff . |
| common.getMobileSession():ExpectAny():Times(0) | ||
| common.getHMIConnection():ExpectAny():Times(0) | ||
| common.sendIgnitionOffSignal() | ||
| common.wait(2000) |
There was a problem hiding this comment.
This test fails on this step, but no information is provided, it might be just considering the shutdown of Core to be a failure.
There was a problem hiding this comment.
@jacobkeeler Could you check please that you use latest version of ATF (smartdevicelink/sdl_atf@900f936)
A fix was implemented in ATF a couple of weeks ago to handle this case in script.
ATF Test Scripts to check
TBDThis PR is ready for review.
Summary
Script for feature "SDL behavior in case of LOW_VOLTAGE event".
Tests #2233
ATF version
Develop branch
CLA