Browse Source

[1522] added note about an assumption on the token length in some tests.

JINMEI Tatuya 13 years ago
parent
commit
1f9be59567
1 changed files with 7 additions and 2 deletions
  1. 7 2
      src/lib/server_common/tests/socket_requestor_test.cc

+ 7 - 2
src/lib/server_common/tests/socket_requestor_test.cc

@@ -420,8 +420,13 @@ private:
         }
         }
     }
     }
 
 
-    // Accept one connection, then send all values from the vector using
-    // send_fd() (prepended by a status code 'ok').
+    // Accept one connection, then for each value of the vector,
+    // read the socket token from the connection and match the string
+    // part of the vector element, and send the integer part of the element
+    // using send_fd() (prepended by a status code 'ok').  For simplicity
+    // we assume the tokens are 4 bytes long; if the test case uses a
+    // different size of token the test will fail.
+    //
     // There are a few specific exceptions;
     // There are a few specific exceptions;
     // when the value is -1, it will send back an error value (signaling
     // when the value is -1, it will send back an error value (signaling
     // CREATOR_SOCKET_UNAVAILABLE)
     // CREATOR_SOCKET_UNAVAILABLE)