diff options
author | Grant Menke <grantmenke@google.com> | 2023-05-02 13:52:30 -0700 |
---|---|---|
committer | Grant Menke <grantmenke@google.com> | 2023-05-02 20:57:40 +0000 |
commit | 1eac75a439c1ee62591bdd3f93ebd0ea0dc6090f (patch) | |
tree | bd7ea01673df3ee3660759e7e385aab52777f9d8 | |
parent | d24120081434e3f4d06a74c82ef50bae6eb636f5 (diff) | |
download | CallLogProvider-1eac75a439c1ee62591bdd3f93ebd0ea0dc6090f.tar.gz |
Report existing calls as successful backups in CallLogBackupAgent.android14-dev
This change ensures that calls that still exist in the current call log are reported as successful backups in addition to new calls that have been made since the last backup.
fixes: 249018618
test: atest CallLogBackupAgentTest#PartialMultipleCall
Change-Id: I40a36f96f3468825751697f6d7f398f25033995f
-rw-r--r-- | src/com/android/calllogbackup/CallLogBackupAgent.java | 1 | ||||
-rw-r--r-- | tests/src/com/android/calllogbackup/CallLogBackupAgentTest.java | 4 |
2 files changed, 5 insertions, 0 deletions
diff --git a/src/com/android/calllogbackup/CallLogBackupAgent.java b/src/com/android/calllogbackup/CallLogBackupAgent.java index b0b0c1c..4bb55b0 100644 --- a/src/com/android/calllogbackup/CallLogBackupAgent.java +++ b/src/com/android/calllogbackup/CallLogBackupAgent.java @@ -302,6 +302,7 @@ public class CallLogBackupAgent extends BackupAgent { // This call still exists in the current call log so delete it from the // "callsToRemove" set since we want to keep it. callsToRemove.remove(call.id); + mBackupRestoreEventLoggerProxy.logItemsBackedUp(CALLLOGS, /* count */ 1); } } diff --git a/tests/src/com/android/calllogbackup/CallLogBackupAgentTest.java b/tests/src/com/android/calllogbackup/CallLogBackupAgentTest.java index 96ac049..de681e5 100644 --- a/tests/src/com/android/calllogbackup/CallLogBackupAgentTest.java +++ b/tests/src/com/android/calllogbackup/CallLogBackupAgentTest.java @@ -371,6 +371,10 @@ public class CallLogBackupAgentTest extends AndroidTestCase { mCallLogBackupAgent.runBackup(state, mBackupDataOutput, calls); + // Ensure the {@link BackupRestoreEventLogger} is informed of the 2 backed up calls: + assertEquals(backupRestoreLoggerSuccessCount, 2); + assertEquals(backupRestoreLoggerFailCount, 0); + InOrder inOrder = Mockito.inOrder(mBackupDataOutput); inOrder.verify(mBackupDataOutput).writeEntityHeader(eq("102"), Matchers.anyInt()); inOrder.verify(mBackupDataOutput). |