Closed Bug 606410 Opened 14 years ago Closed 14 years ago

add tests for calling the callback app and the arguments passed to it

Categories

(Toolkit :: Application Update, defect)

Product:
Toolkit
Component:
Application Update
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla2.0b8

People

(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)

Details

Attachments

(1 file, 2 obsolete files)

patch as checked in
32.55 KB, patch
robert.strong.bugs
: review+
Details | Diff | Splinter Review 
patch coming up
Attached patch patch in progress - sent to try (obsolete) — — Splinter Review 
Assignee: nobody → robert.bugzilla
Status: NEW → ASSIGNED

    
    

    
  

      
      
      
      

        Attached patch
          
          
        patch rev1 (obsolete)
        — 
          — Splinter Review
      

    


  
This tests that the callback app is launched and the command line arguments sent to it. I filed bug 606415 for testing the post update app since it is more complicated and I wanted to get this test in the tree sooner rather than later.

        Attachment #485234 -
        Attachment is obsolete: true

        Attachment #485352 -
        Flags: review?(dtownsend)

    
    

    
  
Comment on attachment 485352 [details] [diff] [review]
patch rev1

Dave, I've renamed TestAUSAfterApply to TestAUSHelper because I want to use it for other things than just the callback app.

        Attachment #485352 -
        Flags: review?(dtownsend) → review+

    
    

    
  


  
I saw an error on win32 debug where the log file contents hadn't been updated when the log files were checked so I made it so the test required the contents to be correct before continuing which is ok since the test will fail due to timing out when the contents are incorrect.

        Attachment #485352 -
        Attachment is obsolete: true

        Attachment #488724 -
        Flags: review+

    
    

    
  
Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/465a7cdf4ab8
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite+
Flags: in-litmus-
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: