Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1527)

Unified Diff: native_client_sdk/src/tests/nacl_io_socket_test/socket_test.cc

Issue 23498015: [NaCl SDK] Support non blocking TCP/UDP (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: native_client_sdk/src/tests/nacl_io_socket_test/socket_test.cc
diff --git a/native_client_sdk/src/tests/nacl_io_socket_test/socket_test.cc b/native_client_sdk/src/tests/nacl_io_socket_test/socket_test.cc
index fe22ae18e03452aabacf4dee31dd011ebb16d8b9..081a50bc526a48d7e995230f6c01ff2438cea46e 100644
--- a/native_client_sdk/src/tests/nacl_io_socket_test/socket_test.cc
+++ b/native_client_sdk/src/tests/nacl_io_socket_test/socket_test.cc
@@ -174,8 +174,13 @@ TEST_F(SocketTestUDP, SendRcv) {
EXPECT_EQ(0, memcmp(outbuf, inbuf, sizeof(outbuf)));
}
-#if 0
TEST_F(SocketTestTCP, Connect) {
+ char outbuf[256];
+ char inbuf[512];
+
+ memset(outbuf, 1, sizeof(outbuf));
+ memset(inbuf, 0, sizeof(inbuf));
+
int sock = socket(AF_INET, SOCK_STREAM, 0);
EXPECT_NE(-1, sock);
@@ -184,8 +189,14 @@ TEST_F(SocketTestTCP, Connect) {
IP4ToSockAddr(LOCAL_HOST, PORT1, &addr);
int err = connect(sock, (sockaddr*) &addr, addrlen);
+
EXPECT_EQ(ENONE, err) << "Failed with errno: " << errno << "\n";
+
+ EXPECT_EQ(sizeof(outbuf), write(sock, outbuf, sizeof(outbuf)));
+ EXPECT_EQ(sizeof(outbuf), read(sock, inbuf, sizeof(inbuf)));
+
+ // Now they should be the same
+ EXPECT_EQ(0, memcmp(outbuf, inbuf, sizeof(outbuf)));
}
-#endif
-#endif // PROVIDES_SOCKETPAIR_API
+#endif // PROVIDES_SOCKET_API

Powered by Google App Engine
This is Rietveld 408576698