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

Support also feature files used by RSpec turnip gem. #919

Merged
merged 1 commit into from Oct 9, 2023

Conversation

simi
Copy link
Contributor

@simi simi commented Oct 7, 2023

fixes #271

ℹ️ tested in real project as well

before

bin/parallel_rspec spec/acceptance/              
0 processes for 0 specs, ~ 0 specs per process 

after

bin/parallel_rspec spec/acceptance/                                                                                                             
1 process for 40 specs, ~ 40 specs per process
...

@grosser
Copy link
Owner

grosser commented Oct 9, 2023

ah that's much simpler than I expected :)

@grosser grosser merged commit 439a9ca into grosser:master Oct 9, 2023
13 checks passed
@grosser
Copy link
Owner

grosser commented Oct 9, 2023

4.3.0

@grosser
Copy link
Owner

grosser commented Oct 9, 2023

I think this could be a bug for users using gherkin and rspec where their .feature files would previously not be run when using rspec, if they keep their files in the spec folder. But let's see if anyone runs into that.

@simi
Copy link
Contributor Author

simi commented Oct 9, 2023

I think this could be a bug for users using gherkin and rspec where their .feature files would previously not be run when using rspec, if they keep their files in the spec folder. But let's see if anyone runs into that.

I was thinking about that, but those features are usually in features folder and are not mixed in spec folder. Alternatively it could be possible to check if Turnip is defined and enhance the lookup regexp only in that case.

@simi simi deleted the turnip-support branch October 9, 2023 08:27
@grosser
Copy link
Owner

grosser commented Oct 9, 2023 via email

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.

Support for Turnip
2 participants