DPL Pipe tests fix
authorTomasz Iwanek <t.iwanek@samsung.com>
Mon, 19 Aug 2013 13:05:12 +0000 (15:05 +0200)
committerTomasz Iwanek <t.iwanek@samsung.com>
Mon, 26 Aug 2013 10:41:57 +0000 (12:41 +0200)
[Issue#]       N/A
[Bug]          2 tests failing
[Cause]        FlattenConsume function assumes presence of data
[Solution]     Test fix
[Verification] Run: wrt-commons-tests-test --output=text --regexp='ProcessPipe_'

Change-Id: I5bc2e9a9cc1e16ecd76eed7002ba0a35babf41ee

tests/test/test_process_pipe.cpp

index ffc8a8d..46405f9 100644 (file)
@@ -58,7 +58,7 @@ RUNNER_TEST(ProcessPipe_echo)
     npp.Close();
 
     char buffer[100] = "";
-    result.FlattenConsume(buffer, 99);
+    result.FlattenConsume(buffer, std::min(result.Size(), sizeof(buffer)));
 
     RUNNER_ASSERT_MSG(strcmp(buffer, "Test echo text\nAnd new line\n") == 0, "Echoed text in not equal");
 }
@@ -113,7 +113,7 @@ RUNNER_TEST(ProcessPipe_pipeerror_pipe)
     readAll(npp, result);
     npp.Close();
     char buffer[100] = "";
-    result.FlattenConsume(buffer, 99);
+    result.FlattenConsume(buffer, std::min(result.Size(), sizeof(buffer)));
 
     RUNNER_ASSERT_MSG(strcmp(buffer, "ls: cannot access /nonexistingdirectory: No such file or directory\n") == 0, "Ls error text in not equal");
 }