-
Notifications
You must be signed in to change notification settings - Fork 136
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
Migrate to Go 1.9.x and Consul 1.0.x #519
Comments
Ok, so I have all tests passing. Interesting side bit... when I bumped Consul to 1.0.0 and made a few small adjustments to how our tests run, it looks like Consul's /cc @misterbisson EDIT: Minor changes to reflect back story and current status. |
@cheapRoc To speed up integration tests and make them more reliable I've initially added an external tool Having said that, if you need a consul instance for your tests you should start one by executing the binary and control the process. I've significantly sped up the startup time for a Just run |
You're correct, I should have phrased that a tad better, it was broken in our use case. The function call worked perfectly fine outside Consul in another test of mine. I also noticed the Nomad team's commits and your added context makes this much clearer.
I'm with you, we shouldn't depend on an internal testing utility of a third party dependency. But I'd also imagine that controlling a Another (long term) answer would be to stub out the Consul API's HTTP client interface, stop testing Consul's API in our own unit tests, then ensure our integration tests continue to exercise the same behavior we expect today. Something for later. Really appreciate the feedback Frank. |
Created #528 to track that testing work separately. |
I think what the nomad team has done is to vendor in the |
That works for me. I should be able to make the change and continue including you in the review. We'll go from there. |
Stub for migrating our build steps and releases to Go 1.9.x.
The text was updated successfully, but these errors were encountered: