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

Scenario Outline Parallelizing #1

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

Conversation

TaysirTayyab
Copy link

This PR adds the ability for the partial runner to also distribute Scenario Outlines across workers. This improves the balancing provided a more even distribution of work between workers.

Each example of Scenario Outline is treated as its own test. So an outline with 3 Example rows on 3 nodes will result in each node executing one Example row. Counting is now done as shown below.

# Assume there are 3 workers total

Scenario:                      # worker 1
    Given something exists
    Then I have something

Scenario Outline:
    Given I have <something>
    When I do action with it

    Examples:
        | something |
        | thing A   |          # worker 2
        | thing B   |          # worker 3
        | thing C   |          # worker 1
        | thing D   |          # worker 2
        | thing E   |          # worker 3

Scenario:                      # worker 1
    When I did that one thing
    Then this other thing happened

There's also some unit tests added to test this behavior.

@TaysirTayyab TaysirTayyab force-pushed the scenario-outline-filtering branch from 271a3c5 to 3064e0c Compare August 9, 2016 16:37
@anton-siardziuk
Copy link
Owner

@TaysirTayyab Hi! Thank you for you help! Sorry for not answering for some time, I've turned off my notifications and kinda lost track of my github account. I will take a look at your PR's soon.

@TaysirTayyab
Copy link
Author

@m00t No problem!

@TaysirTayyab TaysirTayyab force-pushed the scenario-outline-filtering branch from 3064e0c to 9aa085a Compare October 13, 2016 18:05
@TaysirTayyab TaysirTayyab force-pushed the scenario-outline-filtering branch from 9aa085a to f9c3b21 Compare October 13, 2016 18:35
@TaysirTayyab TaysirTayyab force-pushed the scenario-outline-filtering branch from f9c3b21 to 6bf0b9c Compare October 13, 2016 18:38
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