-
Notifications
You must be signed in to change notification settings - Fork 43
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: set timeout in before all #3007
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.
lgtm
|
…onents/ui-inputtranslation-inverted-theme * 'main' of github.com:SAP/open-ux-tools: fix: set timeout in before all (#3007) chore: apply latest changesets fix(ui-components): UISections. Error `Cannot read properties of undefined (reading 'size')` is throws for section on initial rendering (#3014) chore: apply latest changesets feat(preview-middleware):refactor lrep connectors in preview middleware (#3009) chore: apply latest changesets fix(deploy-config): fix errors running standalone (#3012) chore: apply latest changesets fix(deps): update dependency axios to v1.8.2 [security] (#3005) chore: apply latest changesets fix(deploy-config-sub-generator): append destination to backendconfig (#3011) chore: apply latest changesets fix(deploy-gen): fix undefined error when no backend config found (#3008) chore: apply latest changesets fix(cf-deploy-config-writer): add more tests based on testing with CDS generated mta's (#2998) chore: apply latest changesets fix(project-integrity): ensure encoded write for project-integrity (#3004) chore: apply latest changesets fix(project-access): Improve `getModule` to attempt module reinstallation if loading fails (#3000)
Problem:
Timeout was set in config which effects all tests timeout.More on timeout.
If there was any failure, playwright would re-try one more time after waiting
5 min + 30 second
and this caused timeout in pipeline which has only 30 min. Playwright report artifacts were not uploaded due to pipeline time out.Possible solution:
Add time out only in
beforeAll
which affects only before all. More on this check..