Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix execution of scenarios provided as a list of path:line #414

Merged
merged 1 commit into from
Aug 7, 2021

Conversation

vearutop
Copy link
Member

@vearutop vearutop commented Aug 7, 2021

Description

This PR adds line to feature URI to allow executing multiple scenarios identified by line number.

Motivation & context

Fixes #188.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Note to other contributors

No notes.

Update required of cucumber.io/docs

No update.

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@codecov
Copy link

codecov bot commented Aug 7, 2021

Codecov Report

Merging #414 (98b5295) into main (6fb74a5) will decrease coverage by 0.12%.
The diff coverage is 20.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #414      +/-   ##
==========================================
- Coverage   82.88%   82.75%   -0.13%     
==========================================
  Files          26       26              
  Lines        2454     2459       +5     
==========================================
+ Hits         2034     2035       +1     
- Misses        323      327       +4     
  Partials       97       97              
Impacted Files Coverage Δ
internal/parser/parser.go 61.79% <0.00%> (-2.91%) ⬇️
internal/formatters/fmt_pretty.go 80.74% <100.00%> (+0.06%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6fb74a5...98b5295. Read the comment docs.

@vearutop vearutop marked this pull request as ready for review August 7, 2021 20:36
@vearutop vearutop merged commit 09d7d85 into main Aug 7, 2021
@vearutop vearutop deleted the issue-188 branch August 7, 2021 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't execute multiple specific scenarios in the same feature file
1 participant