summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Added engineOilLife parameter to preloaded PT functional groupfeature/EngineOilLifeAKalinich-Luxoft2018-03-011-0/+4
|
* Added CommandHelper namespace with set of common functionsAKalinich-Luxoft2018-01-2610-193/+322
| | | | | Part of functionality were moved out from CommandRequestImpl base class to this helper. Moved functions were updated.
* Fix affected unit testsAKalinich-Luxoft2018-01-261-37/+63
|
* Fix disallowed info message generating for specific RPCsAKalinich-Luxoft2018-01-268-29/+77
| | | | | | | | | | | | | There was found a problem that general info message in case of disallowed by policies params is not applicable for all RPCs. For example SubscribeVehicleData/UnsubscribeVehicleData has own specific format of info message in this case. To fix that following changes were done: - Added virtual function AddSpecificInfoToResponse() which could be overriden in child RPCs in case custom info message required - Removed hardcoded checks of function_id from command_request_impl - all this checks will be done using polymorphism inside aproriate RPCs - Fixed affected UT expectation. Invalid unit test was replaced with actual one.
* Fix all affected UTs related to notifications sendingAKalinich-Luxoft2018-01-2610-16/+77
| | | | Also fixed some unstable tests
* Added param checking by policy for notifications for mobile appsAKalinich-Luxoft2018-01-262-8/+164
| | | | | | There was missed logic to check notification parameters according policy table for notifications. Was added similar functions as for requests.
* Fixed requests param reading from incoming messageAKalinich-Luxoft2018-01-261-1/+4
| | | | | | | | There was a problem that asBool() function of smart object returns false for string/array/object types so this params will no be included to params check list and will not be cutted off after that. This check was replaced to ignore only null/ivalid smart object types.
* Fix info message param for different disallowed param casesAKalinich-Luxoft2018-01-264-3/+38
| | | | | | | | Added generating of info message depending on check permissions result: - for disallowed rpc - for all disallowed params in rpc - for partially disallowed params in rpc
* Add new PermitResult values for PROPRIETARY flowAKalinich-Luxoft2018-01-263-3/+53
| | | | | | | | | Added new values for more detailed CheckPermissions result description. This values will be used in AM layer for generating info message string. Also there was added missed logic for checking RPC params and returning correct PermitResult code and params lists.
* Add new PermitResult values for EXTERNAL_PROPRIETARY flowAKalinich-Luxoft2018-01-263-7/+14
| | | | | | Added new values for more detailed CheckPermissions result description. This values will be used in AM layer for generating info message string.
* Fix SDL behavior in case of param disallowed by policies and doubleAndriy Byzhynar2018-01-263-10/+16
| | | | | | | | | | | subscription - Fixed response to mobile when app tries to subscribe to already subscribed param - Fixed response to mobile when request contains parameter disallowed by policies - Added some logging - Fixed mistake in function name
* Initial implementation of EngineOilLife param supportAndriy Byzhynar2018-01-269-7/+64
|
* Merge pull request #1927 from smartdevicelink/feature/IAP_over_BTJackLivio2018-01-25129-1055/+5850
|\ | | | | Implementation of transport switching from iAP2 Bluetooth to iAP2 USB
| * Fix difference in device hash generationfeature/IAP_over_BTAndriy Byzhynar2018-01-221-2/+5
| | | | | | | | | | Fixed difference in device id hash generation which was different for SDL built with unit-tests and without
| * Remove redundant functionalityAndriy Byzhynar2018-01-223-6/+18
| |
| * Add TODO comment in correct formatAndriy Byzhynar2018-01-191-2/+2
| |
| * Replace deprecation comments with DEPRECATED macroAndriy Byzhynar2018-01-1824-159/+122
| |
| * Extract common code to separate functionAndriy Byzhynar2018-01-181-56/+19
| |
| * Replace pragma with header guardsAndriy Byzhynar2018-01-1819-306/+85
| | | | | | | | | | | | | | Replaced #pragma once usage with header guards Added wrappers for deprecated functions Fixed related unit tests Fixed coding style in places related to code chages
| * Fix issues after rebaseAndriy Byzhynar2018-01-186-8/+9
| |
| * Fix UT for activate_app_requestAndriy Byzhynar2018-01-181-1/+11
| |
| * Add wrappers for deprecated methodsAndriy Byzhynar2018-01-1812-90/+52
| | | | | | | | | | | | | | Added wrappers for some deprecated methods Grouped iface checks in SendRegisterAppInterfaceResponseToMobile Removed duplicated "if" statement Improved readability
| * Fixes thread races in transport manager unit testsAndrey Oleynik (GitHub)2018-01-181-5/+1
| |
| * Revert "Changes return codes for application id replacement"Andrey Oleynik (GitHub)2018-01-181-4/+4
| | | | | | | | | | | | This reverts commit 866dfe849471157d6d0b414cfb013198a879a51f. Returning false in case of inner app_id fields update failed causes many issues on exisiting logic and ATF scripts.
| * Adds error handlingAndrey Oleynik (GitHub)2018-01-181-3/+6
| |
| * Changes return codes for application id replacementAndrey Oleynik (GitHub)2018-01-181-4/+4
| |
| * Fixes potential build warnings with macroAndrey Oleynik (GitHub)2018-01-181-0/+2
| |
| * Fixes thread stopping in case no one connected to the pipeAndrey Oleynik (GitHub)2018-01-181-2/+11
| |
| * Fixes coding styleAndrey Oleynik (GitHub)2018-01-185-31/+25
| |
| * Adds comments, replaces pointer with reference and other minor changesAndrey Oleynik (GitHub)2018-01-182-12/+23
| |
| * Changes implementation of iAP2 transport adapter emulationAndrey Oleynik (GitHub)2018-01-185-3/+165
| | | | | | | | | | | | New implementation considers external switching flow triggering so no implementation includes separate thread which manages incoming signals and starts the flow. Also ACK is being sent back to the system.
| * Adds null pointer checks, fixes typos and other minor changes.Andrey Oleynik (GitHub)2018-01-1810-30/+43
| |
| * Fixes coding styleAndrey Oleynik (GitHub)2018-01-1829-172/+187
| |
| * Updates mocks and unit tests after prior commitAndrey Oleynik (GitHub)2018-01-1815-141/+322
| | | | | | | | | | | | Fixes mock classes interfaces, unit tests and build after changes related to making iAP2 switching triggered externally and using of Bluetooth MAC/USB serial instead of UUID for internal device management.
| * Changes iAP2 Bluetooth to USB switching flowAndrey Oleynik (GitHub)2018-01-1837-106/+501
| | | | | | | | | | | | | | | | | | | | | | These changes update switching flow so now instead of automatic switching start in case of same UUID is detected SDL will wait for external signal from the system to start this flow. Also due to UUID is reliable only while device remains connected (at least on certain systems) SDL now uses Bluetooth MAC and USB serial to manage devices and UUID is used only for detection of devices able to switch their transports. Currently only iAP2 Bluetooth to USB support is implemented.
| * Fixes multiconnection from the same adapter typeAndrey Oleynik (GitHub)2018-01-182-9/+18
| | | | | | | | | | On new device connection SDL has to skip devices already known for the same adapter from update device list update process.
| * Fixes RPCs holding for applications being switchedAndrey Oleynik (GitHub)2018-01-186-55/+221
| |
| * Adds iAP2 BT/USB connection emulation over TCP transportAndrey Oleynik (GitHub)2018-01-186-2/+215
| | | | | | | | | | | | | | | | To be able to check iAP2 transport switch feature these adapters are implemented. Implementation is done in the way of emulating iAP transport over TCP since there is no available iAP library for open source. Since it is just emulation not a real transport is will be built only with other unit tests i.e. with BUILD_TESTS = ON
| * Fixes deletion of commands and submenus on failed resumeAndrey Oleynik (GitHub)2018-01-181-3/+3
| | | | | | | | While adding data from AddCommand/AddSubMenun msg_params are added itself so we have directly address internal fields w/o accessing 'msg_params'
| * Fixes handling of way points un/subscription for transport switch caseAndrey Oleynik (GitHub)2018-01-1815-30/+139
| | | | | | | | | | | | In case app_id is used for way points subscription it being invalidated on transport switch and SDL can't unsubscribe same application after switch is done.
| * Fixes coding style issuesAndrey Oleynik (GitHub)2018-01-181-1/+1
| |
| * Fixes resume issue for application on switching transportAndrey Oleynik (GitHub)2018-01-181-1/+7
| | | | | | | | | | | | In case app has been just registrered in previous session and didn't sent any data to resume later and then re-registers over new transport due to switch there is nothing to resume so RAI result should be SUCCESS
| * Fixes sending 'transport_switch' signal back to adapter/deviceAndrey Oleynik (GitHub)2018-01-182-6/+8
| | | | | | | | | | | | After StopDevice call BT device channel is not usable anymore, so to talk back to system USB channel is used. Fixes unit test as well.
| * Reverts missing code after rebaseAndrey Oleynik (GitHub)2018-01-181-0/+11
| |
| * Renames interfaces according to people desireAndrey Oleynik (GitHub)2018-01-185-34/+35
| |
| * Fixes typos, commentsAndrey Oleynik (GitHub)2018-01-185-7/+6
| |
| * Renames interfaces for consistencyAndrey Oleynik (GitHub)2018-01-1813-21/+21
| |
| * Fixes deadlock on resume data saving during transport switchAndrey Oleynik (GitHub)2018-01-181-5/+7
| | | | | | | | | | | | | | | | | | | | | | Issue was happening due to - resume data restoring thread was acquiring app subscription lock and then application list lock - transport switching thread was acquiring application lock and was saving resume data and trying to get subscription lock Now after filtering switching application from application list its lock will be released
| * Changes CommandHolderImpl usage and removes odd interfaceAndrey Oleynik (GitHub)2018-01-186-38/+22
| | | | | | | | Also typedefs used, typo in description fixed, minor code changes
| * Fixes build w/o unit tests enabledAndrey Oleynik (GitHub)2018-01-181-1/+9
| |