Closed Bug 604604 Opened 14 years ago Closed 14 years ago

Update Mozmill tests to match the new interface of waitFor and assert

Categories

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Product:
Mozilla QA Graveyard
Component:
Mozmill Tests
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED INVALID

People

(Reporter: whimboo, Assigned: whimboo)

References

Details

Attachments

(1 file)

Patch v1
10.15 KB, patch
gmealer
: review-
Details | Diff | Splinter Review 
With the fix on bug 604545 we have to update some of our tests to match the new API. Patch upcoming.
Attached patch Patch v1 — — Splinter Review 

        Attachment #483429 -
        Flags: review?(gmealer)

    
    

    
  
This patch should not be checked-in before Mozmill 1.5.1 is in use.

    
    

    
  
Comment on attachment 483429 [details] [diff] [review]
Patch v1

The patch would look fine, but you had me revert all these calls to waitForEval() (Bug 604743).  My patches for that make this patch invalid.

        Attachment #483429 -
        Flags: review?(gmealer) → review-

    
    

    
  
This bug is useless now with bug 604743 implemented and fixed. Sorry for taking up your time on that. Marking as invalid.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → INVALID
Whiteboard: [needs-mozmill-1.5.1]

    
    

    
  
Move of Mozmill Test related project bugs to newly created components. You can
filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.
Product: Testing → Mozilla QA
Version: Trunk → unspecified

    
  
Product: Mozilla QA → Mozilla QA Graveyard

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: