Bug 342685 - EditorWithStateTest and WorkingSetTest fail on Java 7
Summary: EditorWithStateTest and WorkingSetTest fail on Java 7
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.7 M7   Edit
Assignee: Oleg Besedin CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 342688
  Show dependency tree
 
Reported: 2011-04-13 05:17 EDT by Dani Megert CLA
Modified: 2011-04-26 14:41 EDT (History)
2 users (show)

See Also:

Attachments
UI test results (320.02 KB, text/html)
2011-04-13 05:17 EDT, Dani Megert CLA 		no flags Details
navigator test results (28.60 KB, text/html)
2011-04-13 05:21 EDT, Dani Megert CLA 		no flags Details
Patch (8.62 KB, patch)
2011-04-14 15:07 EDT, Oleg Besedin CLA 		no flags Details | Diff
View All 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 Dani Megert CLA 2011-04-13 05:17:23 EDT 
The EditorWithStateTest test fails on Java 7.

See attached test results.

The reason is that the test assumes to run in a certain order but that is not
guaranteed by JUnit.
Comment 1 Dani Megert CLA 2011-04-13 05:17:48 EDT 
Created attachment 193137 [details]
UI test results
Comment 2 Dani Megert CLA 2011-04-13 05:21:05 EDT 
Created attachment 193138 [details]
navigator test results
Comment 3 Dani Megert CLA 2011-04-13 05:22:09 EDT 
>The EditorWithStateTest test fails on Java 7.
Same for WorkingSetTest.
Comment 4 Paul Webster CLA 2011-04-13 07:49:40 EDT 
I think the solution is to add the tests in order to a suite for that class, instead of relying on reflection on EditorWithStateTest.class.

PW
Comment 5 Dani Megert CLA 2011-04-14 10:33:58 EDT 
> Same for WorkingSetTest.

I've fixed that in HEAD (WorkingSetTest.java, rev. 1.11).
Comment 6 Oleg Besedin CLA 2011-04-14 15:07:20 EDT 
Created attachment 193285 [details]
Patch
Comment 7 Oleg Besedin CLA 2011-04-14 15:07:57 EDT 
Patch applied to CVS Head.
Comment 8 Dani Megert CLA 2011-04-15 03:27:11 EDT 
Thanks for the fast fix Oleg!
Comment 9 Oleg Besedin CLA 2011-04-26 14:41:46 EDT 
Verified that EditorWithStateTest and WorkingSetTest pass on I20110425-1800 using Java7.