807931 - Test failure "Active tab url should equal the page url - 'http://localhost:43336/mozilla_governance.html' should equal 'http://localhost:43336/mozilla_grants.html'" in testAwesomeBar/testSwitchToTab.js

Status: RESOLVED WORKSFORME

(Mozilla QA Graveyard :: Mozmill Tests, defect, P2)

Description

[Andreea Matei [:AndreeaMatei]]

This is happening rarely on OS X 10.6.8 only, with Aurora build.
I assume it's not showing the switch tab in the autocomplete pop-up and has no item to click on in order to change the page.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Well, we call expect.equal() right after controller.click() on the richlist item. This will fail if the expected tab has been selected with a delay. So we have to make use of waitFor() instead.

Comment 2

[Daniela Petrovici]

Attachment: patch v1

replaced expect.equal with expect.waitFor for aurora branch

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Comment on attachment 678658 [details] [diff] [review]
patch v1

Review of attachment 678658 [details] [diff] [review]:
-----------------------------------------------------------------

As mentioned on IRC please create the patch against the default branch. It's something we want to fix across branches and backports will happen once it has been fixed on default.

Comment 4

[Daniela Petrovici]

Attachment: patch v1.1

replaced expect.equal with expect.waitFor for Default

Comment 5

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Comment on attachment 678699 [details] [diff] [review]
patch v1.1

Review of attachment 678699 [details] [diff] [review]:
-----------------------------------------------------------------

It would appear that the patch from bug 792394 caused this patch to conflict. Please pull/update and provide a new patch. Also, please add a space before (and between) the reviewers in the commit message. See https://developer.mozilla.org/en-US/docs/Mozmill_Tests#Commit_message

Comment 6

[Daniela Petrovici]

Attachment: patch v1.2

changed expect.equal with expect.waitFor and used the latest version of testSwitchToTab.js file

Comment 7

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Comment on attachment 679120 [details] [diff] [review]
patch v1.2

Review of attachment 679120 [details] [diff] [review]:
-----------------------------------------------------------------

Landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/4b964ab000ac (default)

Once we can confirm that no regressions are introduced we can transplant this to mozilla-aurora.

Comment 8

[Daniela Petrovici]

Attachment: aurora patch

replaced expect.equal with expect.waitFor in aurora branch

Comment 9

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Comment on attachment 679651 [details] [diff] [review]
aurora patch

Review of attachment 679651 [details] [diff] [review]:
-----------------------------------------------------------------

http://hg.mozilla.org/qa/mozmill-tests/rev/297214f95ddd (aurora)

Comment 10

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Looks like we are done here.

Comment 11

[Andreea Matei [:AndreeaMatei]]

This has failed again on Aurora branch, fr locale, OS X 10.6.8, once today, since Friday:
http://mozmill-ci.blargon7.com/#/functional/report/190d57cf64e3c5c3996c0c1059f72fa9

Comment 12

[Daniela Petrovici]

No more failures reproduced since 11/15/2012

Comment 13

[Alex Lakatos[:AlexLakatos]]

Failed again on the 18th
http://mozmill-ci.blargon7.com/#/functional/failure?branch=All&platform=All&from=2012-11-18&to=&test=%2FtestAwesomeBar%2FtestSwitchToTab.js&func=testSwitchToTab.js%3A%3AtestSwitchToTab

Comment 14

[Daniela Petrovici]

No new failures since 11/18 and unable to reproduce locally or on the VM on which the failure reproduced with Aurora builds (both 19.0a2 and 18.0a2)

Comment 15

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Infrequent crashes here. Whenever it happens it should be checked right away. I wouldn't tend to close this bug yet, given that after 3 or 4 days crashes can still happen again.

Comment 16

[Daniela Petrovici]

I will try to reproduce it again locally

Comment 17

[Daniela Petrovici]

I was unable to reproduce locally and the issue did not happen for 2 weeks.

Comment 18

[Henrik Skupin [:whimboo][⌚️UTC+2]]

It didn't happen in the last month even. So I think we can close as WFM. Reopen if it happens again.

Comment 19

[Andrei Eftimie]

Happened again on 2012-12-29 on Mac OS X 10.8.2 (x86_64) on 18.0

http://mozmill-ci.blargon7.com/#/functional/report/87220f551d53bd61286b3d93d921ab40

Comment 20

[Daniela Petrovici]

This issue did not reproduce for more than a month (last reported in 12/29).