Author: akazakov Date: 2011-09-14 14:45:20 -0400 (Wed, 14 Sep 2011) New Revision: 34741 Modified: trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/core/ICDIElement.java trunk/cdi/tests/org.jboss.tools.cdi.core.test/src/org/jboss/tools/cdi/core/test/CDICoreAllTests.java trunk/cdi/tests/org.jboss.tools.cdi.core.test/src/org/jboss/tools/cdi/core/test/tck/CoreTest.java Log: https://issues.jboss.org/browse/JBIDE-9673 "Show All Generic Configuration Points", "Show CDI Events" and "Show CDI Observer Methods" dialogs should look like "Open Implementation" in JDT Modified: trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/core/ICDIElement.java =================================================================== --- trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/core/ICDIElement.java 2011-09-14 18:36:24 UTC (rev 34740) +++ trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/core/ICDIElement.java 2011-09-14 18:45:20 UTC (rev 34741) @@ -63,6 +63,9 @@ * If the element is a field then the method will return "TypeName.fieldName". * If the bean is a method then the method will return "TypeName.methodName()". * For a method parameter, a name of the parameter will be return. + * + * Implementations may not return null. + * * @return the text representation of this bean. */ String getElementName(); Modified: trunk/cdi/tests/org.jboss.tools.cdi.core.test/src/org/jboss/tools/cdi/core/test/CDICoreAllTests.java =================================================================== --- trunk/cdi/tests/org.jboss.tools.cdi.core.test/src/org/jboss/tools/cdi/core/test/CDICoreAllTests.java 2011-09-14 18:36:24 UTC (rev 34740) +++ trunk/cdi/tests/org.jboss.tools.cdi.core.test/src/org/jboss/tools/cdi/core/test/CDICoreAllTests.java 2011-09-14 18:45:20 UTC (rev 34741) @@ -26,6 +26,7 @@ import org.jboss.tools.cdi.core.test.tck.BeanSpecializationTest; import org.jboss.tools.cdi.core.test.tck.BuiltInBeanInjectionTest; import org.jboss.tools.cdi.core.test.tck.CDIUtilTest; +import org.jboss.tools.cdi.core.test.tck.CoreTest; import org.jboss.tools.cdi.core.test.tck.DecoratorDefinitionTest; import org.jboss.tools.cdi.core.test.tck.DefaultNamedTest; import org.jboss.tools.cdi.core.test.tck.EnterpriseQualifierDefinitionTest; @@ -108,6 +109,7 @@ suite.addTestSuite(UnsatisfiedDependencyTest.class); suite.addTestSuite(ResolvedTypesCacheTest.class); suite.addTestSuite(CDIUtilTest.class); + suite.addTestSuite(CoreTest.class); // Validation tests suite.addTestSuite(DefenitionErrorsValidationTest.class); Modified: trunk/cdi/tests/org.jboss.tools.cdi.core.test/src/org/jboss/tools/cdi/core/test/tck/CoreTest.java =================================================================== --- trunk/cdi/tests/org.jboss.tools.cdi.core.test/src/org/jboss/tools/cdi/core/test/tck/CoreTest.java 2011-09-14 18:36:24 UTC (rev 34740) +++ trunk/cdi/tests/org.jboss.tools.cdi.core.test/src/org/jboss/tools/cdi/core/test/tck/CoreTest.java 2011-09-14 18:45:20 UTC (rev 34741) @@ -41,9 +41,6 @@ for (IInjectionPoint point : points) { assertElementName(point); if(point.getType()!=null) { - if(point.getType().getSimpleName()==null) { - System.out.println("!!!"); - } assertNotNull(point.getType().getSimpleName()); } }