555996 - [mozmill] Implement test for checking multiple search engine providers

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

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

This bug will cover the work which has to be done to automate the following litmus test:

https://litmus.mozilla.org/show_test.cgi?id=10242

Comment 1

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

Attachment: Patch v1 (default)

Comment 2

[u279076]

I'm not sure if this is a problem with the patch or not -- when I try to apply it, I get the following output:

applying search_multiple
patching file shared-modules/testSearchAPI.js
Hunk #7 FAILED at 761
1 out of 7 hunks FAILED -- saving rejects to file shared-modules/testSearchAPI.js.rej
patch failed, unable to continue (try -v)
patch failed, rejects left in working dir
errors during apply, please fix and refresh search_multiple

Applying other patches do not cause this error.

Comment 3

[u279076]

Here is a more detailed output:

applying search_multiple
patching file firefox/testSearch/testSearchSuggestions.js
patching file shared-modules/testSearchAPI.js
Hunk #5 succeeded at 583 (offset 23 lines).
Hunk #6 succeeded at 758 (offset 23 lines).
Hunk #7 FAILED at 761
1 out of 7 hunks FAILED -- saving rejects to file shared-modules/testSearchAPI.js.rej
patch failed to apply
adding firefox/testSearch/testSearchSuggestions.js
firefox/testSearch/testSearchSuggestions.js
shared-modules/testSearchAPI.js
patch failed, rejects left in working dir
errors during apply, please fix and refresh search_multiple

Comment 4

[u279076]

Attachment: Failed Chunk

Here is the chunk of the patch which is failing to apply.

Comment 5

[u279076]

Comment on attachment 436882 [details] [diff] [review]
Patch v1 (default)

r- because patch fails to apply.  r+ if the problems in the previous comment is corrected.

Comment 6

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

Attachment: Patch v1.1 (default)

Sorry, looks like a part has found its way into another patch. Updated the test for your testing.

Comment 7

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

Landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/8533ac2d7595
http://hg.mozilla.org/qa/mozmill-tests/rev/ff16ec5ab09e

Comment 8

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

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.