Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2019 The Android Open Source Project |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
| 16 | |
| 17 | package com.android.server; |
| 18 | |
| 19 | import static com.android.server.PackageWatchdog.TRIGGER_FAILURE_COUNT; |
| 20 | |
| 21 | import static org.junit.Assert.assertEquals; |
| 22 | import static org.junit.Assert.assertNull; |
| 23 | import static org.junit.Assert.assertTrue; |
| 24 | |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 25 | import android.content.pm.VersionedPackage; |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 26 | import android.os.test.TestLooper; |
Brett Chabot | 502ec7a | 2019-03-01 14:43:20 -0800 | [diff] [blame] | 27 | |
| 28 | import androidx.test.InstrumentationRegistry; |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 29 | |
| 30 | import com.android.server.PackageWatchdog.PackageHealthObserver; |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 31 | import com.android.server.PackageWatchdog.PackageHealthObserverImpact; |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 32 | |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 33 | import org.junit.Before; |
| 34 | import org.junit.Test; |
| 35 | |
| 36 | import java.io.File; |
| 37 | import java.util.ArrayList; |
| 38 | import java.util.Arrays; |
| 39 | import java.util.List; |
| 40 | import java.util.concurrent.TimeUnit; |
| 41 | |
| 42 | // TODO(zezeozue): Write test without using PackageWatchdog#getPackages. Just rely on |
| 43 | // behavior of observers receiving crash notifications or not to determine if it's registered |
| 44 | /** |
| 45 | * Test PackageWatchdog. |
| 46 | */ |
| 47 | public class PackageWatchdogTest { |
| 48 | private static final String APP_A = "com.package.a"; |
| 49 | private static final String APP_B = "com.package.b"; |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 50 | private static final String APP_C = "com.package.c"; |
| 51 | private static final String APP_D = "com.package.d"; |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 52 | private static final long VERSION_CODE = 1L; |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 53 | private static final String OBSERVER_NAME_1 = "observer1"; |
| 54 | private static final String OBSERVER_NAME_2 = "observer2"; |
| 55 | private static final String OBSERVER_NAME_3 = "observer3"; |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 56 | private static final String OBSERVER_NAME_4 = "observer4"; |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 57 | private static final long SHORT_DURATION = TimeUnit.SECONDS.toMillis(1); |
| 58 | private static final long LONG_DURATION = TimeUnit.SECONDS.toMillis(5); |
| 59 | private TestLooper mTestLooper; |
| 60 | |
| 61 | @Before |
| 62 | public void setUp() throws Exception { |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 63 | new File(InstrumentationRegistry.getContext().getFilesDir(), |
| 64 | "package-watchdog.xml").delete(); |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 65 | mTestLooper = new TestLooper(); |
| 66 | mTestLooper.startAutoDispatch(); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 67 | } |
| 68 | |
| 69 | /** |
| 70 | * Test registration, unregistration, package expiry and duration reduction |
| 71 | */ |
| 72 | @Test |
| 73 | public void testRegistration() throws Exception { |
| 74 | PackageWatchdog watchdog = createWatchdog(); |
| 75 | TestObserver observer1 = new TestObserver(OBSERVER_NAME_1); |
| 76 | TestObserver observer2 = new TestObserver(OBSERVER_NAME_2); |
| 77 | TestObserver observer3 = new TestObserver(OBSERVER_NAME_3); |
| 78 | |
| 79 | // Start observing for observer1 which will be unregistered |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 80 | watchdog.startObservingHealth(observer1, Arrays.asList(APP_A), SHORT_DURATION, false); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 81 | // Start observing for observer2 which will expire |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 82 | watchdog.startObservingHealth(observer2, Arrays.asList(APP_A, APP_B), SHORT_DURATION, |
| 83 | false); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 84 | // Start observing for observer3 which will have expiry duration reduced |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 85 | watchdog.startObservingHealth(observer3, Arrays.asList(APP_A), LONG_DURATION, false); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 86 | |
| 87 | // Verify packages observed at start |
| 88 | // 1 |
| 89 | assertEquals(1, watchdog.getPackages(observer1).size()); |
| 90 | assertTrue(watchdog.getPackages(observer1).contains(APP_A)); |
| 91 | // 2 |
| 92 | assertEquals(2, watchdog.getPackages(observer2).size()); |
| 93 | assertTrue(watchdog.getPackages(observer2).contains(APP_A)); |
| 94 | assertTrue(watchdog.getPackages(observer2).contains(APP_B)); |
| 95 | // 3 |
| 96 | assertEquals(1, watchdog.getPackages(observer3).size()); |
| 97 | assertTrue(watchdog.getPackages(observer3).contains(APP_A)); |
| 98 | |
| 99 | // Then unregister observer1 |
| 100 | watchdog.unregisterHealthObserver(observer1); |
| 101 | |
| 102 | // Verify observer2 and observer3 left |
| 103 | // 1 |
| 104 | assertNull(watchdog.getPackages(observer1)); |
| 105 | // 2 |
| 106 | assertEquals(2, watchdog.getPackages(observer2).size()); |
| 107 | assertTrue(watchdog.getPackages(observer2).contains(APP_A)); |
| 108 | assertTrue(watchdog.getPackages(observer2).contains(APP_B)); |
| 109 | // 3 |
| 110 | assertEquals(1, watchdog.getPackages(observer3).size()); |
| 111 | assertTrue(watchdog.getPackages(observer3).contains(APP_A)); |
| 112 | |
| 113 | // Then advance time a little and run messages in Handlers so observer2 expires |
| 114 | Thread.sleep(SHORT_DURATION); |
| 115 | mTestLooper.dispatchAll(); |
| 116 | |
| 117 | // Verify observer3 left with reduced expiry duration |
| 118 | // 1 |
| 119 | assertNull(watchdog.getPackages(observer1)); |
| 120 | // 2 |
| 121 | assertNull(watchdog.getPackages(observer2)); |
| 122 | // 3 |
| 123 | assertEquals(1, watchdog.getPackages(observer3).size()); |
| 124 | assertTrue(watchdog.getPackages(observer3).contains(APP_A)); |
| 125 | |
| 126 | // Then advance time some more and run messages in Handlers so observer3 expires |
| 127 | Thread.sleep(LONG_DURATION); |
| 128 | mTestLooper.dispatchAll(); |
| 129 | |
| 130 | // Verify observer3 expired |
| 131 | // 1 |
| 132 | assertNull(watchdog.getPackages(observer1)); |
| 133 | // 2 |
| 134 | assertNull(watchdog.getPackages(observer2)); |
| 135 | // 3 |
| 136 | assertNull(watchdog.getPackages(observer3)); |
| 137 | } |
| 138 | |
| 139 | /** |
| 140 | * Test package observers are persisted and loaded on startup |
| 141 | */ |
| 142 | @Test |
| 143 | public void testPersistence() throws Exception { |
| 144 | PackageWatchdog watchdog1 = createWatchdog(); |
| 145 | TestObserver observer1 = new TestObserver(OBSERVER_NAME_1); |
| 146 | TestObserver observer2 = new TestObserver(OBSERVER_NAME_2); |
| 147 | |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 148 | watchdog1.startObservingHealth(observer1, Arrays.asList(APP_A), SHORT_DURATION, false); |
| 149 | watchdog1.startObservingHealth(observer2, Arrays.asList(APP_A, APP_B), SHORT_DURATION, |
| 150 | false); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 151 | |
| 152 | // Verify 2 observers are registered and saved internally |
| 153 | // 1 |
| 154 | assertEquals(1, watchdog1.getPackages(observer1).size()); |
| 155 | assertTrue(watchdog1.getPackages(observer1).contains(APP_A)); |
| 156 | // 2 |
| 157 | assertEquals(2, watchdog1.getPackages(observer2).size()); |
| 158 | assertTrue(watchdog1.getPackages(observer2).contains(APP_A)); |
| 159 | assertTrue(watchdog1.getPackages(observer2).contains(APP_B)); |
| 160 | |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 161 | // Then advance time and run IO Handler so file is saved |
| 162 | mTestLooper.dispatchAll(); |
| 163 | |
| 164 | // Then start a new watchdog |
| 165 | PackageWatchdog watchdog2 = createWatchdog(); |
| 166 | |
| 167 | // Verify the new watchdog loads observers on startup but nothing registered |
| 168 | assertEquals(0, watchdog2.getPackages(observer1).size()); |
| 169 | assertEquals(0, watchdog2.getPackages(observer2).size()); |
| 170 | // Verify random observer not saved returns null |
| 171 | assertNull(watchdog2.getPackages(new TestObserver(OBSERVER_NAME_3))); |
| 172 | |
| 173 | // Then regiser observer1 |
| 174 | watchdog2.registerHealthObserver(observer1); |
| 175 | watchdog2.registerHealthObserver(observer2); |
| 176 | |
| 177 | // Verify 2 observers are registered after reload |
| 178 | // 1 |
| 179 | assertEquals(1, watchdog1.getPackages(observer1).size()); |
| 180 | assertTrue(watchdog1.getPackages(observer1).contains(APP_A)); |
| 181 | // 2 |
| 182 | assertEquals(2, watchdog1.getPackages(observer2).size()); |
| 183 | assertTrue(watchdog1.getPackages(observer2).contains(APP_A)); |
| 184 | assertTrue(watchdog1.getPackages(observer2).contains(APP_B)); |
| 185 | } |
| 186 | |
| 187 | /** |
| 188 | * Test package failure under threshold does not notify observers |
| 189 | */ |
| 190 | @Test |
| 191 | public void testNoPackageFailureBeforeThreshold() throws Exception { |
| 192 | PackageWatchdog watchdog = createWatchdog(); |
| 193 | TestObserver observer1 = new TestObserver(OBSERVER_NAME_1); |
| 194 | TestObserver observer2 = new TestObserver(OBSERVER_NAME_2); |
| 195 | |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 196 | watchdog.startObservingHealth(observer2, Arrays.asList(APP_A), SHORT_DURATION, false); |
| 197 | watchdog.startObservingHealth(observer1, Arrays.asList(APP_A), SHORT_DURATION, false); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 198 | |
| 199 | // Then fail APP_A below the threshold |
| 200 | for (int i = 0; i < TRIGGER_FAILURE_COUNT - 1; i++) { |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 201 | watchdog.onPackageFailure(Arrays.asList(new VersionedPackage(APP_A, VERSION_CODE))); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 202 | } |
| 203 | |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 204 | // Run handler so package failures are dispatched to observers |
| 205 | mTestLooper.dispatchAll(); |
| 206 | |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 207 | // Verify that observers are not notified |
| 208 | assertEquals(0, observer1.mFailedPackages.size()); |
| 209 | assertEquals(0, observer2.mFailedPackages.size()); |
| 210 | } |
| 211 | |
| 212 | /** |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 213 | * Test package failure and does not notify any observer because they are not observing |
| 214 | * the failed packages. |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 215 | */ |
| 216 | @Test |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 217 | public void testPackageFailureDifferentPackageNotifyNone() throws Exception { |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 218 | PackageWatchdog watchdog = createWatchdog(); |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 219 | TestObserver observer1 = new TestObserver(OBSERVER_NAME_1); |
| 220 | TestObserver observer2 = new TestObserver(OBSERVER_NAME_2); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 221 | |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 222 | |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 223 | watchdog.startObservingHealth(observer2, Arrays.asList(APP_A), SHORT_DURATION, false); |
| 224 | watchdog.startObservingHealth(observer1, Arrays.asList(APP_B), SHORT_DURATION, false); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 225 | |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 226 | // Then fail APP_C (not observed) above the threshold |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 227 | for (int i = 0; i < TRIGGER_FAILURE_COUNT; i++) { |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 228 | watchdog.onPackageFailure(Arrays.asList(new VersionedPackage(APP_C, VERSION_CODE))); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 229 | } |
| 230 | |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 231 | // Run handler so package failures are dispatched to observers |
| 232 | mTestLooper.dispatchAll(); |
| 233 | |
| 234 | // Verify that observers are not notified |
| 235 | assertEquals(0, observer1.mFailedPackages.size()); |
| 236 | assertEquals(0, observer2.mFailedPackages.size()); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 237 | } |
| 238 | |
| 239 | /** |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 240 | * Test package failure and does not notify any observer because the failed package version |
| 241 | * does not match the available rollback-from-version. |
| 242 | */ |
| 243 | @Test |
| 244 | public void testPackageFailureDifferentVersionNotifyNone() throws Exception { |
| 245 | PackageWatchdog watchdog = createWatchdog(); |
| 246 | long differentVersionCode = 2L; |
| 247 | TestObserver observer = new TestObserver(OBSERVER_NAME_1) { |
Zimuzo | 71d931e | 2019-02-01 13:08:16 +0000 | [diff] [blame] | 248 | @Override |
| 249 | public int onHealthCheckFailed(VersionedPackage versionedPackage) { |
| 250 | if (versionedPackage.getVersionCode() == VERSION_CODE) { |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 251 | // Only rollback for specific versionCode |
| 252 | return PackageHealthObserverImpact.USER_IMPACT_MEDIUM; |
| 253 | } |
| 254 | return PackageHealthObserverImpact.USER_IMPACT_NONE; |
| 255 | } |
| 256 | }; |
| 257 | |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 258 | watchdog.startObservingHealth(observer, Arrays.asList(APP_A), SHORT_DURATION, false); |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 259 | |
| 260 | // Then fail APP_A (different version) above the threshold |
| 261 | for (int i = 0; i < TRIGGER_FAILURE_COUNT; i++) { |
| 262 | watchdog.onPackageFailure(Arrays.asList( |
| 263 | new VersionedPackage(APP_A, differentVersionCode))); |
| 264 | } |
| 265 | |
| 266 | // Run handler so package failures are dispatched to observers |
| 267 | mTestLooper.dispatchAll(); |
| 268 | |
| 269 | // Verify that observers are not notified |
| 270 | assertEquals(0, observer.mFailedPackages.size()); |
| 271 | } |
| 272 | |
| 273 | |
| 274 | /** |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 275 | * Test package failure and notifies only least impact observers. |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 276 | */ |
| 277 | @Test |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 278 | public void testPackageFailureNotifyAllDifferentImpacts() throws Exception { |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 279 | PackageWatchdog watchdog = createWatchdog(); |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 280 | TestObserver observerNone = new TestObserver(OBSERVER_NAME_1, |
| 281 | PackageHealthObserverImpact.USER_IMPACT_NONE); |
| 282 | TestObserver observerHigh = new TestObserver(OBSERVER_NAME_2, |
| 283 | PackageHealthObserverImpact.USER_IMPACT_HIGH); |
| 284 | TestObserver observerMid = new TestObserver(OBSERVER_NAME_3, |
| 285 | PackageHealthObserverImpact.USER_IMPACT_MEDIUM); |
| 286 | TestObserver observerLow = new TestObserver(OBSERVER_NAME_4, |
| 287 | PackageHealthObserverImpact.USER_IMPACT_LOW); |
| 288 | |
| 289 | // Start observing for all impact observers |
| 290 | watchdog.startObservingHealth(observerNone, Arrays.asList(APP_A, APP_B, APP_C, APP_D), |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 291 | SHORT_DURATION, false); |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 292 | watchdog.startObservingHealth(observerHigh, Arrays.asList(APP_A, APP_B, APP_C), |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 293 | SHORT_DURATION, false); |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 294 | watchdog.startObservingHealth(observerMid, Arrays.asList(APP_A, APP_B), |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 295 | SHORT_DURATION, false); |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 296 | watchdog.startObservingHealth(observerLow, Arrays.asList(APP_A), |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 297 | SHORT_DURATION, false); |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 298 | |
| 299 | // Then fail all apps above the threshold |
| 300 | for (int i = 0; i < TRIGGER_FAILURE_COUNT; i++) { |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 301 | watchdog.onPackageFailure(Arrays.asList(new VersionedPackage(APP_A, VERSION_CODE), |
| 302 | new VersionedPackage(APP_B, VERSION_CODE), |
| 303 | new VersionedPackage(APP_C, VERSION_CODE), |
| 304 | new VersionedPackage(APP_D, VERSION_CODE))); |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 305 | } |
| 306 | |
| 307 | // Run handler so package failures are dispatched to observers |
| 308 | mTestLooper.dispatchAll(); |
| 309 | |
| 310 | // Verify least impact observers are notifed of package failures |
| 311 | List<String> observerNonePackages = observerNone.mFailedPackages; |
| 312 | List<String> observerHighPackages = observerHigh.mFailedPackages; |
| 313 | List<String> observerMidPackages = observerMid.mFailedPackages; |
| 314 | List<String> observerLowPackages = observerLow.mFailedPackages; |
| 315 | |
| 316 | // APP_D failure observed by only observerNone is not caught cos its impact is none |
| 317 | assertEquals(0, observerNonePackages.size()); |
| 318 | // APP_C failure is caught by observerHigh cos it's the lowest impact observer |
| 319 | assertEquals(1, observerHighPackages.size()); |
| 320 | assertEquals(APP_C, observerHighPackages.get(0)); |
| 321 | // APP_B failure is caught by observerMid cos it's the lowest impact observer |
| 322 | assertEquals(1, observerMidPackages.size()); |
| 323 | assertEquals(APP_B, observerMidPackages.get(0)); |
| 324 | // APP_A failure is caught by observerLow cos it's the lowest impact observer |
| 325 | assertEquals(1, observerLowPackages.size()); |
| 326 | assertEquals(APP_A, observerLowPackages.get(0)); |
| 327 | } |
| 328 | |
| 329 | /** |
| 330 | * Test package failure and least impact observers are notified successively. |
| 331 | * State transistions: |
| 332 | * |
| 333 | * <ul> |
| 334 | * <li>(observer1:low, observer2:mid) -> {observer1} |
| 335 | * <li>(observer1:high, observer2:mid) -> {observer2} |
| 336 | * <li>(observer1:high, observer2:none) -> {observer1} |
| 337 | * <li>(observer1:none, observer2:none) -> {} |
| 338 | * <ul> |
| 339 | */ |
| 340 | @Test |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 341 | public void testPackageFailureNotifyLeastImpactSuccessively() throws Exception { |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 342 | PackageWatchdog watchdog = createWatchdog(); |
| 343 | TestObserver observerFirst = new TestObserver(OBSERVER_NAME_1, |
| 344 | PackageHealthObserverImpact.USER_IMPACT_LOW); |
| 345 | TestObserver observerSecond = new TestObserver(OBSERVER_NAME_2, |
| 346 | PackageHealthObserverImpact.USER_IMPACT_MEDIUM); |
| 347 | |
| 348 | // Start observing for observerFirst and observerSecond with failure handling |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 349 | watchdog.startObservingHealth(observerFirst, Arrays.asList(APP_A), LONG_DURATION, false); |
| 350 | watchdog.startObservingHealth(observerSecond, Arrays.asList(APP_A), LONG_DURATION, false); |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 351 | |
| 352 | // Then fail APP_A above the threshold |
| 353 | for (int i = 0; i < TRIGGER_FAILURE_COUNT; i++) { |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 354 | watchdog.onPackageFailure(Arrays.asList(new VersionedPackage(APP_A, VERSION_CODE))); |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 355 | } |
| 356 | // Run handler so package failures are dispatched to observers |
| 357 | mTestLooper.dispatchAll(); |
| 358 | |
| 359 | // Verify only observerFirst is notifed |
| 360 | assertEquals(1, observerFirst.mFailedPackages.size()); |
| 361 | assertEquals(APP_A, observerFirst.mFailedPackages.get(0)); |
| 362 | assertEquals(0, observerSecond.mFailedPackages.size()); |
| 363 | |
| 364 | // After observerFirst handles failure, next action it has is high impact |
| 365 | observerFirst.mImpact = PackageHealthObserverImpact.USER_IMPACT_HIGH; |
| 366 | observerFirst.mFailedPackages.clear(); |
| 367 | observerSecond.mFailedPackages.clear(); |
| 368 | |
| 369 | // Then fail APP_A again above the threshold |
| 370 | for (int i = 0; i < TRIGGER_FAILURE_COUNT; i++) { |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 371 | watchdog.onPackageFailure(Arrays.asList(new VersionedPackage(APP_A, VERSION_CODE))); |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 372 | } |
| 373 | // Run handler so package failures are dispatched to observers |
| 374 | mTestLooper.dispatchAll(); |
| 375 | |
| 376 | // Verify only observerSecond is notifed cos it has least impact |
| 377 | assertEquals(1, observerSecond.mFailedPackages.size()); |
| 378 | assertEquals(APP_A, observerSecond.mFailedPackages.get(0)); |
| 379 | assertEquals(0, observerFirst.mFailedPackages.size()); |
| 380 | |
| 381 | // After observerSecond handles failure, it has no further actions |
| 382 | observerSecond.mImpact = PackageHealthObserverImpact.USER_IMPACT_NONE; |
| 383 | observerFirst.mFailedPackages.clear(); |
| 384 | observerSecond.mFailedPackages.clear(); |
| 385 | |
| 386 | // Then fail APP_A again above the threshold |
| 387 | for (int i = 0; i < TRIGGER_FAILURE_COUNT; i++) { |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 388 | watchdog.onPackageFailure(Arrays.asList(new VersionedPackage(APP_A, VERSION_CODE))); |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 389 | } |
| 390 | // Run handler so package failures are dispatched to observers |
| 391 | mTestLooper.dispatchAll(); |
| 392 | |
| 393 | // Verify only observerFirst is notifed cos it has the only action |
| 394 | assertEquals(1, observerFirst.mFailedPackages.size()); |
| 395 | assertEquals(APP_A, observerFirst.mFailedPackages.get(0)); |
| 396 | assertEquals(0, observerSecond.mFailedPackages.size()); |
| 397 | |
| 398 | // After observerFirst handles failure, it too has no further actions |
| 399 | observerFirst.mImpact = PackageHealthObserverImpact.USER_IMPACT_NONE; |
| 400 | observerFirst.mFailedPackages.clear(); |
| 401 | observerSecond.mFailedPackages.clear(); |
| 402 | |
| 403 | // Then fail APP_A again above the threshold |
| 404 | for (int i = 0; i < TRIGGER_FAILURE_COUNT; i++) { |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 405 | watchdog.onPackageFailure(Arrays.asList(new VersionedPackage(APP_A, VERSION_CODE))); |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 406 | } |
| 407 | // Run handler so package failures are dispatched to observers |
| 408 | mTestLooper.dispatchAll(); |
| 409 | |
| 410 | // Verify no observer is notified cos no actions left |
| 411 | assertEquals(0, observerFirst.mFailedPackages.size()); |
| 412 | assertEquals(0, observerSecond.mFailedPackages.size()); |
| 413 | } |
| 414 | |
| 415 | /** |
| 416 | * Test package failure and notifies only one observer even with observer impact tie. |
| 417 | */ |
| 418 | @Test |
| 419 | public void testPackageFailureNotifyOneSameImpact() throws Exception { |
| 420 | PackageWatchdog watchdog = createWatchdog(); |
| 421 | TestObserver observer1 = new TestObserver(OBSERVER_NAME_1, |
| 422 | PackageHealthObserverImpact.USER_IMPACT_HIGH); |
| 423 | TestObserver observer2 = new TestObserver(OBSERVER_NAME_2, |
| 424 | PackageHealthObserverImpact.USER_IMPACT_HIGH); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 425 | |
| 426 | // Start observing for observer1 and observer2 with failure handling |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 427 | watchdog.startObservingHealth(observer2, Arrays.asList(APP_A), SHORT_DURATION, false); |
| 428 | watchdog.startObservingHealth(observer1, Arrays.asList(APP_A), SHORT_DURATION, false); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 429 | |
| 430 | // Then fail APP_A above the threshold |
| 431 | for (int i = 0; i < TRIGGER_FAILURE_COUNT; i++) { |
Zimuzo | 972e1cd | 2019-01-28 16:30:01 +0000 | [diff] [blame] | 432 | watchdog.onPackageFailure(Arrays.asList(new VersionedPackage(APP_A, VERSION_CODE))); |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 433 | } |
| 434 | |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 435 | // Run handler so package failures are dispatched to observers |
| 436 | mTestLooper.dispatchAll(); |
| 437 | |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 438 | // Verify only one observer is notifed |
| 439 | assertEquals(1, observer1.mFailedPackages.size()); |
| 440 | assertEquals(APP_A, observer1.mFailedPackages.get(0)); |
| 441 | assertEquals(0, observer2.mFailedPackages.size()); |
| 442 | } |
| 443 | |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 444 | /** |
| 445 | * Test explicit health check status determines package failure or success on expiry |
| 446 | */ |
Zimuzo | 9284e74 | 2019-02-22 12:09:28 +0000 | [diff] [blame] | 447 | @Test |
| 448 | public void testPackageFailureExplicitHealthCheck() throws Exception { |
| 449 | PackageWatchdog watchdog = createWatchdog(); |
| 450 | TestObserver observer1 = new TestObserver(OBSERVER_NAME_1, |
| 451 | PackageHealthObserverImpact.USER_IMPACT_HIGH); |
| 452 | TestObserver observer2 = new TestObserver(OBSERVER_NAME_2, |
| 453 | PackageHealthObserverImpact.USER_IMPACT_HIGH); |
| 454 | TestObserver observer3 = new TestObserver(OBSERVER_NAME_3, |
| 455 | PackageHealthObserverImpact.USER_IMPACT_HIGH); |
| 456 | |
| 457 | |
| 458 | // Start observing with explicit health checks for APP_A and APP_B respectively |
| 459 | // with observer1 and observer2 |
| 460 | watchdog.startObservingHealth(observer1, Arrays.asList(APP_A), SHORT_DURATION, true); |
| 461 | watchdog.startObservingHealth(observer2, Arrays.asList(APP_B), SHORT_DURATION, true); |
| 462 | // Explicit health check passed for APP_A (observer1 is aware) |
| 463 | watchdog.onExplicitHealthCheckPassed(APP_A); |
| 464 | // Start observing APP_A with explicit health checks for observer3. |
| 465 | // Observer3 didn't exist when we got the explicit health check above, so |
| 466 | // it starts out with a non-passing explicit health check and has to wait for a pass |
| 467 | // otherwise it would be notified of APP_A failure on expiry |
| 468 | watchdog.startObservingHealth(observer3, Arrays.asList(APP_A), SHORT_DURATION, true); |
| 469 | |
| 470 | // Then expire observers |
| 471 | Thread.sleep(SHORT_DURATION); |
| 472 | // Run handler so package failures are dispatched to observers |
| 473 | mTestLooper.dispatchAll(); |
| 474 | |
| 475 | // Verify observer1 is not notified |
| 476 | assertEquals(0, observer1.mFailedPackages.size()); |
| 477 | |
| 478 | // Verify observer2 is notifed because health checks for APP_B never passed |
| 479 | assertEquals(1, observer2.mFailedPackages.size()); |
| 480 | assertEquals(APP_B, observer2.mFailedPackages.get(0)); |
| 481 | |
| 482 | // Verify observer3 is notifed because health checks for APP_A did not pass before expiry |
| 483 | assertEquals(1, observer3.mFailedPackages.size()); |
| 484 | assertEquals(APP_A, observer3.mFailedPackages.get(0)); |
| 485 | } |
| 486 | |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 487 | private PackageWatchdog createWatchdog() { |
| 488 | return new PackageWatchdog(InstrumentationRegistry.getContext(), |
| 489 | mTestLooper.getLooper()); |
| 490 | } |
| 491 | |
| 492 | private static class TestObserver implements PackageHealthObserver { |
| 493 | private final String mName; |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 494 | private int mImpact; |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 495 | final List<String> mFailedPackages = new ArrayList<>(); |
| 496 | |
| 497 | TestObserver(String name) { |
| 498 | mName = name; |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 499 | mImpact = PackageHealthObserverImpact.USER_IMPACT_MEDIUM; |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 500 | } |
| 501 | |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 502 | TestObserver(String name, int impact) { |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 503 | mName = name; |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 504 | mImpact = impact; |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 505 | } |
| 506 | |
Zimuzo | 71d931e | 2019-02-01 13:08:16 +0000 | [diff] [blame] | 507 | public int onHealthCheckFailed(VersionedPackage versionedPackage) { |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 508 | return mImpact; |
| 509 | } |
| 510 | |
Zimuzo | 71d931e | 2019-02-01 13:08:16 +0000 | [diff] [blame] | 511 | public boolean execute(VersionedPackage versionedPackage) { |
| 512 | mFailedPackages.add(versionedPackage.getPackageName()); |
Zimuzo | e5009cd | 2019-01-23 18:11:58 +0000 | [diff] [blame] | 513 | return true; |
Zimuzo | 3eee438 | 2019-01-08 20:42:39 +0000 | [diff] [blame] | 514 | } |
| 515 | |
| 516 | public String getName() { |
| 517 | return mName; |
| 518 | } |
| 519 | } |
| 520 | } |