Use a mock for tests about logzio_terraform_client HOT 7 CLOSED

development branch for introducing mocks for testing - release_1_2_mocks

from logzio_terraform_client.

@pengux - agree - which functions would you mock? the ones that make the API HTTP calls? or some higher level of abstraction?

from logzio_terraform_client.

@jonboydell The whole API can be mocked with a simple HTTP server and httptest which just return fixed test data. The mock can be set to expect a specific request and return the appropriate response for it. This way, the one writing the test can for example inject invalid/valid responses to test against.

from logzio_terraform_client.

thanks @pengux - i thought you meant a mock as in https://github.com/golang/mock - i'm not au fait with httptest will look into it

from logzio_terraform_client.

@jonboydell I think something like https://github.com/jarcoal/httpmock is more suitable for mocking APIs

from logzio_terraform_client.

@pengux - didn't see that till just now, have already implemented httptest across the Users part of the lib - going to stick with that for Users but might use the other lib for Endpoints and Alerts and then upgrade - BTW - I am [email protected] - I'd prefer email over Slack ATM, if it doesn't work we can use Slack, but unless you pay for it, it deletes the chat history after 24 hours :(

from logzio_terraform_client.

framework is now in develop, so closing this issue - improvements to test coverage to be tracked as individual issues...

from logzio_terraform_client.