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

Create unit tests #6

Closed
rmccue opened this Issue Oct 10, 2013 · 4 comments

Comments

Projects
None yet
2 participants
@rmccue
Member

rmccue commented Oct 10, 2013

We should start working on unit tests for the API. I'd like to aim for >90% code coverage, ideally 100% if possible. As such, setting the milestone for 3 weeks from now so that we can get this done thoroughly.

@scribu

This comment has been minimized.

Show comment
Hide comment
@scribu

scribu Dec 16, 2013

So do you want to do:

a) integration testing, i.e. you set up a WP instance and do everything via actual HTTP calls or
b) unit testing, i.e. you call the router (or even individual route handlers) directly from PHP?

scribu commented Dec 16, 2013

So do you want to do:

a) integration testing, i.e. you set up a WP instance and do everything via actual HTTP calls or
b) unit testing, i.e. you call the router (or even individual route handlers) directly from PHP?

@rmccue

This comment has been minimized.

Show comment
Hide comment
@rmccue

rmccue Dec 16, 2013

Member

We've already got a start on integration testing via the reference client, but we need unit testing.

We should be able to call the route handlers themselves, as well as testing the routing separately.

Member

rmccue commented Dec 16, 2013

We've already got a start on integration testing via the reference client, but we need unit testing.

We should be able to call the route handlers themselves, as well as testing the routing separately.

@rmccue

This comment has been minimized.

Show comment
Hide comment
@rmccue

rmccue Feb 17, 2014

Member

Made a start in #65, let's crank out the rest of these.

Member

rmccue commented Feb 17, 2014

Made a start in #65, let's crank out the rest of these.

@rmccue

This comment has been minimized.

Show comment
Hide comment
@rmccue

rmccue Mar 4, 2014

Member

Step 1 down! Step 2 is creating more tests, so I've opened #98, #99, #100, #101, #102 and #103.

Go team!

Member

rmccue commented Mar 4, 2014

Step 1 down! Step 2 is creating more tests, so I've opened #98, #99, #100, #101, #102 and #103.

Go team!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment