Closed Bug 926755 Opened 11 years ago Closed 6 years ago

[meta] Screenshot comparison tool for integration tests

Categories

(Testing Graveyard :: JSMarionette, defect)

Product:
Testing Graveyard
Component:
JSMarionette
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED INCOMPLETE

People

(Reporter: evanxd, Assigned: jlee)

References

Details

Attachments

(1 obsolete file) 

Taking screenshots during integration tests to confirm visual changes.
Summary: [marionette-debug-server] Screenshot comparison tool for integration tests → [meta] Screenshot comparison tool for integration tests
Depends on: 916652
Depends on: 926801
No longer blocks: 926802
Depends on: 926802
No longer blocks: 926800
Depends on: 926800
Depends on: 926819
Assignee: nobody → jlee
For example, we could show the screenshots in current and previous version, then we could compare the images in these two versions.

Prototype screenshot:
1. origin: https://docs.google.com/file/d/0B83JBEDsAKxheE9wUjliaDh2Tms/edit?usp=drive_web
2. diff: https://docs.google.com/file/d/0B83JBEDsAKxhUnpaQjExam0tWk0/edit?usp=drive_web
The user story:

After we commit a patch to Gaia, we could compare the screenshot images in current commit and previous commit.

Then we could see the report to know the patch works well or not.
Attached file pull-request.html (obsolete) — 

    
    

    
  
Comment on attachment 817671 [details]
pull-request.html

><html>
>    <head>
>     	<meta http-equiv="Refresh"
>    	content="2; url=https://github.com/evanxd/marionette-debug-server/pull/1/commits" />
>    </head>
>	<body>
>	Redirect to pull request #1
>	</body>
></html>

        Attachment #817671 -
        Attachment mime type: text/plain → text/html

    
  

        Attachment #817671 -
        Flags: review?(evanxd)

    
  

        Attachment #817671 -
        Flags: feedback?(yurenju.mozilla)

    
  

        Attachment #817671 -
        Flags: review?(evanxd)

        Attachment #817671 -
        Flags: feedback?(yurenju.mozilla)

    
    

    
  
The PR should be set in the Bug 926801.

    
  

        Attachment #817671 -
        Attachment is obsolete: true

    
    

    
  
Bulk closed as part of https://bugzilla.mozilla.org/show_bug.cgi?id=1422750
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INCOMPLETE

    
  
Product: Testing → Testing Graveyard

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: