Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 369526 - Session test ordering off on java 7
Summary: Session test ordering off on java 7
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.8   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.2 M5   Edit
Assignee: Paul Webster CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-24 09:54 EST by Paul Webster CLA
Modified: 2012-01-25 11:53 EST (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Webster CLA 2012-01-24 09:54:28 EST 
It looks like the java 7 JRE is returning the session tests in a different order.  The solution is to specify the test order in the test suite itself.
Comment 1 Paul Webster CLA 2012-01-24 10:35:44 EST 
I've released a session test ordering fix to origin/pwebster/bug369526

Remy, Oleg, could you have a look in the context of 3.8 testing and make sure the tests still run?

Our current failure list:
http://download.eclipse.org/eclipse/downloads/drops/I20120123-1300/testresults/html/org.eclipse.ui.tests_win32.win32.x86_7.0.html#SessionTests


PW
Comment 2 Remy Suen CLA 2012-01-24 11:17:40 EST 
Looks okay to me. The failures I get now are from the ones that haven't been restructured.
Comment 3 Oleg Besedin CLA 2012-01-24 15:57:51 EST 
Makes sense. There are two more session tests that fail om my computer if compiled with Java 1.6 and run with Java 1.7:

NonRestorablePropertySheetTest (actual order on my machine: test02SecondOpening run before test01ActivateView)
ArbitraryPropertiesEditorTest (actual order om my machine: testPartInstantiation, testSecondOpening, testOpenEditor)

That was using Eclipse 3.8.
Comment 4 Paul Webster CLA 2012-01-25 08:30:14 EST 
Dean, could you please set yourself up for 3.8 and go through the org.eclipse.ui.tests.session.SessionTests.  The only way to guarantee the correct ordering at runtime is to have each test class return a suite(), and spell out the test order in that suite (see org.eclipse.ui.tests.session.Bug95357Test.suite() for an example).

PW