refactor: blocks transactions endpoint wildcard arguments (#131)

Author: alexbarnsley

client/api/blocks.py

@@ -21,9 +21,5 @@ def first(self):
     def last(self):
         return self.with_endpoint('api').request_get('blocks/last')
 
-    def transactions(self, block_id, page=None, limit=100):
-        params = {
-            'page': page,
-            'limit': limit,
-        }
-        return self.with_endpoint('api').request_get(f'blocks/{block_id}/transactions', params)
+    def transactions(self, block_id, **kwargs):
+        return self.with_endpoint('api').request_get(f'blocks/{block_id}/transactions', kwargs)

tests/api/test_blocks.py

@@ -99,23 +99,6 @@ def test_last_calls_correct_url():
     assert responses.calls[0].request.url == 'http://127.0.0.1:4002/api/blocks/last'
 
 
-def test_transactions_calls_correct_url_with_default_params():
-    block_id = '12345'
-    responses.add(
-        responses.GET,
-        'http://127.0.0.1:4002/api/blocks/{}/transactions'.format(block_id),
-        json={'success': True},
-        status=200
-    )
-
-    client = ArkClient('http://127.0.0.1:4002/api')
-    client.blocks.transactions(block_id)
-    assert len(responses.calls) == 1
-    assert responses.calls[0].request.url == (
-        'http://127.0.0.1:4002/api/blocks/12345/transactions?limit=100'
-    )
-
-
 def test_transactions_calls_correct_url_with_passed_in_params():
     block_id = '12345'
     responses.add(
@@ -126,10 +109,11 @@ def test_transactions_calls_correct_url_with_passed_in_params():
     )
 
     client = ArkClient('http://127.0.0.1:4002/api')
-    client.blocks.transactions(block_id, page=5, limit=69)
+    client.blocks.transactions(block_id, page=5, limit=69, orderBy="timestamp.epoch")
     assert len(responses.calls) == 1
     assert responses.calls[0].request.url.startswith(
         'http://127.0.0.1:4002/api/blocks/12345/transactions?'
     )
     assert 'page=5' in responses.calls[0].request.url
     assert 'limit=69' in responses.calls[0].request.url
+    assert 'orderBy=timestamp.epoch' in responses.calls[0].request.url