Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

New Screenshot tests for Continuous User Interface Testing #4087

Closed
diosmosis opened this Issue · 12 comments

3 participants

@diosmosis
Collaborator

One thing remains to for Piwik's QA suite to be complete: testing the UI. This can be done by generating screenshots of URLs after injecting JavaScript and comparing w/ expected screenshots. This ticket is for the creation of this system.

TODO:

  • Create screenshot compare tests for all of Piwik.
  • Enable screenshot comparison on travis. (Need to make sure screenshots on different machines are the same.)
@anonymous-piwik-user

In 1c274da: Refs #4087, rewrote UI integration tests to use phantomjs or slimerjs and added (disabled) UI tests to travis build config.

Notes:

  • Modified main page-loading JavaScript to use ajaxHelper so it's possible to know when all AJAX requests are made.
  • Add setUrl method to ajaxHelper so it can be used when a doing AJAX w/ a URL instead of an object containing query parameters.
@anonymous-piwik-user

In cee9bc6: Refs #4087, fixing typo.

@anonymous-piwik-user

In 78723a8: Refs #4087, fixing logic error.

@anonymous-piwik-user

In 8bb5f17: Refs #4087, fix bug in capture.js and allow multiple UI test cases to be written.

@anonymous-piwik-user

In 01e5881: Refs #4087, remove recursive links and create/remove them only when doing UI tests.

@anonymous-piwik-user

In d57f4a8: Refs #4087, add UI screenshot tests to new repository, enable tests on travis and make sure new repo is built every time integration tests pass on piwik build.

@anonymous-piwik-user

In 7770c12: Refs #4087, tweak tests README.

@anonymous-piwik-user

In 07d058f: Refs #4087, adding new screenshot tests for visitor profile, transitions and user settings admin page.

@mattab
Owner

Congrats on closing this ticket!!

@mattab
Owner

In fdb2cba: Documenting required imagick lib for screenshot comparison generate refs #4087

@mattab
Owner

In 5e644be: Refs #4087 #4189 We would like ui tests build to run every commit even when the Integration tests fail. This will help know exactly which commit introduced a regression.

@diosmosis diosmosis added this to the 2.0 - Piwik 2.0 milestone
@mattab mattab referenced this issue
Closed

UI unit tests #4189

@sabl0r sabl0r referenced this issue from a commit in sabl0r/piwik
@diosmosis diosmosis Refs #4087, rewrote UI integration tests to use phantomjs or slimerjs…
… and added (disabled) UI tests to travis build config.

Notes:
  * Modified main page-loading JavaScript to use ajaxHelper so it's possible to know when all AJAX requests are made.
  * Add setUrl method to ajaxHelper so it can be used when a doing AJAX w/ a URL instead of an object containing query parameters.
1c274da
@sabl0r sabl0r referenced this issue from a commit in sabl0r/piwik
@diosmosis diosmosis Refs #4087, fixing typo. cee9bc6
@sabl0r sabl0r referenced this issue from a commit in sabl0r/piwik
@diosmosis diosmosis Refs #4087, fixing logic error. 78723a8
@sabl0r sabl0r referenced this issue from a commit in sabl0r/piwik
@diosmosis diosmosis Refs #4087, fix bug in capture.js and allow multiple UI test cases to…
… be written.
8bb5f17
@sabl0r sabl0r referenced this issue from a commit in sabl0r/piwik
@diosmosis diosmosis Refs #4087, remove recursive links and create/remove them only when d…
…oing UI tests.
01e5881
@sabl0r sabl0r referenced this issue from a commit in sabl0r/piwik
@diosmosis diosmosis Refs #4087, add UI screenshot tests to new repository, enable tests o…
…n travis and make sure new repo is built every time integration tests pass on piwik build.
d57f4a8
@sabl0r sabl0r referenced this issue from a commit in sabl0r/piwik
@diosmosis diosmosis Refs #4087, tweak tests README. 7770c12
@sabl0r sabl0r referenced this issue from a commit in sabl0r/piwik
@diosmosis diosmosis Refs #4087, adding new screenshot tests for visitor profile, transiti…
…ons and user settings admin page.
07d058f
@sabl0r sabl0r referenced this issue from a commit in sabl0r/piwik
@mattab mattab Refs #4087 #4189 We would like ui tests build to run every commit eve…
…n when the Integration tests fail. This will help know exactly which commit introduced a regression.
5e644be
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.