Bug 161555 - org.eclipse.jdt.core.tests.model.ReconcilerTests#testDeleteTwoMethods is failing on IBM 1.5 VM
Summary: org.eclipse.jdt.core.tests.model.ReconcilerTests#testDeleteTwoMethods is fail...
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.2.2   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 161674 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-10-19 08:09 EDT by Olivier Thomann CLA
Modified: 2007-01-16 04:43 EST (History)
2 users (show)

See Also:


Attachments
Proposed fix (1.04 KB, patch)
2006-10-19 19:03 EDT, Olivier Thomann CLA
no flags Details | Diff
Complete patch (all tests updated) (12.58 KB, patch)
2006-10-19 21:32 EDT, Olivier Thomann CLA
no flags Details | Diff
Regression tests updated for 3.2.2 (12.56 KB, patch)
2006-10-20 16:51 EDT, Olivier Thomann CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Olivier Thomann CLA 2006-10-19 08:09:29 EDT
This test seems to fail because the two deltas are reversed.
We might want to make sure that we sort the deltas in order to be VM independant.

I also got this stack trace in the console:
java.lang.Throwable: Caller of IElementChangedListener#elementChanged
	at org.eclipse.jdt.core.tests.model.AbstractJavaModelTests$DeltaListener.elementChanged(AbstractJavaModelTests.java:106)
	at org.eclipse.jdt.internal.core.DeltaProcessor$3.run(DeltaProcessor.java:1510)
	at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:37)
	at org.eclipse.jdt.internal.core.DeltaProcessor.notifyListeners(DeltaProcessor.java:1500)
	at org.eclipse.jdt.internal.core.DeltaProcessor.fireReconcileDelta(DeltaProcessor.java:1366)
	at org.eclipse.jdt.internal.core.DeltaProcessor.fire(DeltaProcessor.java:1324)
	at org.eclipse.jdt.internal.core.JavaModelOperation.run(JavaModelOperation.java:760)
	at org.eclipse.jdt.internal.core.JavaModelOperation.runOperation(JavaModelOperation.java:779)
	at org.eclipse.jdt.internal.core.CompilationUnit.reconcile(CompilationUnit.java:1132)
	at org.eclipse.jdt.internal.core.CompilationUnit.reconcile(CompilationUnit.java:1104)
	at org.eclipse.jdt.core.tests.model.ReconcilerTests.testDeleteTwoMethods(ReconcilerTests.java:1075)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:64)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:615)
	at junit.framework.TestCase.runTest(TestCase.java:164)
	at junit.framework.TestCase.runBare(TestCase.java:130)
	at junit.framework.TestResult$1.protect(TestResult.java:106)
	at junit.framework.TestResult.runProtected(TestResult.java:124)
	at junit.framework.TestResult.run(TestResult.java:109)
	at junit.framework.TestCase.run(TestCase.java:120)
	at junit.framework.TestSuite.runTest(TestSuite.java:230)
	at org.eclipse.jdt.core.tests.model.SuiteOfTestCases$Suite.runTest(SuiteOfTestCases.java:99)
	at junit.framework.TestSuite.run(TestSuite.java:225)
	at org.eclipse.jdt.core.tests.model.SuiteOfTestCases$Suite.superRun(SuiteOfTestCases.java:83)
	at org.eclipse.jdt.core.tests.model.SuiteOfTestCases$1.protect(SuiteOfTestCases.java:71)
	at junit.framework.TestResult.runProtected(TestResult.java:124)
	at org.eclipse.jdt.core.tests.model.SuiteOfTestCases$Suite.run(SuiteOfTestCases.java:80)
	at org.eclipse.jdt.internal.junit.runner.junit3.JUnit3TestReference.run(JUnit3TestReference.java:128)
	at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38)
	at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:460)
	at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:673)
	at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:386)
	at org.eclipse.pde.internal.junit.runtime.RemotePluginTestRunner.main(RemotePluginTestRunner.java:58)
	at org.eclipse.pde.internal.junit.runtime.CoreTestApplication.run(CoreTestApplication.java:24)
	at org.eclipse.core.internal.runtime.PlatformActivator$1.run(PlatformActivator.java:78)
	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:104)
	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:74)
	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:348)
	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:165)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:64)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:615)
	at org.eclipse.core.launcher.Main.invokeFramework(Main.java:341)
	at org.eclipse.core.launcher.Main.basicRun(Main.java:285)
	at org.eclipse.core.launcher.Main.run(Main.java:987)
	at org.eclipse.core.launcher.Main.main(Main.java:962)
Comment 1 Olivier Thomann CLA 2006-10-19 08:12:22 EDT
This exception doesn't occur with Sun VM.
Comment 2 Olivier Thomann CLA 2006-10-19 19:03:18 EDT
Created attachment 52354 [details]
Proposed fix
Comment 3 Olivier Thomann CLA 2006-10-19 21:32:25 EDT
Created attachment 52362 [details]
Complete patch (all tests updated)
Comment 4 Olivier Thomann CLA 2006-10-19 21:33:36 EDT
*** Bug 161674 has been marked as a duplicate of this bug. ***
Comment 5 Olivier Thomann CLA 2006-10-19 21:42:14 EDT
Released for 3.3M3.
Comment 6 Olivier Thomann CLA 2006-10-20 14:41:59 EDT
Philippe,

Should I backport to 3.2.2?
Comment 7 Olivier Thomann CLA 2006-10-20 16:51:26 EDT
Created attachment 52429 [details]
Regression tests updated for 3.2.2
Comment 8 Philipe Mulet CLA 2006-10-23 10:25:29 EDT
+1 for 3.2.2
Comment 9 Olivier Thomann CLA 2006-10-23 16:40:11 EDT
Released for 3.2.2
Comment 10 David Audel CLA 2006-10-30 09:00:45 EST
Verified for 3.3 M3 using build I20061030-0010
Comment 11 Maxime Daniel CLA 2007-01-16 04:43:42 EST
Verified for 3.2.2 using build M20070112-1200.