Open
Bug 1996125
Opened 6 months ago
Updated 5 months ago
Refactor test_moz_input_elems_in_form.html
Categories
(Toolkit :: UI Widgets, task, P3)
Toolkit
UI Widgets
Tracking
()
ASSIGNED
People
(Reporter: tgiles, Assigned: dansinger)
References
Details
(Whiteboard: [recomp])
There's a lot of duplicated logic in this test that should be refactored. For example:
- The template functions
- The only difference is one has the form elements within the
<form>tags, and the other has the input elements outside of the form with these elements using theformattribute
- The only difference is one has the form elements within the
- The various
assert*event handlers can most likely be pulled out of the tasks and used as global functions - The
*with_form_attributetasks since they're copies of the submission and reset tasks respectively - Most of the
isasserts can probably be pulled into various functions so that we can reduce the number of lines
Additionally, we should have some dynamic way of determining valid moz-* input elements since this test will need to be modified as soon as a new moz input element is created.
Updated•6 months ago
|
| Assignee | ||
Comment 1•5 months ago
|
||
Hi there,
I'd like to take on this patch. I'll let you know if I have any questions.
Thanks!
Flags: needinfo?(tgiles)
| Reporter | ||
Updated•5 months ago
|
Assignee: nobody → dansinger
Status: NEW → ASSIGNED
Flags: needinfo?(tgiles)
You need to log in
before you can comment on or make changes to this bug.
Description
•