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

Feature: Add used tags to the meta information #9

Closed
eisenreich opened this issue Nov 4, 2020 · 5 comments · Fixed by #13
Closed

Feature: Add used tags to the meta information #9

eisenreich opened this issue Nov 4, 2020 · 5 comments · Fixed by #13

Comments

@eisenreich
Copy link

For each scenario I add a tag to be able to exclude them in the test pipeline if some of them is flaky.

When now a error happens in the pipeline I always have to compare the whole string (name of the scenario) to find the appropriate test in the source code and retrieve the tag. If I use some variables, then it gets even harder.

What would help me a lot is, if the tags of a test case are added to a meta tag like: tags and the value is all meta tags concatenated like @TAG1,@TAG2.

Ref.: https://devexpress.github.io/testcafe/documentation/reference/test-api/fixture/meta.html

@Arthy000
Copy link
Owner

Arthy000 commented Nov 5, 2020

Hello,

Would it work for you if the "tags" metadata was an array instead of a string of concatenated tags ?
[ "@TAG1", "@TAG2" ]

@eisenreich
Copy link
Author

Hello,

if I understood the doc right, then in this object should only be a string placed:
image

@Arthy000
Copy link
Owner

Arthy000 commented Nov 5, 2020

True, my bad :D

Could you please provide an example of what you are trying to achieve, so I can test my changes?

Arthy000 pushed a commit that referenced this issue Nov 6, 2020
@Arthy000
Copy link
Owner

@eisenreich would you rather test the changes yourself? You can pull the branch and use it by changing the dependency to gherkin-testcafe in your package.json

@eisenreich
Copy link
Author

eisenreich commented Jan 7, 2021

I added the changes from 08c4cd2 manually and it works as expected.

Thank you very much and a happy new year! 🎉

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 a pull request may close this issue.

2 participants