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

Commit af3598cf authored by Henry Liu's avatar Henry Liu Committed by Android (Google) Code Review
Browse files

Merge "Fix [kv-4] PerformBackupTask possible NPE/Wrong package report."

parents dda35b31 87f1209a

services/backup/java/com/android/server/backup/internal/PerformBackupTask.java

+1 −1
Original line number Original line Diff line number Diff line
@@ -528,7 +528,7 @@ public class PerformBackupTask implements BackupRestoreTask { 
                    // we're otherwise fine.  Just drop it and go on to the next as usual.

                    // we're otherwise fine.  Just drop it and go on to the next as usual.

                    mStatus = BackupTransport.TRANSPORT_OK;

                    mStatus = BackupTransport.TRANSPORT_OK;

                    BackupObserverUtils

                    BackupObserverUtils

                            .sendBackupOnPackageResult(mObserver, mCurrentPackage.packageName,

                            .sendBackupOnPackageResult(mObserver, request.packageName,

                                    BackupManager.ERROR_PACKAGE_NOT_FOUND);

                                    BackupManager.ERROR_PACKAGE_NOT_FOUND);

                } else {

                } else {

                    // Transport-level failure means we reenqueue everything

                    // Transport-level failure means we reenqueue everything

services/robotests/src/com/android/server/backup/PerformBackupTaskTest.java

+0 −3
Original line number Original line Diff line number Diff line
@@ -468,9 +468,6 @@ public class PerformBackupTaskTest { 
        verify(agentMock.agent).onQuotaExceeded(anyLong(), anyLong());

        verify(agentMock.agent).onQuotaExceeded(anyLong(), anyLong());

    }

    }





    // TODO: Giving NPE at PerformBackupTask:524 because mCurrentPackage is null (PackageManager

    // rightfully threw NameNotFoundException). Remove @Ignore when fixed.

    @Ignore

    @Test

    @Test

    public void testRunTask_whenAgentUnknown() throws Exception {

    public void testRunTask_whenAgentUnknown() throws Exception {

        // Not calling setUpAgent()

        // Not calling setUpAgent()