diff options
author | Jeff Sharkey <jsharkey@android.com> | 2017-11-27 15:02:10 -0700 |
---|---|---|
committer | Jeff Sharkey <jsharkey@android.com> | 2017-11-27 17:07:40 -0700 |
commit | c204c223a77237e13d084d58590acb16b025c676 (patch) | |
tree | 5c9a11890a222d60deabfccef5292c3a63d9f11f /test-runner | |
parent | 71b4882da60867e948bbaaf86c03066bc07ccc85 (diff) |
Deeper sanity checking of emitted types.
When emitting a method or field, verify that we're able to reference
all mentioned types. This ensures that we don't accidentally
reference undefined classes/interfaces.
Test: manual inspection of API files
Bug: 69791141
Change-Id: I84e0c87fe83daa118661f61dbdf17b58ea5282d4
Diffstat (limited to 'test-runner')
-rw-r--r-- | test-runner/api/android-test-runner-current.txt | 2 | ||||
-rw-r--r-- | test-runner/src/android/test/suitebuilder/TestSuiteBuilder.java | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/test-runner/api/android-test-runner-current.txt b/test-runner/api/android-test-runner-current.txt index 905cfe701ab6..1170eb53ab7f 100644 --- a/test-runner/api/android-test-runner-current.txt +++ b/test-runner/api/android-test-runner-current.txt @@ -271,8 +271,6 @@ package android.test.suitebuilder { public deprecated class TestSuiteBuilder { ctor public TestSuiteBuilder(java.lang.Class); ctor public TestSuiteBuilder(java.lang.String, java.lang.ClassLoader); - method public android.test.suitebuilder.TestSuiteBuilder addRequirements(java.util.List<com.android.internal.util.Predicate<android.test.suitebuilder.TestMethod>>); - method public final android.test.suitebuilder.TestSuiteBuilder addRequirements(com.android.internal.util.Predicate<android.test.suitebuilder.TestMethod>...); method public final junit.framework.TestSuite build(); method public android.test.suitebuilder.TestSuiteBuilder excludePackages(java.lang.String...); method protected java.lang.String getSuiteName(); diff --git a/test-runner/src/android/test/suitebuilder/TestSuiteBuilder.java b/test-runner/src/android/test/suitebuilder/TestSuiteBuilder.java index 6158e0cf14f4..2857696ef2ff 100644 --- a/test-runner/src/android/test/suitebuilder/TestSuiteBuilder.java +++ b/test-runner/src/android/test/suitebuilder/TestSuiteBuilder.java @@ -119,6 +119,7 @@ public class TestSuiteBuilder { * * @param predicates Predicates to add to the list of requirements. * @return The builder for method chaining. + * @hide */ public TestSuiteBuilder addRequirements(List<Predicate<TestMethod>> predicates) { this.predicates.addAll(predicates); @@ -156,7 +157,7 @@ public class TestSuiteBuilder { /** * Override the default name for the suite being built. This should generally be called if you - * call {@link #addRequirements(com.android.internal.util.Predicate[])} to make it clear which + * call {@code addRequirements(com.android.internal.util.Predicate[])} to make it clear which * tests will be included. The name you specify is automatically prefixed with the package * containing the tests to be run. If more than one package is specified, the first is used. * @@ -215,6 +216,7 @@ public class TestSuiteBuilder { * * @param predicates Predicates to add to the list of requirements. * @return The builder for method chaining. + * @hide */ public final TestSuiteBuilder addRequirements(Predicate<TestMethod>... predicates) { ArrayList<Predicate<TestMethod>> list = new ArrayList<Predicate<TestMethod>>(); |