-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
e2e: add new test package "global" with TestTLSVerify #2196
Conversation
Signed-off-by: Tibor Vass <tibor@docker.com>
a005f3a
to
e692381
Compare
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.
LGTM w.r.t. getting tests in the appropriate repository
|
||
func TestTLSVerify(t *testing.T) { | ||
// Remote daemons use TLS and this test is not applicable when TLS is required. | ||
skip.If(t, environment.RemoteDaemon()) |
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.
Perhaps put the "skip" description as third argument to skip.If
, then it shows up in the logs as reason why it's skipped
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.
LGTM
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.
LGTM
Signed-off-by: Tibor Vass tibor@docker.com
This is to move TestRunTLSVerify from moby integration-cli tests.