Removing unwanted code and fixing ? error for lower node versions

Author: karanshah-browserstack

README.md

@@ -106,7 +106,7 @@ If you are using Playwright Test Runner to run your Playwright tests, you can ru
 3. Install the dependencies using `npm install`
 4. Put in your credentials in the file `fixtures.js` in the caps part.
 5. If you are trying to run your own tests on BrowserStack, then you need to ensure that you have configured the `projects` correctly in `playwright.config.js` file.
-6. Run the sample test script using `npm run test` which runs all the tests inside tests directory across 5 browsers in BrowserStack.
+6. Run the sample test script using `npm test` which runs all the tests inside tests directory across 5 browsers in BrowserStack.
 7. Run the sample test script using `npm run test:local` and add `browserstack.local:true` in the file `fixture.js` in caps part which runs all the tests inside tests directory across 5 browsers in BrowserStack.
 8. If you want to run your tests locally, you just need to configure the `projects` without name `@browserstack` in `playwright.config.js` file.
 

playwright-test/fixtures.js

@@ -40,6 +40,9 @@ const patchCaps = (name) => {
   caps.name = name;
 };
 
+const isHash = (entity) => Boolean(entity && typeof(entity) === "object" && !Array.isArray(entity));
+const nestedKeyValue = (hash, keys) => keys.reduce((hash, key) => (isHash(hash) ? hash[key] : undefined), hash);
+
 exports.test = base.test.extend({
   browser: async ({ playwright, browser }, use, workerInfo) => {
     if (workerInfo.project.name.match(/browserstack/)) {
@@ -62,7 +65,7 @@ exports.test = base.test.extend({
         action: 'setSessionStatus',
         arguments: {
           status: testInfo.status,
-          reason: testInfo?.error?.message,
+          reason: nestedKeyValue(testInfo, ['error', 'message'])
         },
       };
       await vPage.evaluate(() => {},

playwright-test/tests/test_1.spec.js

@@ -1,4 +1,3 @@
-// example.spec.js
 const { test } = require('../fixtures');
 const { expect } = require('@playwright/test');
 test.describe('feature foo', () => {

playwright-test/tests/test_10.spec.js

@@ -1,4 +1,3 @@
-// example.spec.js
 const { test } = require('../fixtures');
 const { expect } = require('@playwright/test');
 test.describe('feature foo', () => {

playwright-test/tests/test_2.spec.js

@@ -1,4 +1,3 @@
-// example.spec.js
 const { test } = require('../fixtures');
 const { expect } = require('@playwright/test');
 test.describe('feature foo', () => {

playwright-test/tests/test_3.spec.js

@@ -1,4 +1,3 @@
-// example.spec.js
 const { test } = require('../fixtures');
 const { expect } = require('@playwright/test');
 test.describe('feature foo', () => {

playwright-test/tests/test_4.spec.js

@@ -1,4 +1,3 @@
-// example.spec.js
 const { test } = require('../fixtures');
 const { expect } = require('@playwright/test');
 test.describe('feature foo', () => {

playwright-test/tests/test_5.spec.js

@@ -1,4 +1,3 @@
-// example.spec.js
 const { test } = require('../fixtures');
 const { expect } = require('@playwright/test');
 test.describe('feature foo', () => {

playwright-test/tests/test_6.spec.js

@@ -1,4 +1,3 @@
-// example.spec.js
 const { test } = require('../fixtures');
 const { expect } = require('@playwright/test');
 test.describe('feature foo', () => {

playwright-test/tests/test_7.spec.js

@@ -1,4 +1,3 @@
-// example.spec.js
 const { test } = require('../fixtures');
 const { expect } = require('@playwright/test');
 test.describe('feature foo', () => {