From e2e557976f1d68bf4d6a147ec484e5455f15eeef Mon Sep 17 00:00:00 2001 From: Paul Duffin Date: Tue, 20 Jun 2017 14:53:48 +0100 Subject: Cleanup a.t.TestGrouping TestGrouping is not part of the API but is used by classes that are and so will need to be included in the static library we ship as part of the SDK. As that library will be built directly from the source (as opposed to android.jar which is built from stubs) developers will be able to see classes and methods that are hidden and so not present in the stubs. This change makes TestGrouping and its members as inaccessible as possible. Unused methods and fields were removed and the TestGrouping class was simplified by passing the ClassLoader into the constructor, initializing the classLoader, making the classLoader field final, and removing the setClassLoader(ClassLoader) method. Bug: 30188076 Test: make checkbuild Change-Id: Ie0545b1a61b74cdbc78667417f360b5197e2e128 --- test-runner/tests/src/android/test/suitebuilder/TestGroupingTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'test-runner/tests') diff --git a/test-runner/tests/src/android/test/suitebuilder/TestGroupingTest.java b/test-runner/tests/src/android/test/suitebuilder/TestGroupingTest.java index f4477d1c780e..972bfb49e75f 100644 --- a/test-runner/tests/src/android/test/suitebuilder/TestGroupingTest.java +++ b/test-runner/tests/src/android/test/suitebuilder/TestGroupingTest.java @@ -30,7 +30,7 @@ public class TestGroupingTest extends TestCase { @Override protected void setUp() throws Exception { super.setUp(); - mGrouping = new TestGrouping(TestGrouping.SORT_BY_SIMPLE_NAME); + mGrouping = new TestGrouping(TestGrouping.SORT_BY_SIMPLE_NAME, getClass().getClassLoader()); } /** -- cgit v1.2.3-59-g8ed1b