BIND 10 trac3264, updated. 8edba8799ee54f28e6ab8ee9abe0b60e033ca954 [3264] One more review change.

[BIND 10 source code commits]

The branch, trac3264 has been updated
       via  8edba8799ee54f28e6ab8ee9abe0b60e033ca954 (commit)
      from  432acd7ecbdbe741d07c20f8b43be107d9f761b2 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 8edba8799ee54f28e6ab8ee9abe0b60e033ca954
Author: Thomas Markwalder <tmark at isc.org>
Date:   Wed Jan 15 08:19:14 2014 -0500

    [3264] One more review change.
    
    Forgot to document params to doOneExchange.

-----------------------------------------------------------------------

Summary of changes:
 src/bin/d2/tests/nc_trans_unittests.cc |    7 +++++++
 1 file changed, 7 insertions(+)

-----------------------------------------------------------------------
diff --git a/src/bin/d2/tests/nc_trans_unittests.cc b/src/bin/d2/tests/nc_trans_unittests.cc
index 7045dfb..010d197 100644
--- a/src/bin/d2/tests/nc_trans_unittests.cc
+++ b/src/bin/d2/tests/nc_trans_unittests.cc
@@ -302,6 +302,9 @@ public:
     /// This method is used to build and send and update request. It is used
     /// in conjuction with FauxServer to test various message response
     /// scenarios.
+    /// @param name_change Transaction under test
+    /// @param run_time Maximum time to permit IO processing to run before
+    /// timing out (in milliseconds)
     void doOneExchange(NameChangeStubPtr name_change,
                        unsigned int run_time = 500) {
         // Create a valid request for the transaction.
@@ -856,6 +859,10 @@ TEST_F(NameChangeTransactionTest, sendUpdateTimeout) {
     // Build a valid request, call sendUpdate and process the response.
     // Note we have to wait for DNSClient timeout plus a bit more to allow
     // DNSClient to timeout.
+    // The method, doOneExchange, can suffer fatal assertions which invalidate
+    // not only it but the invoking test as well. In other words, if the
+    // doOneExchange blows up the rest of test is pointless. I use
+    // ASSERT_NO_FATAL_FAILURE to abort the test immediately.
     ASSERT_NO_FATAL_FAILURE(doOneExchange(name_change,
                                           NameChangeTransaction::
                                           DNS_UPDATE_DEFAULT_TIMEOUT + 100));