Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 356639

Summary: ModelElementTest tests are failing
Product: [Eclipse Project] Platform Reporter: Remy Suen <remy.suen>
Component: UIAssignee: Eric Moffatt <emoffatt>
Status: VERIFIED FIXED QA Contact: Eric Moffatt <emoffatt>
Severity: normal    
Priority: P3 CC: emoffatt
Version: 4.2   
Target Milestone: 4.2 M4   
Hardware: All   
OS: All   
Whiteboard:

Description Remy Suen CLA 2011-09-02 16:12:06 EDT 
It seems SnippetContainer is not abstract. I believe it should be.

junit.framework.AssertionFailedError: The following concrete classes don't implement 'MApplicationElement':
* SnippetContainer

	at junit.framework.Assert.fail(Assert.java:47)
	at org.eclipse.e4.ui.tests.application.ModelElementTest.testForMApplicationInterface(ModelElementTest.java:38)

junit.framework.AssertionFailedError: The following concrete classes have a mixin as base 'MApplicationElement':
* SnippetContainer

	at junit.framework.Assert.fail(Assert.java:47)
	at org.eclipse.e4.ui.tests.application.ModelElementTest.testForOptimalBaseClass(ModelElementTest.java:73)
Comment 1 Remy Suen CLA 2011-09-02 16:12:18 EDT 
Eric, please confirm.
Comment 2 Eric Moffatt CLA 2011-11-16 13:55:47 EST 
Pushed in >20111116.

commit 7ed043e1cc641d1e64fd5aec7dda371f4505c33b

I tested to make sure that the change is backwards compatible for models that have saved perspectives...
Comment 3 Eric Moffatt CLA 2011-12-08 15:06:29 EST 
Verified in I20111205-2330.