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 sends GetInteriorVehicleData response with moduleData = null in case of erroneous response type for GetInteriorVehicleData from HMI #3514

Closed
AKalinich-Luxoft opened this issue Sep 24, 2020 · 5 comments

Comments

@AKalinich-Luxoft
Copy link
Contributor

Preconditions:

  1. sdl_preloaded_pt.json default app_policies
    groups: [ "Base-4", "RemoteControl" ]
    "moduleType": [ "RADIO", "CLIMATE", "SEAT", "AUDIO", "LIGHT", "HMI_SETTINGS" ]
    HMI supports RC functionality
  2. SDL and HMI started

Reproduction Steps:

  1. App→SDL: GetInteriorVehicleData("moduleType": "module_1", subscribe = true )
    SDL→HMI: RC.GetInteriorVehicleData("moduleType": "module_1", subscribe = true )
    HMI→SDL: RC.GetInteriorVehicleData( GENERIC_ERROR)
    {"error":{"code":22,"message":"Erroneous response is assigned by settings","data": {"method":"RC.GetInteriorVehicleData"}
    }}

Observed Behavior:
SDL→App: GetInteriorVehicleData("moduleData":null, "resultCode": "GENERIC_ERROR", "success": "false", info )

Expected Behavior:
SDL→App: GetInteriorVehicleData( "resultCode": "GENERIC_ERROR", "success": "false", info )

OS & Version Information
https://github.com/LuxoftSDL/sdl_core/tree/develop (a36ea82)
https://github.com/LuxoftSDL/sdl_hmi/tree/develop (6d32f1b)

@dboltovskyi
Copy link
Contributor

@theresalech Please notice this is not regression issue and it's applicable for Release 6.1.1

@theresalech
Copy link
Contributor

@dboltovskyi, thank you for letting us know. We are currently focused on regression issues (issues introduced during Core 7.0 development), and existing issues that were previously prioritized by the Steering Committee for the Core 7.0 release. We'll therefore look at this issue/PR when time allows, following the regression and previously prioritized issues. Thanks!

@AKalinich-Luxoft
Copy link
Contributor Author

@theresalech Please notice PR is ready for PM review.

@theresalech
Copy link
Contributor

@AKalinich-Luxoft, thank you for letting us know! This issue was not requested as priority for the next Core release, but we will try to review as time allows. If it's imperative this be included in the next release, please raise as priority to the SDLC Steering Committee. Thanks!

@ShobhitAd
Copy link
Contributor

Closed via #3565

@Jack-Byrne Jack-Byrne mentioned this issue Apr 13, 2021
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants