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
tests(smoke): handle warn-not-offline-capable in smoketest #11799
Conversation
errors: { | ||
length: '>= 1', | ||
0: { | ||
// In m89 the `warn-not-offline-capable` error was added. |
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.
COMPAT? ;)
errors: [ | ||
{errorId: 'no-icon-available'}, | ||
], | ||
errors: { |
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.
I assume the audit(s) will have to account for this starting in #11745?
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.
yes indeed. i was thinking it'd be a followup PR. but yeah it'll have to be handled.
another smoke failure with ToT chrome.. this time around the new
warn-not-offline-capable
. There's an internal thread about this installability errorId. and here's the crbug: https://bugs.chromium.org/p/chromium/issues/detail?id=965802#c46smoke failure example from #11790:
This PR doesn't do much except allow this smoketest assertion to be a bit more lenient.
fwiw i could not reproduce receiving this errorId locally