Skip to content

Commit

Permalink
Rebrand non_windows_sock_test->unix_only_sock_test
Browse files Browse the repository at this point in the history
  • Loading branch information
webknjaz committed Dec 31, 2018
1 parent 2232d5d commit 78fa844
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions cheroot/test/test_server.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
)


non_windows_sock_test = pytest.mark.skipif(
unix_only_sock_test = pytest.mark.skipif(
not hasattr(socket, 'AF_UNIX'),
reason='UNIX domain sockets are only available under UNIX-based OS',
)
Expand Down Expand Up @@ -106,7 +106,7 @@ def test_bind_addr_inet(http_server, ip_addr):
assert httpserver.bind_addr[1] != EPHEMERAL_PORT


@non_windows_sock_test
@unix_only_sock_test
def test_bind_addr_unix(http_server, unix_sock_file):
"""Check that bound UNIX socket address is stored in server."""
httpserver = http_server.send(unix_sock_file)
Expand All @@ -115,7 +115,7 @@ def test_bind_addr_unix(http_server, unix_sock_file):


@pytest.mark.skip(reason="Abstract sockets don't work currently")
@non_windows_sock_test
@unix_only_sock_test
def test_bind_addr_unix_abstract(http_server):
"""Check that bound UNIX socket address is stored in server."""
unix_abstract_sock = b'\x00cheroot/test/socket/here.sock'
Expand Down Expand Up @@ -158,7 +158,7 @@ def peercreds_enabled_server_and_client(http_server, unix_sock_file):
return httpserver, get_server_client(httpserver)


@non_windows_sock_test
@unix_only_sock_test
@non_macos_sock_test
def test_peercreds_unix_sock(peercreds_enabled_server_and_client):
"""Check that peercred lookup works when enabled."""
Expand All @@ -185,7 +185,7 @@ def test_peercreds_unix_sock(peercreds_enabled_server_and_client):
reason='Modules `grp` and `pwd` are not available '
'under the current platform',
)
@non_windows_sock_test
@unix_only_sock_test
@non_macos_sock_test
def test_peercreds_unix_sock_with_lookup(peercreds_enabled_server_and_client):
"""Check that peercred resolution works when enabled."""
Expand Down

0 comments on commit 78fa844

Please sign in to comment.