Skip to content

Try to figure out some percy problems #1948

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

Closed

Conversation

carols10cents
Copy link
Member

@carols10cents carols10cents commented Dec 12, 2019

I'm fixing some issues with our tests and trying to figure out some percy weirdness in the process.

@carols10cents carols10cents changed the title Remove a test that's duplicated Remove some duplicate tests; fix a test going to the wrong page Dec 12, 2019
@carols10cents
Copy link
Member Author

The search test wasn't actually going to the search page 😰

and the search page actually has some accessibility problems 🤦‍♀

@carols10cents carols10cents force-pushed the fix-probable-copypasta branch 3 times, most recently from e962d1e to 4a2ccdd Compare December 13, 2019 15:56
@carols10cents
Copy link
Member Author

I really don't understand what the problem could be.

The chart for /crates/:crate always fails to load on travis now (and github actions on that PR), but never fails to load locally for me, either while running the tests locally or visiting the page.

The chart on /crates/:crate/:version always successfully loads on travis, so it's not a "google charts js isn't accessible on travis anymore" problem.

@carols10cents
Copy link
Member Author

I found these docs on how to just hide an element from percy: https://docs.percy.io/docs/percy-specific-css#section-percy-specific-css

Also learned here that percy takes DOM snapshots, not screenshots, and then renders the page on its own infrastructure. That page also says:

We skip assets that are hosted on a different domain from what the snapshot came from.

so I'm not sure how the Google charts ever worked, or work on some pages but not others!!

@carols10cents
Copy link
Member Author

Ok, the download graph was removed as I expected for Acceptance | crate page | /crates/:crate is accessible

But I don't understand why it still shows up for Acceptance | crate page | /crates/:crate/:version is accessible and percy reports "no changes" 😭

This test does an accessibility audit on the home page, which is already
being done in front-page-test.js.
@carols10cents carols10cents changed the title Remove some duplicate tests; fix a test going to the wrong page Try to figure out some percy problems Dec 19, 2019
@carols10cents
Copy link
Member Author

I'm going to close this and redo it once we figure out the direction we want to go with percy.

@carols10cents carols10cents deleted the fix-probable-copypasta branch December 20, 2019 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants