Last modified: 2014-06-28 20:58:52 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T57751, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 55751 - ProveIt gadget test causes page.feature etc. to fail in Chrome
ProveIt gadget test causes page.feature etc. to fail in Chrome
Status: REOPENED
Product: Wikimedia
Classification: Unclassified
Quality Assurance (Other open bugs)
unspecified
All All
: Lowest enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-10-15 19:58 UTC by Chris McMahon
Modified: 2014-06-28 20:58 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
ProveIt floats over Save (31.22 KB, image/png)
2013-10-15 19:58 UTC, Chris McMahon
Details
ProveIt really floats over Save (34.92 KB, image/png)
2013-10-18 14:37 UTC, Chris McMahon
Details

Description Chris McMahon 2013-10-15 19:58:57 UTC
Created attachment 13494 [details]
ProveIt floats over Save

The ProveIt tool floats on top of the wikitext editor controls and makes the test fail like so attempting to click 'Save page': 

$ bundle exec cucumber features/page.feature:38
Using the default profile...
....F------

(::) failed steps (::)

unknown error: Element is not clickable at point (259, 764). Other element would receive the click: <div id="tabs" class="ui-tabs ui-widget ui-widget-content ui-corner-all">...</div>
  (Session info: chrome=30.0.1599.66)
  (Driver info: chromedriver=2.3,platform=Linux 3.2.0-54-generic x86_64) (Selenium::WebDriver::Error::UnknownError)
./features/step_definitions/page_steps.rb:9:in `/^I click Save page button$/'
./features/step_definitions/page_steps.rb:51:in `/^I create the page to be moved$/'
features/page.feature:38:in `And I create the page to be moved'

Failing Scenarios:
cucumber features/page.feature:35 # Scenario: Move existing page dialog

1 scenario (1 failed)
10 steps (1 failed, 6 skipped, 3 passed)
0m36.042s
Comment 1 Tomislav Plavcic 2013-10-17 19:21:10 UTC
What is the best possible solution for this?
Is it to implement something like Page Down keypress before clicking the save button (user would probably scroll up-down to see where are the buttons) or something else?
Comment 2 Željko Filipin 2013-10-18 05:37:43 UTC
We could enable the gadget in a "before" hook before each ProveIt scenario and disable it an "after" hook after each scenario, so the gadget is active only when the tests for it are running.
Comment 3 Matthew Flaschen 2013-10-18 05:53:19 UTC
That screenshot doesn't show ProveIt blocking the save button.

It could be a bug in the test, or perhaps it needs a moment to position ProveIt before you do the save.
Comment 4 Chris McMahon 2013-10-18 14:37:55 UTC
Created attachment 13521 [details]
ProveIt really floats over Save

This is arguably a bug in ChromeDriver. This part of the WebDriver W3C standard is being discussed as we speak, actually.  Firefox is liberal about what can be clicked, Chrome is not.
Comment 5 Chris McMahon 2013-10-18 14:39:33 UTC
(In reply to comment #1)
> What is the best possible solution for this?
> Is it to implement something like Page Down keypress before clicking the save
> button (user would probably scroll up-down to see where are the buttons) or
> something else?

I think I would like to just run the test for Firefox and not for Chrome.  (And see what happens with Internet Explorer)
Comment 6 Gerrit Notification Bot 2013-10-22 13:46:23 UTC
Change 91180 had a related patch set uploaded by Cmcmahon:
don't run the ProveIt test for Chrome because of Bug 55751

https://gerrit.wikimedia.org/r/91180
Comment 7 Gerrit Notification Bot 2013-10-22 14:02:17 UTC
Change 91180 abandoned by Cmcmahon:
don't run the ProveIt test for Chrome because of Bug 55751

Reason:
mistake

https://gerrit.wikimedia.org/r/91180
Comment 8 Gerrit Notification Bot 2013-10-22 14:05:54 UTC
Change 91182 had a related patch set uploaded by Cmcmahon:
ProveIt gadget causes Page test to fail in Chrome Bug 55751

https://gerrit.wikimedia.org/r/91182
Comment 9 Gerrit Notification Bot 2013-10-22 14:52:25 UTC
Change 91182 merged by Cmcmahon:
ProveIt gadget causes Page test to fail in Chrome Bug 55751

https://gerrit.wikimedia.org/r/91182
Comment 10 Željko Filipin 2013-10-25 20:28:11 UTC
Can this be closed now?
Comment 11 Matthew Flaschen 2013-10-26 00:36:37 UTC
It hasn't been fixed, right?

My understanding was just that you stopped running the tests for now.  So how about just marking it lowest priority?
Comment 12 Željko Filipin 2013-12-02 15:33:24 UTC
Closing since this seems to be fixed. Please reopen if the problem is not fixed.
Comment 13 Matthew Flaschen 2013-12-02 22:10:17 UTC
I checked with Chris.  They are still not running, and this is not fixed.

Stopping them from running was just a workaround.
Comment 14 Željko Filipin 2014-06-28 20:58:52 UTC
Chris, is this resolved now?

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links