Commit 97af747e authored by Orvid King's avatar Orvid King Committed by Facebook Github Bot

Shift SocketAddressTest from socketpair to netops::socketpair

Summary: I missed a use of file descriptors in the last diff to clean up this test.

Reviewed By: yfeldblum

Differential Revision: D12990054

fbshipit-source-id: 433f5b624eaa5b4fb8b757ba0b503354d5da0623
parent cf5b2228
...@@ -640,8 +640,8 @@ TEST(SocketAddress, Unix) { ...@@ -640,8 +640,8 @@ TEST(SocketAddress, Unix) {
TEST(SocketAddress, AnonymousUnix) { TEST(SocketAddress, AnonymousUnix) {
// Create a unix socket pair, and get the addresses. // Create a unix socket pair, and get the addresses.
int fds[2]; NetworkSocket fds[2];
int rc = socketpair(AF_UNIX, SOCK_STREAM, 0, fds); int rc = netops::socketpair(AF_UNIX, SOCK_STREAM, 0, fds);
EXPECT_EQ(rc, 0); EXPECT_EQ(rc, 0);
SocketAddress addr0; SocketAddress addr0;
...@@ -652,8 +652,8 @@ TEST(SocketAddress, AnonymousUnix) { ...@@ -652,8 +652,8 @@ TEST(SocketAddress, AnonymousUnix) {
peer0.setFromPeerAddress(fds[0]); peer0.setFromPeerAddress(fds[0]);
addr1.setFromLocalAddress(fds[1]); addr1.setFromLocalAddress(fds[1]);
peer1.setFromPeerAddress(fds[1]); peer1.setFromPeerAddress(fds[1]);
close(fds[0]); netops::close(fds[0]);
close(fds[1]); netops::close(fds[1]);
EXPECT_EQ(addr0.describe(), "<anonymous unix address>"); EXPECT_EQ(addr0.describe(), "<anonymous unix address>");
EXPECT_EQ(addr1.describe(), "<anonymous unix address>"); EXPECT_EQ(addr1.describe(), "<anonymous unix address>");
......
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