Fix for Bug 179584 OpenCloseViewTest#showView:org.eclipse.ui.views.ResourceNavigator() should be removed from the fingerprints
diff --git a/tests/org.eclipse.ui.tests.performance/src/org/eclipse/ui/tests/performance/OpenCloseViewTest.java b/tests/org.eclipse.ui.tests.performance/src/org/eclipse/ui/tests/performance/OpenCloseViewTest.java
index d01c59f..02058aa 100644
--- a/tests/org.eclipse.ui.tests.performance/src/org/eclipse/ui/tests/performance/OpenCloseViewTest.java
+++ b/tests/org.eclipse.ui.tests.performance/src/org/eclipse/ui/tests/performance/OpenCloseViewTest.java
@@ -41,8 +41,7 @@
processEvents();
tagIfNecessary("UI - Open/Close " + view1.getTitle(), Dimension.ELAPSED_PROCESS);
- setDegradationComment("<a href=https://bugs.eclipse.org/bugs/show_bug.cgi?id=98144>See Bug 98144</a> ");
-
+
for (int j = 0; j < 100; j++) {
startMeasuring();
diff --git a/tests/org.eclipse.ui.tests.performance/src/org/eclipse/ui/tests/performance/ViewPerformanceSuite.java b/tests/org.eclipse.ui.tests.performance/src/org/eclipse/ui/tests/performance/ViewPerformanceSuite.java
index 484da0e..9ac8e20 100644
--- a/tests/org.eclipse.ui.tests.performance/src/org/eclipse/ui/tests/performance/ViewPerformanceSuite.java
+++ b/tests/org.eclipse.ui.tests.performance/src/org/eclipse/ui/tests/performance/ViewPerformanceSuite.java
@@ -26,14 +26,11 @@
*/
public class ViewPerformanceSuite extends TestSuite {
- public static final String RESOURCE_NAVIGATOR = "org.eclipse.ui.views.ResourceNavigator";
+ public static final String PROJECT_EXPLORER = "org.eclipse.ui.navigator.ProjectExplorer";
public static final String BASIC_PATH = "org.eclipse.ui";
-
- public static final String VIEWS_PATTERN = "org.eclipse.ui.views";
- // public static final String [] VIEW_IDS = {BASIC_VIEW,
- // IPageLayout.ID_RES_NAV, MockViewPart.ID};
+ public static final String VIEWS_PATTERN = "org.eclipse.ui.views";
/**
* Returns the suite. This is required to use the JUnit Launcher.
@@ -60,11 +57,8 @@
for (int i = 0; i < ids.length; i++) {
String id = ids[i];
- // Use the BASIC_VIEW as the fingerprint test.
- // Do not change this as this is an empty view
- // and not dependant on other components
addTest(new OpenCloseViewTest(id,
- id.equals(RESOURCE_NAVIGATOR) ? BasicPerformanceTest.GLOBAL
+ id.equals(PROJECT_EXPLORER) ? BasicPerformanceTest.GLOBAL
: BasicPerformanceTest.NONE));
}
}
@@ -87,6 +81,7 @@
.getViewRegistry().getViews();
for (int i = 0; i < descriptors.length; i++) {
IViewDescriptor descriptor = descriptors[i];
+
String[] categoryPath = descriptor.getCategoryPath();
if (categoryPath == null)
@@ -94,10 +89,12 @@
for (int j = 0; j < categoryPath.length; j++) {
// Only test basic views
- if (categoryPath[j].equals(BASIC_PATH)){
- if(descriptor.getId().indexOf(VIEWS_PATTERN) >= 0) {
+ if (categoryPath[j].equals(BASIC_PATH)) {
+ if (descriptor.getId().indexOf(VIEWS_PATTERN) >= 0
+ || descriptor.getId().equals(PROJECT_EXPLORER)) {
result.add(descriptor.getId());
- }}
+ }
+ }
}
}