Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 321563 - [Unit Test Failure] org.eclipse.gmt.modisco.infra.browser.custom.tests.BrowserCustomTest.testWorkspaceCustomizationHidesRegistryCustomization
Summary: [Unit Test Failure] org.eclipse.gmt.modisco.infra.browser.custom.tests.Browse...
Status: CLOSED FIXED
Alias: None
Product: MoDisco
Classification: Modeling
Component: Infrastructure (show other bugs)
Version: 0.9.0   Edit
Hardware: PC Windows XP
: P1 normal (vote)
Target Milestone: 0.9.0 M2   Edit
Assignee: Gregoire Dupe CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-03 05:51 EDT by Gregoire Dupe CLA
Modified: 2010-10-18 08:35 EDT (History)
4 users (show)

See Also:
gdupe: helios+
gdupe: indigo+

Attachments
details (9.17 KB, text/plain)
2010-08-03 05:51 EDT, Gregoire Dupe CLA 		no flags Details
Log of build 25 (65.06 KB, text/plain)
2010-08-05 08:23 EDT, Gregoire Dupe 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.
Comment 1 Gregoire Dupe CLA 2010-08-03 05:54:02 EDT 
In revision 2812, I’ve improved the error message to help to debug.
Comment 2 Gregoire Dupe CLA 2010-08-04 04:40:52 EDT 
In revision 2819, I've enabled the tracing in the AllTest's launch configuration.
Comment 3 Gregoire Dupe CLA 2010-08-04 06:13:31 EDT 
I’m not able to reproduce the bug in the modisco-nightly job but the modisco-integration job (for the version 0.8.0) has the same problem.

In revision 2820, I’ve enable tracing in the AllTest launch configuration in the 0.8.0 branch.
Comment 4 Gregoire Dupe CLA 2010-08-04 08:46:29 EDT 
In revision 2821, I've added debug messages to understand the bug. (in the branch 0.8.0)
Comment 5 Gregoire Dupe CLA 2010-08-04 10:12:22 EDT 
In revision 2831:
 - I've added synchronized key word on CatalogJob::getInstance() and CatalogJob::run(...).
 - I've added the option -consoleLog in the AllTest's launch configuration.
Comment 6 Gregoire Dupe CLA 2010-08-05 08:23:45 EDT 
Created attachment 175924 [details]
Log of build 25

In this log we can see that something wrong happened with SWT during the CatalogJob execution:
 org.eclipse.swt.SWTError: No more handles [gtk_init_check() failed]
Comment 7 Gregoire Dupe CLA 2010-08-06 12:35:36 EDT 
I’ve removed the synchronized key word. This key word may cause a dead lock the unit test run on build.eclipse.org. 

Cf. :
 - https://build.eclipse.org/hudson/job/modisco-integration/26/console
 - https://build.eclipse.org/hudson/job/modisco-integration/27/console
Comment 8 Gregoire Dupe CLA 2010-08-20 03:45:28 EDT 
In the revision 2952, I've added the keyword "synchronized" to the method CatalogJob.getInstance() in the trunk.
Comment 9 Nicolas Bros CLA 2010-09-09 08:55:02 EDT 
This test now passes successfully on Hudson.
Comment 10 Hugo Bruneliere CLA 2010-10-05 04:30:16 EDT 
Bug fixed.