Author: dazarov Date: 2012-01-25 12:58:33 -0500 (Wed, 25 Jan 2012) New Revision: 38180 Modified: trunk/cdi/plugins/org.jboss.tools.cdi.ui/src/org/jboss/tools/cdi/ui/marker/ChangeAnnotationMarkerResolution.java trunk/cdi/tests/org.jboss.tools.cdi.ui.test/src/org/jboss/tools/cdi/ui/test/marker/CDIMarkerResolutionTest.java Log: Add the code which is supposed to be inserted by the Quick Fix into its description https://issues.jboss.org/browse/JBIDE-10636 Modified: trunk/cdi/plugins/org.jboss.tools.cdi.ui/src/org/jboss/tools/cdi/ui/marker/ChangeAnnotationMarkerResolution.java =================================================================== --- trunk/cdi/plugins/org.jboss.tools.cdi.ui/src/org/jboss/tools/cdi/ui/marker/ChangeAnnotationMarkerResolution.java 2012-01-25 17:52:35 UTC (rev 38179) +++ trunk/cdi/plugins/org.jboss.tools.cdi.ui/src/org/jboss/tools/cdi/ui/marker/ChangeAnnotationMarkerResolution.java 2012-01-25 17:58:33 UTC (rev 38180) @@ -56,6 +56,7 @@ String shortName = CDIMarkerResolutionUtils.getShortName(parameter); changeString += "("+shortName+")"; + init(); } public ChangeAnnotationMarkerResolution(IAnnotation annotation, String[] typeNames){ @@ -68,11 +69,13 @@ totalList = CDIMarkerResolutionUtils.OPEN_BRACE+totalList+CDIMarkerResolutionUtils.CLOSE_BRACE; changeString += "("+totalList+")"; + init(); } public ChangeAnnotationMarkerResolution(IAnnotation annotation, String[] typeNames, boolean useBraces){ this(annotation, typeNames); this.useBraces = useBraces; + init(); } @Override Modified: trunk/cdi/tests/org.jboss.tools.cdi.ui.test/src/org/jboss/tools/cdi/ui/test/marker/CDIMarkerResolutionTest.java =================================================================== --- trunk/cdi/tests/org.jboss.tools.cdi.ui.test/src/org/jboss/tools/cdi/ui/test/marker/CDIMarkerResolutionTest.java 2012-01-25 17:52:35 UTC (rev 38179) +++ trunk/cdi/tests/org.jboss.tools.cdi.ui.test/src/org/jboss/tools/cdi/ui/test/marker/CDIMarkerResolutionTest.java 2012-01-25 17:58:33 UTC (rev 38180) @@ -39,6 +39,8 @@ import org.jboss.tools.cdi.ui.marker.DeleteAllInjectedConstructorsMarkerResolution; import org.jboss.tools.cdi.ui.marker.DeleteAllOtherAnnotationsFromParametersMarkerResolution; import org.jboss.tools.cdi.ui.marker.DeleteAnnotationMarkerResolution; +import org.jboss.tools.cdi.ui.marker.MakeBeanScopedDependentMarkerResolution; +import org.jboss.tools.cdi.ui.marker.MakeFieldProtectedMarkerResolution; import org.jboss.tools.cdi.ui.marker.MakeFieldStaticMarkerResolution; import org.jboss.tools.cdi.ui.marker.MakeMethodBusinessMarkerResolution; import org.jboss.tools.cdi.ui.marker.MakeMethodPublicMarkerResolution; @@ -1049,4 +1051,27 @@ CDIValidationErrorManager.MULTIPLE_OBSERVING_PARAMETERS_ID, DeleteAllOtherAnnotationsFromParametersMarkerResolution.class); } + + public void testMakeFieldProtectedResolution() throws CoreException{ + checkResolution(tckProject, + new String[]{ + "JavaSource/org/jboss/jsr299/tck/tests/jbt/quickfixes/Leopard_Broken.java" + }, + CDICoreValidator.PROBLEM_TYPE, + CDIValidationErrorManager.MESSAGE_ID_ATTRIBUTE_NAME, + CDIValidationErrorManager.ILLEGAL_SCOPE_FOR_MANAGED_BEAN_WITH_PUBLIC_FIELD_ID, + MakeFieldProtectedMarkerResolution.class); + } + + public void testMakeBeanScopedDependentResolution() throws CoreException{ + checkResolution(tckProject, + new String[]{ + "JavaSource/org/jboss/jsr299/tck/tests/jbt/quickfixes/Leopard_Broken.java" + }, + CDICoreValidator.PROBLEM_TYPE, + CDIValidationErrorManager.MESSAGE_ID_ATTRIBUTE_NAME, + CDIValidationErrorManager.ILLEGAL_SCOPE_FOR_MANAGED_BEAN_WITH_PUBLIC_FIELD_ID, + MakeBeanScopedDependentMarkerResolution.class); + } + } \ No newline at end of file