Skip to content

(wdio-allure-reporter): CompoundError: Also print error message if present [v8]#12378

Merged
christian-bromann merged 1 commit intowebdriverio:v8from
torbenkohlmeier:improve-compound-error-v8
Feb 28, 2024
Merged

(wdio-allure-reporter): CompoundError: Also print error message if present [v8]#12378
christian-bromann merged 1 commit intowebdriverio:v8from
torbenkohlmeier:improve-compound-error-v8

Conversation

@torbenkohlmeier
Copy link
Copy Markdown
Contributor

Proposed changes

Currently, the Allure reports only contain the stack traces of inner errors of CompoundErrors. Depending on the environment, stack traces sometimes do not contain error messages. This makes such errors hard to analyze.

This change adds error messages if present and if the stack trace does not already contain the error message.

Types of changes

  • Polish (an improvement for an existing feature)
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (improvements to the project's docs)
  • Specification changes (updates to WebDriver command specifications)
  • Internal updates (everything related to internal scripts, governance documentation and CI files)

Checklist

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have added the necessary documentation (if appropriate)
  • I have added proper type definitions for new commands (if appropriate)

Reviewers: @webdriverio/project-committers

@torbenkohlmeier torbenkohlmeier changed the base branch from main to v8 February 28, 2024 13:00
Copy link
Copy Markdown
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Polish 💅 PRs that contain improvements on existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants