Bug 366625 - [Compatibility] Failures in MultiPageEditorTestSuite
Summary: [Compatibility] Failures in MultiPageEditorTestSuite
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.2   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 4.2 M6   Edit
Assignee: Paul Webster CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 366451 381873
  Show dependency tree
 
Reported: 2011-12-13 15:54 EST by Remy Suen CLA
Modified: 2012-06-06 11:11 EDT (History)
1 user (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 Remy Suen CLA 2011-12-13 15:54:40 EST 
MultiVariablePageTest's testContextActivation is failing.

junit.framework.AssertionFailedError: org.eclipse.ui.tests.multipageeditor.contextEditor expected:<true> but was:<false>
	at junit.framework.Assert.fail(Assert.java:47)
	at junit.framework.Assert.failNotEquals(Assert.java:283)
	at junit.framework.Assert.assertEquals(Assert.java:64)
	at junit.framework.Assert.assertEquals(Assert.java:143)
	at org.eclipse.ui.tests.multipageeditor.MultiVariablePageTest.checkActiveContext(MultiVariablePageTest.java:247)
	at org.eclipse.ui.tests.multipageeditor.MultiVariablePageTest.testContextActivation(MultiVariablePageTest.java:220)
Comment 1 Remy Suen CLA 2011-12-14 12:34:22 EST 
Reverting ec87e24a07c23cb6144b4cdb77bd3303b7530171 didn't help.
Comment 2 Remy Suen CLA 2011-12-14 13:36:01 EST 
(In reply to comment #1)
> Reverting ec87e24a07c23cb6144b4cdb77bd3303b7530171 didn't help.

Reverting and spinning the event loop will get the test to pass. Also see bug 366606.
Comment 3 Remy Suen CLA 2011-12-14 14:21:06 EST 
(In reply to comment #2)
> (In reply to comment #1)
> > Reverting ec87e24a07c23cb6144b4cdb77bd3303b7530171 didn't help.
> 
> Reverting and spinning the event loop will get the test to pass.

I take that back. Just adding the event loop processing code will get it to pass. Reverting is not a necessity.
Comment 5 Paul Webster CLA 2012-03-13 11:13:04 EDT 
In I20120313-0610
PW