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

added donate landing and help page to percy tests #11598

Merged
merged 5 commits into from
Jan 4, 2024

Conversation

danielfmiranda
Copy link
Collaborator

@danielfmiranda danielfmiranda commented Dec 21, 2023

Description

Link to sample test page:
Related PRs/issues: #11174

This PR adds the Donate Landing Page and Donate Help Page to the list of pages to be checked by Percy. That way we can flag any unintended changes to the donate pages in future updates!

Please note that this change does not include the donate "Ways to give" page as there is currently no factory, and also, that page is built using the "default page" model. However, ticket #11497 has been created to capture that work/discussion 👍

@danielfmiranda
Copy link
Collaborator Author

Leaving a note before the holiday break:

I am able to get the recaptcha box for the donate help form consistently on percy's chrome screenshots.

However for some reason, the box does not render on percys screenshots using FireFox 🤔

I have tried the following things in visual.spec.js, all which produce the "render on chrome, but not on firefox" issue:

  • Wait for Iframe to be visible, scroll Iframe into view, wait 2s
  • Wait for Iframe to be visible, scroll Iframe into view, and wait 3s
  • Wait for Iframe to be visible, wait for Iframes recaptcha label to load in, wait 2s
  • Scroll to bottom of page, wait for iframe and recaptcha label, scroll into view, scroll to top, wait 2s

Whats interesting to me is that the await statements that wait for the recaptcha box/label do not fail. I have double checked and if the recaptcha box was really not rendering, these statements would return an error during the CI tests.

Going to bring this up to discuss with other devs after the break, to see if we want to dedicate more time to getting the recaptcha box to render on FF.

@danielfmiranda danielfmiranda changed the title added donate landing and help page to percy tests [WIP] - added donate landing and help page to percy tests Dec 21, 2023
@danielfmiranda danielfmiranda force-pushed the feature/11174-donate-percy-test branch from 4172d8d to 2f8b6c6 Compare January 2, 2024 21:37
@danielfmiranda danielfmiranda changed the title [WIP] - added donate landing and help page to percy tests added donate landing and help page to percy tests Jan 3, 2024
@danielfmiranda
Copy link
Collaborator Author

danielfmiranda commented Jan 3, 2024

Hi Everyone!

As mentioned in the comment above, we now have the recaptcha box consistently rendering as expected on Percy's screenshots for Google Chrome:

Screenshot 2024-01-03 at 11-59-15 Build #15500 - Mozilla Foundation - Percy Visual testing as a service

On FireFox screenshots, the page content and spacing consistently renders as if the recaptcha box is there, and, JavaScript that is awaiting the Recaptcha's inner iframe contents to be loaded in is also not failing. The only difference is that the recaptcha box is not being shown in the screenshot for some reason:

Screenshot 2024-01-03 at 11-59-25 Build #15500 - Mozilla Foundation - Percy Visual testing as a service

After discussion with @mtdenton, we have decided that this work is good to go as is.

Submitting for review!

Copy link
Contributor

@mtdenton mtdenton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@danielfmiranda danielfmiranda merged commit 0328756 into 2023-eoy Jan 4, 2024
4 checks passed
@danielfmiranda danielfmiranda deleted the feature/11174-donate-percy-test branch January 4, 2024 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants