diff options
| author | 2023-02-03 17:25:14 +0000 | |
|---|---|---|
| committer | 2024-04-10 17:42:14 +0100 | |
| commit | f98823a3aa1d31189fcedd05f62d963781e766f3 (patch) | |
| tree | 1bb793e76d79f7c2e0b3d59304ba5a4433e71e27 /libs/androidfw/StringPool.cpp | |
| parent | efc35f89fd67ad7a66a42df7ddef51f3d44d48c8 (diff) | |
Improve `dumpsys backup`
This is a re-submit of the previously reverted ag/22327091 by
piee@google.com.
Original description:
This change refactors `dumpsys backup` code and adds some improvements -
- Removes `isUserReadyForBackup()` check on calling user. This check is not needed, as it prevents a User with INTERACT_ACROSS_USERS_FULL permission, but not Backup activated, to get info about Backup service on other users. This is particularly relevant in case of headless mode, as Backup service isn't running for SYSTEM_USER.
- Adds `--user <userId>` optional parameter for `dumpsys backup`, which will return info about Backup service for the user passed in argument. This is also helpul in case of multiple users on the device.
Changes by sarpm@google.com over the original:
- Some test improvements: remove @VisibleForTesting, increase
coverage slightly.
- Remove redundant robo tests that are now covered by the unit
tests.
Bug: 267755698
Test: atest BackupManagerServiceTest
Manual test by running dumpsys backup with optional arguments.
Change-Id: Id7790a501a8befb12a212f66adfc0cd8cc9f9e8b
Diffstat (limited to 'libs/androidfw/StringPool.cpp')
0 files changed, 0 insertions, 0 deletions