Merge pull request #19 from cardforcoin/master

Fix CoinbaseError

Author: mhluongo

coinbase/mock.py

@@ -113,10 +113,12 @@ def get_transaction(self, transaction_id):
         return self._transactions[transaction_id]
 
     def get_user_details(self):
-        raise NotImplementedError  # todo
+        # todo - this could be mocked better
+        return CoinbaseUser(email='[email protected]')
 
     def generate_receive_address(self, callback_url=None):
-        raise NotImplementedError  # todo
+        # todo - this could be mocked better
+        return '1DzkRzSUqm8jGhT8wp7E8XNMcr9J3nT3SX'
 
     def create_button(self, button, account_id=None):
         id = random_button_id()

coinbase/models/error.py

@@ -4,4 +4,4 @@
 class CoinbaseError(Exception):
 
     def __init__(self, message, errors=None):
-        super(CoinbaseError, Exception).__init__(self, ' '.join([message] + (errors or [])))
+        super(CoinbaseError, self).__init__(self, ' '.join([message] + (errors or [])))

coinbase/tests/test_error.py

@@ -0,0 +1,6 @@
+from coinbase.models import CoinbaseError
+
+
+def test_coinbase_error_instantiation():
+    CoinbaseError("message")
+    CoinbaseError("message", ["abc", "def"])