Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 348484 - Runtime GMFGraph tests are failing under the modeling (non Tycho) build
Summary: Runtime GMFGraph tests are failing under the modeling (non Tycho) build
Status: ASSIGNED
Alias: None
Product: GMF-Tooling
Classification: Modeling
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Michael Golubev CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-06 19:15 EDT by Michael Golubev CLA
Modified: 2012-05-30 01:27 EDT (History)
1 user (show)

See Also:

Attachments
correctly generated code for failing testFlowLayout (1.73 KB, text/x-java)
2011-06-06 19:23 EDT, Michael Golubev CLA 		no flags Details
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 Michael Golubev CLA 2011-06-06 19:15:49 EDT 
The failure logs are avaialble here: http://modeling.eclipse.org/modeling/gmp/gmf-tooling/downloads/drops//2.4.0/N201106061733/testresults/xml/org.eclipse.gmf.tests_linux.gtk.xml

Exactly the same tests not failing under the Tycho build and when running from local workspace. 

All 7 failures are for checking the foreground color of the generated figures: no matter what is the value of the color set in model, the runtime color is always black (RGB {0, 0, 0}).
Comment 1 Michael Golubev CLA 2011-06-06 19:21:30 EDT 
I've verified that the generated code is the same under the build and for local runs (attached).
So, it explicitly contains the setting of the color like this: 

	/**
	 * @generated
	 */
	static final Color THIS_FORE = new Color(null, 0, 0, 255);
    
	this.setForegroundColor(THIS_FORE);

So this is the problem with test checker not with the code being tested.
Comment 2 Michael Golubev CLA 2011-06-06 19:23:40 EDT 
Created attachment 197470 [details]
correctly generated code for failing testFlowLayout
Comment 3 Michael Golubev CLA 2012-05-30 01:27:47 EDT 
batch update: Verified in RC2