summaryrefslogtreecommitdiffstats
path: root/test
Commit message (Collapse)AuthorAgeFilesLines
...
* Clean useless extern "C" and wrong documentationKevin Rocard2014-02-121-10/+0
| | | | | | | | | | | | | | | | | BZ: 122982 Doxygen documentation was wrongly formated. An extern "C" was useless. Fix it. Change-Id: I1c144af30c692dc124bda90058f8caff1090ae3d Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/130520 Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* Add the setFailureOnFailedSettingsLoad commandKevin Rocard2014-02-122-19/+47
| | | | | | | | | | | | | | | | | | | BZ: 122982 The failure on failed settings load option was added by the previous patch (Ifd3cd859ad0eaef266be4c9245ebd28c9e9c0a74) but not expose in test-platform. Create command to expose the boolean getters and setters of the ParameterMgrConector in a generic way. Change-Id: I438ab36c2d1141dc139d528f83b570f4d5078d19 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/119925 Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* Wrap lines too longs and add doxygenKevin Rocard2014-02-122-45/+105
| | | | | | | | | | | | | | | | BZ: 122982 Some lines were not compliant with the coding style. Wrap them and add doxigen. Change-Id: I5271b78b1e879a39914a4405a58d7366148b6972 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/119424 Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* Add a command to set missing subsystem policyKevin Rocard2014-02-124-4/+70
| | | | | | | | | | | | | | | | | | | BZ: 122982 When the PFW starts, it loads subsystems using their corresponding plugins. If a requested plugin is not found, the start fails. This is a problem, as for host, plugins are not compiled. Add a command in test platform to use the feature continue load on missing subsystem. Change-Id: Ib533123683b860bfbe0fea069f4bb3c1cc1ef2d0 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/119305 Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* parameter-framework: Remove the hardcoding from TOP of treeChetan Rao2014-02-121-2/+2
| | | | | | | | | | | | | | | | | | | BZ: 120651 Remove the hard coding of path from TOP of android build tree. pathmap_INCL has been extended to add AOSP include dirs used by parameter framework. Need to use the MACRO include-path-for defined in build/core/pathmap.mk to include the AOSP paths Change-Id: Ic7c9eb4bcf39db5e31d14c785d8d2e37ab53af78 Signed-off-by: Chetan Rao <chetan.rao@intel.com> Reviewed-on: http://android.intel.com:8080/105311 Reviewed-by: Dixon, CharlesX <charlesx.dixon@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: Rocard, KevinX <kevinx.rocard@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [Makefile] Fix useless CFLAGSKevin Rocard2014-02-121-4/+0
| | | | | | | | | | | | | | | | | | | BZ: 115743 The patch e2c35a01 deleted -Wno-non-virtual-dtor flags by deleting the common_cflags definition but it was still used in the rest of the makefiles. Delete the common_cflags occurrences. Change-Id: If4301a774c45594b36962ce5b82ae5656c6a5bb1 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/115292 Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [Makefile] Format variables to lower_snake_caseKevin Rocard2014-02-121-18/+18
| | | | | | | | | | | | | | | | | BZ: 115743 Some variables where not formatted as requested by the coding style. Format them to lower_snake_case. Change-Id: I31c152f161cde469bcd4c00953e95274c60868ca Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/115175 Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [Makefile] Append "_host" to host modules namesKevin Rocard2014-02-121-2/+3
| | | | | | | | | | | | | | | | | | | | | BZ: 115743 In some parameter framework makefiles, host and target modules (with the same name) are defined in the same file. External prebuilt framework properly handle this case when modules are defined in different Android.mk, but there is a limitation when modules are defined in same Android.mk. Append _host to all host modules. Change-Id: Ib594dabd86617cc40993066338f095b49afb2ea2 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/113537 Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* Remove no-non-virtual-dtor flagSebastien Gonzalve2014-02-122-4/+0
| | | | | | | | | | | | | | BZ: 111473 The warning non-virtual-dtor should not be ignored for the whole project, but only in the few cases where it it necessary. Change-Id: I0ede82a30f2cd055b789f7fb56104fb227934e6f Signed-off-by: Sebastien Gonzalve <sebastien.gonzalve@intel.com> Signed-off-by: Guilhem IMBERTON <guilhem.imberton@intel.com> Reviewed-on: http://android.intel.com:8080/110191 Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [Parameter] Move public header to include dirKevin Rocard2014-02-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | BZ: 107991 The parameter-framework core public headers were mixed up with all other private headers. Making it dificult for other components to include PFW public headers. Move public headers - ParameterHandle - ParameterMgrPlatformConnector - SelectionCriterionInterface - SelectionCriterionTypeInterface to the "include" dir. Change-Id: I3a4840345a26ec9bd848179bdd4966d44d97bede Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/109757 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* Change TARGET_ERROR_FLAGS to LOCAL_CFLAGSKevin Rocard2014-02-121-3/+3
| | | | | | | | | | | | | | | | | | BZ: 107991 All plugin Android.mk were copy pasted resulting in use of TARGET_ERROR_FLAGS instead of LOCAL_CFLAGS. Change TARGET_ERROR_FLAGS to LOCAL_CFLAGS Change-Id: I792164e8c056be5dbfee35d07e7dd21b3137f4f6 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/109723 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [test-platform] Add support for host buildKevin Rocard2014-02-121-17/+43
| | | | | | | | | | | | | | | | | | BZ: 107991 In order to generate setting xml at build time, one need to build and run PFW environement on host. Add an host compilation target. Change-Id: I8f736652f34c412dab20c185098abb4319bb001f Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/107406 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* Fix incorect use of tabs, replace by spacesKevin Rocard2014-02-121-10/+10
| | | | | | | | | | | | | | | | | | BZ: 97960 Incorect use of tabs Replace by spaces Change-Id: I44e4b374e7f79ad268366a00b80b3409ab676cc2 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/102855 Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: Do not stop on hard sync errorsKevin Rocard2014-02-121-10/+25
| | | | | | | | | | | | | | | | BZ: 76263 When synchronization fails, the parameter-framework should log a warning and continue synchronization instead of stopping Change-Id: Ic12d8798ea25584db714ee26e644fac793c28881 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/81825 Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* PFW: Correct test-platform compilationKevin Rocard2014-02-122-12/+14
| | | | | | | | | | | | | | | | | | BZ: 67916 Add libstlport in test-plaform's Android.mk. Add a Android.mk that includes test-plaform/Android.mk in the test directory be able to compile test-platform from the source root (m test-platform). Change-Id: I878c4010aa8be4f9de0c4dd6666b9f512e26a309 Origin-Change-Id: Ieeb3bc7e908a2b35f0e0603917b80343a75259e8 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/76889 Reviewed-by: Barthes, FabienX <fabienx.barthes@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* PFW: Allow custom port number in test-platformFrédéric Boisnard2014-02-123-5/+8
| | | | | | | | | | | | | | | | | | BZ: 56440 This patch aims to allow custom port number in test-platform: test-platform <file path> [port number, default 5001] Change-Id: Ia617507045a58d0e79a5dc1df83b0173c968853a Signed-off-by: Frédéric Boisnard <fredericx.boisnard@intel.com> Reviewed-on: http://android.intel.com:8080/65412 Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: Rocard, KevinX <kevinx.rocard@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Mendi, EduardoX <eduardox.mendi@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: setCriterionState now support lexical stateKevin Rocard2014-02-122-1/+82
| | | | | | | | | | | | | | | | | | | | | | | | BZ: 54174 If the test-platform setCriterionState command is unable to convert the supply state to integer, it tries to consider it as a lexical state. Before setCriterionState SelectedOutputDevice 0x3 After setCriterionState SelectedOutputDevice IHF Headset Change-Id: Ib981eb955afd2acfa6e3d60d11828fcc4fc8860a Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/63478 Reviewed-by: Baron, Georges-henriX <georges-henrix.baron@intel.com> Reviewed-by: Boisnard, FredericX <fredericx.boisnard@intel.com> Reviewed-by: Benavoli, Patrick <patrick.benavoli@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: Remove names from PFW license headerKevin Rocard2014-02-123-18/+3
| | | | | | | | | | | | | | | BZ: 47701 As parameter framework code is proprietary, it should not be signed (patrick Benavoli name inside the header). Change-Id: I198f2851ee2a6cffed64a552fa399b072a0cbd3e orig-Change-Id: I335ecce2fa22ad11d6fa24f57c7cbbae3423bf1e Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/59560 Reviewed-by: Mendi, EduardoX <eduardox.mendi@intel.com> Tested-by: Mendi, EduardoX <eduardox.mendi@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: Make test-platform run on androidKevin Rocard2014-02-121-0/+38
| | | | | | | | | | | | | | | BZ: 34880 For routing testing purpose, the PFW test program : test-platform needs to run on the target. Change-Id: Iab8d8baf42a5db99550e1cba921f80fdb0a69374 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/47500 Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Mendi, EduardoX <eduardox.mendi@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: Add CriterionFromStateList to test-platformKevin Rocard2014-02-122-0/+88
| | | | | | | | | | | | | | | | | | BZ: 34881 Add commands : createExclusiveSelectionCriterionFromStateList and createInclusiveSelectionCriterionFromStateList For routing testing purpose, the PFW test program "test-platform" need commands to create criterion from a state list, so that each criterion state could have a specific name. Change-Id: Ibf4214723e085069ea9513efb099034478fa3b73 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/47499 Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Mendi, EduardoX <eduardox.mendi@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: Add test meansGuillaume Denneulin2014-02-103-0/+353
BZ: 12873 In order to test the parameter-framework, new means have been developped. they consist in 2 distinct interfaces: - Simulate platform behavior through command-line - Simulate HW behavior through test-plugin performing R/W in file system. Change-Id: I6efc9b5489155d9f334a72d7ed65324c3e52690f Signed-off-by: Guillaume Denneulin <guillaumex.denneulin@intel.com> Reviewed-on: http://android.intel.com:8080/35904 Reviewed-by: Barthes, FabienX <fabienx.barthes@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>