Skip to content

wdio-cucumber-framework: Don't attempt to filter empty specs#10368

Merged
christian-bromann merged 1 commit intowebdriverio:mainfrom
nextlevelbeard:hotfix/empty-specs-tags
May 12, 2023
Merged

wdio-cucumber-framework: Don't attempt to filter empty specs#10368
christian-bromann merged 1 commit intowebdriverio:mainfrom
nextlevelbeard:hotfix/empty-specs-tags

Conversation

@nextlevelbeard
Copy link
Member

@nextlevelbeard nextlevelbeard commented May 11, 2023

Proposed changes

fixes #10367 #10366

Adds a safe guard to the work done in #10330
Further filtering might be done after filtering all the specs that match the tag expression.
But it shouldn't be attempted when there are no specs to run.
Prevents cases where it was listening for events forever.

Types of changes

  • 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

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 necessary documentation (if appropriate)
  • I have added proper type definitions for new commands (if appropriate)

Further comments

Reviewers: @webdriverio/project-committers

Copy link
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.

LGTM, thank you 👍

@christian-bromann christian-bromann merged commit 8e1d621 into webdriverio:main May 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Bug Fix 🐛 PRs that contain bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🐛 Bug]: Cucumber+JS: Feature file without target tag pauses the execution forever

2 participants