Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SDL does not respond info message in case GENERIC_ERROR watchdog timeout from HM #1012

Closed
6 of 9 tasks
KhrystynaDubovyk opened this issue Nov 7, 2016 · 2 comments
Closed
6 of 9 tasks
Labels
Projects

Comments

@KhrystynaDubovyk
Copy link

KhrystynaDubovyk commented Nov 7, 2016

Occurrence

Always.

Preconditions

  1. Move IsReady_Template folder into sdl_atf\user_modules.

Steps to reproduce

Or: (Go to atf folder -> Run command: ./start.sh ../ATF_VehicleInfo_IsReady_missing_SingleRPC_empty.lua)

  1. RegisterApp in case VehicleInfo.IsReady=false
  2. Activate app
  3. Send READDID with "didLocation":[56832],"ecuName":2000 from mobile app
  4. HMI does not respond during SDL`s watchdog

Actual result

SDL respond
{"resultCode":"GENERIC_ERROR","success":false}
to mobile app.

Expected result

SDL should respond
{"resultCode":"GENERIC_ERROR","success":false,"info":"VehicleInfo component does not respond"}

Notes: It also happens with other APIs: , GetDTCs, DiagnosticMessage, SubscribeVehicleData, GetVehicleData, UnsubscribeVehicleData

Environment

SDL branch: master
SDL commit: a6c32b4
HMI: HTML5
OS: Ubuntu 14.04 LTS
Transport: WiFi
Mobile device: HTC One
Mobile OS: Android 4.3
Mobile App version: SPT v.2.1.81 Code: 239
Mobile App type (media/non-media/navi/any): any

Attachments

Uploading att_issue#1012.zip…

Expected delivery

  • Source code updates 
  • Code comments 
  • UTs add/update 
  • ATF tests add/update (not required) 
  • Manual tests (not required) 
  • Add/update CI plans/jobs (not required) 
  • SDD updates (not required) 
  • Guidelines update (sdl_core_guides(not required) 
  • Guidelines update (sdl_hmi_integration_guidelines) (not required)
@jacobkeeler
Copy link
Contributor

@KhrystynaDubovyk The ATF script that was supposed to be included with this issue was never uploaded.

@jacobkeeler
Copy link
Contributor

Fixed with the merge of #1640

@theresalech theresalech added this to Completed Bug Fixes in 4.5.0 Nov 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
4.5.0
Completed Bug Fixes
Development

No branches or pull requests

3 participants