added error handling during trace attach (#115)

* added error handling during trace attach
* added node 22

Author: AlexGalichenko

.github/pull_request_template.md

@@ -1,19 +1,13 @@
-Before submitting this PR, please ensure that you have completed the following:
+## Description
+
+Please describe your changes and any new features you're introducing, or issues you're fixing.
 
 - [ ] Updated the CHANGELOG.md.
 - [ ] Checked that there aren't other open pull requests for the same issue/update.
 - [ ] Checked that your contribution follows the project's contribution guidelines.
 - [ ] Added corresponding unit/E2E tests
 - [ ] Unit and E2E pass
 
-## Description
-
-Please describe your changes and any new features you're introducing, or issues you're fixing.
-
 ### Related Issues
 
 Please link any related issues or bug reports that this PR will address.
-
----
-
-Thank you for your contribution! 

.github/workflows/npm-publish.yml

@@ -14,7 +14,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v4
         with:
-          node-version: 18
+          node-version: 20
           registry-url: https://registry.npmjs.org/
       - run: npm ci
       - run: npm run build

.github/workflows/pull-request.yml

@@ -9,7 +9,7 @@ jobs:
   test:
     strategy:
       matrix:
-        version: [ 18, 20 ]
+        version: [ 18, 20, 22 ]
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4

CHANGELOG.md

@@ -10,7 +10,8 @@ Check [Keep a Changelog](http://keepachangelog.com/) for recommendations on how
 :pencil: - chore
 :microscope: - experimental
 
-## [Unreleased]
+## [0.46.2]
+- :beetle: added error handling during trace attach
 - :pencil: updated dependencies
 
 ## [0.46.1]

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@qavajs/steps-playwright",
-  "version": "0.46.1",
+  "version": "0.46.2",
   "description": "steps to interact with playwright",
   "main": "./index.js",
   "scripts": {
@@ -28,7 +28,7 @@
   },
   "homepage": "https://github.com/qavajs/steps-playwright#readme",
   "devDependencies": {
-    "@cucumber/cucumber": "^10.6.0",
+    "@cucumber/cucumber": "^10.7.0",
     "@qavajs/cli": "^0.37.0",
     "@qavajs/console-formatter": "^0.7.2",
     "@qavajs/html-formatter": "^0.15.4",

package.json

@@ -20,10 +20,14 @@ class TracingManager {
     async stop(driverConfig: any, world: any, scenario: any) {
         if (saveTrace(config.driverConfig, scenario)) {
             const path = traceArchive(config.driverConfig, scenario);
-            await context.tracing.stopChunk({ path });
-            if (driverConfig?.trace.attach) {
-                const zipBuffer: Buffer = await readFile(path);
-                world.attach(zipBuffer.toString('base64'), 'base64:application/zip');
+            try {
+                await context.tracing.stopChunk({ path });
+                if (driverConfig?.trace.attach) {
+                    const zipBuffer: Buffer = await readFile(path);
+                    world.attach(zipBuffer.toString('base64'), 'base64:application/zip');
+                }
+            } catch (err) {
+                console.warn('Trace was not recorded');
             }
         }
     }