From 7d742ad9f4acf9e48aca40a49037ec565559e6e5 Mon Sep 17 00:00:00 2001 From: cTangonan123 Date: Wed, 26 Feb 2025 11:28:39 -0800 Subject: [PATCH] feat(json-reporter): modified test for parallelIndex to account for scheduling required when ran with 2 workers Co-authored-by: Neel Deshmukh Co-authored-by: Marcelo Villalobos Diaz --- tests/playwright-test/reporter-json.spec.ts | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/tests/playwright-test/reporter-json.spec.ts b/tests/playwright-test/reporter-json.spec.ts index 27910e40ca..89b995ff35 100644 --- a/tests/playwright-test/reporter-json.spec.ts +++ b/tests/playwright-test/reporter-json.spec.ts @@ -334,19 +334,24 @@ test('should report parallelIndex', async ({ runInlineTest }, testInfo) => { import { test, expect } from '@playwright/test'; const fs = require('fs'); test.describe.configure({ mode: 'parallel' }); - test('test 1 passes!', async ({}) => {}); + test('test 1 passes!', async ({}) => { + await new Promise(f => setTimeout(f, 1000)); + }); test('test 2 fails!', async ({}) => { expect(1 + 1).toBe(3); + await new Promise(f => setTimeout(f, 1000)); + }); + test('test 3 passes!', async ({}) => { + await new Promise(f => setTimeout(f, 1000)); }); - test('test 3 passes!', async ({}) => {}); ` }, { 'workers': '2', 'reporter': 'json' }); expect(result.passed).toBe(2); expect(result.failed).toBe(1); expect(result.report.suites[0].specs[0].tests[0].results[0].workerIndex).toBe(0); expect(result.report.suites[0].specs[1].tests[0].results[0].workerIndex).toBe(1); - expect(result.report.suites[0].specs[2].tests[0].results[0].workerIndex).toBe(0); + expect(result.report.suites[0].specs[2].tests[0].results[0].workerIndex).toBe(2); expect(result.report.suites[0].specs[0].tests[0].results[0].parallelIndex).toBe(0); expect(result.report.suites[0].specs[1].tests[0].results[0].parallelIndex).toBe(1); - expect(result.report.suites[0].specs[2].tests[0].results[0].parallelIndex).toBe(0); + expect(result.report.suites[0].specs[2].tests[0].results[0].parallelIndex).toBe(1); });