diff --git a/CHANGELOG.md b/CHANGELOG.md index 449e4256f907780bfbd0ce3d6a23853f20de0ae6..fcdf01692fe8dbcd0dfa40ae7ae9eb7f596a5cf3 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -19,4 +19,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ### Fixed +* Fixed failing async test. + ### Security diff --git a/test/test_async.cpp b/test/test_async.cpp index c8049865abfb74b662c9f2e63619da80fcc7376b..bf437f6065f386ccfd49b01c56bc6b6d31848af0 100644 --- a/test/test_async.cpp +++ b/test/test_async.cpp @@ -53,13 +53,19 @@ TEST(test_async, retrieve_non_existing) { auto status = transaction->GetStatus(); EXPECT_EQ(status.GetCode(), StatusCode::EXECUTING); - // wait some time - std::this_thread::sleep_for(1000ms); + // 1000ms are not always sufficient, when there is very high pipeline load. + auto count = 1000; + auto code = transaction->GetStatus().GetCode(); + do { + // wait some time + std::this_thread::sleep_for(10ms); - // DONT call WaitForIt -> the transaction finishes in the back-ground - status = transaction->GetStatus(); - EXPECT_EQ(status.GetCode(), TransactionStatus::TRANSACTION_ERROR().GetCode()); - ASSERT_EQ(status.GetCode(), StatusCode::GENERIC_TRANSACTION_ERROR); + // DONT call WaitForIt -> the transaction finishes in the background + code = transaction->GetStatus().GetCode(); + } while (--count > 0 && code == StatusCode::EXECUTING); + ASSERT_GT(count, 0) << "ERROR: Timeout while waiting for transaction."; + EXPECT_EQ(code, TransactionStatus::TRANSACTION_ERROR().GetCode()); + ASSERT_EQ(code, StatusCode::GENERIC_TRANSACTION_ERROR); const auto &result_set = transaction->GetResultSet();