-
Notifications
You must be signed in to change notification settings - Fork 38
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix failing e2e test #335
Fix failing e2e test #335
Conversation
timeout: 30 * 1000, | ||
expect: { | ||
timeout: 30 * 1000, | ||
timeout: 10 * 1000, |
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.
Unrelated but lowering some timeouts after the latest issue which took way too long cause of these being so high.
@@ -393,7 +393,7 @@ test.describe('switching products', () => { | |||
await expect(devPage.page.locator('#calls-widget')).toBeVisible(); | |||
|
|||
await devPage.page.locator('#calls-widget-participants-button').click(); | |||
const participantsList = devPage.page.locator('#calls-widget-participants-menu'); | |||
const participantsList = devPage.page.locator('#calls-widget-participants-list'); |
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.
Just curious, what did boards due to affect this?
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.
No idea to be honest I just wanted to fix it fast. Maybe something on .Menu
class again. But it started failing all of the sudden, I guess it could have been webapp as well, didn't investigate further.
I have no idea why the pipeline fails to be honest. Plugin is building fine but somehow it breaks at uploading the npm cache. I'll try to trigger it once more but otherwise will merge this. |
Summary
Fixing some new failures after the latest boards/webapp update 馃槵