Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Write new visual diff tests for AMP components owned by @aghassemi #11415

Closed
14 tasks
rsimha opened this issue Sep 25, 2017 · 4 comments
Closed
14 tasks

Write new visual diff tests for AMP components owned by @aghassemi #11415

rsimha opened this issue Sep 25, 2017 · 4 comments
Assignees
Milestone

Comments

@rsimha
Copy link
Contributor

rsimha commented Sep 25, 2017

The following components owned by @aghassemi do not have visual diff test coverage.

  • amp-video
  • amp-vimeo
  • amp-youtube
  • amp-app-banner
  • amp-fx-parallax
  • amp-lightbox
  • amp-lightbox-viewer
  • amp-viz-vega
  • amp-form
  • amp-facebook
  • amp-social-share
  • amp-izlesene
  • amp-nexxtv-player
  • amp-gist

How to add a visual test:

  1. Create a simple test page for your component. You could use a portion of an existing page in examples/, or create a new one based on AMP by Example.
  2. Create a new directory in examples/visual-tests, for the test page (+ assets)
  3. Add an entry for the test page to test/visual-diff/visual-tests.json
  4. Create a PR and wait for it to be tested on Travis (the percy/amphtml check will fail)
  5. Click the details link and approve the Percy build that was generated on your PR branch
  6. Merge your PR and approve the next Percy build on master. Until you do so, master will be red, and this is expected.

Tip: Use #11413 as an example of how to add a new visual test. If your test page needs time to reach a steady state before a snapshot is taken, you can specify css classes that need to be visible, eventually appear, or eventually disappear. See test/visual-diff/visual-tests.json for examples.

Documentation: See this page on the AMP developer guide.

Master issue: #10155

@ampprojectbot
Copy link
Member

This issue doesn't have a category which makes it harder for us to keep track of it. @aghassemi Please add an appropriate category.

@ampprojectbot
Copy link
Member

This issue hasn't been updated in awhile. @aghassemi Do you have any updates?

@aghassemi
Copy link
Contributor

/cc @nainar

@rsimha rsimha removed their assignment Jun 4, 2018
@ampprojectbot
Copy link
Member

This issue hasn't been updated in awhile. @aghassemi Do you have any updates?

levidurfee added a commit to levidurfee/amphtml that referenced this issue Sep 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants