summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMathieu Chartier <mathieuc@google.com>2020-03-06 09:19:42 -0800
committerMathieu Chartier <mathieuc@google.com>2020-03-06 09:19:42 -0800
commitd5d442ddd9b59528f84ae84b27b05795c04ef7cb (patch)
tree335cf56f991cf9bc59e796e35438af05accbcfd6
parentf134ea991dcc44b002d9126d8599c88ab3fe1766 (diff)
Make BootImageProfileTest less flaky
Number one flake is when the system server process is killed before SIGUSR1. Change this case to be non fatal. Increase number of retry iterations in case the package manager is very slow to start and get a JIT sample. Hopefully fixes the other flake where PackageManagerService.<init> is not seen in the profile. Test: atest BootImageProfileTest Bug: 149802360 Change-Id: I52ac80f15d8efe152dea81b75a82faa4da1b8600
-rw-r--r--tests/BootImageProfileTest/src/com/android/bootimageprofile/BootImageProfileTest.java5
1 files changed, 2 insertions, 3 deletions
diff --git a/tests/BootImageProfileTest/src/com/android/bootimageprofile/BootImageProfileTest.java b/tests/BootImageProfileTest/src/com/android/bootimageprofile/BootImageProfileTest.java
index 61f3dbaff1a3..4ecca2dc4c39 100644
--- a/tests/BootImageProfileTest/src/com/android/bootimageprofile/BootImageProfileTest.java
+++ b/tests/BootImageProfileTest/src/com/android/bootimageprofile/BootImageProfileTest.java
@@ -78,13 +78,12 @@ public class BootImageProfileTest implements IDeviceTest {
return false;
}
String res = mTestDevice.executeShellCommand("kill -s SIGUSR1 " + pid).trim();
- assertTrue("kill SIGUSR1: " + res, res.length() == 0);
- return true;
+ return res.length() == 0;
}
@Test
public void testSystemServerProfile() throws Exception {
- final int numIterations = 20;
+ final int numIterations = 30;
String res;
// Set properties and wait for them to be readable.
for (int i = 1; i <= numIterations; ++i) {