-
Notifications
You must be signed in to change notification settings - Fork 40k
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
cmd/e2e/e2e.go work items #3490
Comments
I'd rather move away from a monolithic test file into distinct files and On Wed, Jan 14, 2015 at 4:15 PM, Eric Tune notifications@github.com wrote:
|
Agreed, but I think that's even longer term and not a contradictory goal. |
Linking relevant other thread: #3447 (comment) |
I am about to submit and update to my PR with all the Go tests split into S On Wed, Jan 14, 2015 at 4:32 PM, Zach Loafman notifications@github.com
|
Snazzy! I should get hopped up on DayQuil more often! |
@thockin : I think it is fine (in fact, preferable) to have all the Go e2e tests in one package as long as we have a way to conditionally execute each test. Which is a feature that I (or someone else) will add shortly to our Go e2e tests. |
Also, I am trying to chip away at the e2e tests with a view to getting them into shape for use with the Go |
Is this monolithic issue still needed? |
@zmerlynn @satnam6502 @filbranden Closing as obsolete. |
Recording these for posterity.
The text was updated successfully, but these errors were encountered: