Merge branch '4.x' of github.com:codeceptjs/CodeceptJS into 4.x

Author: DavertMik

docs/plugins.md

@@ -569,6 +569,49 @@ More config options are available:
 
 *   `config`   (optional, default `{}`)
 
+## junitReporter
+
+Generates a JUnit-compatible XML report after a test run.
+
+Unlike Mocha's `mocha-junit-reporter`, this plugin understands CodeceptJS steps and substeps.
+For every `<testcase>` it includes:
+
+*   `<properties>` — the test's meta information: every `meta` key from `Scenario('...', { meta })`, plus its `tags` and `retries`
+*   `<system-out>` — an indented step/substep log (substeps are nested under their meta step); only failed steps are marked
+*   `<failure>` — for failed tests: the error message, type, stack trace and (optionally) the step trace
+
+The produced file is consumable by Jenkins, GitLab CI, CircleCI, GitHub Actions test reporters, etc.
+
+#### Configuration
+
+```js
+"plugins": {
+   "junitReporter": {
+     "enabled": true
+   }
+ }
+```
+
+Possible config options:
+
+*   `outputName`: file name for the report. Default: `report.xml`.
+*   `output`: directory where the report is stored, relative to the project root. Default: the `output` directory.
+*   `testGroupName`: value of the `name` attribute on the root `<testsuites>` element. Default: `CodeceptJS`.
+*   `attachMeta`: add the test's meta information (`meta` keys, `tags`, `retries`) as `<properties>`. Default: true.
+*   `attachSteps`: add the step/substep log as `<system-out>`. Default: true.
+*   `stepsInFailure`: append the step trace to the `<failure>` body. Default: true.
+
+CLI examples:
+
+    npx codeceptjs run -p junitReporter
+    npx codeceptjs run -p junitReporter:outputName=junit.xml
+
+> ℹ When running with `run-workers`, steps are serialized between processes and substep nesting is flattened.
+
+### Parameters
+
+*   `config` **any**  (optional, default `{}`)
+
 ## pageInfo
 
 Collects information from web page after each failed test and adds it to the test as an artifact.

lib/actor.js

@@ -94,7 +94,7 @@ export default function (obj = {}, container) {
             actor[action] = actor[actionAlias] = function () {
               const step = new Step(helper, action)
               if (translation.loaded) {
-                step.name = actionAlias
+                step.title = actionAlias
                 step.actor = translation.I
               }
               // add methods to promise chain

lib/command/workers/runTests.js

@@ -54,11 +54,7 @@ process.on('unhandledRejection', (reason, promise) => {
     process.stderr.write(`${reason.stack}\n`)
   }
 
-  // Don't exit on test-related rejections
-  if (msg.includes('expected') || msg.includes('AssertionError')) {
-    return
-  }
-  process.exit(1)
+  // Do not exit — killing the worker silently drops every remaining test from the report.
 })
 
 // hide worker output

lib/heal.js

@@ -49,12 +49,12 @@ class Heal {
   }
 
   hasCorrespondingRecipes(step) {
-    return matchRecipes(this.recipes, this.contextName).filter(r => !r.steps || r.steps.includes(step.name)).length > 0
+    return matchRecipes(this.recipes, this.contextName).filter(r => !r.steps || r.steps.includes(step.title)).length > 0
   }
 
   async getCodeSuggestions(context) {
     const suggestions = []
-    const stepName = context.step?.name
+    const stepName = context.step?.title
     const recipes = matchRecipes(this.recipes, this.contextName)
       .filter(r => !r.steps || !stepName || r.steps.includes(stepName))
 

lib/plugin/aiTrace.js

@@ -195,7 +195,7 @@ export default function (config = {}) {
     } else {
       if (stepNum === -1) return
       if (isStepIgnored(step)) return
-      if (step.metaStep && step.metaStep.name === 'BeforeSuite') return
+      if (step.metaStep && step.metaStep.title === 'BeforeSuite') return
 
       const stepPrefix = generateStepPrefix(step, stepNum)
       stepNum++
@@ -247,7 +247,7 @@ export default function (config = {}) {
   async function persistStep(step) {
     if (stepNum === -1) return
     if (isStepIgnored(step)) return
-    if (step.metaStep && step.metaStep.name === 'BeforeSuite') return
+    if (step.metaStep && step.metaStep.title === 'BeforeSuite') return
 
     const stepKey = step.toString()
 
@@ -437,8 +437,9 @@ export default function (config = {}) {
 
   function isStepIgnored(step) {
     if (!config.ignoreSteps) return false
+    if (!step.title) return false
     for (const pattern of config.ignoreSteps || []) {
-      if (step.name.match(pattern)) return true
+      if (step.title.match(pattern)) return true
     }
     return false
   }

lib/plugin/analyze.js

@@ -155,10 +155,9 @@ const defaultConfig = {
       if (config.vision && test.artifacts.screenshot) {
         debug('Adding screenshot to prompt')
         messages[0].content.push({
-          type: 'image_url',
-          image_url: {
-            url: 'data:image/png;base64,' + base64EncodeFile(test.artifacts.screenshot),
-          },
+          type: 'image',
+          image: base64EncodeFile(test.artifacts.screenshot),
+          mediaType: 'image/png',
         })
       }