diff options
author | Mathieu Chartier <mathieuc@google.com> | 2020-03-06 09:19:42 -0800 |
---|---|---|
committer | Mathieu Chartier <mathieuc@google.com> | 2020-03-06 09:19:42 -0800 |
commit | d5d442ddd9b59528f84ae84b27b05795c04ef7cb (patch) | |
tree | 335cf56f991cf9bc59e796e35438af05accbcfd6 | |
parent | f134ea991dcc44b002d9126d8599c88ab3fe1766 (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.java | 5 |
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) { |