-
Notifications
You must be signed in to change notification settings - Fork 244
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
Cleanup test naming - e2e vs integration #1780
Comments
/project "Sprint 168" |
Issue description is really clear in this case! For example: https://github.com/openshift/odo/blob/009f9d7e775caef0dc63d2a361f2eeb423398a05/Makefile#L106-L109 has the name |
/label state/Ready |
1 similar comment
/label state/Ready |
The bot's on a PTO, maybe? 🤔 |
This should be also updated in https://github.com/openshift/release |
Right, I will fix all those reference of e2e related to test. |
Pr sent openshift/release#4268 for OpenShift CI job and config changes |
Is there a PR for this on odo side? |
PR for this issue #1878 |
/close |
@kadel: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
There is still a lot of places (like Makefile, or names of Describe blocks) where integration tests are named as e2e or have e2e in the name.
This should be cleaned to make it clear that those are two different tests approaches.
/kind bug
/area testing
/priority medium
/project Sprint 168
The text was updated successfully, but these errors were encountered: