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

Unittesting #87

Closed
Alkalit opened this issue Nov 24, 2015 · 3 comments
Closed

Unittesting #87

Alkalit opened this issue Nov 24, 2015 · 3 comments

Comments

@Alkalit
Copy link
Contributor

Alkalit commented Nov 24, 2015

How about to write some tests for this app?

@avelis
Copy link
Collaborator

avelis commented Nov 24, 2015

@Alkalit First, I appreciate you expressing interest in making a contribution. Always refreshing to see.

We currently have about 110 tests passing thus far. However, I always say you can never have to many unit tests.

If you make a pull request with any tests you write I can be happy to take a look and merge them into the master branch.

@Alkalit
Copy link
Contributor Author

Alkalit commented Nov 24, 2015

@avelis Deal.

@avelis
Copy link
Collaborator

avelis commented Jan 19, 2016

Addressed in pull request: #93

@avelis avelis closed this as completed Jan 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants