Closed Bug 543796 Opened 13 years ago Closed 13 years ago

Clean up test_role_table_cells.html

Categories

(Core :: Disability Access APIs, defect)

Product:
Core
Component:
Disability Access APIs
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
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: nobody → marco.zehe
Status: NEW → ASSIGNED

        Attachment #424842 -
        Flags: review?(surkov.alexander)

    
    

    
  
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+

    
    

    
  
(In reply to comment #2)

> So I think the nicer cleanin up is the test removing :)

However it's probably ok to save it.

    
    

    
  
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.

    
    

    
  
Cool thanks!

    
    

    
  
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

    
    

    
  
Resolving WONTFIX since this will be fixed in a different way as part of bug 423409.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX

          You need to log in
          before you can comment on or make changes to this bug.