-
Notifications
You must be signed in to change notification settings - Fork 771
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
We don't have any tests #34
Comments
I am working on creating functional tests right now, will work on the unit tests once, the code refactor and code moving thing that @kadel and @ngtuna are working on, is done. So one question: What framework are we using to do the functional testing? Plain shell scripts or framework like avocado or anything else? |
Added functional tests: #89 |
We also need to add unit tests. |
@ngtuna yes will work on them once your refactor is in! So we can keep this open. |
I'm adding unit test for komposeConvert in #104 |
Thanks @janetkuo for adding some unit tests and we will keep adding more. This issue can be closed now. |
We should add a few test fixtures and run tests on Travis/circleCI.
The text was updated successfully, but these errors were encountered: