Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 338d3328 authored by Mike Tsao's avatar Mike Tsao
Browse files

Cleanup that wasn't uploaded with ag/2094132

I saw that Tree Hugger had verified, and hit submit without
remembering to repo upload the last patch set containing toddke's
most recent set of suggestions.

Bug: 37192695
Test: runtest -c com.android.server.pm.PackageManagerServiceTest frameworks-services
Change-Id: I4d6376165aaa615007891ba045c9b5c34e4b0394
parent 9c1a1327
Loading
Loading
Loading
Loading
+6 −6
Original line number Diff line number Diff line
@@ -71,17 +71,17 @@ public class PackageManagerServiceTest extends AndroidTestCase {
          "primaryCpuAbiString", "secondaryCpuAbiString",
          "cpuAbiOverrideString", 0, 0, 0, "parentPackageName", null, 0,
          null, null);
      pri.removedUsers = new int[] {1, 2, 3, 4, 5};
      pri.populateUsers(pri.removedUsers, setting);
      pri.populateUsers(new int[] {1, 2, 3, 4, 5}, setting);
      assertNotNull(pri.broadcastUsers);
      assertEquals(pri.removedUsers.length, pri.broadcastUsers.length);
      assertEquals(5, pri.broadcastUsers.length);

      // Exclude a user
      pri.broadcastUsers = null;
      setting.setInstantApp(true, 4);
      pri.populateUsers(pri.removedUsers, setting);
      final int EXCLUDED_USER_ID = 4;
      setting.setInstantApp(true, EXCLUDED_USER_ID);
      pri.populateUsers(new int[] {1, 2, 3, EXCLUDED_USER_ID, 5}, setting);
      assertNotNull(pri.broadcastUsers);
      assertEquals(pri.removedUsers.length - 1, pri.broadcastUsers.length);
      assertEquals(5 - 1, pri.broadcastUsers.length);

      // TODO: test that sendApplicationHiddenForUser() actually fills in
      // broadcastUsers