> === modified file 'lib/lp/bugs/windmill/tests/test_bugs/test_filebug_dupe_finder.py'
> --- lib/lp/bugs/windmill/tests/test_bugs/test_filebug_dupe_finder.py 2009-06-25 00:40:31 +0000
> +++ lib/lp/bugs/windmill/tests/test_bugs/test_filebug_dupe_finder.py 2009-08-19 08:53:28 +0000
> @@ -8,7 +8,7 @@
> WAIT_PAGELOAD = u'20000'
> WAIT_ELEMENT_COMPLETE = u'20000'
> WAIT_CHECK_CHANGE = u'1000'
> -FILEBUG_URL = 'http://launchpad.dev:8085/firefox/+filebug'
> +FILEBUG_URL = 'http://bugs.launchpad.dev:8085/firefox/+filebug'
>
> FORM_OVERLAY = u'//div[@id="duplicate-overlay-bug-4"]/table'
> FORM_OVERLAY_CANCEL = (
> @@ -24,6 +24,9 @@
> FORM_VISIBLE = (
> u'element.className.search("yui-lazr-formoverlay-hidden") == -1')
>
> +BUG_INFO_HIDDEN = 'style.height|0px'
> +BUG_INFO_SHOWN_JS = 'element.style.height != "0px"'
> +
>
> def test_duplicate_finder():
> """Test the +filebug duplicate finder.
> @@ -47,10 +50,9 @@
> client.type(text=u'problem', id=u'field.title')
> client.click(xpath=u'//input[@id="field.actions.search"]')
> client.waits.forPageLoad(timeout=WAIT_PAGELOAD)
> -
> # The details div for the duplicate bug should not be shown.
> client.asserts.assertProperty(
> - id='details-for-bug-4', validator='style.display|none')
> + id='details-for-bug-4', validator=BUG_INFO_HIDDEN)
>
> # The expander for the duplicate should be collapsed.
> client.asserts.assertProperty(
> @@ -64,8 +66,8 @@
> client.waits.sleep(milliseconds=WAIT_CHECK_CHANGE)
> client.asserts.assertProperty(
> id='bug-details-expander-bug-4', validator='src|/@@/treeExpanded')
> - client.asserts.assertProperty(
> - id='details-for-bug-4', validator='style.display|block')
> + client.asserts.assertElemJS(
> + id='details-for-bug-4', js=BUG_INFO_SHOWN_JS)
>
> # Clicking the expander again will hide the details div and collapse
> # the expander.
> @@ -74,15 +76,15 @@
> client.asserts.assertProperty(
> id='bug-details-expander-bug-4', validator='src|/@@/treeCollapsed')
> client.asserts.assertProperty(
> - id='details-for-bug-4', validator='style.display|none')
> + id='details-for-bug-4', validator=BUG_INFO_HIDDEN)
>
> # Clicking it yet again will reopen it.
> client.click(id='bug-details-expander-bug-4')
> client.waits.sleep(milliseconds=WAIT_CHECK_CHANGE)
> client.asserts.assertProperty(
> id='bug-details-expander-bug-4', validator='src|/@@/treeExpanded')
> - client.asserts.assertProperty(
> - id='details-for-bug-4', validator='style.display|block')
> + client.asserts.assertElemJS(
> + id='details-for-bug-4', js='element.style.height != "0px"')
I guess this should be: js=BUG_INFO_SHOWN_JS
>
> # Clicking "No, I need to file a new bug" will collapse the
> # duplicate details and expander and will show the filebug form.
> @@ -91,7 +93,7 @@
> client.asserts.assertProperty(
> id='bug-details-expander-bug-4', validator='src|/@@/treeCollapsed')
> client.asserts.assertProperty(
> - id='details-for-bug-4', validator='style.display|none')
> + id='details-for-bug-4', validator=BUG_INFO_HIDDEN)
> client.asserts.assertProperty(
> id='bug_reporting_form', validator='style.display|block')
>
> @@ -103,8 +105,8 @@
> id='bug_reporting_form', validator='style.display|none')
> client.asserts.assertProperty(
> id='bug-details-expander-bug-4', validator='src|/@@/treeExpanded')
> - client.asserts.assertProperty(
> - id='details-for-bug-4', validator='style.display|block')
> + client.asserts.assertElemJS(
> + id='details-for-bug-4', js=BUG_INFO_SHOWN_JS)
>
> # Clicking on the "Yes, this is my bug button" will show a form
> # overlay, which will offer the user the option to subscribe to the
>
It's good to have these fixed :)
One small ateration and it's all good.
Gavin.
> === modified file 'lib/lp/ bugs/windmill/ tests/test_ bugs/test_ filebug_ dupe_finder. py' bugs/windmill/ tests/test_ bugs/test_ filebug_ dupe_finder. py 2009-06-25 00:40:31 +0000 bugs/windmill/ tests/test_ bugs/test_ filebug_ dupe_finder. py 2009-08-19 08:53:28 +0000 COMPLETE = u'20000' launchpad. dev:8085/ firefox/ +filebug' bugs.launchpad. dev:8085/ firefox/ +filebug' @id="duplicate- overlay-bug-4"]/table' className. search( "yui-lazr- formoverlay- hidden" ) == -1') style.height != "0px"' finder( ): type(text= u'problem' , id=u'field.title') click(xpath= u'//input[ @id="field. actions. search" ]') waits.forPageLo ad(timeout= WAIT_PAGELOAD) asserts. assertProperty( 'style. display| none') BUG_INFO_ HIDDEN) asserts. assertProperty( waits.sleep( milliseconds= WAIT_CHECK_ CHANGE) asserts. assertProperty( details- expander-bug-4', validator= 'src|/@ @/treeExpanded' ) asserts. assertProperty( 'style. display| block') asserts. assertElemJS( INFO_SHOWN_ JS) asserts. assertProperty( details- expander-bug-4', validator= 'src|/@ @/treeCollapsed ') asserts. assertProperty( 'style. display| none') BUG_INFO_ HIDDEN) click(id= 'bug-details- expander-bug-4') waits.sleep( milliseconds= WAIT_CHECK_ CHANGE) asserts. assertProperty( details- expander-bug-4', validator= 'src|/@ @/treeExpanded' ) asserts. assertProperty( 'style. display| block') asserts. assertElemJS( style.height != "0px"')
> --- lib/lp/
> +++ lib/lp/
> @@ -8,7 +8,7 @@
> WAIT_PAGELOAD = u'20000'
> WAIT_ELEMENT_
> WAIT_CHECK_CHANGE = u'1000'
> -FILEBUG_URL = 'http://
> +FILEBUG_URL = 'http://
>
> FORM_OVERLAY = u'//div[
> FORM_OVERLAY_CANCEL = (
> @@ -24,6 +24,9 @@
> FORM_VISIBLE = (
> u'element.
>
> +BUG_INFO_HIDDEN = 'style.height|0px'
> +BUG_INFO_SHOWN_JS = 'element.
> +
>
> def test_duplicate_
> """Test the +filebug duplicate finder.
> @@ -47,10 +50,9 @@
> client.
> client.
> client.
> -
> # The details div for the duplicate bug should not be shown.
> client.
> - id='details-for-bug-4', validator=
> + id='details-for-bug-4', validator=
>
> # The expander for the duplicate should be collapsed.
> client.
> @@ -64,8 +66,8 @@
> client.
> client.
> id='bug-
> - client.
> - id='details-for-bug-4', validator=
> + client.
> + id='details-for-bug-4', js=BUG_
>
> # Clicking the expander again will hide the details div and collapse
> # the expander.
> @@ -74,15 +76,15 @@
> client.
> id='bug-
> client.
> - id='details-for-bug-4', validator=
> + id='details-for-bug-4', validator=
>
> # Clicking it yet again will reopen it.
> client.
> client.
> client.
> id='bug-
> - client.
> - id='details-for-bug-4', validator=
> + client.
> + id='details-for-bug-4', js='element.
I guess this should be: js=BUG_ INFO_SHOWN_ JS
> asserts. assertProperty( details- expander-bug-4', validator= 'src|/@ @/treeCollapsed ') asserts. assertProperty( 'style. display| none') BUG_INFO_ HIDDEN) asserts. assertProperty( reporting_ form', validator= 'style. display| block') reporting_ form', validator= 'style. display| none') asserts. assertProperty( details- expander-bug-4', validator= 'src|/@ @/treeExpanded' ) asserts. assertProperty( 'style. display| block') asserts. assertElemJS( INFO_SHOWN_ JS)
> # Clicking "No, I need to file a new bug" will collapse the
> # duplicate details and expander and will show the filebug form.
> @@ -91,7 +93,7 @@
> client.
> id='bug-
> client.
> - id='details-for-bug-4', validator=
> + id='details-for-bug-4', validator=
> client.
> id='bug_
>
> @@ -103,8 +105,8 @@
> id='bug_
> client.
> id='bug-
> - client.
> - id='details-for-bug-4', validator=
> + client.
> + id='details-for-bug-4', js=BUG_
>
> # Clicking on the "Yes, this is my bug button" will show a form
> # overlay, which will offer the user the option to subscribe to the
>