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

first pass of parallelization #416

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jtushman
Copy link

[Do not merge]

first pass of adding parallelization

  • adding hooks for before.batch and after.batch (think I should rename to before.fork and after.hook)
  • added a new runner -- ParallelRunner, which loops over all of the scenarios (not the features) batches them up for n workers to have at them, then joins the threads and regroups the scenarios into featureResults
  • adds option --parallel / -p which you can pass -n- for the number of processes you want to have processing your tests

but with these hooks the implementer can set up there forked process. I also set a world.port_number for the implementer to use that to setup webservers and other resources specifically to each fork.

Still very much in beta -- and I have some ideas to make it faster -- but seems promising

Known issues:

  • need docs
  • need tests

Love Feedback -- is this something that you would consider pulling in. If so I can work on the known issues, and other feedback that you might have.

Happy Holidays!

@ashrafuzzaman
Copy link

Guys what is the update on this issue. It would be great to run tests in parallel.

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.

2 participants