-
Notifications
You must be signed in to change notification settings - Fork 137
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
No tests! #12
Comments
Sounds good to me - just having tried the demo key using Postman it appears for GET calls anyway the results are repeatable. |
That's good - I think we can avoid any difficulties with repeatability by designing the tests so we only rely on the data we create, e.g.
|
Is there a preference as to which testing framework to use? |
I'm pretty flexible really - we use VSTest with Moq for some internal projects but have seen a lot of XUnit around too. Do you have a preference or any relevant experience? |
The code for our websites use VSTest and Moq but was added later.
I'm currently working on a new project and am using test-driven development
on it with xUnit.
…On Wed, Jun 6, 2018, 6:08 AM Dave Clarke ***@***.***> wrote:
I'm pretty flexible really - we use VSTest with Moq for some internal
projects but have seen a lot of XUnit around too. Do you have a preference
or any relevant experience?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#12 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ANXycVAJlXHdkIwfP2jfS-wlZxLnQLizks5t56omgaJpZM4TtzCi>
.
|
To keep the barrier to entry low and enable anyone to write tests I think looking at VSTest would be a great start. Are you thinking of kicking something off? I'd been to help get the ball rolling so let me know if there's anything I can provide! |
As soon as I can find a spare moment... :) |
We don't currently have any unit tests - with API wrappers like this, they can be difficult to add as you'd have to mock the HubSpot API.
But... we could look at using the HubSpot demo instance (they provide a demo API key etc.) so we could make actual API calls etc.
Any thoughts?
The text was updated successfully, but these errors were encountered: