From a1d701ef2f0302ad18ff356c697466f87214a924 Mon Sep 17 00:00:00 2001
From: Brett Chabot <>
Date: Tue, 31 Mar 2009 17:03:46 -0700
Subject: [PATCH] AI 143869: Change InstrumentationTestRunner so all tests in
 the application are run when no other arguments are supplied, instead of
 running only tests in the app's package.   BUG=1749513

Automated import of CL 143869
---
 test-runner/android/test/ClassPathPackageInfoSource.java |  5 ++++-
 test-runner/android/test/InstrumentationTestRunner.java  | 15 ++++++++-------
 2 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/test-runner/android/test/ClassPathPackageInfoSource.java b/test-runner/android/test/ClassPathPackageInfoSource.java
index 12bc7f3a00e4..877075f2d2af 100644
--- a/test-runner/android/test/ClassPathPackageInfoSource.java
+++ b/test-runner/android/test/ClassPathPackageInfoSource.java
@@ -226,8 +226,11 @@ public class ClassPathPackageInfoSource {
                 String className = apkClassNames.nextElement();
 
                 if (className.startsWith(packageName)) {
+                    String subPackageName = packageName;
                     int lastPackageSeparator = className.lastIndexOf('.');
-                    String subPackageName = className.substring(0, lastPackageSeparator);
+                    if (lastPackageSeparator > 0) {
+                        subPackageName = className.substring(0, lastPackageSeparator);
+                    }
                     if (subPackageName.length() > packageName.length()) {
                         subpackageNames.add(subPackageName);
                     } else if (isToplevelClass(className)) {
diff --git a/test-runner/android/test/InstrumentationTestRunner.java b/test-runner/android/test/InstrumentationTestRunner.java
index f038612cbd0e..044f55581a3d 100644
--- a/test-runner/android/test/InstrumentationTestRunner.java
+++ b/test-runner/android/test/InstrumentationTestRunner.java
@@ -300,16 +300,17 @@ public class InstrumentationTestRunner extends Instrumentation implements TestSu
         }
 
         if (testClassesArg == null) {
-            TestSuite testSuite = null;
             if (mPackageOfTests != null) {
                 testSuiteBuilder.includePackages(mPackageOfTests);
             } else {
-                testSuite = getTestSuite();
-                testSuiteBuilder.addTestSuite(testSuite);
-            }
-
-            if (testSuite == null) {
-                testSuiteBuilder.includePackages(getTargetContext().getPackageName());
+                TestSuite testSuite = getTestSuite();
+                if (testSuite != null) {
+                    testSuiteBuilder.addTestSuite(testSuite);
+                } else {
+                    // no package or class bundle arguments were supplied, and no test suite 
+                    // provided so add all tests in application
+                    testSuiteBuilder.includePackages("");
+                }
             }
         } else {
             parseTestClasses(testClassesArg, testSuiteBuilder);
-- 
2.11.0