Closed
Bug 543796
Opened 14 years ago
Closed 14 years ago
Clean up test_role_table_cells.html
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: MarcoZ, Assigned: MarcoZ)
References
(Blocks 1 open bug)
Details
Attachments
(1 obsolete file)
This has some bogus references and a wrong comment about what it is actually testing.
Assignee | ||
Comment 1•14 years ago
|
||
Assignee: nobody → marco.zehe
Status: NEW → ASSIGNED
Attachment #424842 -
Flags: review?(surkov.alexander)
Comment 2•14 years ago
|
||
Comment on attachment 424842 [details] [diff] [review] Clean up the file for its new sole purpose of testing the roles. r=me but in general we don't need this test at all since it is covered by table structure tests (test_table_struct.html and test_table_struct_ariagrid.html). So I think the nicer cleanin up is the test removing :)
Attachment #424842 -
Flags: review?(surkov.alexander) → review+
Comment 3•14 years ago
|
||
(In reply to comment #2) > So I think the nicer cleanin up is the test removing :) However it's probably ok to save it.
Comment 4•14 years ago
|
||
Again changing opinion :) I think it's worth to remove it since it was intendent originally to test name calculation. I'm working on bug 423409, let me fix this as part of that bug.
Assignee | ||
Comment 5•14 years ago
|
||
Cool thanks!
Assignee | ||
Comment 6•14 years ago
|
||
Comment on attachment 424842 [details] [diff] [review] Clean up the file for its new sole purpose of testing the roles. Obsoleting because this will be fixed in a different way.
Attachment #424842 -
Attachment is obsolete: true
Assignee | ||
Comment 7•14 years ago
|
||
Resolving WONTFIX since this will be fixed in a different way as part of bug 423409.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•