[test optimization] Fix latest playwright@1.56.0 release (#6611)

juan-fernandez · web-flow · commit b0f79f9ca4cf · 2025-10-07T12:57:43.000+02:00
diff --git a/packages/datadog-instrumentations/src/playwright.js b/packages/datadog-instrumentations/src/playwright.js
@@ -772,7 +772,10 @@ addHook({
     if (!isKnownTestsEnabled && !isTestManagementTestsEnabled && !isImpactedTestsEnabled) {
       return oldCreateRootSuite.apply(this, arguments)
     }
-    const rootSuite = await oldCreateRootSuite.apply(this, arguments)
+
+    const createRootSuiteReturnValue = await oldCreateRootSuite.apply(this, arguments)
+    // From v1.56.0 on, createRootSuite returns `{ rootSuite, topLevelProjects }`
+    const rootSuite = createRootSuiteReturnValue.rootSuite || createRootSuiteReturnValue
 
     const allTests = rootSuite.allTests()
 
@@ -861,7 +864,7 @@ addHook({
       }
     }
 
-    return rootSuite
+    return createRootSuiteReturnValue
   }
 
   // We need to proxy the createRootSuite function because the function is not configurable

Original file line number	Diff line number	Diff line change
`@@ -772,7 +772,10 @@ addHook({`
`772`	`772`	`if (!isKnownTestsEnabled && !isTestManagementTestsEnabled && !isImpactedTestsEnabled) {`
`773`	`773`	`return oldCreateRootSuite.apply(this, arguments)`
`774`	`774`	`}`
`775`		`- const rootSuite = await oldCreateRootSuite.apply(this, arguments)`
	`775`	`+`
	`776`	`+ const createRootSuiteReturnValue = await oldCreateRootSuite.apply(this, arguments)`
	`777`	+ // From v1.56.0 on, createRootSuite returns `{ rootSuite, topLevelProjects }`
	`778`	`+ const rootSuite = createRootSuiteReturnValue.rootSuite \|\| createRootSuiteReturnValue`
`776`	`779`
`777`	`780`	`const allTests = rootSuite.allTests()`
`778`	`781`
`@@ -861,7 +864,7 @@ addHook({`
`861`	`864`	`}`
`862`	`865`	`}`
`863`	`866`
`864`		`- return rootSuite`
	`867`	`+ return createRootSuiteReturnValue`
`865`	`868`	`}`
`866`	`869`
`867`	`870`	`// We need to proxy the createRootSuite function because the function is not configurable`