diff options
| author | David Wagner <david.wagner@intel.com> | 2015-01-02 14:26:27 +0100 |
|---|---|---|
| committer | David Wagner <david.wagner@intel.com> | 2015-01-02 15:29:03 +0100 |
| commit | cf428e5ee22d8bb8a182d81f0433aeccf89fa810 (patch) | |
| tree | 8d0b71f85e07f2932c75a6bae1a6995a860aa131 /test/test-platform | |
| parent | 1404cf730742a1adb0359bbc8c0ff79f216c4cb6 (diff) | |
| download | external_parameter-framework-cf428e5ee22d8bb8a182d81f0433aeccf89fa810.zip external_parameter-framework-cf428e5ee22d8bb8a182d81f0433aeccf89fa810.tar.gz external_parameter-framework-cf428e5ee22d8bb8a182d81f0433aeccf89fa810.tar.bz2 | |
test-platform: fix boolean remote getter commands output
Getter commands return a boolean value ("true" or "false") were implemented by
a generic method. However, this method returned a "Done" result, which
suppresses the normal output and prints "Done" instead.
This is fixed by making this method return a "Success" result, which is
interpreted as "give the output to the user".
In the same time, we remove a useless "cast to void" which is usually a
workaround for unused variables.
Change-Id: I49115d3b29967fd455ffc0b703dfb4f4d438280e
Signed-off-by: David Wagner <david.wagner@intel.com>
Diffstat (limited to 'test/test-platform')
| -rw-r--r-- | test/test-platform/TestPlatform.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/test/test-platform/TestPlatform.cpp b/test/test-platform/TestPlatform.cpp index f1e99b6..b2b895c 100644 --- a/test/test-platform/TestPlatform.cpp +++ b/test/test-platform/TestPlatform.cpp @@ -245,11 +245,10 @@ CTestPlatform::CommandReturn CTestPlatform::getter( const IRemoteCommand& remoteCommand, string& strResult) { (void)remoteCommand; - (void)strResult; strResult = (_pParameterMgrPlatformConnector->*getFunction)() ? "true" : "false"; - return CTestPlatform::CCommandHandler::EDone; + return CTestPlatform::CCommandHandler::ESucceeded; } CTestPlatform::CommandReturn CTestPlatform::setCriterionState( |