contiamo / contiamo-client-python

MIT License
2 stars 2 forks source link

Fix broken tests #13

Closed bmwilly closed 6 years ago

bmwilly commented 6 years ago

Two tests are currently broken:

➜  contiamo-client-python git:(update-contracts) ✗ pytest contiamo/tests
======================================================== test session starts ========================================================
platform darwin -- Python 3.6.4, pytest-3.4.1, py-1.5.2, pluggy-0.6.0
rootdir: /Users/brandon/projects/contiamo-client-python, inifile:
plugins: nbval-0.9.0, aspectlib-1.4.2
collected 34 items / 2 errors

============================================================== ERRORS ===============================================================
_________________________________________ ERROR collecting contiamo/tests/test_requests.py __________________________________________
contiamo/tests/test_requests.py:28: in <module>
    data_client = DataClient(contract_id, contract_token, api_base=api_base)
E   NameError: name 'contract_id' is not defined
---------------------------------------------------------- Captured stderr ----------------------------------------------------------
/Users/brandon/projects/contiamo-client-python/contiamo/tests/test_requests.py:25: UserWarning: These tests will be skipped, as they require configuration information that is unavailable.
  warnings.warn('These tests will be skipped, as they require configuration information that is unavailable.')
_________________________________________ ERROR collecting contiamo/tests/test_resources.py _________________________________________
contiamo/tests/test_resources.py:28: in <module>
    class ResourceTestCase(unittest.TestCase):
contiamo/tests/test_resources.py:30: in ResourceTestCase
    client = Client(api_key, api_base=api_base)
E   NameError: name 'api_key' is not defined
---------------------------------------------------------- Captured stderr ----------------------------------------------------------
/Users/brandon/projects/contiamo-client-python/contiamo/tests/test_resources.py:23: UserWarning: These tests will be skipped, as they require configuration information that is unavailable.
  warnings.warn('These tests will be skipped, as they require configuration information that is unavailable.')
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 2 errors during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
====================================================== 2 error in 0.86 seconds ======================================================
bmwilly commented 6 years ago

The above errors occur because you need contiamo/tests/data/test_config.yml to run the tests. More informative errors are in f93343ffbf3d653d9551ba84f307585bea77702d and ae74c86282147e9320cdf2b5adbd2ab66d7334f6.