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

41: Added contextual and handling #63

Merged
merged 2 commits into from
May 8, 2017
Merged

41: Added contextual and handling #63

merged 2 commits into from
May 8, 2017

Conversation

TitouanVanBelle
Copy link
Contributor

@TitouanVanBelle TitouanVanBelle commented May 5, 2017

This PR brings contextual And handling as suggested in #41

Given a scenario with the following steps

Given a given state
And another given state

The second step another given state can be defined the following ways

Given(@"another given state", ^(NSArray<NSString *> *args, NSDictionary *userInfo) {});
And(@"another given state", ^(NSArray<NSString *> *args, NSDictionary *userInfo) {});

@brentleyjones
Copy link
Collaborator

So this essentially makes a statement like And(@"another given state", ^(NSArray<NSString *> *args, NSDictionary *userInfo) {}); a catch all And, correct? Would be good to update documentation to reflect this, and recommend that existing And or Match(@[x,@"And"] be migrated to just x.

@brentleyjones brentleyjones changed the base branch from master to develop May 5, 2017 14:08
@TitouanVanBelle
Copy link
Contributor Author

TitouanVanBelle commented May 7, 2017

Sorry what do you mean with 'a catch all And'? I updated the documentation locally but I'm not sure how to push it or create a PR for it. Any suggestion?

@brentleyjones
Copy link
Collaborator

Ahh, a deprecation is better than what I was thinking 👍.

@brentleyjones brentleyjones merged commit 5c88c7e into Ahmed-Ali:develop May 8, 2017
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