-
Notifications
You must be signed in to change notification settings - Fork 210
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
Fix for E2Es failing on webp test #11700
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Co-authored-by: Karina Thomas <58214768+karinathomasbbc@users.noreply.github.com>
Co-authored-by: Karina Thomas <58214768+karinathomasbbc@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might want to remove a commented line with a path in it. Otherwise #shipit™
@@ -15,7 +15,7 @@ describe('getIchefURL', () => { | |||
|
|||
describe('builds WebP ichef img url based on originCode, locator and resolution passed', () => { | |||
const BASE_IMAGE_URL = 'https://ichef.bbci.co.uk'; | |||
|
|||
// https://ichef.bbci.co.uk/ace/ws/400/cpsdevpb/2F4D/test/_63490121_110329105535_soweto_304x171_b_nocredit.gif.webp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Get rid?
// you leave the Cypress command queue and are using plain JS. Using .wrap converts | ||
// the JQuery element into a Cypress wrapped element so we can execute Cypress commands on it | ||
|
||
// Images are lazy loaded so we need to scroll to them, check they have loaded before getting currentSrc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lovely use of commenting in this file. Love it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙌
The sport page has no image on. The tests now check for the presence of at least one img in the document before they run.
There were .gif images in the frosted promos for one test asset. '.gif' was not included in the isWebSupported array of accepted url regexes. I added it in the regex.
Helpful Links
Add Links to useful resources related to this PR if applicable.
Coding Standards
Repository use guidelines