Skip to content
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

Closed
2 tasks done
jbrooksuk opened this issue Nov 25, 2014 · 8 comments
Closed
2 tasks done

Tests #39

jbrooksuk opened this issue Nov 25, 2014 · 8 comments
Labels
Enhancement Help Wanted Issues that need more help
Milestone

Comments

@jbrooksuk
Copy link
Member

Since Cachet is being developed to be deployed live (straight from master) it should at least go through some tests.

  • Create test suite
  • Link Travis CI

I'm not too worried about 100% coverage straightaway. We can spend time writing tests later on.

@jbrooksuk jbrooksuk added Enhancement Help Wanted Issues that need more help labels Nov 25, 2014
@Ehesp
Copy link
Contributor

Ehesp commented Nov 25, 2014

How far are we going in order to test the actual application, say via unit testing?

@jbrooksuk
Copy link
Member Author

PHPUnit, mock tests, test for expected outputs etc.

@jbrooksuk
Copy link
Member Author

The guys at @codeclimate have an awesome system for showing code coverage too.

@Ehesp
Copy link
Contributor

Ehesp commented Nov 25, 2014

Okay, quite a bit of work to do then on the controllers, DI etc.

@jbrooksuk
Copy link
Member Author

Yeah, it's not a high priority but it would help catch any silly mistakes.

@Ehesp
Copy link
Contributor

Ehesp commented Nov 27, 2014

I think as the code has already been written I'll config Mockery to only allow existing methods on Mocked classes... Mockery::getConfiguration()->allowMockingNonExistentMethods(false);

@jbrooksuk
Copy link
Member Author

@Ehesp I've setup Travis and the tests are now running.

@jbrooksuk
Copy link
Member Author

We have a test suite coming along let's close this and just add tests as we go now.

@CuReSoft CuReSoft mentioned this issue Jan 4, 2015
@GrahamCampbell GrahamCampbell modified the milestone: V0.1.0 Alpha Jul 25, 2015
@efriel efriel mentioned this issue May 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Help Wanted Issues that need more help
Projects
None yet
Development

No branches or pull requests

3 participants