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

Test cases #4

Open
farhaanbukhsh opened this issue Aug 15, 2018 · 6 comments
Open

Test cases #4

farhaanbukhsh opened this issue Aug 15, 2018 · 6 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@farhaanbukhsh
Copy link
Member

There are no test cases yet each function has to have test cases.

@hellozee
Copy link
Collaborator

Right now the code is just a set of unorganized functions, test cases can be written after the code has been refactored properly

@hellozee hellozee added enhancement New feature or request help wanted Extra attention is needed labels Oct 23, 2018
@farhaanbukhsh
Copy link
Member Author

I guess after @hellozee made the changes we are up for a test sprint :)

@sourabh1031
Copy link

@farhaanbukhsh i will work on this . can you provide me guide of how to write unit test cases in go

@hellozee
Copy link
Collaborator

@sourabh1031 I guess you can take a look at how the tests for the logger and the config manager is written. But I myself am not sure how to write the tests for the server and the indexer or the fact that they need any tests or not.

Testing seems kind of overkill at this stage of the program, though, what are you thoughts on it @farhaanbukhsh

@farhaanbukhsh
Copy link
Member Author

what I think is we can have a basic frame for tests.

@sourabh1031
Copy link

i will work on the issue. @farhaanbukhsh @hellozee can you suggest me resources to fix the issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants