-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
refactored test-cmd/edit to use new helper methods #6012
refactored test-cmd/edit to use new helper methods #6012
Conversation
[test] |
|
||
[ "$(OC_EDITOR=cat oc edit pod/hello-openshift 2>&1 | grep 'Edit cancelled')" ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2>&1
is not necessary any longer, as it was previously used for the grep
and the helper methods implicitly will check both stdout
and stderr
for the desired text.
ac80dd8
to
4221622
Compare
Changed |
Tagging @Kargakis for review - please review that we are expecting success and/or failure correctly, as the exit codes were masked previously with the |
Evaluated for origin test up to 4221622 |
lgtm [merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/4118/) (Image: devenv-rhel7_2782) |
Evaluated for origin merge up to 4221622 |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/7369/) |
…md-edit Merged by openshift-bot
/cc @deads2k @smarterclayton