Skip to content

Conversation

@mpkorstanje
Copy link
Contributor

@mpkorstanje mpkorstanje commented Jun 21, 2024

🤔 What's changed?

Include the pickle name in the test name, if the pickle was parameterized.

📋 Checklist:

mpkorstanje added a commit to cucumber/compatibility-kit that referenced this pull request Jun 21, 2024
When using a Scenario Outline it is possible to use a placeholder in the
name of that Scenario Outline. The resulting pickle will have the
placeholder replaced with a value from the example tables.

This allows us to test use cases where this matter such as:
- cucumber/junit-xml-formatter#34
- cucumber/testng-xml-formatter#4

Fixes: #96
mpkorstanje added a commit to cucumber/compatibility-kit that referenced this pull request Jun 22, 2024
When using a Scenario Outline it is possible to use a placeholder in the
name of that Scenario Outline. The resulting pickle will have the
placeholder replaced with a value from the example tables.

This allows us to test use cases where this matter such as:
- cucumber/junit-xml-formatter#34
- cucumber/testng-xml-formatter#4

Fixes: #96
mpkorstanje added a commit to cucumber/compatibility-kit that referenced this pull request Jun 22, 2024
When using a Scenario Outline it is possible to use a placeholder in the
name of that Scenario Outline. The resulting pickle will have the
placeholder replaced with a value from the example tables.

This allows us to test use cases where this matter such as:
- cucumber/junit-xml-formatter#34
- cucumber/testng-xml-formatter#4

Fixes: #96
@mpkorstanje mpkorstanje force-pushed the include-pickle-name-if-parameterized branch 4 times, most recently from d2b4d4c to f07adaf Compare June 22, 2024 12:51
Include the pickle name in the test name, if the pickle was parameterized.
@mpkorstanje mpkorstanje force-pushed the include-pickle-name-if-parameterized branch from f07adaf to 2df5b93 Compare June 22, 2024 13:24
@mpkorstanje mpkorstanje marked this pull request as ready for review June 22, 2024 13:51
@mpkorstanje mpkorstanje merged commit af58833 into main Jun 22, 2024
@mpkorstanje mpkorstanje deleted the include-pickle-name-if-parameterized branch June 22, 2024 13:51
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.

1 participant