Closed
Bug 1442157
Opened 8 years ago
Closed 8 years ago
Add automated test for " The removal of a bookmark can be undone"
Categories
(Firefox :: Bookmarks & History, enhancement, P3)
Firefox
Bookmarks & History
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: gechim, Assigned: gechim)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
Attachments
(1 file)
|
3.76 KB,
patch
|
standard8
:
review-
|
Details | Diff | Splinter Review |
TestRail link: https://testrail.stage.mozaws.net/index.php?/cases/view/4156
Steps:
1. Launch Firefox.
2. Enable the Bookmarks Toolbar.
3. Right-click a bookmark from the toolbar and select Delete.
4. Press ctrl or cmd + z to undo this action.
Expected results:
1. The browser is successfully launched.
2. The Bookmarks Toolbar is enabled and displayed below the Location Bar.
3. The bookmark is successfully deleted and no errors are thrown for this action.
4. The previously deleted bookmark is restored.
| Assignee | ||
Updated•8 years ago
|
Assignee: nobody → gechim
| Assignee | ||
Comment 1•8 years ago
|
||
Hi Mark,
I added a skip until https://bugzilla.mozilla.org/show_bug.cgi?id=381770 is fixed.
Attachment #8955053 -
Flags: review?(standard8)
| Assignee | ||
Updated•8 years ago
|
Comment 2•8 years ago
|
||
Comment on attachment 8955053 [details] [diff] [review]
mochitest_browser_bookmarks_undo_delete
Review of attachment 8955053 [details] [diff] [review]:
-----------------------------------------------------------------
I don't think we should land this until bug 381770 is fixed. I'm not sure why there was a test for it in the QA tests if it was known that it never worked...
The problem with skipped tests is that they only bitrot and are unlikely to get enabled unless the bug is fixed.
I think we should close this bug, and then just put a reference on bug 381770 that there's a prototype test case here.
Attachment #8955053 -
Flags: review?(standard8) → review-
Updated•8 years ago
|
Priority: -- → P3
Comment 3•8 years ago
|
||
I've added a reference on bug 381770.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•