From b271610ba7d00c795aa8358d31de306f415f64b4 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Tue, 27 Jan 2026 18:08:38 +0000 Subject: [PATCH] perf: Use Promise.all for concurrent assertions in e2e tests Wraps sequential expect(...).toBeVisible() calls for social links in Promise.all to allow concurrent execution. This is a best practice for independent assertions to potentially improve test performance. Co-authored-by: xRahul <1639945+xRahul@users.noreply.github.com> --- tests/e2e.spec.ts | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/tests/e2e.spec.ts b/tests/e2e.spec.ts index 1d920bea..e237e7f5 100644 --- a/tests/e2e.spec.ts +++ b/tests/e2e.spec.ts @@ -18,8 +18,10 @@ test('has social links', async ({ page }) => { await page.goto('/'); // Check for social links - await expect(page.getByLabel('Stack Overflow')).toBeVisible(); - await expect(page.getByLabel('Linkedin')).toBeVisible(); - await expect(page.getByLabel('Github')).toBeVisible(); - await expect(page.getByLabel('Twitter')).toBeVisible(); + await Promise.all([ + expect(page.getByLabel('Stack Overflow')).toBeVisible(), + expect(page.getByLabel('Linkedin')).toBeVisible(), + expect(page.getByLabel('Github')).toBeVisible(), + expect(page.getByLabel('Twitter')).toBeVisible(), + ]); });