Introduce `@Disabled` annotation to disable a `@Suite`

[mpkorstanje]

With JUnit 4 it was possible to ignore a suite. For example:

package io.cucumber.examples.calculator;

...

@Ignore
@RunWith(Cucumber.class)
public class RunCucumberTest {

}

This allowed people to disable a specific execution when running all tests and still have ability to run it manually from their IDE (ignoring the annotation).

With JUnit 5 people then imagine the equivalent would be

package io.cucumber.examples.calculator;

...

@Disabled
@Suite
@IncludeEngines("cucumber")
@SelectPackages("io.cucumber.examples.calculator")
@ConfigurationParameter(key = GLUE_PROPERTY_NAME, value = "io.cucumber.examples.calculator")
public class RunCucumberTest {
}

And then expect org.junit.jupiter.api.Disabled to work. Which naturally doesn't work because ExecutionCondition is a JUpiter extension, which do not apply to Suite engines.

Originally reported as cucumber/cucumber-jvm#3033 and as indicated there this feature request is not urgent. I think we can let this sit to gauge interest.

Deliverables

• Add org.junit.platform.suite.api.Disabled and implement in the Suite Engine.

[Ardenntu]

Hi @mpkorstanje! I'm interested in becoming a new contributor to JUnit 5.

I noticed this issue hasn't been assigned yet. Would it be possible for me to work on this as my first contribution? I understand this involves:

• Adding org.junit.platform.suite.api.Disabled annotation
• Implementing support in the Suite Engine

I'm willing to follow the contribution guidelines and would appreciate any guidance you might have.

Thank you!

[mpkorstanje]

I'm not part of the JUnit team, I only contribute occasionally. So I can't make that decision.

The usual workflow is that the JUnit team has an internal discussion about the feature. Then depending on importance they'll pick it up themselves or label it as "up for grabs".

However, and as I indicated in the issue, the utility of this feature is uncertain. So I imagine they will want to wait for more people to show interest in this feature first.

[bankmonitor-gabor-viczai]

I'm interested.