summaryrefslogtreecommitdiffstats
path: root/test
diff options
context:
space:
mode:
authorDavid Wagner <david.wagner@intel.com>2015-01-02 14:26:27 +0100
committerDavid Wagner <david.wagner@intel.com>2015-01-02 15:29:03 +0100
commitcf428e5ee22d8bb8a182d81f0433aeccf89fa810 (patch)
tree8d0b71f85e07f2932c75a6bae1a6995a860aa131 /test
parent1404cf730742a1adb0359bbc8c0ff79f216c4cb6 (diff)
downloadexternal_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')
-rw-r--r--test/test-platform/TestPlatform.cpp3
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(