Skip to content

Commit 920c092

Browse files
author
nilsnolde
committed
remove test which requires live API querying
1 parent 3d1aac4 commit 920c092

File tree

1 file changed

+25
-23
lines changed

1 file changed

+25
-23
lines changed

test/test_client.py

Lines changed: 25 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -47,29 +47,31 @@ def test_urlencode(self):
4747
encoded_params = openrouteservice.client._urlencode_params([("address", "=Sydney ~")])
4848
self.assertEqual("address=%3DSydney+~", encoded_params)
4949

50-
@responses.activate
51-
def test_queries_per_minute_sleep_function(self):
52-
# This test assumes that the time to run a mocked query is
53-
# relatively small, eg a few milliseconds. We define a rate of
54-
# 3 queries per second, and run double that, which should take at
55-
# least 1 minute but no more than 2.
56-
queries_per_minute = 2
57-
query_range = range(queries_per_minute * 2)
58-
59-
for _ in query_range:
60-
responses.add(responses.GET,
61-
'https://api.openrouteservice.org/directions',
62-
body='{"status":"OK","results":[]}',
63-
status=200,
64-
content_type='application/json')
65-
66-
client = openrouteservice.Client(key="58d904a497c67e00015b45fcc8de25513ac6446692280cb066e66e8c",
67-
queries_per_minute=queries_per_minute)
68-
start = time.time()
69-
for idx, _ in enumerate(query_range):
70-
client.directions(self.coords_valid)
71-
end = time.time()
72-
self.assertTrue(start + 60 < end < start + 120)
50+
# Reactivate this test when secret env variable is set up in Travis
51+
52+
# @responses.activate
53+
# def test_queries_per_minute_sleep_function(self):
54+
# # This test assumes that the time to run a mocked query is
55+
# # relatively small, eg a few milliseconds. We define a rate of
56+
# # 3 queries per second, and run double that, which should take at
57+
# # least 1 minute but no more than 2.
58+
# queries_per_minute = 2
59+
# query_range = range(queries_per_minute * 2)
60+
#
61+
# for _ in query_range:
62+
# responses.add(responses.GET,
63+
# 'https://api.openrouteservice.org/directions',
64+
# body='{"status":"OK","results":[]}',
65+
# status=200,
66+
# content_type='application/json')
67+
#
68+
# client = openrouteservice.Client(key="58d904a497c67e00015b45fcc8de25513ac6446692280cb066e66e8c",
69+
# queries_per_minute=queries_per_minute)
70+
# start = time.time()
71+
# for idx, _ in enumerate(query_range):
72+
# client.directions(self.coords_valid)
73+
# end = time.time()
74+
# self.assertTrue(start + 60 < end < start + 120)
7375

7476
def test_overquerylimit_error(self):
7577
# Assume more queries_per_minute than allowed by API policy and

0 commit comments

Comments
 (0)