-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Tests #39
Comments
How far are we going in order to test the actual application, say via unit testing? |
PHPUnit, mock tests, test for expected outputs etc. |
The guys at @codeclimate have an awesome system for showing code coverage too. |
Okay, quite a bit of work to do then on the controllers, DI etc. |
Yeah, it's not a high priority but it would help catch any silly mistakes. |
I think as the code has already been written I'll config Mockery to only allow existing methods on Mocked classes... |
@Ehesp I've setup Travis and the tests are now running. |
We have a test suite coming along let's close this and just add tests as we go now. |
Since Cachet is being developed to be deployed live (straight from
master
) it should at least go through some tests.I'm not too worried about 100% coverage straightaway. We can spend time writing tests later on.
The text was updated successfully, but these errors were encountered: