test: improve ui test stability and remove unnecessary code (#3945)

packages/integration-tests/src/tests/ui/user-management.test.ts

@@ -194,7 +194,6 @@ describe('user management', () => {
     await page.waitForSelector('form');
     // Conflicted email
     await expect(page).toFillForm('form', { primaryEmail: 'jdoe@gmail.com' });
-    await page.screenshot({ path: 'user-conflict-email.png' });
     await expect(page).toClick('form div[class$=actionBar] button:nth-of-type(2)');
     await expect(page).toMatchElement('div[class$=error] div[class$=message]', {
       text: 'This email is associated with an existing account.',

packages/integration-tests/src/tests/ui/webhooks.test.ts

@@ -11,6 +11,7 @@ const createWebhookFromWebhooksPage = async () => {
   await expect(page).toFill('input[name=name]', 'hook_name');
   await expect(page).toFill('input[name=url]', 'https://example.com/webhook');
   await expect(page).toClick('button[type=submit]');
+  await page.waitForSelector('div[class$=header] div[class$=metadata] div[class$=title]');
 };
 
 describe('webhooks', () => {
@@ -38,7 +39,6 @@ describe('webhooks', () => {
     await createWebhookFromWebhooksPage();
 
     // Go to webhook details page
-    await page.waitForSelector('div[class$=header] div[class$=metadata] div[class$=title]');
     await expect(page).toMatchElement('div[class$=main] div[class$=metadata] div[class$=title]', {
       text: 'hook_name',
     });