A JUnit Rule to Conditionally Ignore Tests

I always believed that using @Ignore to deactivate tests is a bad idea. Except, maybe as one way to put tests that fail intermittently into quarantine to attend to them later (as Martin Fowler describes it here). This bears the danger that the test suite decays as more and more tests keep getting ignored and forgotten. Therefore you should have a policy in place to ensure that tests aren’t quarantined for too long. Well, so I thought until recently when we ran into this:

In a project that Frank and I work on, we ran into an SWT issue described here.
On non-Windows platforms, asserting whether an SWT widget has got the input focus does not work with automated tests.

We decided to ignore focus-related tests on non-Windows platforms for now. Though our build server runs on Linux we found it safe enough as both our development environments run on Windows.

In JUnit, assumptions are the means to skip tests that aren’t meaningful under the given condition. Expressed that way, our test would look like this:

public void testFocus() {
  assumeTrue( isRunningOnWindows() );
  // ...
}

But we didn’t want the test code mingled with conditions if it should be executed at all. The code that decides whether the test is ignored should be separated from the test code itself.

This led us to creating a ConditionalIgnore annotation and a corresponding rule to hook it into the JUnit runtime. The thing is simple and best explained with an example:

public class SomeTest {
  @Rule
  public ConditionalIgnoreRule rule = new ConditionalIgnoreRule();

  @Test
  @ConditionalIgnore( condition = NotRunningOnWindows.class )
  public void testFocus() {
    // ...
  }
}

public class NotRunningOnWindows implements IgnoreCondition {
  public boolean isSatisfied() {
    return !System.getProperty( "os.name" ).startsWith( "Windows" );
  }
}

The ConditionalIgnore annotation requires a ‘condition’ property that points to a class that implements IgnoreContition. At runtime, an instance of the IgnoreCondition implementation is created and its isSatisfied() method decides whether the test is ignored (returns true) or not (returns false). Finally there is an IgnoreConditionRule that hooks the annotations into the JUnit runtime.

If the IgnoreCondition implementation decides to ignore a test case, an AssumptionViolatedException is thrown. Therefore the ConditionalIgnore annotation has the same effect as if an Assume condition would return false. With a slight difference that we consider an advantage: @Before and @After methods are not executed for ignored tests.

The source code of the rule and its related classes can be found here.

The remaining issue with Assume is that it affects the test statistics. If an Assume condition is found to be false, it treats the test as having passed even though it hasn’t run. To overcome that, you’d have to provide your own runner that handles AssumptionViolatedException the way you want.

Even though I just wrote about ignoring tests in length, I am still convinced that tests at best should not be ignored and if so only in exceptional cases.

Rüdiger Herrmann

Rüdiger started writing software more than 20 years ago. From the very beginning he focused on a pragmatic approach to writing and delivering software.

Currently he is co-leading the Rich Ajax Platform (RAP) Project and
as one of the initial committers on the RAP project, Rüdiger was responsible for the development of RWT, the RAP side of SWT (Standard Widget Toolkit). His interests include promoting test driven development and agile methods to deliver clean code that works.

(EMail: rherrmann@codeaffine.com)

Latest posts by Rüdiger Herrmann (see all)

5 Responses to “A JUnit Rule to Conditionally Ignore Tests”

  • Tomek says:

    Looks like a good example of JUnit rule usage! Thanks for sharing!


    Tomek Kaczanowski
    http://practicalunittesting.com

  • Vijaya says:

    By Using the above sample, if a testcase is ignored, will that appear in Skipped one, if so, how to make sure that test case is ignored and does not appear in Skipped one

    • Rüdiger Herrmann says:

      Vijaya,

      if you refer to Maven ‘skipped’ tests, then yes, the maven test runner lists ignored tests as skipped. But as I said above, how assumption violations are handled is up to the test runner. Does that answer your question?

      Rüdiger

  • Matt Morrissette says:

    The above example can cause an error using the referenced gist on GitHub for the ConditionalIgnoreRule if your class that extends IgnoreCondition is declared inside your test case.

    Please use the gist at:
    https://gist.github.com/yinzara/9980184

    • Rüdiger Herrmann says:

      Good catch, Matt. Thank you for this hint. I have also updated the original gist so that readers who don’t follow down to the comments benefit from your enhancement.


Leave a Reply