Commit 432acd7e authored by Thomas Markwalder's avatar Thomas Markwalder
Browse files

[3264] Addressed review comments.

Added missing test method commentary.
parent faf86ad3
......@@ -297,6 +297,11 @@ public:
forward_domain_, reverse_domain_)));
}
/// @brief Builds and then sends an update request
///
/// This method is used to build and send and update request. It is used
/// in conjuction with FauxServer to test various message response
/// scenarios.
void doOneExchange(NameChangeStubPtr name_change,
unsigned int run_time = 500) {
// Create a valid request for the transaction.
......@@ -852,8 +857,8 @@ TEST_F(NameChangeTransactionTest, sendUpdateTimeout) {
// Note we have to wait for DNSClient timeout plus a bit more to allow
// DNSClient to timeout.
ASSERT_NO_FATAL_FAILURE(doOneExchange(name_change,
NameChangeTransaction::DNS_UPDATE_DEFAULT_TIMEOUT
+ 100));
NameChangeTransaction::
DNS_UPDATE_DEFAULT_TIMEOUT + 100));
// Verify that next event is IO_COMPLETED_EVT and DNS status is TIMEOUT.
ASSERT_EQ(NameChangeTransaction::IO_COMPLETED_EVT,
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment