[richfaces-svn-commits] JBoss Rich Faces SVN: r19019 - in trunk/ui/input/ui/src/test: resources/org/richfaces/renderkit and 1 other directory.

richfaces-svn-commits at lists.jboss.org richfaces-svn-commits at lists.jboss.org
Mon Aug 30 10:26:50 EDT 2010


Author: abelevich
Date: 2010-08-30 10:26:50 -0400 (Mon, 30 Aug 2010)
New Revision: 19019

Added:
   trunk/ui/input/ui/src/test/resources/org/richfaces/renderkit/inplaceInputTest.xhtml
Removed:
   trunk/ui/input/ui/src/test/resources/org/richfaces/renderkit/rendererTest.xhtml
Modified:
   trunk/ui/input/ui/src/test/java/org/richfaces/renderkit/InplaceInputRendererTest.java
Log:
test refactoring

Modified: trunk/ui/input/ui/src/test/java/org/richfaces/renderkit/InplaceInputRendererTest.java
===================================================================
--- trunk/ui/input/ui/src/test/java/org/richfaces/renderkit/InplaceInputRendererTest.java	2010-08-30 14:23:21 UTC (rev 19018)
+++ trunk/ui/input/ui/src/test/java/org/richfaces/renderkit/InplaceInputRendererTest.java	2010-08-30 14:26:50 UTC (rev 19019)
@@ -31,7 +31,6 @@
 import org.jboss.test.faces.htmlunit.HtmlUnitEnvironment;
 import org.junit.After;
 import org.junit.Before;
-import org.junit.Ignore;
 import org.junit.Test;
 
 import com.gargoylesoftware.htmlunit.html.DomNode;
@@ -52,17 +51,14 @@
         environment = new HtmlUnitEnvironment();
         environment.withWebRoot(new File("src/test/resources"));
         environment.withResource("/WEB-INF/faces-config.xml", "org/richfaces/renderkit/faces-config.xml");
-        environment.withResource("/test.xhtml", "org/richfaces/renderkit/rendererTest.xhtml");
+        environment.withResource("/test.xhtml", "org/richfaces/renderkit/inplaceInputTest.xhtml");
         environment.start();
     }
     
     @Test
