thereadme/hapi-sequelize-crud
1
0
Fork 0
Code Issues4 Pull Requests1 Projects Releases Wiki Activity

add tests #19

New Issue
Closed
opened 2016-09-03 20:23:44 +00:00 by joeybaker · 2 comments
joeybaker commented 2016-09-03 20:23:44 +00:00 (Migrated from github.com)

I've run into enough edge cases in the code that it would be really nice to have tests to give assurance that the right things are happening. Though I'm a tape/tap fan usually, I've been wanting to give Ava or Jest a try.

@mdibaiee what do you think of adding tests and do you care about the framework?

I've run into enough edge cases in the code that it would be really nice to have tests to give assurance that the right things are happening. Though I'm a tape/tap fan usually, I've been wanting to give Ava or Jest a try. @mdibaiee what do you think of adding tests and do you care about the framework?
👍 1
mdibaiee commented 2016-09-04 04:03:36 +00:00 (Migrated from github.com)

Sure thing tests are great! I have no preference for any test framework here, please use the one you think is appropriate.

Thanks!

Sure thing tests are great! I have no preference for any test framework here, please use the one you think is appropriate. Thanks!
mdibaiee commented 2016-09-06 05:29:34 +00:00 (Migrated from github.com)

fixed in .

fixed in #22.
Sign in to join this conversation.
No Label
enhancement
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: thereadme/hapi-sequelize-crud#19
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?