On Feb 2, 2009, at 12:56 PM, mflorea (SVN) wrote:

> Author: mflorea
> Date: 2009-02-02 12:56:41 +0100 (Mon, 02 Feb 2009)
> New Revision: 16027
>
> Modified:
>   enterprise/trunk/distribution-test/wysiwyg-tests/src/test/it/com/ 
> xpn/xwiki/it/selenium/StandardFeaturesTest.java
> Log:
> XWIKI-3053: When a HR is inserted at the beginning of a paragraph an  
> extra empty paragraph is generated before that HR
>
>
> Modified: enterprise/trunk/distribution-test/wysiwyg-tests/src/test/ 
> it/com/xpn/xwiki/it/selenium/StandardFeaturesTest.java
> ===================================================================
> --- enterprise/trunk/distribution-test/wysiwyg-tests/src/test/it/com/ 
> xpn/xwiki/it/selenium/StandardFeaturesTest.java       2009-02-02 11:52:59  
> UTC (rev 16026)
> +++ enterprise/trunk/distribution-test/wysiwyg-tests/src/test/it/com/ 
> xpn/xwiki/it/selenium/StandardFeaturesTest.java       2009-02-02 11:56:41  
> UTC (rev 16027)
> @@ -613,4 +613,35 @@
>         switchToWikiEditor();
>         assertEquals("before\n\n|=Space|=Page\n|Main|WebHome\n 
> \nafter", getFieldValue("content"));
>     }
> +
> +    /**
> +     * @see XWIKI-3053: When a HR is inserted at the beginning of a  
> paragraph an extra empty paragraph is generated
> +     *      before that HR
> +     */
> +    public void testInsertHRInsideParagraph()
> +    {
> +        typeText("xy");
> +        applyStyleParagraph();
> +
> +        // Insert HR at the end of the paragraph.
> +        clickHRButton();
> +
> +        // More the caret between x and y.
> +        runScript("var range = XWE.selection.getRangeAt(0);\n" +  
> "range.setStart(XWE.body.firstChild.firstChild, 1);\n"
> +            + "range.collapse(true);");

Is there a way move to this to a DSL method for moving the caret?

> +
> +        // Insert HR in the middle of the paragraph.
> +        clickHRButton();
> +
> +        // Move the caret before x.
> +        runScript("var range = XWE.selection.getRangeAt(0);\n" +  
> "range.setStart(XWE.body.firstChild.firstChild, 0);\n"
> +            + "range.collapse(true);");
> +
> +        // Insert HR at the beginning of the paragraph.
> +        clickHRButton();
> +
> +        // We have to assert the XHTML because the arrow keys don't  
> move the caret so we can't test if the user can edit
> +        // the generated empty paragraphs. The fact that they  
> contain a BR proves this.
> +        assertXHTML("<p><br class=\"emptyLine\"></p><hr><p>x</ 
> p><hr><p>y</p><hr><p><br class=\"emptyLine\"></p>");
> +    }
> }

Thanks
-Vincent
http://xwiki.com
http://xwiki.org
http://massol.net






_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs

Reply via email to