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

Commit 138856dc authored by Felka Chang's avatar Felka Chang
Browse files

Organize services/tests/PackageManager* for PackageManager

This patch adds 3 CTS test cases for PackageManager. They are the
following.
presubmit:
* PackageInstallerTests: Pixel 3 takes less than 5 seconds
* PackageManagerServiceUnitTests: Pixel 3 takes less than 5 seconds
* PackageManagerComponentOverrideTests: Pixel 3 takes less than 5
                                        seconds
* AppEnumerationInternalTests(moving location): Pixel 3 takes 10
                                                seconds

postsubmit:
* PackageManagerServiceHostTests: Pixel 4 5G takes 1 hour 17 minutes

When the developers change any code in
frameworks/base/service/core/java/com/android/server/pm and
frameworks/base/service/core/java/android/content/pm, it should invoke
all of the tests under cts/hostsidetests/packagemanager.

Why does this patch change AppEnumerationInternalTests from
frameworks/base/services/core/java/com/android/server/pm/TEST_MAPPING
to
frameworks/base/services/tests/PackageManagerServiceTests/TEST_MAPPING?
When developers uploads the gerrit patches to modify the content of
AppEnumerationInternalTests, presubmit process won't trigger
AppEnumerationInternalTests except for moving to
frameworks/base/services/tests/PackageManagerServiceTests/TEST_MAPPING.

Test: atest -c --dry-run -p \
    frameworks/base/services/tests/PackageManagerServiceTests:presubmit
Test: atest -c --dry-run -p \
    frameworks/base/services/tests/PackageManagerServiceTests
Test: atest -c --dry-run -p \
    frameworks/base/services/tests/PackageManager:presubmit
Test: TEST_BASE="frameworks/base/services" ;\
      atest -c --dry-run -p \
    ${TEST_BASE}/testsPackageManagerComponentOverrideTests:presubmit

Bug: 180650365
Change-Id: I62de692f7b5a33e3737effea622ed3f2460c64c3
parent 7933c5da
Loading
Loading
Loading
Loading
+9 −0
Original line number Diff line number Diff line
{
  "imports": [
    {
      "path": "frameworks/base/services/tests/PackageManagerServiceTests"
    },
    {
      "path": "frameworks/base/services/tests/PackageManager"
    },
    {
      "path": "frameworks/base/services/tests/PackageManagerComponentOverrideTests"
    },
    {
      "path": "cts/tests/tests/packageinstaller"
    },
+0 −3
Original line number Diff line number Diff line
@@ -20,9 +20,6 @@
    {
      "name": "CtsAppEnumerationTestCases"
    },
    {
      "name": "AppEnumerationInternalTests"
    },
    {
      "name": "CtsMatchFlagTestCases"
    },
+8 −0
Original line number Diff line number Diff line
{
  "presubmit": [
    {
      "name": "PackageInstallerTests"
    }
  ]
}
+8 −0
Original line number Diff line number Diff line
{
  "presubmit": [
    {
      "name": "PackageManagerComponentOverrideTests"
    }
  ]
}
+18 −0
Original line number Diff line number Diff line
{
  "presubmit": [
    {
      "name": "AppEnumerationInternalTests"
    }
  ],
  "postsubmit": [
    {
      "name": "PackageManagerServiceHostTests"
    }
  ],
  "imports": [
    {
      "path": "frameworks/base/services/tests/PackageManagerServiceTests/unit"
    }
  ]
}