Problem: Driver tests failing (#380)

* Problem: Driver tests failing

Solution: Fix necessary tests for release.

* Problem: Fixture transaction not commited

Solution: Post the transaction with 'commit' mode

* Fix attempt 1

* Problem: bdb_node_pubkey is not available,

Solution: Remove bdb_node_pubkey.
Update the fixture transactions_api_full_url to have paramter 'mode=commit

Author: kansi

tests/conftest.py

@@ -171,11 +171,6 @@ def bdb_port():
     return environ.get('BDB_PORT', '9984')
 
 
-@fixture
-def bdb_node_pubkey():
-    return environ['BIGCHAINDB_KEYPAIR_PUBLIC']
-
-
 @fixture
 def bdb_node(bdb_host, bdb_port):
     return 'http://{host}:{port}'.format(host=bdb_host, port=bdb_port)
@@ -194,7 +189,7 @@ def api_root(bdb_node):
 
 @fixture
 def transactions_api_full_url(api_root):
-    return api_root + '/transactions'
+    return api_root + '/transactions?mode=commit'
 
 
 @fixture
@@ -239,7 +234,6 @@ def signed_alice_transaction(alice_privkey, alice_transaction_obj):
 
 
 @fixture
-@await_transaction
 def persisted_alice_transaction(signed_alice_transaction,
                                 transactions_api_full_url):
     response = requests.post(transactions_api_full_url,
@@ -310,7 +304,6 @@ def signed_carol_bicycle_transaction(request, carol_keypair,
 
 
 @fixture
-@await_transaction
 def persisted_carol_bicycle_transaction(transactions_api_full_url,
                                         signed_carol_bicycle_transaction):
     response = requests.post(

tests/test_driver.py

@@ -34,13 +34,11 @@ def test_driver_init(self, nodes, headers, normalized_nodes):
         assert driver.transactions
         assert driver.outputs
 
-    def test_info(self, driver, bdb_node_pubkey):
+    def test_info(self, driver):
         response = driver.info()
         assert 'api' in response
         assert 'docs' in response
         assert response['keyring'] == []
-        # TODO: update when server exposes public key
-        # assert response['public_key'] == bdb_node_pubkey
         assert response['software'] == 'BigchainDB'
         assert 'version' in response
 
@@ -146,9 +144,9 @@ def test_get_empty(self, driver, query_params):
         response = driver.transactions.get(asset_id='a' * 64)
         assert response == []
 
-    @mark.parametrize('operation,tx_qty', (
+    @mark.parametrize('operation,tx_qty', [
         (None, 3), ('CREATE', 1), ('TRANSFER', 2)
-    ))
+    ])
     @mark.usefixtures('persisted_transfer_dimi_car_to_ewy')
     def test_get(self, driver,
                  signed_carol_car_transaction, operation, tx_qty):