Open Bug 1160326 Opened 11 years ago Updated 3 years ago

split browser_bookmarksProperties.js into single tests and add further tests

Categories

(Firefox :: Bookmarks & History, defect, P3)

Product:
Firefox
Component:
Bookmarks & History
Type:
defect
Points:
5

Tracking

()

People

(Reporter: mak, Unassigned)

References

(
URL
)

Details

We need more tests (title editing, bookmark all tabs, insert into new folder...) but the current test is verbose is complex to maintain. We can likely refactor it using add_task and avoiding multiple parts per test.
Flags: qe-verify-
Flags: in-testsuite?
Flags: firefox-backlog+
for a list of additional tests to add, see the regressions caused by bug 951651
URL: https://bugzilla.mozilla.org/show_bug...
Thanks for adding this Marco, re Bookmark edit regression in bug 1163341.
Depends on: 1167915
Blocks: 1178709
Priority: -- → P2
Summary: refactor browser_bookmarksProperties.js and add more tests to it → split browser_bookmarksProperties.js into single tests
Flags: in-testsuite?
Priority: P2 → P3
Currently it contains 3 tests
Priority: P3 → P2
Summary: split browser_bookmarksProperties.js into single tests → split browser_bookmarksProperties.js into single tests and add further tests
not a top priority atm, the test seems to work fine for now.
No longer blocks: PlacesAsyncTransact
Priority: P2 → P3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.