Skip to content

Commit 4c485cc

Browse files
authored
coverage: pin higher coverage (#1006)
* coverage: pin higher coverage * add more ignores * not change previous code * remove marker on httptools * fix requirement
1 parent abde790 commit 4c485cc

File tree

8 files changed

+10
-10
lines changed

8 files changed

+10
-10
lines changed

scripts/coverage

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
#!/bin/sh -e
22

33
export PREFIX=""
4-
if [ -d 'venv' ] ; then
4+
if [ -d 'venv' ]; then
55
export PREFIX="venv/bin/"
66
fi
77
export SOURCE_FILES="uvicorn tests"
88

99
set -x
1010

11-
${PREFIX}coverage report --show-missing --skip-covered --fail-under=80
11+
${PREFIX}coverage report --show-missing --skip-covered --fail-under=95

scripts/test

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#!/bin/sh
22

33
export PREFIX=""
4-
if [ -d 'venv' ] ; then
4+
if [ -d 'venv' ]; then
55
export PREFIX="venv/bin/"
66
fi
77

setup.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ def get_packages(package):
5353

5454
extra_requirements = [
5555
"websockets>=8.*",
56-
"httptools==0.2.* ;" + env_marker_cpython,
56+
"httptools==0.2.*",
5757
"uvloop>=0.14.0,!=0.15.0,!=0.15.1; " + env_marker_cpython,
5858
"colorama>=0.4;" + env_marker_win,
5959
"watchgod>=0.6",

tests/protocols/test_utils.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ def test_get_local_addr_with_socket():
4040
)
4141
assert get_local_addr(transport) == ("123.45.6.7", 123)
4242

43-
if hasattr(socket, "AF_UNIX"):
43+
if hasattr(socket, "AF_UNIX"): # pragma: no cover
4444
transport = MockTransport(
4545
{"socket": MockSocket(family=socket.AF_UNIX, sockname=("127.0.0.1", 8000))}
4646
)
@@ -61,7 +61,7 @@ def test_get_remote_addr_with_socket():
6161
)
6262
assert get_remote_addr(transport) == ("123.45.6.7", 123)
6363

64-
if hasattr(socket, "AF_UNIX"):
64+
if hasattr(socket, "AF_UNIX"): # pragma: no cover
6565
transport = MockTransport(
6666
{"socket": MockSocket(family=socket.AF_UNIX, peername=("127.0.0.1", 8000))}
6767
)

uvicorn/loops/asyncio.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
import sys
55

66

7-
def asyncio_setup():
7+
def asyncio_setup(): # pragma: no cover
88
if (
99
sys.version_info.major >= 3
1010
and sys.version_info.minor >= 8

uvicorn/loops/auto.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ def auto_loop_setup():
55
from uvicorn.loops.asyncio import asyncio_setup as loop_setup
66

77
loop_setup()
8-
else:
8+
else: # pragma: no cover
99
from uvicorn.loops.uvloop import uvloop_setup
1010

1111
uvloop_setup()

uvicorn/protocols/http/auto.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
from uvicorn.protocols.http.h11_impl import H11Protocol
55

66
AutoHTTPProtocol = H11Protocol
7-
else:
7+
else: # pragma: no cover
88
from uvicorn.protocols.http.httptools_impl import HttpToolsProtocol
99

1010
AutoHTTPProtocol = HttpToolsProtocol

uvicorn/server.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,7 @@ def install_signal_handlers(self) -> None:
279279
try:
280280
for sig in HANDLED_SIGNALS:
281281
loop.add_signal_handler(sig, self.handle_exit, sig, None)
282-
except NotImplementedError:
282+
except NotImplementedError: # pragma: no cover
283283
# Windows
284284
for sig in HANDLED_SIGNALS:
285285
signal.signal(sig, self.handle_exit)

0 commit comments

Comments
 (0)