From 3be88c8cbfc3f10f6359f43957e64a01ce21bbd3 Mon Sep 17 00:00:00 2001 From: Luna Date: Wed, 17 Oct 2018 21:17:14 +0100 Subject: [PATCH] removed redundant import in routes.py --- responder/routes.py | 5 +++-- tests/test_routes.py | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/responder/routes.py b/responder/routes.py index b28791a..53019f4 100644 --- a/responder/routes.py +++ b/responder/routes.py @@ -1,5 +1,5 @@ import re -from parse import parse, search +from parse import parse def memoize(f): @@ -60,7 +60,8 @@ class Route: return url def _weight(self): - params_count = -len(set(self._param_pattern.findall(self.route))) + params = set(self._param_pattern.findall(self.route)) + params_count = -len(params) or 0 return params_count != 0, params_count @property diff --git a/tests/test_routes.py b/tests/test_routes.py index c3e0b00..0ae2606 100644 --- a/tests/test_routes.py +++ b/tests/test_routes.py @@ -97,7 +97,7 @@ def test_does_match_with_route(route, match, expected): pytest.param( "/{greetings}_{name}", (True, -2), id="with 2 param and underscore" ), - pytest.param("/hello", (False, -0), id="with 2 param and underscore"), + pytest.param("/hello", (False, 0), id="with 2 param and underscore"), ], ) def test_weight(path_param, expected_weight):