-
Notifications
You must be signed in to change notification settings - Fork 211
Test Coverage #55
Comments
Since I'm using Ardent in a production environment I wouldn't mind writing tests that covers the class. But it would require some refactoring and that could perhaps be reviewed by the maintainer. So if @laravelbook is fine with that, I could get on it. |
@modess Man you're a godsend! I've been extremely busy working on an enterprisey project (my dayjob) lately. It'd be nice to have you guys pick up the slack! ;) Please do go ahead with the test coverage. Thanks! |
I'm also available and interested to help out where needed. This is a great product that I'm getting a lot of use out of and I've really appreciated the work you've done on this so far @laravelbook |
Ok, I'm really starting to see the need of tests to make the library maintainable. I'm willing to do some structural changes but tests are essential to make sure I'm not destroying the library after the changes. However, I know nothing about mocking and stuff, so I may take some more time to write those things since I'll need to study some stuff first. But I'll work on this! |
I'm in the process of writing tests and am actively avoiding making structural changes. This, of course, will affect how tidy the tests look but will be less invasive. I'll make sure to document well and keep it as organized as possible. We can always refactor later after the package has coverage. |
This is exactly the idea, Dustin. I tried to start testing some days ago but had little success, since I have Igor Santos On Wed, Oct 16, 2013 at 6:24 PM, Dustin Wheeler [email protected]:
|
I'd like to see some test coverage for Ardent. Has anyone written any tests for it already?
The text was updated successfully, but these errors were encountered: