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

@Ignore annotation migration #15

Open
Philzen opened this issue Jun 14, 2024 · 0 comments
Open

@Ignore annotation migration #15

Philzen opened this issue Jun 14, 2024 · 0 comments
Labels
Feature New feature or request must-have

Comments

@Philzen
Copy link
Owner

Philzen commented Jun 14, 2024

Migrate @Ignore to @Disabled (class-level and method-level).

@Ignore does not disable test execution on inner public static classes. As there is no known workaround (ideas welcome!), the recipe must not migrate a class-level annotation if it contains inner public static classes unless there is a also an @Ignore annotation on every inner class or every @Test within the inner classes.

To clarify if this behavior is intended (and potentially verify if it will stay that way) i've reported this over at testng-team/testng#3138

@Philzen Philzen added Feature New feature or request must-have labels Jun 14, 2024
@Philzen Philzen moved this to Ready in TestNG → Jupiter Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature or request must-have
Projects
Status: Ready
Development

No branches or pull requests

1 participant