tequilaOS/platform_hardware_interfaces
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Increate testcase timeout for VtsHalHealthStorageV1_0TargetTest

Browse source 
The test is flaky with the current timeout and this CL
increases it to a safe value.

Bug: 120994008
Test: test passes

Change-Id: I63e0e11fcc1eb3bed490cdb21193e2c332cccf80
This commit is contained in:
Hridya Valsaraju 2019-02-13 17:17:56 -08:00
parent 7081726008
commit bbab8f2a3e
1 changed files with 3 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
health/storage/1.0/vts/functional/VtsHalHealthStorageV1_0TargetTest.cpp
View file

@ -36,7 +36,7 @@ using ::std::literals::chrono_literals::operator""ms;
const uint64_t kDevGcTimeoutSec = 120;
const std::chrono::seconds kDevGcTimeout{kDevGcTimeoutSec};
// Time accounted for RPC calls.
const std::chrono::milliseconds kRpcTime{100};
const std::chrono::milliseconds kRpcTime{1000};
template <typename R>
std::string toString(std::chrono::duration<R, std::milli> time) {
@ -90,11 +90,8 @@ class GcCallback : public IGarbageCollectCallback, public Flag {
template <typename R, typename P>
void waitForResult(std::chrono::duration<R, P> timeout, Result expected) {
std::unique_lock<std::mutex> lock(mMutex);
if (waitLocked(&lock, timeout)) {
EXPECT_EQ(expected, mResult);
} else {
LOG(INFO) << "timeout after " << toString(timeout);
}
ASSERT_TRUE(waitLocked(&lock, timeout)) << "timeout after " << toString(timeout);
EXPECT_EQ(expected, mResult);
}
private: