1701250 - Move browser_search_discovery.js and browser_addKeywordSearch.js to browser/components/search

Status: RESOLVED FIXED

(Firefox :: Search, task, P3)

Description

[Mark Banner (:standard8)]

I just noticed that in browser/base/content/test/general/, we have browser_search_discovery.js and browser_addKeywordSearch.js.

We should move those across to browser/components/search as I think that's more appropriate (especially for the search discovery).

Comment 1

[Mark Banner (:standard8)]

I'm happy to mentor this. The basic work that needs to be done here:

• hg mv the two test files from browser/base/content/test/general/ to browser/components/search/test/browser/.
• hg mv the support file as well browser/base/content/test/general/discovery.html
• Move the references for the test and support files from browser/base/content/test/general/browser.ini to browser/components/search/test/browser/browser.ini (to equivalent places, please keep alphabetical ordering)
• Update the file discovery.html reference in browser_search_discovery.js.
• In browser_addKeywordSearch.js change the TEST_URL reference to http://example.org/browser/browser/components/search/test/browser/test.html
• Do a build (./mach build)
• Make sure the tests pass in the new location (./mach mochitest browser/components/search/test/browser/)

Comment 2

[u682976]

Can I be assigned this task?

Comment 3

[Mark Banner (:standard8)]

Hi Dawit, sure. Issues will also be automatically assigned when the first patch is posted, but I'll set you as assignee anyway for this. Please let us know if you happen to stop working on it so we can reassign.

Comment 4

[u682976]

What passes as a success when I run the test command (./mach mochitest browser/components/search/test/browser/)?
I finished all the tasks and when I run that command, it opens up Firefox with 4 tabs and it crashes.

Comment 5

[Mark Banner (:standard8)]

(In reply to Dawit from comment #4)

What passes as a success when I run the test command (./mach mochitest browser/components/search/test/browser/)?
I finished all the tasks and when I run that command, it opens up Firefox with 4 tabs and it crashes.

Normally there should be a success/fail indication at the end of the run. Maybe try it without your changes (commit the changes, then update to the current tip and rebuild), and see if that helps.

If you're still hitting issues, then the #introduction channel on Matrix is a good place to ask for help.

Comment 6

[u682976]

Attachment: Bug 1701250 - Move some test files to their respective locations. r?Standard8

Update references to the files in all occurences

Comment 7

[u682976]

I just submitted a patch to Phabricator (https://phabricator.services.mozilla.com/D111944)

Comment 8

[Mark Banner (:standard8)]

Thank you - the patch is looking good, though there's an extra added file testing/web-platform/tests/websockets/Create-verify-url-set-non-default-port.any.js-53d60826 - please can you remove that? It shouldn't be necessary.

I'd normally comment on the patch itself, but as it is in draft mode, you wouldn't get notified of the comments. For the reviewer, you can request review from me, e.g. add "r=Standard8" in the commit message and re-submit (After also having removed the extra file).

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e82b7fa25ac9
Move some test files to their respective locations. r=Standard8

Comment 10

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/e82b7fa25ac9