Closed Bug 1642880 Opened 1 month ago Closed 1 month ago

Test names in mochitest.sh are incorrect

Categories

(Firefox :: New Tab Page, defect)

defect

Tracking

()

RESOLVED FIXED
Firefox 79
Tracking Status
firefox79 --- fixed

People

(Reporter: pbone, Assigned: pbone)

References

Details

Attachments

(1 file)

At least one test name in browser/components/newtab/mochitest.sh is incorrect, referring to a test that has been renamed. Now it appears this test is not covered by this script.

browser_new_tab_in_privileged_process_pref.js has been renamed to browser/base/content/test/tabs/browser_new_tab_in_privilegedabout_process_pref.js.

This reference to this test uses it's old name, it can't possibly be doing
anything useful now so I'm removing it.

@mardak says this file can probably be removed. See the patch & review comments.

I can do that or feel free to take the bug from me.

Attachment #9153729 - Attachment description: Bug 1642880 - Remove an old reference to browser_new_tab_in_privileged_process_pref.js r=thecount → Bug 1642880 - Remove an old reference to browser_new_tab_in_privileged_process_pref.js r=Mardak
Depends on: 1619405
Attachment #9153729 - Attachment description: Bug 1642880 - Remove an old reference to browser_new_tab_in_privileged_process_pref.js r=Mardak → Bug 1642880 - Remove unused mochitest.sh with old reference to browser_new_tab_in_privileged_process_pref.js r?andreio
Pushed by elee@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/dae19c12350d
Remove unused mochitest.sh with old reference to browser_new_tab_in_privileged_process_pref.js r=andreio
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 79
You need to log in before you can comment on or make changes to this bug.