Merge pull request #470 from tutorcruncher/double-hlr-search-time

Author: tomhamiltonstubber

src/version.py

@@ -1 +1 @@
-VERSION = '1.0.33'
+VERSION = '1.0.34'

src/worker/sms.py

@@ -187,7 +187,7 @@ async def _messagebird_get_number_cost(self, number: Number):
                 hlr_id = hlr_data.get('id')
 
                 network, hlr = None, None
-                for i in range(30):
+                for i in range(60):
                     try:
                         r = await self.messagebird.get(f'hlr/{hlr_id}')
                     except ApiError:

tests/test_sms.py

@@ -244,7 +244,7 @@ def test_messagebird_no_hlr(cli, tmpdir, dummy_server, worker, caplog, loop):
     assert worker.test_run() == 1
     assert [
         'POST /messagebird/hlr > 201',
-        *['GET /messagebird/hlr/447888888888 > 200' for _ in range(30)],
+        *['GET /messagebird/hlr/447888888888 > 200' for _ in range(60)],
     ] == dummy_server.log
     dummy_server.log = []
     assert 'No HLR result found for +447888888888 after 30 attempts' in caplog.messages
@@ -263,7 +263,7 @@ def test_messsagebird_no_hlr_found(cli, tmpdir, dummy_server, worker, caplog, lo
     assert worker.test_run() == 1
     assert [
         'POST /messagebird/hlr > 201',
-        *['GET /messagebird/hlr/447877777777 > 404' for _ in range(30)],
+        *['GET /messagebird/hlr/447877777777 > 404' for _ in range(60)],
     ] == dummy_server.log
     dummy_server.log = []
     assert 'No HLR result found for +447877777777 after 30 attempts' in caplog.messages