Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 362595 - orion test framework starts a test before finishing the previous (async) test
Summary: orion test framework starts a test before finishing the previous (async) test
Status: RESOLVED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Client (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 0.4 RC1   Edit
Assignee: Simon Kaegi CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-01 15:48 EDT by Felipe Heidrich CLA
Modified: 2012-02-06 14:22 EST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Felipe Heidrich CLA 2011-11-01 15:48:39 EDT
We are in the process to move the textview unit tests to the orion test framework.
Unfortunately, besides Bug344344#c7, we are also having trouble with our async tests.

The framework will essentially start all the tests in the suite at the same time, and try to run them in parallel. This is a big problem because tests can share data. In our case, the view.

For example, take the pageDown and the pageUp test. They both scroll the same view and they are both async (they need to return the control to the ua after each operation).


The correct behavior is to run all async operations for the pageDown first, then run all the async operations for the pageUp.
The current implementation alternates pageDown,pageUp operations never ending.

Note that jstestdrive does solve this problem:
"The test runner will not execute any subsequent step in the queue until all outstanding callbacks of the current step are complete. If the callbacks are not a called for an egregious amount of time, currently set to 30 seconds, the test fails. "
http://code.google.com/p/js-test-driver/wiki/AsyncTestCase

Note that changing our tests to not share the view is not ideal, running performance tests in parallel is wrong.
Comment 1 Simon Kaegi CLA 2012-02-06 14:22:12 EST
Fixed.