Author: vrubezhny Date: 2011-04-28 07:17:28 -0400 (Thu, 28 Apr 2011) New Revision: 30899 Modified: trunk/seam/tests/org.jboss.tools.seam.ui.test/src/org/jboss/tools/seam/ui/test/ca/SeamELContentAssistJbide1676Test.java Log: JBIDE-8806 testSeamELContentAssistJbide1676 test case fails Issue is fixed Modified: trunk/seam/tests/org.jboss.tools.seam.ui.test/src/org/jboss/tools/seam/ui/test/ca/SeamELContentAssistJbide1676Test.java =================================================================== --- trunk/seam/tests/org.jboss.tools.seam.ui.test/src/org/jboss/tools/seam/ui/test/ca/SeamELContentAssistJbide1676Test.java 2011-04-28 09:59:05 UTC (rev 30898) +++ trunk/seam/tests/org.jboss.tools.seam.ui.test/src/org/jboss/tools/seam/ui/test/ca/SeamELContentAssistJbide1676Test.java 2011-04-28 11:17:28 UTC (rev 30899) @@ -45,7 +45,7 @@ private static final String[] VALID_SEAM_EL_PROPOSALS = new String[] { "TestSeamELContentAssistEntityManagerFactory", "fullPostList", - "a4j", + "a4j", "a4jSkin", "actor", "ajaxContext", @@ -302,13 +302,20 @@ if (filteredValidProposals.contains(proposalString)) { existingProposals.add(proposalString); filteredValidProposals.remove(proposalString); - } else { + } else if (!existingProposals.contains(proposalString)){ nonExistingProposals.add(proposalString); } } } assertTrue("Some Seam EL proposals werent\'t shown in the Content Assistant", filteredValidProposals.isEmpty()); - assertTrue("Some Seam EL proposals were shown in the Content Assistant but they shouldn\'t", nonExistingProposals.isEmpty()); + StringBuffer sb = new StringBuffer(); + if (nonExistingProposals != null && !nonExistingProposals.isEmpty()) { + for (String np : nonExistingProposals) { + sb.append(np).append(' '); + } + } + + assertTrue("Some Seam EL proposals were shown in the Content Assistant but they shouldn\'t: " + sb.toString(), nonExistingProposals.isEmpty()); } finally { PlatformUI.getWorkbench().getActiveWorkbenchWindow().getActivePage() .closeEditor(editorPart, false);