diff options
author | Sudheer Shanka <sudheersai@google.com> | 2017-07-12 18:12:31 -0700 |
---|---|---|
committer | Sudheer Shanka <sudheersai@google.com> | 2017-07-12 18:15:02 -0700 |
commit | 2d0278b09f89a13a0ec4be91fdde2e7a682c1197 (patch) | |
tree | b0dce49cd47117fc42710f7a1aa86ffe893719ec /apct-tests | |
parent | 363c238b5edf58029d46d3730ac78effe7f2b268 (diff) |
Update user perf tests to report only mean.
Test: bit MultiUserPerfTests:android.multiuser.UserLifecycleTests
Change-Id: Ib70123217c54b469844ce65ec52b7e7ed5b5998d
Diffstat (limited to 'apct-tests')
-rw-r--r-- | apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResults.java | 8 | ||||
-rw-r--r-- | apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResultsReporter.java | 8 | ||||
-rw-r--r-- | apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkRunner.java | 8 | ||||
-rw-r--r-- | apct-tests/perftests/multiuser/src/android/multiuser/UserLifecycleTests.java (renamed from apct-tests/perftests/multiuser/src/android/multiuser/UserLifecycleTest.java) | 12 |
4 files changed, 25 insertions, 11 deletions
diff --git a/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResults.java b/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResults.java index 0d764ce29c74..e417ca791c45 100644 --- a/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResults.java +++ b/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResults.java @@ -28,7 +28,13 @@ public class BenchmarkResults { mResults.add(TimeUnit.NANOSECONDS.toMillis(duration)); } - public Bundle getStats() { + public Bundle getStatsToReport() { + final Bundle stats = new Bundle(); + stats.putDouble("Mean (ms)", mean()); + return stats; + } + + public Bundle getStatsToLog() { final Bundle stats = new Bundle(); stats.putDouble("Mean (ms)", mean()); stats.putDouble("Median (ms)", median()); diff --git a/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResultsReporter.java b/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResultsReporter.java index 7472865e9a5a..d3a3ce54e378 100644 --- a/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResultsReporter.java +++ b/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResultsReporter.java @@ -40,9 +40,11 @@ public class BenchmarkResultsReporter implements TestRule { @Override public void evaluate() throws Throwable { base.evaluate(); - final Bundle stats = mRunner.getStats(); - final String summary = getSummaryString(description.getMethodName(), stats); - logSummary(description.getTestClass().getSimpleName(), summary, mRunner.getAllDurations()); + final Bundle stats = mRunner.getStatsToReport(); + final String summary = getSummaryString(description.getMethodName(), + mRunner.getStatsToLog()); + logSummary(description.getTestClass().getSimpleName(), summary, + mRunner.getAllDurations()); stats.putString(Instrumentation.REPORT_KEY_STREAMRESULT, summary); InstrumentationRegistry.getInstrumentation().sendStatus( Activity.RESULT_OK, stats); diff --git a/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkRunner.java b/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkRunner.java index ccadc9a8f6a9..c7bebf3847fe 100644 --- a/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkRunner.java +++ b/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkRunner.java @@ -93,8 +93,12 @@ public class BenchmarkRunner { mState = RUNNING; } - public Bundle getStats() { - return mResults.getStats(); + public Bundle getStatsToReport() { + return mResults.getStatsToReport(); + } + + public Bundle getStatsToLog() { + return mResults.getStatsToLog(); } public ArrayList<Long> getAllDurations() { diff --git a/apct-tests/perftests/multiuser/src/android/multiuser/UserLifecycleTest.java b/apct-tests/perftests/multiuser/src/android/multiuser/UserLifecycleTests.java index 6e802a9fc0c8..855be0859520 100644 --- a/apct-tests/perftests/multiuser/src/android/multiuser/UserLifecycleTest.java +++ b/apct-tests/perftests/multiuser/src/android/multiuser/UserLifecycleTests.java @@ -49,19 +49,21 @@ import java.util.concurrent.TimeUnit; * make MultiUserPerfTests && * adb install -r \ * ${ANDROID_PRODUCT_OUT}/data/app/MultiUserPerfTests/MultiUserPerfTests.apk && - * adb shell am instrument -e class android.multiuser.UserLifecycleTest \ + * adb shell am instrument -e class android.multiuser.UserLifecycleTests \ * -w com.android.perftests.multiuser/android.support.test.runner.AndroidJUnitRunner * * or * - * bit MultiUserPerfTests:android.multiuser.UserLifecycleTest + * bit MultiUserPerfTests:android.multiuser.UserLifecycleTests * * Note: If you use bit for running the tests, benchmark results won't be printed on the host side. - * But in either case, results can be checked on the device side 'adb logcat -s UserLifecycleTest' + * But in either case, results can be checked on the device side 'adb logcat -s UserLifecycleTests' */ @LargeTest @RunWith(AndroidJUnit4.class) -public class UserLifecycleTest { +public class UserLifecycleTests { + private static final String TAG = UserLifecycleTests.class.getSimpleName(); + private final int TIMEOUT_IN_SECOND = 30; private final int CHECK_USER_REMOVED_INTERVAL_MS = 200; @@ -276,7 +278,7 @@ public class UserLifecycleTest { bootCompleteLatch.countDown(); } } - }, "UserLifecycleTest"); + }, TAG); } private void registerBroadcastReceiver(final String action, final CountDownLatch latch, |