-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add tests #13
Comments
@virresh helo, I can work on it. I'm going to work at java_client. What classes do you prefer to cover with tests first? Assign me to this task if your pleasure). |
Hi @EEM86, Thanks a lot for volunteering! The java_client has been deprecated and is present for legacy purposes (noted in README here). However, I'd like to recommend you the Python version. If you're even a bit comfortable with Python, you can start looking at https://github.com/virresh/StockD/blob/master/python_client/app/routes.py The application is mainly based on a web-development framework flask, so all the UI-handling portion is done through html-css in static folder. Since there isn't any existing test framework in use, feel free to use any framework if you feel comfortable in that. Also, let me know if you're taking up the task of python testing, I'll assign you then :D |
@virresh Oh, I'm terribly sorry. I read the README file but missed the info that java_client is deprecated. Unfortunately, I'm not familiar with Python, but I have it on my to-do list. So, maybe next time, when I'll be ready for Python tasks... |
sure, no worries |
Currently there are no tests whatsoever.
It'd be great if someone can help adding unit, integration and end-to-end tests.
Any contribution in this regard will be appreciated.
The text was updated successfully, but these errors were encountered: