Skip to content

Commit

Permalink
579498 filter dialog types #144
Browse files Browse the repository at this point in the history
Squashed commit of the following:

commit 24db6db
Author: Enda O Brien <[email protected]>
Date:   Fri Sep 20 13:19:55 2024 +0100

    579498 filter dialog types #144

    Squashed commit of the following:

    commit 94f352d
    Author: Enda O Brien <[email protected]>
    Date:   Thu Apr 4 17:07:02 2024 +0100

        Fixed a couple of warnings in test code

    commit 49a195d
    Author: Enda O Brien <[email protected]>
    Date:   Wed Aug 24 15:54:38 2022 +0100

        put add and addAll back in previous order

    commit 7dd8032
    Author: Enda O Brien <[email protected]>
    Date:   Wed Aug 24 15:11:23 2022 +0100

        Moved copyright to be first lines in files

    commit 87763b5
    Author: Enda O Brien <[email protected]>
    Date:   Wed Aug 24 14:41:46 2022 +0100

        Modified copyright message in test code to include my name

    commit 1b08d4a
    Author: Enda O Brien <[email protected]>
    Date:   Wed Aug 24 14:09:12 2022 +0100

        updated documentation of markerSupport.exsd

    commit 579fca0
    Author: Enda O Brien <[email protected]>
    Date:   Wed Aug 24 12:04:44 2022 +0100

        Added copyright notice to test code

    commit 1271c95
    Author: Enda O Brien <[email protected]>
    Date:   Wed Aug 24 11:49:13 2022 +0100

        Fixed whitespace differences

    commit 36e50cd
    Author: Enda O Brien <[email protected]>
    Date:   Thu Jun 9 14:16:44 2022 +0100

        Test code for application attribute on marker type reference

    commit 01940b2
    Author: Enda O Brien <[email protected]>
    Date:   Thu Jun 9 10:57:55 2022 +0100

        Added application attribute to type reference in marker content generator
  • Loading branch information
eobrienPilz committed Sep 24, 2024
1 parent 0986115 commit 99c91af
Show file tree
Hide file tree
Showing 9 changed files with 346 additions and 4 deletions.
20 changes: 20 additions & 0 deletions bundles/org.eclipse.ui.ide/schema/markerSupport.exsd
Original file line number Diff line number Diff line change
Expand Up @@ -192,6 +192,26 @@ ON_ANY_IN_SAME_CONTAINER: on any item with the same top level container as the s
</appinfo>
</annotation>
</attribute>
<attribute name="application">
<annotation>
<documentation>
The application attribute describes how the reference should be applied.
i.e. Does it refer to type only, type and subtypes or subtypes only.
It is optionally included.
If it is not specified it defaults to typeAndSubTypes.
</documentation>
</annotation>
<simpleType>
<restriction base="string">
<enumeration value="subTypesOnly">
</enumeration>
<enumeration value="typeOnly">
</enumeration>
<enumeration value="typeAndSubTypes">
</enumeration>
</restriction>
</simpleType>
</attribute>
</complexType>
</element>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,26 @@ public class MarkerSupportInternalUtilities {
*/
public static final String ATTRIBUTE_ID = "id"; //$NON-NLS-1$

/**
* The application attribute from a configuration element.
*/
public static final String APPLICATION = "application"; //$NON-NLS-1$

/**
* The sub type only attribute value from the application attribute.
*/
public static final String SUB_TYPES_ONLY = "subTypesOnly"; //$NON-NLS-1$

/**
* The type only attribute value from the application attribute.
*/
public static final String TYPE_ONLY = "typeOnly"; //$NON-NLS-1$

/**
* The type and subtype attribute value from the application attribute.
*/
public static final String TYPE_AND_SUBTYPE = "typeAndSubTypes"; //$NON-NLS-1$

/**
* The name attribute name from a configuration element.
*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -190,9 +190,25 @@ public Collection<MarkerType> getMarkerTypes() {
IConfigurationElement[] markerTypeElements = configurationElement.getChildren(MarkerSupportRegistry.MARKER_TYPE_REFERENCE);
for (IConfigurationElement configElement : markerTypeElements) {
String elementName = configElement.getAttribute(MarkerSupportInternalUtilities.ATTRIBUTE_ID);
MarkerType[] types = MarkerTypesModel.getInstance().getType(elementName).getAllSubTypes();
markerTypes.addAll(Arrays.asList(types));
markerTypes.add(MarkerTypesModel.getInstance().getType(elementName));

String application = configElement.getAttribute(MarkerSupportInternalUtilities.APPLICATION) == null
? MarkerSupportInternalUtilities.TYPE_AND_SUBTYPE
: configElement.getAttribute(MarkerSupportInternalUtilities.APPLICATION);

switch (application) {
case MarkerSupportInternalUtilities.TYPE_ONLY:
markerTypes.add(MarkerTypesModel.getInstance().getType(elementName));
break;
case MarkerSupportInternalUtilities.SUB_TYPES_ONLY:
markerTypes.addAll(
Arrays.asList(MarkerTypesModel.getInstance().getType(elementName).getAllSubTypes()));
break;
case MarkerSupportInternalUtilities.TYPE_AND_SUBTYPE:
default:
markerTypes.addAll(
Arrays.asList(MarkerTypesModel.getInstance().getType(elementName).getAllSubTypes()));
markerTypes.add(MarkerTypesModel.getInstance().getType(elementName));
}
}
if (markerTypes.isEmpty()) {
MarkerType[] types = MarkerTypesModel.getInstance().getType(IMarker.PROBLEM).getAllSubTypes();
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
package org.eclipse.ui.tests;

import org.eclipse.ui.views.markers.MarkerSupportView;

/**
* @since 3.5
*
*/
public class NoApplicationAttribTestView extends MarkerSupportView {
public static final String ID = "org.eclipse.ui.tests.noApplicationAttribTestView";

static final String CONTENT_GEN_ID = "org.eclipse.ui.tests.noApplicationAttribTestViewContentGenerator";

public NoApplicationAttribTestView() {
super(CONTENT_GEN_ID);
}

}
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
/*******************************************************************************
* Copyright (c) 2022 Enda O'Brien and others.
*
* This program and the accompanying materials are made available under the
* terms of the Eclipse Public License 2.0 which accompanies this distribution,
* and is available at https://www.eclipse.org/legal/epl-2.0/
*
* SPDX-License-Identifier: EPL-2.0
*
* Contributors: IBM Corporation - initial API and implementation
*******************************************************************************/
package org.eclipse.ui.tests;

import org.eclipse.ui.views.markers.MarkerSupportView;

public class SubTypeOnlyTestView extends MarkerSupportView {
public static final String ID = "org.eclipse.ui.tests.subTypeOnlyTestView";

static final String CONTENT_GEN_ID = "org.eclipse.ui.tests.subTypeOnlyTestViewContentGenerator";

public SubTypeOnlyTestView() {
super(CONTENT_GEN_ID);
}

}
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
/*******************************************************************************
* Copyright (c) 2022 Enda O'Brien and others.
*
* This program and the accompanying materials are made available under the
* terms of the Eclipse Public License 2.0 which accompanies this distribution,
* and is available at https://www.eclipse.org/legal/epl-2.0/
*
* SPDX-License-Identifier: EPL-2.0
*
* Contributors: IBM Corporation - initial API and implementation
*******************************************************************************/
package org.eclipse.ui.tests;

import org.eclipse.ui.views.markers.MarkerSupportView;

public class TypeAndSubTypeTestView extends MarkerSupportView {
public static final String ID = "org.eclipse.ui.tests.typeAndSubTypeTestView";

static final String CONTENT_GEN_ID = "org.eclipse.ui.tests.typeAndSubTypeTestViewContentGenerator";

public TypeAndSubTypeTestView() {
super(CONTENT_GEN_ID);
}

}
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
/*******************************************************************************
* Copyright (c) 2022 Enda O'Brien and others.
*
* This program and the accompanying materials are made available under the
* terms of the Eclipse Public License 2.0 which accompanies this distribution,
* and is available at https://www.eclipse.org/legal/epl-2.0/
*
* SPDX-License-Identifier: EPL-2.0
*
* Contributors: IBM Corporation - initial API and implementation
*******************************************************************************/
package org.eclipse.ui.tests;

import org.eclipse.ui.views.markers.MarkerSupportView;

public class TypeOnlyTestView extends MarkerSupportView {

public static final String ID = "org.eclipse.ui.tests.typeOnlyTestView";

static final String CONTENT_GEN_ID = "org.eclipse.ui.tests.typeOnlyTestViewContentGenerator";

public TypeOnlyTestView() {
super(CONTENT_GEN_ID);
}

}
Original file line number Diff line number Diff line change
@@ -0,0 +1,142 @@
/*******************************************************************************
* Copyright (c) 2022 Enda O'Brien and others.
*
* This program and the accompanying materials are made available under the
* terms of the Eclipse Public License 2.0 which accompanies this distribution,
* and is available at https://www.eclipse.org/legal/epl-2.0/
*
* SPDX-License-Identifier: EPL-2.0
*
* Contributors: IBM Corporation - initial API and implementation
*******************************************************************************/

package org.eclipse.ui.tests.markers;

import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertTrue;

import java.lang.reflect.Field;
import java.util.Arrays;
import java.util.Collection;
import java.util.HashSet;

import org.eclipse.ui.PlatformUI;
import org.eclipse.ui.internal.views.markers.ExtendedMarkersView;
import org.eclipse.ui.internal.views.markers.MarkerContentGenerator;
import org.eclipse.ui.tests.NoApplicationAttribTestView;
import org.eclipse.ui.tests.SubTypeOnlyTestView;
import org.eclipse.ui.tests.TypeAndSubTypeTestView;
import org.eclipse.ui.tests.TypeOnlyTestView;
import org.eclipse.ui.views.markers.MarkerSupportView;
import org.eclipse.ui.views.markers.internal.ContentGeneratorDescriptor;
import org.eclipse.ui.views.markers.internal.MarkerType;
import org.eclipse.ui.views.markers.internal.MarkerTypesModel;
import org.junit.Test;

public class MarkerTypeTests {

static final String PROBLEM_MARKER = "org.eclipse.core.resources.problemmarker";

@Test
public void canIncludeTypeOnly() throws Exception {
MarkerSupportView view = (MarkerSupportView) PlatformUI.getWorkbench().getActiveWorkbenchWindow()
.getActivePage().showView(TypeOnlyTestView.ID);

MarkerContentGenerator generator = getMarkerContentGenerator(view);
Collection<MarkerType> filterDialogTypes = getMarkerTypes(generator);

assertEquals(1, filterDialogTypes.size());
assertEquals(PROBLEM_MARKER, filterDialogTypes.stream().map(type -> type.getId()).findFirst().get());
}

@Test
public void canIncludeTypeAndSubTypes() throws Exception {
MarkerSupportView view = (MarkerSupportView) PlatformUI.getWorkbench().getActiveWorkbenchWindow()
.getActivePage().showView(TypeAndSubTypeTestView.ID);

MarkerContentGenerator generator = getMarkerContentGenerator(view);
Collection<MarkerType> filterDialogTypes = getMarkerTypes(generator);

Collection<MarkerType> markerTypes = new HashSet<>();
markerTypes.add(MarkerTypesModel.getInstance().getType(PROBLEM_MARKER));
markerTypes.addAll(Arrays.asList(MarkerTypesModel.getInstance().getType(PROBLEM_MARKER).getAllSubTypes()));

assertEquals(markerTypes.size(), filterDialogTypes.size());
assertEquals(PROBLEM_MARKER, filterDialogTypes.stream().map(type -> type.getId())
.filter(s -> s.equals(PROBLEM_MARKER)).findFirst().get());

for (MarkerType type : markerTypes) {
assertTrue(filterDialogTypes.contains(type));
}
}

@Test
public void canIncludeSubtypesOnly() throws Exception {
MarkerSupportView view = (MarkerSupportView) PlatformUI.getWorkbench().getActiveWorkbenchWindow()
.getActivePage().showView(SubTypeOnlyTestView.ID);

MarkerContentGenerator generator = getMarkerContentGenerator(view);
Collection<MarkerType> filterDialogTypes = getMarkerTypes(generator);

Collection<MarkerType> markerTypes = new HashSet<>();
markerTypes.addAll(Arrays.asList(MarkerTypesModel.getInstance().getType(PROBLEM_MARKER).getAllSubTypes()));

assertEquals(markerTypes.size(), filterDialogTypes.size());
assertTrue(PROBLEM_MARKER, filterDialogTypes.stream().map(type -> type.getId())
.filter(s -> s.equals(PROBLEM_MARKER)).findFirst().isEmpty());
for (MarkerType type : markerTypes) {
assertTrue(filterDialogTypes.contains(type));
}
}

@Test
public void typeAndSubTypesIsDefault() throws Exception {
MarkerSupportView view = (MarkerSupportView) PlatformUI.getWorkbench().getActiveWorkbenchWindow()
.getActivePage().showView(NoApplicationAttribTestView.ID);

MarkerContentGenerator generator = getMarkerContentGenerator(view);
Collection<MarkerType> filterDialogTypes = getMarkerTypes(generator);

Collection<MarkerType> markerTypes = new HashSet<>();
markerTypes.add(MarkerTypesModel.getInstance().getType(PROBLEM_MARKER));
markerTypes.addAll(Arrays.asList(MarkerTypesModel.getInstance().getType(PROBLEM_MARKER).getAllSubTypes()));

assertEquals(markerTypes.size(), filterDialogTypes.size());
assertEquals(PROBLEM_MARKER, filterDialogTypes.stream().map(type -> type.getId())
.filter(s -> s.equals(PROBLEM_MARKER)).findFirst().get());

for (MarkerType type : markerTypes) {
assertTrue(filterDialogTypes.contains(type));
}
}

public static MarkerContentGenerator getMarkerContentGenerator(MarkerSupportView view) {
MarkerContentGenerator generator = null;
try {
Field fieldGenerator = ExtendedMarkersView.class.getDeclaredField("generator");
fieldGenerator.setAccessible(true);
generator = (MarkerContentGenerator) fieldGenerator.get(view);
} catch (NoSuchFieldException | SecurityException | IllegalArgumentException | IllegalAccessException e) {
}
return generator;
}

@SuppressWarnings("unchecked")
public static Collection<MarkerType> getMarkerTypes(MarkerContentGenerator generator) {
Collection<MarkerType> selectedTypesCollection = null;
try {
Field generatorDescriptor = MarkerContentGenerator.class.getDeclaredField("generatorDescriptor");
generatorDescriptor.setAccessible(true);

ContentGeneratorDescriptor contentGeneratorDescriptor = (ContentGeneratorDescriptor) generatorDescriptor
.get(generator);

Field markerTypesField = ContentGeneratorDescriptor.class.getDeclaredField("markerTypes");
markerTypesField.setAccessible(true);

selectedTypesCollection = (Collection<MarkerType>) markerTypesField.get(contentGeneratorDescriptor);
} catch (NoSuchFieldException | SecurityException | IllegalArgumentException | IllegalAccessException e) {
}
return selectedTypesCollection;
}
}
52 changes: 51 additions & 1 deletion tests/org.eclipse.ui.tests/plugin.xml
Original file line number Diff line number Diff line change
Expand Up @@ -190,7 +190,26 @@
class="org.eclipse.ui.tests.api.workbenchpart.ViewWithCreateControlsException"
id="org.eclipse.ui.tests.api.workbenchpart.ViewWithCreateControlsException">
</view>

<view
class="org.eclipse.ui.tests.TypeOnlyTestView"
id="org.eclipse.ui.tests.typeOnlyTestView"
name="TypeOnlyTestView">
</view>
<view
class="org.eclipse.ui.tests.SubTypeOnlyTestView"
id="org.eclipse.ui.tests.subTypeOnlyTestView"
name="SubTypeOnlyTestView">
</view>
<view
class="org.eclipse.ui.tests.TypeAndSubTypeTestView"
id="org.eclipse.ui.tests.typeAndSubTypeTestView"
name="TypeAndSubTypeTestView">
</view>
<view
class="org.eclipse.ui.tests.NoApplicationAttribTestView"
id="org.eclipse.ui.tests.noApplicationAttribTestView"
name="NoApplicationAttribTestView">
</view>
</extension>

<extension
Expand Down Expand Up @@ -3408,6 +3427,37 @@
</markerContentGeneratorExtension>
<markerContentGenerator
id="org.eclipse.ui.tests.filterHelpContentGenerator">
</markerContentGenerator>
<markerContentGenerator
id="org.eclipse.ui.tests.typeAndSubTypeTestViewContentGenerator"
name="typeAndSubTypeTestViewContentGenerator">
<markerTypeReference
application="typeAndSubTypes"
id="org.eclipse.core.resources.problemmarker">
</markerTypeReference>
</markerContentGenerator>
<markerContentGenerator
id="org.eclipse.ui.tests.typeOnlyTestViewContentGenerator"
name="typeOnlyTestViewContentGenerator">
<markerTypeReference
application="typeOnly"
id="org.eclipse.core.resources.problemmarker">
</markerTypeReference>
</markerContentGenerator>
<markerContentGenerator
id="org.eclipse.ui.tests.subTypeOnlyTestViewContentGenerator"
name="subTypeOnlyTestViewContentGenerator">
<markerTypeReference
application="subTypesOnly"
id="org.eclipse.core.resources.problemmarker">
</markerTypeReference>
</markerContentGenerator>
<markerContentGenerator
id="org.eclipse.ui.tests.noApplicationAttribTestViewContentGenerator"
name="noApplicationAttribTestViewContentGenerator">
<markerTypeReference
id="org.eclipse.core.resources.problemmarker">
</markerTypeReference>
</markerContentGenerator>
<markerContentGenerator
id="org.eclipse.ui.tests.customScopeContentGenerator">
Expand Down

0 comments on commit 99c91af

Please sign in to comment.