diff --git a/tests/org.eclipse.ui.tests.performance/src/org/eclipse/ui/tests/performance/ProblemsViewPerformanceTest.java b/tests/org.eclipse.ui.tests.performance/src/org/eclipse/ui/tests/performance/ProblemsViewPerformanceTest.java index a4fd01f2b69..1d5a1d24f5f 100644 --- a/tests/org.eclipse.ui.tests.performance/src/org/eclipse/ui/tests/performance/ProblemsViewPerformanceTest.java +++ b/tests/org.eclipse.ui.tests.performance/src/org/eclipse/ui/tests/performance/ProblemsViewPerformanceTest.java @@ -16,7 +16,7 @@ import static org.eclipse.ui.tests.harness.util.UITestUtil.openTestWindow; import static org.eclipse.ui.tests.harness.util.UITestUtil.processEvents; -import static org.junit.Assert.fail; +import static org.junit.jupiter.api.Assertions.fail; import java.util.HashMap; import java.util.Map; @@ -26,25 +26,30 @@ import org.eclipse.core.resources.IWorkspaceRoot; import org.eclipse.core.resources.ResourcesPlugin; import org.eclipse.core.runtime.CoreException; -import org.eclipse.test.performance.PerformanceTestCaseJunit4; +import org.eclipse.test.performance.Performance; +import org.eclipse.test.performance.PerformanceMeter; import org.eclipse.ui.IWorkbenchPage; import org.eclipse.ui.IWorkbenchWindow; import org.eclipse.ui.PartInitException; -import org.eclipse.ui.tests.harness.util.CloseTestWindowsRule; +import org.eclipse.ui.tests.harness.util.CloseTestWindowsExtension; import org.eclipse.ui.tests.performance.parts.PerformanceProblemsView; import org.eclipse.ui.texteditor.MarkerUtilities; -import org.junit.Rule; -import org.junit.Test; +import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.TestInfo; +import org.junit.jupiter.api.extension.RegisterExtension; /** * The ProblemsViewPerformanceTest is a test of population of the problems view. * * @since 3.2 */ -public class ProblemsViewPerformanceTest extends PerformanceTestCaseJunit4 { +public class ProblemsViewPerformanceTest { - @Rule - public final CloseTestWindowsRule closeTestWindows = new CloseTestWindowsRule(); + @RegisterExtension + static UIPerformanceTestRule uiPerformanceTestRule = new UIPerformanceTestRule(); + + @RegisterExtension + CloseTestWindowsExtension closeTestWindows = new CloseTestWindowsExtension(); private final String EMPTY_PERSPECTIVE_ID = "org.eclipse.ui.tests.harness.util.EmptyPerspective"; @@ -54,33 +59,41 @@ public class ProblemsViewPerformanceTest extends PerformanceTestCaseJunit4 { * Test the population of the problems view. */ @Test - public void testPopulation() { - IWorkbenchWindow window = openTestWindow(EMPTY_PERSPECTIVE_ID); - final IWorkbenchPage page = window.getActivePage(); - - PerformanceProblemsView view; + public void testPopulation(TestInfo testInfo) { + Performance perf = Performance.getDefault(); + String scenarioId = getClass().getName() + "." + testInfo.getDisplayName(); + PerformanceMeter meter = perf.createPerformanceMeter(scenarioId); try { - view = (PerformanceProblemsView) page.showView(PROBLEMS_VIEW_ID); - } catch (PartInitException e) { - e.printStackTrace(); - fail(); - return; - } + IWorkbenchWindow window = openTestWindow(EMPTY_PERSPECTIVE_ID); + final IWorkbenchPage page = window.getActivePage(); - for (int i = 0; i < 100; i++) { - createMarkers(); - processEvents(); - startMeasuring(); - while (view.getTreeWidget().getItemCount() == 0) - processEvents(); - stopMeasuring(); - removeMarkers(); - while (view.getTreeWidget().getItemCount() > 0) + PerformanceProblemsView view; + + try { + view = (PerformanceProblemsView) page.showView(PROBLEMS_VIEW_ID); + } catch (PartInitException e) { + e.printStackTrace(); + fail(); + return; + } + + for (int i = 0; i < 100; i++) { + createMarkers(); processEvents(); + meter.start(); + while (view.getTreeWidget().getItemCount() == 0) + processEvents(); + meter.stop(); + removeMarkers(); + while (view.getTreeWidget().getItemCount() > 0) + processEvents(); + } + meter.commit(); + perf.assertPerformance(meter); + } finally { + meter.dispose(); } - commitMeasurements(); - assertPerformance(); }