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

Alert, AlertManeuver, PerformAudioPassThru: SDL sends success = false in case of UNSUPPORTED_RESOURCE #1597

Closed
5 of 9 tasks
DrachenkoAnastasiia opened this issue Jun 7, 2017 · 1 comment
Labels

Comments

@DrachenkoAnastasiia
Copy link

Occurrence

Always

Preconditions

  1. SDL is built with Proprietary flag.
  2. SDL and HMI are started. First ignition cycle.
  3. Connect device
  4. Register new application.

Steps to reproduce

  1. App->SDL: Alert()
    SDL->HMI:UI.Alert()
    SDL->HMI:TTS.Speak()
  2. HMI Responses:
    HMI->SDL: SUCCESS: UI.Alert()
    HMI->SDL: UNSUPPORTED_RESOURCE: TTS.Speak() + info: "Error message"
  3. SDL->App: response

Actual result

{ success = false, resultCode = "WARNINGS", info = "Unsupported phoneme type sent in a prompt" }
"success" and "info" parameters are incorrect

Expected result

{ success = true, resultCode = "WARNINGS", info = "Error message"}

Notes::

  • Issue is reproduced for PerformAudioPassThru RPC as well.
  • Issue is reproduced for AlertManeuver RPC as well (but with another actual result).

Environment

SDL: https://github.com/LuxoftAKutsan/sdl_core/tree/integration/4.3.1 (35344c8)
ATF: https://github.com/smartdevicelink/sdl_atf/tree/develop (fe002fb)
ATF Scripts: https://github.com/smartdevicelink/sdl_atf_test_scripts/tree/develop (4ba374c)
OS: Ubuntu 14.04 LTS x64

Attachments

SDLOPEN-2454.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

Closing with the merge of #3587

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants