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] Rename @TmsLink annotation to @Tms #987

Open
2 of 4 tasks
harmin-parra opened this issue Dec 27, 2023 · 2 comments
Open
2 of 4 tasks

🐞: [Feature] Rename @TmsLink annotation to @Tms #987

harmin-parra opened this issue Dec 27, 2023 · 2 comments
Labels
triage type:bug Something isn't working

Comments

@harmin-parra
Copy link

harmin-parra commented Dec 27, 2023

What happened?

I'm submitting a ...

  • bug report
  • feature request
  • support request => Please do not submit support request here, see note at the top of this template.

What is the current behavior?

Allure API implementations use different names for TMS links:

JUnit: @TmsLink

Cucumber-JVM: Allure.tms

NUnit: [AllureTms]

Behave: @allure.tms

Playwright: allure.tms

Jest: allure.tms

Cucumber.js and Mocha: this.tms

What is the expected behavior?

For the sake of coherence, could we rename the @TmsLink annotation to @Tms for Java JUnit ?

What is the motivation / use case for changing the behavior?

To have a coherent API

Please tell us about your environment:

  • Allure version: 2.25.0
  • Test framework: junit 5.10.1
  • Allure adaptor: allure-junit5 2.25.0

Other information

What Allure Integration are you using?

allure-junit5

What version of Allure Integration you are using?

2.25.0

What version of Allure Report you are using?

2.25.0

Code of Conduct

  • I agree to follow this project's Code of Conduct
@harmin-parra
Copy link
Author

See also allure-python/issues/786

@Scot3004
Copy link

I do have a question around this issue, since I'm seeing that currently on allure-cucumber7-jvm We have a class named LabelBuilder  where is possible to map the annotation vs the tms link

The thing is that ZephyrReport uses instead @TestCaseKey=ABC-T123

https://support.smartbear.com/zephyr-scale-cloud/docs/en/test-automation/cucumber-integration.html

And maybe it would be good to have the possibility to configure the Tag that will be used on the tests, because for example in my team we are comfortable with the provided @TestCaseKey=ABC-T123 and I don't feel that make full refactor to adapt the current implementation to support the allure automatic link with @TMSLink=ABC-T123 is viable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage type:bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants