-
Notifications
You must be signed in to change notification settings - Fork 49
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
[SURVEY] Important Test Cases #211
Comments
This is a good practice! I was on vacation and couldn't answer right away. I think everyone will agree that tests run for every PR.
(there may be problems with the architecture, as function calls are looping) I think that we will fix them in the near future, and when all tests pass successfully, we will add them to the test workflow. And this will allow us to review only successfully tested code, and also raise the level of the best practices of our project. Thank @t25kim for this proposal, testing is very important for open source projects. |
@tdrozdovsky Thank you for the comment and PR! |
Signed-off-by: Taras Drozdovskyi <t.drozdovsky@samsung.com> - During the launch of a new PR or push command, a test suite (unit testing) will be launched (Fixes #211). - It should be noted that testing for the following packages (src/controller/discoverymgr, src/controller/servicemgr/executor/containerexecutor src/orchestrationapi) is temporary excluded due to non-working tests!
I'd like to know core test cases to add on build workflows like below in order to check the riskiness of PR.
Please leave comments on important test cases, for example
common/sigmgr
.The text was updated successfully, but these errors were encountered: