Closed Bug 1296800 Opened 9 years ago Closed 9 years ago

Move non-proxy, non-doorhanger tests from test_prompt.html to its own file

Categories

(Toolkit :: Password Manager, defect, P3)

Product:
Toolkit
Component:
Password Manager
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla51
Tracking Flags:
Tracking Status
firefox51 --- fixed

People

(Reporter: MattN, Assigned: MattN)

References

Details

Attachments

(3 files)

Bug 1296800 - Output test numbers in test_prompt.html.
58 bytes, text/x-review-board-request
Dolske
: review+
Details
Bug 1296800 - Use separate setup and test tasks in test_prompt.html.
58 bytes, text/x-review-board-request
Dolske
: review+
Details
Bug 1296800 - test_prompt.html: Move promptAuth tests to their own test file.
58 bytes, text/x-review-board-request
Dolske
: review+
Details
In preparation for making the tests work in e10s (bug 1266618), move the simplest test_prompt.html tests to their own file.
Comment on attachment 8783164 [details] Bug 1296800 - Output test numbers in test_prompt.html. https://reviewboard.mozilla.org/r/73104/#review71804 In one sense the numbers are not useful, since the old testNum counter was removed. I think in other tests we converted it to an actual description of the test. OTOH, there's a lot of individual tests here, so it makes it easy to jump to the right spot. Would be nice if there was something that could just, I dunno, output a line number?
Attachment #8783164 - Flags: review?(dolske) → review+
Comment on attachment 8783165 [details] Bug 1296800 - Use separate setup and test tasks in test_prompt.html. https://reviewboard.mozilla.org/r/73106/#review71806
Attachment #8783165 - Flags: review?(dolske) → review+
Comment on attachment 8783166 [details] Bug 1296800 - test_prompt.html: Move promptAuth tests to their own test file. https://reviewboard.mozilla.org/r/73108/#review71816 ::: toolkit/components/passwordmgr/test/mochitest/test_prompt_promptAuth.html:1 (Diff revision 1) > +<!DOCTYPE HTML> Oops, please |hg copy| this file, looks like you didn't. (Or mozreview is confused, it's showing it as a new file with complete additions, not diffs from test_prompt).
Attachment #8783166 - Flags: review?(dolske) → review+
Comment on attachment 8783164 [details] Bug 1296800 - Output test numbers in test_prompt.html. https://reviewboard.mozilla.org/r/73104/#review71804 Yeah, the remaining ones should be switched to add_task too which then outputs the test name. This was just an easy first step to help more easily know where the test fails.
Comment on attachment 8783166 [details] Bug 1296800 - test_prompt.html: Move promptAuth tests to their own test file. https://reviewboard.mozilla.org/r/73108/#review71816 > Oops, please |hg copy| this file, looks like you didn't. (Or mozreview is confused, it's showing it as a new file with complete additions, not diffs from test_prompt). Oops, I guess this got messed up in a rebase?
Pushed by mozilla@noorenberghe.ca: https://hg.mozilla.org/integration/autoland/rev/2c7cfad1bc2b Output test numbers in test_prompt.html. r=Dolske https://hg.mozilla.org/integration/autoland/rev/62ada473cc52 Use separate setup and test tasks in test_prompt.html. r=Dolske https://hg.mozilla.org/integration/autoland/rev/eb7d22f1a38d test_prompt.html: Move promptAuth tests to their own test file. r=Dolske
https://hg.mozilla.org/mozilla-central/rev/2c7cfad1bc2b https://hg.mozilla.org/mozilla-central/rev/62ada473cc52 https://hg.mozilla.org/mozilla-central/rev/eb7d22f1a38d
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox51: --- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Blocks: 1298193
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: