Running tests not against the live HubSpot demo account about hubspot3

[rheinwerk-mp]

Great that you are open changing the tests.

Regarding testing against the live API: Yes, but I would suggest to create a different set of tests (integration tests) for this purpose.
The unit tests on the other hand would only test the integrity of the code itself.

[jpetrucciani]

Thanks for bringing this issue up!

Yeah it'd probably be best to mock those all out - I had started on rebuilding the test suite and quickly ran into all sorts of problems trying to get how I wanted to test things working (including a nasty workaround that you've found in a couple different places for loading a record on boot to use for testing

hubspot3/hubspot3/test/test_contacts.py

Lines 12 to 14 in 6791c78

	# since we need to have an id to submit and to attempt to get a contact,
	# we need to be hacky here and fetch one upon loading this file.
	BASE_CONTACT = CONTACTS.get_all(limit=1)[0]

).

Good point on the test_get_all tests as well - there's a lot of the testing that I hadn't completely thought through when I started writing the tests!

I'd agree with you that mocking the calls would probably be best - I'm open to any methods of doing so that you guys are comfortable with!

[jpetrucciani]

I've also created a topic on the HubSpot developer forum about this, and asked if we could potentially have a different key specific for testing - I do feel that mocking it is fine, but not entirely optimal. If we could be hitting the real API during tests, that'd allow us to test for any changes that they've made as well.

[jpetrucciani]

Good point - that makes a lot of sense!

[jpetrucciani]

As of the latest commit/release 3.2.17, I've included my test account key in the tests, so we should be able to use that key for integration tests (so at least we can get the old tests back up and running on Travis!). I've also switched the CI to use tox across the three supported versions we've specified.

I'm still on board with also building out a bunch of unit tests for the library that use your mocking methods - I'll just start slowly writing out some new tests when I get some more time!