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

Promise flaky test #637

Closed
jenny opened this issue Apr 19, 2013 · 9 comments
Closed

Promise flaky test #637

jenny opened this issue Apr 19, 2013 · 9 comments
Assignees

Comments

@jenny
Copy link
Member

jenny commented Apr 19, 2013

The following test passes locally, but fails with Yeti in CI and Yeti locally. We are addressing all such flaky tests in the repo by refactoring tests, increasing timeouts, etc. Please let me know if you have any questions on how best to address this issue. Improving the test helps us have a reliable CI system. Thanks!

Firefox (19_0) / Windows XP.Promise.Promise: Y.batch: batch should fail as fast as possible

@ghost ghost assigned juandopazo Apr 19, 2013
@juandopazo
Copy link
Member

I'll set up Yeti and give it a try.

Does it happen only in Windows XP?

@juandopazo
Copy link
Member

I'm running into an issue with Yeti when trying to test this. After starting Yeti, Firefox in Windows XP navigates to a URL like /agent/123/batch/123/test/src\promise\tests\unit\promise.html instead of /agent/123/batch/123/test/src/promise/tests/unit/promise.html.

@juandopazo
Copy link
Member

Jenny, could I see the result of the assertions?

@derek
Copy link
Contributor

derek commented Apr 21, 2013

Tested and works for me on Firefox (20) and XP.

Juan, the VMs that conduct the testing tend to be a little less powerful than we we use for development, so tests relying on timeouts may need a little extra time to finish. This test times out after 100ms, can it be bumped up to 300ms just to be on the safe side?

@derek
Copy link
Contributor

derek commented Apr 21, 2013

Also, a ping to @reid for the Yeti issue.

@juandopazo
Copy link
Member

Yeah, I want to see the assertion result to see if the test timed out or if the assertion failed. If the assertion failed it would be really weird because even if the browser clamped the minimum timeout or took longer to finish, the order of execution of that test shouldn't change. If it times out, I'll just increase the timeout.

@jenny
Copy link
Member Author

jenny commented Apr 24, 2013

Here it is:

Firefox (19_0) / Windows XP.Promise.Promise: Y.batch: batch should fail as fast as possible

@juandopazo
Copy link
Member

I guess that means it's timing out. I'll take a look at the internals of YUI Test to see why its own timeout could be running before the others and I'll increase it.

juandopazo added a commit to juandopazo/yui3 that referenced this issue Apr 24, 2013
Also change the test value from a string to a number as it should have
been to avoid possible failures
@juandopazo
Copy link
Member

These unit tests were rewritten.

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

3 participants