fix: browser type detection (#90)

Brightspace · Feb 2, 2024 · c587849 · c587849
1 parent 0f00b2b
commit c587849
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/reporters/playwright.js b/src/reporters/playwright.js
@@ -20,7 +20,10 @@ const getRootParent = (test) => {
 const getBrowserType = (project) => {
 	// won't work for merge-reports workflow, all information under `use` will be undefined
 	// hopefully get something included as part of https://github.com/microsoft/playwright/issues/29174
-	const { use: { browserName, defaultBrowserType }, metadata: { browserType } } = project;
+	const {
+		use: { browserName, defaultBrowserType } = {},
+		metadata: { browserType } = {}
+	} = project;
 
 	if (browserType) {
 		return browserType;

diff --git a/test/integration/data/config/playwright.js b/test/integration/data/config/playwright.js
@@ -18,12 +18,9 @@ export default defineConfig({
 	fullyParallel: true,
 	testDir: '../',
 	testMatch: 'playwright-*.test.js',
+	use: deviceTypeChrome,
 	projects: [{
-		name: 'chromium',
-		use: deviceTypeChrome,
-		metadata: {
-			browserType: deviceTypeChrome.defaultBrowserType
-		}
+		name: 'chromium'
 	}, {
 		name: 'firefox',
 		testMatch: 'playwright-2.test.js',