split browser_bookmarksProperties.js into single tests and add further tests

NEW
Unassigned

Status

()

P3
normal
4 years ago
a year ago

People

(Reporter: mak, Unassigned)

Tracking

unspecified
Points:
5
Dependency tree / graph
Bug Flags:
firefox-backlog +
qe-verify -

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

4 years ago
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+
(Reporter)

Comment 1

4 years ago
for a list of additional tests to add, see the regressions caused by bug 951651
Thanks for adding this Marco, re Bookmark edit regression in bug 1163341.
(Reporter)

Updated

3 years ago
Depends on: 1167915
(Reporter)

Updated

3 years ago
Blocks: 1178709
(Reporter)

Updated

3 years ago
Priority: -- → P2
(Reporter)

Updated

3 years ago
Summary: refactor browser_bookmarksProperties.js and add more tests to it → split browser_bookmarksProperties.js into single tests
(Reporter)

Updated

3 years ago
Flags: in-testsuite?
(Reporter)

Updated

a year ago
Priority: P2 → P3
(Reporter)

Comment 3

a year ago
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
(Reporter)

Comment 4

a year ago
not a top priority atm, the test seems to work fine for now.
No longer blocks: 979280
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.