-    @Ignore("Updates to RF-9132 broke this test - it needs to be fixed, but is blocking M2 release")
     public void testRenderDefaultState() throws Exception {
         HtmlPage page = environment.getPage("/test.jsf");
-        List<?> nodes = page.getByXPath("//*[@id = 'form:input_default']");
-        assertEquals(1, nodes.size());
-        HtmlElement span = (HtmlElement) nodes.get(0);
+        HtmlElement span = page.getFirstByXPath("//*[@id = 'form:input_default']");
         assertEquals("span", span.getNodeName());
         assertEquals("rf-ii-d-s", span.getAttribute(HTML.CLASS_ATTRIBUTE));
         
@@ -72,10 +68,7 @@
         DomNode text = label.getFirstChild();
         assertEquals(DomNode.TEXT_NODE, text.getNodeType());
         
-        List<?> editNodes = page.getByXPath("//*[@id = 'form:input_default:edit']");
-        assertEquals(1, editNodes.size());
-        HtmlElement edit = (HtmlElement) editNodes.get(0);
-        
+        HtmlElement edit = page.getFirstByXPath("//*[@id = 'form:input_default:edit']");
         assertEquals("span", edit.getNodeName());
         assertEquals("rf-ii-e-s rf-ii-none", edit.getAttribute(HTML.CLASS_ATTRIBUTE));
         
@@ -93,9 +86,7 @@
         
         HtmlPage page = environment.getPage("/test.jsf");
     
-        List<?> nodes = page.getByXPath("//*[@id = 'form:input_edit']");
-        assertEquals(1, nodes.size());
-        HtmlElement span = (HtmlElement) nodes.get(0);
+        HtmlElement span = page.getFirstByXPath("//*[@id = 'form:input_edit']");
         assertEquals("span", span.getNodeName());
         assertEquals("rf-ii-d-s", span.getAttribute(HTML.CLASS_ATTRIBUTE));
 
@@ -105,9 +96,7 @@
         DomNode text = label.getFirstChild();
         assertEquals(DomNode.TEXT_NODE, text.getNodeType());
         
-        List<?> editNodes = page.getByXPath("//*[@id = 'form:input_edit:edit']");
-        assertEquals(1, editNodes.size());
-        HtmlElement edit = (HtmlElement) editNodes.get(0);
+        HtmlElement edit = page.getFirstByXPath("//*[@id = 'form:input_edit:edit']");
         assertEquals("span", edit.getNodeName());
         assertEquals("rf-ii-e-s", edit.getAttribute(HTML.CLASS_ATTRIBUTE));
         
@@ -116,24 +105,15 @@
         assertEquals("rf-ii-f", input.getAttribute(HTML.CLASS_ATTRIBUTE));
         assertEquals(text.getNodeValue(), input.getAttribute(HTML.VALUE_ATTRIBUTE));
 
-        List<?> buttonNodes = page.getByXPath("//*[@id = 'form:input_edit:btn']");
-        assertEquals(1, buttonNodes.size());
-
-        HtmlElement button = (HtmlElement) buttonNodes.get(0);
+        HtmlElement button = page.getFirstByXPath("//*[@id = 'form:input_edit:btn']");
         assertEquals("span", button.getNodeName());
         
-        List<?> okButtonNodes = page.getByXPath("//*[@id = 'form:input_edit:okbtn']");
-        assertEquals(1, okButtonNodes.size());
-        
-        HtmlElement okButton = (HtmlElement) okButtonNodes.get(0);
+        HtmlElement okButton = page.getFirstByXPath("//*[@id = 'form:input_edit:okbtn']");
         assertEquals("input", okButton.getNodeName());
         assertEquals("rf-ii-btn", okButton.getAttribute(HTML.CLASS_ATTRIBUTE));
         assertEquals("image", okButton.getAttribute(HTML.TYPE_ATTR));
 
-        List<?> cancelButtonNodes = page.getByXPath("//*[@id = 'form:input_edit:cancelbtn']");
-        assertEquals(1, cancelButtonNodes.size());
-        
-        HtmlElement cancelButton = (HtmlElement) cancelButtonNodes.get(0);
+        HtmlElement cancelButton = page.getFirstByXPath("//*[@id = 'form:input_edit:cancelbtn']");
         assertEquals("input", cancelButton.getNodeName());
         assertEquals("rf-ii-btn", cancelButton.getAttribute(HTML.CLASS_ATTRIBUTE));
         assertEquals("image", cancelButton.getAttribute(HTML.TYPE_ATTR));
@@ -144,97 +124,62 @@
     public void testEdit() throws Exception {
         HtmlPage page = environment.getPage("/test.jsf");
         edit(page, "input_default", "Another Test String");  
-
         blur(page);
-
-        List<?> labelNodes = page.getByXPath("//*[@id = 'form:input_default:label']/text()");
-        assertEquals(1, labelNodes.size());
-        DomText text = (DomText) labelNodes.get(0);
+        DomText text = page.getFirstByXPath("//*[@id = 'form:input_default:label']/text()");
         assertEquals("Another Test String", text.getTextContent());
-        
-        List<?>nodes = page.getByXPath("//*[@id = 'form:input_default']");
-        assertEquals(1, nodes.size());
-        HtmlElement span = (HtmlElement) nodes.get(0);
+        HtmlElement span = page.getFirstByXPath("//*[@id = 'form:input_default']");
         assertEquals("rf-ii-d-s rf-ii-c-s", span.getAttribute(HTML.CLASS_ATTRIBUTE));
-
     }
     
     private void blur(HtmlPage page) throws Exception {
-        List<?> panelNodes = page.getByXPath("//*[@id = 'form:panel']");
-        assertEquals(1, panelNodes.size());
-        HtmlElement panel = (HtmlElement) panelNodes.get(0);
+        HtmlElement panel = page.getFirstByXPath("//*[@id = 'form:panel']");
         panel.click();
     }
     
     private void typeNewValue(HtmlPage page, String inplaceInputId, String value) throws Exception {
-        List<?> inputNodes = page.getByXPath("//*[@id = 'form:" + inplaceInputId + ":input']");
-        assertEquals(1, inputNodes.size());
-        HtmlElement input = (HtmlElement) inputNodes.get(0);
+        HtmlElement input = page.getFirstByXPath("//*[@id = 'form:" + inplaceInputId + ":input']");
         input.setAttribute(HTML.VALUE_ATTRIBUTE, "");
         input.type(value);
     }
     
     private void edit(HtmlPage page, String inplaceInputId, String value) throws Exception {
-        List<?> nodes = page.getByXPath("//*[@id = 'form:" + inplaceInputId + "']");
-        assertEquals(1, nodes.size());
-        HtmlElement span = (HtmlElement) nodes.get(0);
+        HtmlElement span = page.getFirstByXPath("//*[@id = 'form:" + inplaceInputId + "']");
         span.click();
-        
-        List<?> editNodes = page.getByXPath("//*[@id = 'form:" + inplaceInputId + ":edit']");
-        assertEquals(1, editNodes.size());
-        HtmlElement edit = (HtmlElement) editNodes.get(0);
+        HtmlElement edit = page.getFirstByXPath("//*[@id = 'form:" + inplaceInputId + ":edit']");
         assertEquals("rf-ii-e-s", edit.getAttribute(HTML.CLASS_ATTRIBUTE));
-
         typeNewValue(page, inplaceInputId, value);
     }
     
     @Test
     public void testEditWithControls() throws Exception {
         HtmlPage page = environment.getPage("/test.jsf");
-        
         edit(page, "input_controls", "Another Test String");
-        
-        List<?> cancelNodes = page.getByXPath("//*[@id = 'form:input_controls:cancelbtn']");
-        assertEquals(1, cancelNodes.size());
-        HtmlElement cancel = (HtmlElement) cancelNodes.get(0);
-       
+
+        HtmlElement cancel = page.getFirstByXPath("//*[@id = 'form:input_controls:cancelbtn']");
         cancel.mouseDown();
         
-        List<?> labelNodes = page.getByXPath("//*[@id = 'form:input_controls:label']/text()");
-        assertEquals(1, labelNodes.size());
-        DomText text = (DomText) labelNodes.get(0);
+        DomText text = page.getFirstByXPath("//*[@id = 'form:input_controls:label']/text()");
         assertEquals("Test String", text.getTextContent());
                 
-        List<?> nodes = page.getByXPath("//*[@id = 'form:input_controls']");
-        assertEquals(1, nodes.size());
-        HtmlElement span = (HtmlElement) nodes.get(0);
+        HtmlElement span = page.getFirstByXPath("//*[@id = 'form:input_controls']");
         assertEquals("rf-ii-d-s", span.getAttribute(HTML.CLASS_ATTRIBUTE));
         
         edit(page, "input_controls",  "Another Test String");
                 
-        List<?> okNodes = page.getByXPath("//*[@id = 'form:input_controls:okbtn']");
-        assertEquals(1, okNodes.size());
-        HtmlElement ok = (HtmlElement) okNodes.get(0);
-      
+        HtmlElement ok = page.getFirstByXPath("//*[@id = 'form:input_controls:okbtn']");
         ok.mouseDown();
         
-        labelNodes = page.getByXPath("//*[@id = 'form:input_controls:label']/text()");
-        assertEquals(1, labelNodes.size());
-        text = (DomText) labelNodes.get(0);
+        text = page.getFirstByXPath("//*[@id = 'form:input_controls:label']/text()");
         assertEquals("Another Test String", text.getTextContent());
         
-        nodes = page.getByXPath("//*[@id = 'form:input_controls']");
-        assertEquals(1, nodes.size());
-        span = (HtmlElement) nodes.get(0);
+        span = page.getFirstByXPath("//*[@id = 'form:input_controls']");
         assertEquals("rf-ii-d-s rf-ii-c-s", span.getAttribute(HTML.CLASS_ATTRIBUTE));
         
         edit(page, "input_controls", "Test String");
         
         blur(page);
         
-        labelNodes = page.getByXPath("//*[@id = 'form:input_controls:label']/text()");
-        assertEquals(1, labelNodes.size());
-        text = (DomText) labelNodes.get(0);
+        text = page.getFirstByXPath("//*[@id = 'form:input_controls:label']/text()");
         assertEquals("Test String", text.getTextContent());
     }
 

Added: trunk/ui/input/ui/src/test/resources/org/richfaces/renderkit/inplaceInputTest.xhtml
===================================================================
--- trunk/ui/input/ui/src/test/resources/org/richfaces/renderkit/inplaceInputTest.xhtml	                        (rev 0)
+++ trunk/ui/input/ui/src/test/resources/org/richfaces/renderkit/inplaceInputTest.xhtml	2010-08-30 14:26:50 UTC (rev 19019)
@@ -0,0 +1,26 @@
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml"
+	xmlns:h="http://java.sun.com/jsf/html"
+	xmlns:f="http://java.sun.com/jsf/core"
+	xmlns:ui="http://java.sun.com/jsf/facelets"
+	xmlns:in="http://richfaces.org/input">
+<f:view contentType="text/html" />
+
+<h:head>
+	<title>Richfaces InplaceInput</title>
+</h:head>
+
+<h:body>
+	<h:form id="form">
+		<in:inplaceInput id="input_default" value="#{dataBean.value}"/>
+	
+		<in:inplaceInput id="input_controls" showControls="true" value="#{dataBean.value}"/>
+	
+		<in:inplaceInput id="input_edit" state="edit" showControls="true" value="#{dataBean.value}"/>
+		
+		<h:panelGroup id="panel">
+			<!-- click here for blur -->
+		</h:panelGroup>
+	</h:form>
+</h:body>
+</html>

Deleted: trunk/ui/input/ui/src/test/resources/org/richfaces/renderkit/rendererTest.xhtml
===================================================================
--- trunk/ui/input/ui/src/test/resources/org/richfaces/renderkit/rendererTest.xhtml	2010-08-30 14:23:21 UTC (rev 19018)
+++ trunk/ui/input/ui/src/test/resources/org/richfaces/renderkit/rendererTest.xhtml	2010-08-30 14:26:50 UTC (rev 19019)
@@ -1,26 +0,0 @@
-<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
-<html xmlns="http://www.w3.org/1999/xhtml"
-	xmlns:h="http://java.sun.com/jsf/html"
-	xmlns:f="http://java.sun.com/jsf/core"
-	xmlns:ui="http://java.sun.com/jsf/facelets"
-	xmlns:in="http://richfaces.org/input">
-<f:view contentType="text/html" />
-
-<h:head>
-	<title>Richfaces InplaceInput</title>
-</h:head>
-
-<h:body>
-	<h:form id="form">
-		<in:inplaceInput id="input_default" value="#{dataBean.value}"/>
-	
-		<in:inplaceInput id="input_controls" showControls="true" value="#{dataBean.value}"/>
-	
-		<in:inplaceInput id="input_edit" state="edit" showControls="true" value="#{dataBean.value}"/>
-		
-		<h:panelGroup id="panel">
-			<!-- click here for blur -->
-		</h:panelGroup>
-	</h:form>
-</h:body>
-</html>



More information about the richfaces-svn-commits mailing list