Skip to content
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

Set up TLS for TestDefaultClientRejectSelfSigned #11

Merged
merged 3 commits into from
Dec 2, 2016

Conversation

tgross
Copy link
Contributor

@tgross tgross commented Nov 14, 2016

For #10

As in an issue with gomanta the test suite was not updated with the new router that was added in f676d62. This makes sure we're using the same library in this test suite.

Additionally, in a previous commit (5642c3e) we accidentally removed the setup of the TLS test suite which is used to verify the client behavior on self-signed TLS cert. This restores the correct behavior by splitting out a second test suite.

cc @misterbisson

Test run:

$ go test ./...
ok      github.com/joyent/gocommon      0.011s
ok      github.com/joyent/gocommon/client       0.830s
ok      github.com/joyent/gocommon/errors       0.007s
ok      github.com/joyent/gocommon/http 0.121s
?       github.com/joyent/gocommon/jpc  [no test files]
ok      github.com/joyent/gocommon/testing      0.079s

In a previous commit we accidentally removed the setup of the TLS test suite
which is used to verify the client behavior on self-signed TLS cert. This
restores the correct behavior by splitting out a second test suite.

previous ref 5642c3e
@tgross
Copy link
Contributor Author

tgross commented Nov 15, 2016

I've added support for Wercker CI to this PR as well. Successful builds look like this: https://app.wercker.com/joyent/gocommon/runs/build/582b7fb1718dd00100502abb

@tgross tgross merged commit b787089 into TritonDataCenter:master Dec 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant