Closed Bug 527981 Opened 15 years ago Closed 15 years ago

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

Categories

(Mozilla QA Graveyard :: Mozmill Tests, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: whimboo, Assigned: whimboo)

References

Details

(Keywords: regression, Whiteboard: [mozmill-test-failure])

Attachments

(1 file)

Attached patch Patch v1Splinter Review
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.
Attachment #411739 - Flags: review?(adesai)
Blocks: 527983
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
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.
Attachment #411739 - Flags: review?(adesai) → review-
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.
(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.
Attachment #411739 - Flags: review- → review?(adesai)
Comment on attachment 411739 [details] [diff] [review]
Patch v1

Downloaded and installed 1.3 and they're working fine now.
Attachment #411739 - Flags: review?(adesai) → review+
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
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Keywords: regression
Whiteboard: [mozmill-test-failure]
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.
Component: General → Mozmill Tests
Product: Firefox → Mozilla QA
QA Contact: general → mozmill-tests
Version: 3.5 Branch → unspecified
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: