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

Interpolate placeholders in tables #95

Merged
merged 2 commits into from
Mar 5, 2018

Conversation

marnen
Copy link
Contributor

@marnen marnen commented Mar 1, 2018

<Placeholders> in tables within scenario outlines were being treated as literal text. I've fixed that.


then_ ~r/^the table data should (?<negation>not )?contain "(?<string>[^"]+)"$/,
fn %{table_data: table_data} = state, %{negation: negation, string: string} ->
contains = (for %{data: data} <- table_data, do: data) |> Enum.any?(&contains?(&1, string))
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This actually won't quite work. Will fix in a few minutes.

@marnen
Copy link
Contributor Author

marnen commented Mar 1, 2018

OK, I think this is ready to merge now.

@marnen
Copy link
Contributor Author

marnen commented Mar 5, 2018

Looks like the red build was an error, not a failure. @meadsteve Can you restart that build job? I don't have the permissions to.

Copy link
Collaborator

@meadsteve meadsteve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍 I'll merge once the restarted build passes

@meadsteve meadsteve merged commit 902223e into meadery:master Mar 5, 2018
@meadsteve
Copy link
Collaborator

Nice. Thanks for this 💯 I'll try and get it released later today.

@marnen
Copy link
Contributor Author

marnen commented Mar 6, 2018

Great!

@marnen marnen deleted the placeholders-in-tables branch March 6, 2018 14:32
@meadsteve
Copy link
Collaborator

@marnen now available as https://hex.pm/packages/white_bread/4.1.1

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