summaryrefslogtreecommitdiff
path: root/apct-tests/perftests/multiuser
diff options
context:
space:
mode:
authorSudheer Shanka <sudheersai@google.com>2017-07-12 18:12:31 -0700
committerSudheer Shanka <sudheersai@google.com>2017-07-12 18:15:02 -0700
commit2d0278b09f89a13a0ec4be91fdde2e7a682c1197 (patch)
treeb0dce49cd47117fc42710f7a1aa86ffe893719ec /apct-tests/perftests/multiuser
parent363c238b5edf58029d46d3730ac78effe7f2b268 (diff)
Update user perf tests to report only mean.
Test: bit MultiUserPerfTests:android.multiuser.UserLifecycleTests Change-Id: Ib70123217c54b469844ce65ec52b7e7ed5b5998d
Diffstat (limited to 'apct-tests/perftests/multiuser')
-rw-r--r--apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResults.java8
-rw-r--r--apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResultsReporter.java8
-rw-r--r--apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkRunner.java8
-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,