aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/java_tools/junitrunner/java/com/google/testing/junit/runner/junit4/JUnit4TestModelBuilder.java
diff options
context:
space:
mode:
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.java13
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);
}
}