diff --git a/test/functional/interface_rest.py b/test/functional/interface_rest.py --- a/test/functional/interface_rest.py +++ b/test/functional/interface_rest.py @@ -6,6 +6,7 @@ from codecs import encode from decimal import Decimal +from enum import Enum import http.client from io import BytesIO import json @@ -22,26 +23,22 @@ ) -def http_get_call(host, port, path, response_object=0): - """Make a simple HTTP GET request.""" - conn = http.client.HTTPConnection(host, port) - conn.request('GET', path) +class ReqType(Enum): + JSON = 1 + BIN = 2 + HEX = 3 - if response_object: - return conn.getresponse() - return conn.getresponse().read().decode('utf-8') +class RetType(Enum): + OBJ = 1 + BYTES = 2 + JSON = 3 -def http_post_call(host, port, path, requestdata='', response_object=0): - """Make a simple HTTP POST request with a request body.""" - conn = http.client.HTTPConnection(host, port) - conn.request('POST', path, requestdata) - - if response_object: - return conn.getresponse() - - return conn.getresponse().read() +def filter_output_indices_by_value(vouts, value): + for vout in vouts: + if vout['value'] == value: + yield vout['n'] class RESTTest (BitcoinTestFramework): @@ -53,10 +50,35 @@ def setup_network(self, split=False): super().setup_network() connect_nodes_bi(self.nodes[0], self.nodes[2]) + self.url = urllib.parse.urlparse(self.nodes[0].url) + + def test_rest_request(self, uri, http_method='GET', req_type=ReqType.JSON, body='', status=200, ret_type=RetType.JSON): + rest_uri = '/rest' + uri + if req_type == ReqType.JSON: + rest_uri += '.json' + elif req_type == ReqType.BIN: + rest_uri += '.bin' + elif req_type == ReqType.HEX: + rest_uri += '.hex' + + conn = http.client.HTTPConnection(self.url.hostname, self.url.port) + self.log.debug('{} {} {}'.format(http_method, rest_uri, body)) + if http_method == 'GET': + conn.request('GET', rest_uri) + elif http_method == 'POST': + conn.request('POST', rest_uri, body) + resp = conn.getresponse() + + assert_equal(resp.status, status) + + if ret_type == RetType.OBJ: + return resp + elif ret_type == RetType.BYTES: + return resp.read() + elif ret_type == RetType.JSON: + return json.loads(resp.read().decode('utf-8'), parse_float=Decimal) def run_test(self): - url = urllib.parse.urlparse(self.nodes[0].url) - self.log.info("Mine blocks and send Bitcoin Cash to node 1") self.nodes[0].generate(1) @@ -76,40 +98,32 @@ self.log.info("Load the transaction using the /tx URI") - json_request = "/rest/tx/{}.json".format(txid) - json_string = http_get_call(url.hostname, url.port, json_request) - json_obj = json.loads(json_string) + json_obj = self.test_rest_request("/tx/{}".format(txid)) # Get the vin to later check for utxo (should be spent by then) - vintx = json_obj['vin'][0]['txid'] + spent = (json_obj['vin'][0]['txid'], json_obj['vin'][0]['vout']) # Get n of 0.1 outpoint - n = 0 - for vout in json_obj['vout']: - if vout['value'] == 0.1: - n = vout['n'] + n, = filter_output_indices_by_value(json_obj['vout'], Decimal('0.1')) + spending = (txid, n) self.log.info("Query an unspent TXO using the /getutxos URI") - json_request = "/rest/getutxos/{}-{}.json".format(txid, str(n)) - json_string = http_get_call(url.hostname, url.port, json_request) - json_obj = json.loads(json_string) + json_obj = self.test_rest_request("/getutxos/{}-{}".format(*spending)) # Check chainTip response assert_equal(json_obj['chaintipHash'], bb_hash) # Make sure there is one utxo assert_equal(len(json_obj['utxos']), 1) - assert_equal(json_obj['utxos'][0]['value'], 0.1) + assert_equal(json_obj['utxos'][0]['value'], Decimal('0.1')) self.log.info("Query a spent TXO using the /getutxos URI") - json_request = "/rest/getutxos/{}-0.json".format(vintx) - json_string = http_get_call(url.hostname, url.port, json_request) - json_obj = json.loads(json_string) + json_obj = self.test_rest_request("/getutxos/{}-{}".format(*spent)) # Check chainTip response assert_equal(json_obj['chaintipHash'], bb_hash) - # Make sure there is no utox in the response because this oupoint has + # Make sure there is no utxo in the response because this outpoint has # been spent assert_equal(len(json_obj['utxos']), 0) @@ -118,10 +132,9 @@ self.log.info("Query two TXOs using the /getutxos URI") - json_request = "/rest/getutxos/{}-{}/{}-0.json".format( - txid, str(n), vintx) - json_string = http_get_call(url.hostname, url.port, json_request) - json_obj = json.loads(json_string) + json_obj = self.test_rest_request( + "/getutxos/{}-{}/{}-{}".format(*(spending + spent))) + assert_equal(len(json_obj['utxos']), 1) assert_equal(json_obj['bitmap'], "10") @@ -131,13 +144,12 @@ bb_hash = self.nodes[0].getbestblockhash() bin_request = b'\x01\x02' - bin_request += hex_str_to_bytes(txid) - bin_request += pack("i", n) - bin_request += hex_str_to_bytes(vintx) - bin_request += pack("i", 0) + for txid, n in [spending, spent]: + bin_request += hex_str_to_bytes(txid) + bin_request += pack("i", n) - bin_response = http_post_call( - url.hostname, url.port, '/rest/getutxos.bin', bin_request) + bin_response = self.test_rest_request( + "/getutxos", http_method='POST', req_type=ReqType.BIN, body=bin_request, ret_type=RetType.BYTES) output = BytesIO() output.write(bin_response) output.seek(0) @@ -156,84 +168,53 @@ # Do a tx and don't sync txid = self.nodes[0].sendtoaddress(self.nodes[1].getnewaddress(), 0.1) - json_request = "/rest/tx/{}.json".format(txid) - json_string = http_get_call(url.hostname, url.port, json_request) - json_obj = json.loads(json_string) + json_obj = self.test_rest_request("/tx/{}".format(txid)) # Get the spent output to later check for utxo (should be spent by then) - spent = '{}-{}'.format(json_obj['vin'][0] - ['txid'], json_obj['vin'][0]['vout']) + spent = (json_obj['vin'][0]['txid'], json_obj['vin'][0]['vout']) # Get n of 0.1 outpoint - n = 0 - for vout in json_obj['vout']: - if vout['value'] == 0.1: - n = vout['n'] - spending = '{}-{}'.format(txid, n) - - json_request = '/rest/getutxos/{}.json'.format(spending) - json_string = http_get_call(url.hostname, url.port, json_request) - json_obj = json.loads(json_string) + n, = filter_output_indices_by_value(json_obj['vout'], Decimal('0.1')) + spending = (txid, n) + + json_obj = self.test_rest_request("/getutxos/{}-{}".format(*spending)) assert_equal(len(json_obj['utxos']), 0) - json_request = '/rest/getutxos/checkmempool/{}.json'.format(spending) - json_string = http_get_call(url.hostname, url.port, json_request) - json_obj = json.loads(json_string) + json_obj = self.test_rest_request( + "/getutxos/checkmempool/{}-{}".format(*spending)) assert_equal(len(json_obj['utxos']), 1) - json_request = '/rest/getutxos/{}.json'.format(spent) - json_string = http_get_call(url.hostname, url.port, json_request) - json_obj = json.loads(json_string) + json_obj = self.test_rest_request("/getutxos/{}-{}".format(*spent)) assert_equal(len(json_obj['utxos']), 1) - json_request = '/rest/getutxos/checkmempool/{}.json'.format(spent) - json_string = http_get_call(url.hostname, url.port, json_request) - json_obj = json.loads(json_string) + json_obj = self.test_rest_request( + "/getutxos/checkmempool/{}-{}".format(*spent)) assert_equal(len(json_obj['utxos']), 0) self.nodes[0].generate(1) self.sync_all() - json_request = '/rest/getutxos/{}.json'.format(spending) - json_string = http_get_call(url.hostname, url.port, json_request) - json_obj = json.loads(json_string) + json_obj = self.test_rest_request("/getutxos/{}-{}".format(*spending)) assert_equal(len(json_obj['utxos']), 1) - json_request = '/rest/getutxos/checkmempool/{}.json'.format(spending) - json_string = http_get_call(url.hostname, url.port, json_request) - json_obj = json.loads(json_string) + json_obj = self.test_rest_request( + "/getutxos/checkmempool/{}-{}".format(*spending)) assert_equal(len(json_obj['utxos']), 1) # Do some invalid requests - json_request = '{"checkmempool' - response = http_post_call( - url.hostname, url.port, '/rest/getutxos.json', json_request, True) - # Must be a 400 because we send an invalid json request - assert_equal(response.status, 400) - - json_request = '{"checkmempool' - response = http_post_call( - url.hostname, url.port, '/rest/getutxos.bin', json_request, True) - # Must be a 400 because we send an invalid bin request - assert_equal(response.status, 400) - - response = http_post_call( - url.hostname, url.port, '/rest/getutxos/checkmempool.json', '', True) - # Must be a 400 because we send an invalid bin request - assert_equal(response.status, 400) + self.test_rest_request("/getutxos", http_method='POST', req_type=ReqType.JSON, + body='{"checkmempool', status=400, ret_type=RetType.OBJ) + self.test_rest_request("/getutxos", http_method='POST', req_type=ReqType.BIN, + body='{"checkmempool', status=400, ret_type=RetType.OBJ) + self.test_rest_request("/getutxos/checkmempool", http_method='POST', + req_type=ReqType.JSON, status=400, ret_type=RetType.OBJ) # Test limits - json_request = '/rest/getutxos/checkmempool/' + \ - '/'.join(["{}-{}".format(txid, n) for n in range(20)]) + '.json' - response = http_post_call( - url.hostname, url.port, json_request, '', True) - # Must be a 400 because we exceeding the limits - assert_equal(response.status, 400) - - json_request = '/rest/getutxos/checkmempool/' + \ - '/'.join(['{}-{}'.format(txid, n) for n in range(15)]) + '.json' - response = http_post_call( - url.hostname, url.port, json_request, '', True) - # Must be a 200 because we are within the limits - assert_equal(response.status, 200) + long_uri = '/'.join(["{}-{}".format(txid, n) for n in range(20)]) + self.test_rest_request("/getutxos/checkmempool/{}".format(long_uri), + http_method='POST', status=400, ret_type=RetType.OBJ) + + long_uri = '/'.join(['{}-{}'.format(txid, n) for n in range(15)]) + self.test_rest_request( + "/getutxos/checkmempool/{}".format(long_uri), http_method='POST', status=200) # Generate block to not affect upcoming tests self.nodes[0].generate( @@ -243,33 +224,29 @@ self.log.info("Test the /block and /headers URIs") # Check binary format - response = http_get_call( - url.hostname, url.port, '/rest/block/{}.bin'.format(bb_hash), True) - assert_equal(response.status, 200) + response = self.test_rest_request( + "/block/{}".format(bb_hash), req_type=ReqType.BIN, ret_type=RetType.OBJ) assert_greater_than(int(response.getheader('content-length')), 80) response_str = response.read() # Compare with block header - response_header = http_get_call( - url.hostname, url.port, '/rest/headers/1/{}.bin'.format(bb_hash), True) - assert_equal(response_header.status, 200) + response_header = self.test_rest_request( + "/headers/1/{}".format(bb_hash), req_type=ReqType.BIN, ret_type=RetType.OBJ) assert_equal(int(response_header.getheader('content-length')), 80) response_header_str = response_header.read() assert_equal(response_str[0:80], response_header_str) # Check block hex format - response_hex = http_get_call( - url.hostname, url.port, '/rest/block/{}.hex'.format(bb_hash), True) - assert_equal(response_hex.status, 200) + response_hex = self.test_rest_request( + "/block/{}".format(bb_hash), req_type=ReqType.HEX, ret_type=RetType.OBJ) assert_greater_than(int(response_hex.getheader('content-length')), 160) response_hex_str = response_hex.read() assert_equal( encode(response_str, "hex_codec")[0:160], response_hex_str[0:160]) # Compare with hex block header - response_header_hex = http_get_call( - url.hostname, url.port, '/rest/headers/1/{}.hex'.format(bb_hash), True) - assert_equal(response_header_hex.status, 200) + response_header_hex = self.test_rest_request( + "/headers/1/{}".format(bb_hash), req_type=ReqType.HEX, ret_type=RetType.OBJ) assert_greater_than( int(response_header_hex.getheader('content-length')), 160) response_header_hex_str = response_header_hex.read() @@ -278,17 +255,11 @@ 0:160], response_header_hex_str[0:160]) # Check json format - block_json_string = http_get_call( - url.hostname, url.port, '/rest/block/{}.json'.format(bb_hash)) - block_json_obj = json.loads(block_json_string) + block_json_obj = self.test_rest_request("/block/{}".format(bb_hash)) assert_equal(block_json_obj['hash'], bb_hash) # Compare with json block header - response_header_json = http_get_call( - url.hostname, url.port, '/rest/headers/1/{}.json'.format(bb_hash), True) - assert_equal(response_header_json.status, 200) - response_header_json_str = response_header_json.read().decode('utf-8') - json_obj = json.loads(response_header_json_str, parse_float=Decimal) + json_obj = self.test_rest_request("/headers/1/{}".format(bb_hash)) # Ensure that there is one header in the json response assert_equal(len(json_obj), 1) # Request/response hash should be the same @@ -302,27 +273,20 @@ # See if we can get 5 headers in one response self.nodes[1].generate(5) self.sync_all() - response_header_json = http_get_call( - url.hostname, url.port, '/rest/headers/5/{}.json'.format(bb_hash), True) - assert_equal(response_header_json.status, 200) - response_header_json_str = response_header_json.read().decode('utf-8') - json_obj = json.loads(response_header_json_str) + json_obj = self.test_rest_request("/headers/5/{}".format(bb_hash)) # Now we should have 5 header objects assert_equal(len(json_obj), 5) self.log.info("Test the /tx URI") tx_hash = block_json_obj['tx'][0]['txid'] - json_string = http_get_call( - url.hostname, url.port, '/rest/tx/{}.json'.format(tx_hash)) - json_obj = json.loads(json_string) + json_obj = self.test_rest_request("/tx/{}".format(tx_hash)) assert_equal(json_obj['txid'], tx_hash) # Check hex format response - hex_string = http_get_call( - url.hostname, url.port, '/rest/tx/{}.hex'.format(tx_hash), True) - assert_equal(hex_string.status, 200) - assert_greater_than(int(response.getheader('content-length')), 10) + hex_response = self.test_rest_request( + "/tx/{}".format(tx_hash), req_type=ReqType.HEX, ret_type=RetType.OBJ) + assert_greater_than(int(hex_response.getheader('content-length')), 10) self.log.info("Test tx inclusion in the /mempool and /block URIs") @@ -338,19 +302,15 @@ # Check that there are exactly 3 transactions in the TX memory pool # before generating the block - json_string = http_get_call( - url.hostname, url.port, '/rest/mempool/info.json') - json_obj = json.loads(json_string) + json_obj = self.test_rest_request("/mempool/info") assert_equal(json_obj['size'], 3) # The size of the memory pool should be greater than 3x ~100 bytes assert_greater_than(json_obj['bytes'], 300) # Check that there are our submitted transactions in the TX memory pool - json_string = http_get_call( - url.hostname, url.port, '/rest/mempool/contents.json') - json_obj = json.loads(json_string) + json_obj = self.test_rest_request("/mempool/contents") for i, tx in enumerate(txs): - assert_equal(tx in json_obj, True) + assert tx in json_obj assert_equal(json_obj[tx]['spentby'], txs[i + 1:i + 2]) assert_equal(json_obj[tx]['depends'], txs[i - 1:i]) @@ -359,27 +319,22 @@ self.sync_all() # Check if the 3 tx show up in the new block - json_string = http_get_call( - url.hostname, url.port, '/rest/block/{}.json'.format(newblockhash[0])) - json_obj = json.loads(json_string) - for tx in json_obj['tx']: - if not 'coinbase' in tx['vin'][0]: # exclude coinbase - assert_equal(tx['txid'] in txs, True) + json_obj = self.test_rest_request("/block/{}".format(newblockhash[0])) + non_coinbase_txs = {tx['txid'] + for tx in json_obj['tx'] if 'coinbase' not in tx['vin'][0]} + assert_equal(non_coinbase_txs, set(txs)) # Check the same but without tx details - json_string = http_get_call( - url.hostname, url.port, '/rest/block/notxdetails/{}.json'.format(newblockhash[0])) - json_obj = json.loads(json_string) + json_obj = self.test_rest_request( + "/block/notxdetails/{}".format(newblockhash[0])) for tx in txs: - assert_equal(tx in json_obj['tx'], True) + assert tx in json_obj['tx'] self.log.info("Test the /chaininfo URI") bb_hash = self.nodes[0].getbestblockhash() - json_string = http_get_call( - url.hostname, url.port, '/rest/chaininfo.json') - json_obj = json.loads(json_string) + json_obj = self.test_rest_request("/chaininfo") assert_equal(json_obj['bestblockhash'], bb_hash)