fixed for release (#3301)

Author: DavertMik

CHANGELOG.md

@@ -1,10 +1,36 @@
 ## 3.3.1
+
 🛩️ Features:
+
 * Add option to avoid duplicate gherkin step definitions (#3257) - @raywiis
+* Added `step.*` for run-workers #3272. Thanks to @abhimanyupandian
+* Fixed loading tests for `codecept run` using glob patterns. By @jayudey-wf 
+
+```
+npx codeceptjs run test-dir/*"
+```
+
+* [Playwright] **Possible breaking change.** By default `timeout` is changed to 5000ms. The value set in 3.3.0 was too low. Please set `timeout` explicitly to not depend on release values.
+* [Playwright] Added for color scheme option by @PeterNgTr
+
+```js
+ helpers: {
+  Playwright : {
+    url: "http://localhost",
+    colorScheme: "dark",
+  }
+ }
+```
+
 
 🐛 Bugfixes:
-* Changing the defunct faker library. See #3232 - @cybersonic 
+
+* [Playwright] Fixed `Cannot read property 'video' of undefined`
 * Fixed haveRequestHeaders() and amBearerAuthenticated() of REST helper (#3260) - @mirao
+* Fixed: allure attachment fails if screenshot failed #3298 by @ruudvanderweijde 
+* Fixed #3105 using autoLogin() plugin with TypeScript. Fix #3290 by @PeterNgTr 
+* [Playwright] Added extra params for click and dragAndDrop to type definitions by @mirao
+
 
 📖 Documentation
 * Improving the typings in many places

lib/command/workers/runTests.js

@@ -127,7 +127,7 @@ function initializeListeners() {
       parent,
     };
   }
-  
+
   function simplifyStep(step, err = null) {
     step = { ...step };
 
@@ -156,10 +156,10 @@ function initializeListeners() {
       duration: step.duration || 0,
       err,
       parent,
-      test: simplifyTest(step.test)
+      test: simplifyTest(step.test),
     };
   }
-  
+
   collectStats();
   // suite
   event.dispatcher.on(event.suite.before, suite => sendToParentThread({ event: event.suite.before, workerIndex, data: simplifyTest(suite) }));
@@ -178,12 +178,12 @@ function initializeListeners() {
   event.dispatcher.on(event.test.started, test => sendToParentThread({ event: event.test.started, workerIndex, data: simplifyTest(test) }));
   event.dispatcher.on(event.test.skipped, test => sendToParentThread({ event: event.test.skipped, workerIndex, data: simplifyTest(test) }));
 
-  // steps 
+  // steps
   event.dispatcher.on(event.step.finished, step => sendToParentThread({ event: event.step.finished, workerIndex, data: simplifyStep(step) }));
   event.dispatcher.on(event.step.started, step => sendToParentThread({ event: event.step.started, workerIndex, data: simplifyStep(step) }));
   event.dispatcher.on(event.step.passed, step => sendToParentThread({ event: event.step.passed, workerIndex, data: simplifyStep(step) }));
   event.dispatcher.on(event.step.failed, step => sendToParentThread({ event: event.step.failed, workerIndex, data: simplifyStep(step) }));
-  
+
   event.dispatcher.on(event.hook.failed, (test, err) => sendToParentThread({ event: event.hook.failed, workerIndex, data: simplifyTest(test, err) }));
   event.dispatcher.on(event.hook.passed, (test, err) => sendToParentThread({ event: event.hook.passed, workerIndex, data: simplifyTest(test, err) }));
   event.dispatcher.on(event.all.failures, (data) => sendToParentThread({ event: event.all.failures, workerIndex, data }));

lib/helper/Playwright.js

@@ -225,7 +225,7 @@ const { createValueEngine, createDisabledEngine } = require('./extras/Playwright
  *  }
  * }
  * ```
- * 
+ *
  * * #### Example #8: Launch test with a specific color scheme
  *
  * ```js
@@ -287,7 +287,7 @@ class Playwright extends Helper {
       waitForAction: 100,
       waitForTimeout: 1000,
       pressKeyDelay: 10,
-      timeout: 1000,
+      timeout: 5000,
       fullPageScreenshots: false,
       disableScreenshots: false,
       ignoreLog: ['warning', 'log'],
@@ -2025,7 +2025,7 @@ class Playwright extends Helper {
       test.artifacts = {};
     }
 
-    if (this.options.recordVideo && this.page.video()) {
+    if (this.options.recordVideo && this.page && this.page.video()) {
       test.artifacts.video = await this.page.video().path();
     }
 
@@ -2037,7 +2037,7 @@ class Playwright extends Helper {
   }
 
   async _passed(test) {
-    if (this.options.recordVideo && this.page.video()) {
+    if (this.options.recordVideo && this.page && this.page.video()) {
       if (this.options.keepVideoForPassedTests) {
         test.artifacts.video = await this.page.video().path();
       } else {

package.json

@@ -154,4 +154,4 @@
       "pre-push": "npm run test:unit"
     }
   }
-}
+}