Author: max.andersen(a)jboss.com Date: 2008-01-21 14:32:54 -0500 (Mon, 21 Jan 2008) New Revision: 5856 Modified: trunk/jst/plugins/org.jboss.tools.jst.jsp/src/org/jboss/tools/jst/jsp/jspeditor/JSPTextEditor.java trunk/seam/tests/org.jboss.tools.seam.ui.test/src/org/jboss/tools/seam/ui/test/ca/SeamELContentAssistTest.java Log: if we must have stupid test only methods then let us clearly mark them as such. Modified: trunk/jst/plugins/org.jboss.tools.jst.jsp/src/org/jboss/tools/jst/jsp/jspeditor/JSPTextEditor.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.jsp/src/org/jboss/tools/jst/jsp/jspeditor/JSPTextEditor.java 2008-01-21 19:25:44 UTC (rev 5855) +++ trunk/jst/plugins/org.jboss.tools.jst.jsp/src/org/jboss/tools/jst/jsp/jspeditor/JSPTextEditor.java 2008-01-21 19:32:54 UTC (rev 5856) @@ -167,7 +167,8 @@ super.setSourceViewerConfiguration(config); } - public SourceViewerConfiguration getSVConfiguration () { + /** This is *only* for allowing unit tests to access the source configuration. */ + public SourceViewerConfiguration getSourceViewerConfigurationForTest () { return getSourceViewerConfiguration(); } Modified: trunk/seam/tests/org.jboss.tools.seam.ui.test/src/org/jboss/tools/seam/ui/test/ca/SeamELContentAssistTest.java =================================================================== --- trunk/seam/tests/org.jboss.tools.seam.ui.test/src/org/jboss/tools/seam/ui/test/ca/SeamELContentAssistTest.java 2008-01-21 19:25:44 UTC (rev 5855) +++ trunk/seam/tests/org.jboss.tools.seam.ui.test/src/org/jboss/tools/seam/ui/test/ca/SeamELContentAssistTest.java 2008-01-21 19:32:54 UTC (rev 5856) @@ -454,7 +454,7 @@ JSPTextEditor jspTextEditor = jspEditor.getJspEditor(); StructuredTextViewer viewer = jspTextEditor.getTextViewer(); IDocument document = viewer.getDocument(); - IContentAssistant contentAssistant = jspTextEditor.getSVConfiguration().getContentAssistant(viewer); + IContentAssistant contentAssistant = jspTextEditor.getSourceViewerConfigurationForTest().getContentAssistant(viewer); List<IRegion> regionsToTest = getELRegionsToTest(document); if (regionsToTest != null) {