Community
Participate
Working Groups
N20110110-2000. no exception should be thrown junit.framework.AssertionFailedError: no exception should be thrown at org.eclipse.ui.tests.statushandlers.StatusDialogManagerTest.testBug276371(StatusDialogManagerTest.java:638) at org.eclipse.test.EclipseTestRunner.run(EclipseTestRunner.java:416) at org.eclipse.test.EclipseTestRunner.run(EclipseTestRunner.java:249) at org.eclipse.test.UITestApplication$2.run(UITestApplication.java:197) at org.eclipse.swt.widgets.RunnableLock.run(RunnableLock.java:35) at org.eclipse.swt.widgets.Synchronizer.runAsyncMessages(Synchronizer.java:134) at org.eclipse.swt.widgets.Display.runAsyncMessages(Display.java:4064) at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3683) at org.eclipse.ui.internal.Workbench.runEventLoop(Workbench.java:2697) at org.eclipse.ui.internal.Workbench.runUI(Workbench.java:2661) at org.eclipse.ui.internal.Workbench.access$4(Workbench.java:2495) at org.eclipse.ui.internal.Workbench$7.run(Workbench.java:674) at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:332) at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:667) at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:149) at org.eclipse.ui.internal.ide.application.IDEApplication.start(IDEApplication.java:115) at org.eclipse.test.UITestApplication.runApplication(UITestApplication.java:140) at org.eclipse.test.UITestApplication.run(UITestApplication.java:62) at org.eclipse.test.UITestApplication.start(UITestApplication.java:212) at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:196) at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:110) at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:79) at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:344) at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:179) at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:622) at org.eclipse.equinox.launcher.Main.basicRun(Main.java:577) at org.eclipse.equinox.launcher.Main.run(Main.java:1410) at org.eclipse.equinox.launcher.Main.main(Main.java:1386) at org.eclipse.core.launcher.Main.main(Main.java:34)
The try/catch block with the fail(String) method needs to be deleted. The stack trace as it is tells us absolutely nothing. We should let the exception be propagated up to JUnit so that it can report the trace back in its entirety.
The test was introduced to ensure that no exception during displaying status dialog will cause an endless loop of exceptions nor prevent further exceptions from displaying. To cause an exception, I have used custom IStatus which exploded when the message was get. The code in status has not changed, the exception is properly caught and then logged via WorkbenchPlugin.log(statusAdapter.getStatus()); Due to changes introduced by bug 292135, following code will fail: PlatformLogWriter. IStatus status = new Status(IStatus.ERROR, "org.eclipse.ui.tests", "bomb"){ public String getMessage() { throw new RuntimeException("the bomb!"); } }; PlatformLogWriter.getLog(status). Thomas, is this a valid test case for you?
I am not sure what the correct coarse of action is here. If someone supplies a bad status we are supposed to catch the exception and let the status through the logging system without propagating the exception to the caller? John, only CC'ing you in case I am not available to fix this and it turns out to be something we need to address in common.
Ok, the issue is caused by the bug in the test itself. The fake status throws exception all the time, and therefore it cannot be logged properly. I have updated the status to fail only once and therefore simulate an exception during exception handling only in status dialog. The test passes now. The question what to do with such a status during osgi logging is a separate issue.
(In reply to comment #4) > Ok, the issue is caused by the bug in the test itself. The fake status throws > exception all the time, and therefore it cannot be logged properly. > > I have updated the status to fail only once and therefore simulate an exception > during exception handling only in status dialog. FYI: I've updated copyright date and bundle version in HEAD.
Verified in I20110118-0800.
