527981 - [mozmill] Fix broken tests under testLayout (Amazon, Yahoo)

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

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

Attachment: Patch v1

Two tests under testLayout (Amazon and Yahoo) are failing because the layout has been changed a bit. Further Yahoo is redirecting to a local version too so we have to make sure to load the correct page before starting the test.

Comment 1

[Aakash Desai [:aakashd]]

Comment on attachment 411739 [details] [diff] [review]
Patch v1

The changes made for testCheckAmazon run fine on OSX and XP, testCheckYahoo fails on a check of assertJS("subject.indexOf('" + url + "')....

I get the following failure: http://pastebin.mozilla.org/683247 through IDE.

Comment 2

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

Aakash, have you installed Mozmill 1.3b1? It's a new feature in that version so we can use the subject there too. Looks like you have an older version.

Comment 3

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

(In reply to comment #2)
> Aakash, have you installed Mozmill 1.3b1? It's a new feature in that version so
> we can use the subject there too. Looks like you have an older version.

Comment 4

[Aakash Desai [:aakashd]]

Comment on attachment 411739 [details] [diff] [review]
Patch v1

Downloaded and installed 1.3 and they're working fine now.

Comment 5

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

Sounds good. So you didn't upgraded the pyPI package? Would be great if we could talk about that beside this bug.

Landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/ab95dbad9adf
http://hg.mozilla.org/qa/mozmill-tests/rev/05e5c3724fc3

Comment 6

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

Mass move of Mozmill Test related project bugs to newly created components. You can filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.