bd96f408085701fe04fb46acdd3a101cfdebe064 |
|
22-Jun-2017 |
Paul Duffin <paulduffin@google.com> |
Clean up TestCaseUtil Part of the work of removing JUnit and dependent android.test classes from the Android API involves providing a static library that developers can include in their test applications to ease migration. That library will be built directly from the source (as opposed to android.jar which is built from stubs) and so developers will be able to see classes and methods that are not present in the stubs. This change is one of a number of similar changes that cleanup the existing non-API code in order to minimize the additional methods and classes exposed externally. The basic approach is to remove unused classes and methods, use least visible access modifier possible and generally minimize the amount of publicly visible code. TestCaseUtil.getTestCaseNames() is only used by tests but its tests did provide some coverage of the getTests() method so remove the method and the tests the method was simply moved into TestCaseUtilTest and the tests renamed to make it clearer that they are testing TestCaseUtil.getTests(). Similarly, TestCaseUtil.createTestSuite() was only used by tests but its tests did provide some coverage of the invokeSuiteMethodIfPossible() method so the tests were modified and renamed to preserve that coverage. TestCaseUtil.getTestAtIndex() was completely unused so was just removed. Bug: 30188076 Test: make checkbuild and ran FrameworkTestRunnerTests Change-Id: I62bbdbab428d7560f0c7df11f313fe60cfd31d13
/frameworks/base/test-runner/tests/src/android/test/TestCaseUtilTest.java
|
2a637cf9b0f2c2ebaf573a1f478e31dc1e6a8354 |
|
22-Jun-2017 |
Paul Duffin <paulduffin@google.com> |
Ensure that android.test tests pass It looks as though the tests in test-runner/tests have not actually been run successfully for over 7 years. As a result they have degraded. This change fixes the tests so that they will pass and provides instructions on how to run them. TestCaseUtilTest.testGetTestCaseNamesForTestSuiteWithSuiteMethod This fails because it expected 2 names to be returned but only returns 0. The reason for that is although TwoTestsInTestSuite has a Test suite() method that does create a TestSuite with two tests the TestCaseUtil method does not actually call suite(). Instead, because TwoTestsInTestSuite is a TestSuite it just calls the tests() method on it which returns an empty Enumeration because TwoTestsInTestSuite is empty. The support for "static Test suite() {}" is broken but fixing this will affect the behavior of InstrumentationTestRunner which is used in thousands of different places both in and outside Google and so could cause untold problems. TestSuiteBuilderTest.testIncludeAllPackagesUnderHere Reformat the list, one per line and then add missing name "testPublicConstructor". ErrorTest/FailingTest These tests are not meant to be run on their own, only as part of a separate test. The RunAsPartOfSeparateTest annotation was added to allow these to be excluded using notAnnotation as shown in the instructions for running the tests. Bug: 30188076 Test: followed new instructions in test-runner/tests/Android.mk Change-Id: I60e7bee9cd08a9ab7777a2578fc58da772de5c1f
/frameworks/base/test-runner/tests/src/android/test/TestCaseUtilTest.java
|
e70f61b1160e953e5e4d18d30a463fa9ba821779 |
|
19-Feb-2010 |
Brett Chabot <brettchabot@android.com> |
Retry test-runner tests move. This time change the frameworks makefile so it only includes test-runner/src in the public API.
/frameworks/base/test-runner/tests/src/android/test/TestCaseUtilTest.java
|
c1ca8c51c616d1517b18bf24feaa9f8f1430c835 |
|
19-Feb-2010 |
Brett Chabot <brettchabot@android.com> |
Revert "Move framework test-runner unit tests to be closer to their source." This reverts commit 12093976a4842a795491cfd2b1d3b71e18503f2d.
/frameworks/base/test-runner/tests/src/android/test/TestCaseUtilTest.java
|
12093976a4842a795491cfd2b1d3b71e18503f2d |
|
19-Feb-2010 |
Brett Chabot <brettchabot@android.com> |
Move framework test-runner unit tests to be closer to their source. Move the test-runner source into a separate src folder to accommodate the test move.
/frameworks/base/test-runner/tests/src/android/test/TestCaseUtilTest.java
|