-
Notifications
You must be signed in to change notification settings - Fork 205
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
Address some of the flaky E2E tests #5106
Conversation
Preview URL 🚀 : https://blurts-server-pr-5106-mgjlpikfea-uk.a.run.app |
c7bb0ce
to
9c43b59
Compare
9c43b59
to
bb3b15d
Compare
a04b8ee
to
68f1879
Compare
src/e2e/specs/dashboard.spec.ts
Outdated
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.
Test groups from dashboard.spec.ts
were split into multiple dashboard-*.spec.ts
files.
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.
Test groups from landing.spec.ts
were split into multiple landing-*.spec.ts
files.
}); | ||
const resJson = await res.json(); | ||
if (resJson.length) { | ||
try { |
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.
Handle unexpected errors and increase waiting time for retrieving the verification code from restmail.
Cleanup completed - database 'blurts-server-pr-5106' destroyed, cloud run service 'blurts-server-pr-5106' destroyed |
This PR addresses some of the flaky E2E tests. The
dashboard
andlanding
test groups were split into their own.spec
files for better parallel execution.