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

pkg/analysis_server/test/integration/analysis/update_content_test times out on Windows #20575

Closed
whesse opened this issue Aug 19, 2014 · 5 comments
Assignees

Comments

@whesse
Copy link
Member

whesse commented Aug 19, 2014

The test pkg/analysis_server/test/integration/analysis/update_content_test is timing out on Windows on the buildbots. Skipping for now.

@whesse
Copy link
Member Author

whesse commented Aug 19, 2014

cc @stereotype441.

@stereotype441
Copy link
Member

Thanks for the heads up. I am planning to talk to Rico about this issue when I'm in Aarhus next week.


cc @ricowind.
Set owner to @stereotype441.
Added Accepted label.

@bwilkerson
Copy link
Member

I believe that the problem is in the test framework, not the tests. If you disagree, please start an e-mail conversation with me.

@ricowind
Copy link
Contributor

ricowind commented Nov 9, 2015

I don't know if this is a problem in the test framework or not, but for completeness could you elaborate on what you think the problem is?

@whesse
Copy link
Member Author

whesse commented Nov 9, 2015

I believe that the test.py scripts had a timeout of 1 minute, and many analysis server tests are long-running. We increased the timeout to 3 or 5 minutes, for this particular test suite, so the problem should be resolved. The flaw was not in the framework, but in our assumptions about how people would be using it. We assumed lots of small tests, in separate files. Some users want to put all tests in one large file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants