Skip to content

Fix tests: getsockname() can return None on OS X on unbound sockets#1400

Merged
pitrou merged 1 commit into
python:masterfrom
pitrou:getsockname_none
May 2, 2017
Merged

Fix tests: getsockname() can return None on OS X on unbound sockets#1400
pitrou merged 1 commit into
python:masterfrom
pitrou:getsockname_none

Conversation

@pitrou

@pitrou pitrou commented May 2, 2017

Copy link
Copy Markdown
Member

No description provided.

@mention-bot

Copy link
Copy Markdown

@pitrou, thanks for your PR! By analyzing the history of the files in this pull request, we identified @asvetlov, @tiran and @ncoghlan to be potential reviewers.

@pitrou pitrou added trivial tests Tests in the Lib/test dir labels May 2, 2017
@pitrou

pitrou commented May 2, 2017

Copy link
Copy Markdown
Member Author

@ned-deily, @brettcannon, @ronaldoussoren, can one of you test this quickly on OS X? It's supposed to fix a test_socket failure on OS X, see http://bugs.python.org/issue30205

Comment thread Lib/test/test_socket.py
def testUnbound(self):
# Issue #30205
self.assertEqual(self.sock.getsockname(), '')
# Issue #30205 (note getsockname() can return None on OS X)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are now using the "bpo-xxx" syntax instead of "issue #xxx" to prevent GitHub to creating a link to the GitHub bug tracker.

@pitrou pitrou merged commit 0360a9d into python:master May 2, 2017
pitrou added a commit to pitrou/cpython that referenced this pull request May 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants