Merge "Fix JobStoreTest after adding defensive copy to NetworkRequest.Builder"

This commit is contained in:
Treehugger Robot
2022-05-23 22:44:22 +00:00
committed by Gerrit Code Review

View File

@@ -475,15 +475,16 @@ public class JobStoreTest {
* Helper function to kick a {@link JobInfo} through a persistence cycle and * Helper function to kick a {@link JobInfo} through a persistence cycle and
* assert that it's unchanged. * assert that it's unchanged.
*/ */
private void assertPersistedEquals(JobInfo first) throws Exception { private void assertPersistedEquals(JobInfo firstInfo) throws Exception {
mTaskStoreUnderTest.clear(); mTaskStoreUnderTest.clear();
mTaskStoreUnderTest.add(JobStatus.createFromJobInfo(first, SOME_UID, null, -1, null)); JobStatus first = JobStatus.createFromJobInfo(firstInfo, SOME_UID, null, -1, null);
mTaskStoreUnderTest.add(first);
waitForPendingIo(); waitForPendingIo();
final JobSet jobStatusSet = new JobSet(); final JobSet jobStatusSet = new JobSet();
mTaskStoreUnderTest.readJobMapFromDisk(jobStatusSet, true); mTaskStoreUnderTest.readJobMapFromDisk(jobStatusSet, true);
final JobStatus second = jobStatusSet.getAllJobs().iterator().next(); final JobStatus second = jobStatusSet.getAllJobs().iterator().next();
assertTasksEqual(first, second.getJob()); assertTasksEqual(first.getJob(), second.getJob());
} }
/** /**