diff options
Diffstat (limited to 'src/java_tools/junitrunner/java/com/google/testing/junit/runner/junit4/JUnit4TestModelBuilder.java')
-rw-r--r-- | src/java_tools/junitrunner/java/com/google/testing/junit/runner/junit4/JUnit4TestModelBuilder.java | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/src/java_tools/junitrunner/java/com/google/testing/junit/runner/junit4/JUnit4TestModelBuilder.java b/src/java_tools/junitrunner/java/com/google/testing/junit/runner/junit4/JUnit4TestModelBuilder.java index 56d57c1d86..75199cf144 100644 --- a/src/java_tools/junitrunner/java/com/google/testing/junit/runner/junit4/JUnit4TestModelBuilder.java +++ b/src/java_tools/junitrunner/java/com/google/testing/junit/runner/junit4/JUnit4TestModelBuilder.java @@ -17,12 +17,10 @@ package com.google.testing.junit.runner.junit4; import com.google.testing.junit.runner.model.TestSuiteModel; import com.google.testing.junit.runner.model.TestSuiteModel.Builder; import com.google.testing.junit.runner.util.Supplier; - -import org.junit.runner.Description; -import org.junit.runner.Request; - import javax.inject.Inject; import javax.inject.Singleton; +import org.junit.runner.Description; +import org.junit.runner.Request; /** * Builds a {@link TestSuiteModel} for JUnit4 tests. @@ -49,9 +47,12 @@ class JUnit4TestModelBuilder implements Supplier<TestSuiteModel> { @Override public TestSuiteModel get() { Description root = request.getRunner().getDescription(); + // A test class annotated with @Ignore effectively has no test methods, + // which is what isSuite() tests for. if (!root.isSuite()) { - throw new IllegalArgumentException("Top test must be a suite"); + return builder.build(suiteName); + } else { + return builder.build(suiteName, root); } - return builder.build(suiteName, root); } } |