Closed
Bug 1107178
Opened 10 years ago
Closed 5 years ago
add test coverage for one-off buttons
Categories
(Firefox :: Search, defect, P4)
Firefox
Search
Tracking
()
RESOLVED
INCOMPLETE
People
(Reporter: mossop, Unassigned)
References
Details
(Whiteboard: [ui][fxsearch])
No description provided.
Updated•10 years ago
|
Flags: qe-verify-
Flags: firefox-backlog+
Comment 1•10 years ago
|
||
Let's define this a bit more completely - what tests do we most need? What's the minimal set that will offer the most regression-protection for common areas that we're currently changing?
Flags: needinfo?(florian)
Flags: needinfo?(dtownsend)
Comment 2•10 years ago
|
||
(In reply to :Gavin Sharp [email: gavin@gavinsharp.com] from comment #1)
> Let's define this a bit more completely - what tests do we most need? What's
> the minimal set that will offer the most regression-protection for common
> areas that we're currently changing?
Dave already added a good set of tests about the search panel showing/hiding. The tests we need the most at this point are tests for the one-off buttons, and more generally the new pieces of UI we added with bug 1088660 (that includes for example the headers shown in the search panel).
Flags: needinfo?(florian)
Updated•10 years ago
|
Points: --- → 8
Summary: New search UI needs tests → add test coverage for one-off buttons and new search UI headers
Reporter | ||
Comment 3•10 years ago
|
||
I filed bug 1120641 and bug 1120642 to cover other pieces.
Points: 8 → 5
Flags: needinfo?(dtownsend)
Summary: add test coverage for one-off buttons and new search UI headers → add test coverage for one-off buttons
Reporter | ||
Updated•10 years ago
|
Priority: -- → P4
Whiteboard: [fxsearch][searchui]
Updated•10 years ago
|
Rank: 45
Whiteboard: [fxsearch][searchui] → [ui][fxsearch]
Comment 4•10 years ago
|
||
Tests here should not only ensure that the one-off buttons work, but also verify that the layout is correct, so that in the figure we could catch immediately regressions like bug 1170500. We can probably do this by checking the computed height of the description element containing the one-off buttons.
Updated•5 years ago
|
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INCOMPLETE
You need to log in
before you can comment on or make changes to this bug.
Description
•