Rename Alarm to AlarmJob

pull/618/head
Alex Baker 7 years ago
parent 6574a02f45
commit 5af31fcbff

@ -14,7 +14,7 @@ import org.junit.runner.RunWith;
import org.mockito.InOrder;
import org.tasks.injection.InjectingTestCase;
import org.tasks.injection.TestComponent;
import org.tasks.jobs.Alarm;
import org.tasks.jobs.AlarmJob;
import org.tasks.jobs.JobQueue;
import org.tasks.time.DateTime;
@ -28,7 +28,7 @@ import static org.tasks.makers.TaskMaker.REMINDER_LAST;
import static org.tasks.makers.TaskMaker.newTask;
@RunWith(AndroidJUnit4.class)
public class AlarmServiceTest extends InjectingTestCase {
public class AlarmJobServiceTest extends InjectingTestCase {
@Inject MetadataDao metadataDao;
@Inject TaskDao taskDao;
@ -64,7 +64,7 @@ public class AlarmServiceTest extends InjectingTestCase {
alarmService.scheduleAllAlarms();
InOrder order = inOrder(jobs);
order.verify(jobs).add(new Alarm(alarm));
order.verify(jobs).add(new AlarmJob(alarm));
}
@Test

@ -58,13 +58,13 @@ public class JobQueueTest {
long now = currentTimeMillis();
queue.add(new Reminder(1, now, TYPE_DUE));
queue.add(new Alarm(1, 1, now));
queue.add(new AlarmJob(1, 1, now));
verify(jobManager).schedule(TAG, now);
Freeze.freezeAt(now).thawAfter(new Snippet() {{
assertEquals(
newHashSet(new Alarm(1, 1, now),
newHashSet(new AlarmJob(1, 1, now),
new Reminder(1, now, TYPE_DUE)),
newHashSet(queue.getOverdueJobs()));
}});
@ -75,11 +75,11 @@ public class JobQueueTest {
long now = currentTimeMillis();
queue.add(new Reminder(1, now, TYPE_DUE));
queue.add(new Alarm(1, 1, now));
queue.add(new AlarmJob(1, 1, now));
verify(jobManager).schedule(TAG, now);
queue.remove(singletonList(new Alarm(1, 1, now)));
queue.remove(singletonList(new AlarmJob(1, 1, now)));
Freeze.freezeAt(now).thawAfter(new Snippet() {{
assertEquals(
@ -93,7 +93,7 @@ public class JobQueueTest {
long now = currentTimeMillis();
queue.add(new Reminder(1, now, TYPE_DUE));
queue.add(new Alarm(1, 1, now));
queue.add(new AlarmJob(1, 1, now));
verify(jobManager).schedule(TAG, now);
@ -101,7 +101,7 @@ public class JobQueueTest {
Freeze.freezeAt(now).thawAfter(new Snippet() {{
assertEquals(
singletonList(new Alarm(1, 1, now)),
singletonList(new AlarmJob(1, 1, now)),
queue.getOverdueJobs());
}});
}

@ -1,6 +1,6 @@
package org.tasks.injection;
import com.todoroo.astrid.alarms.AlarmServiceTest;
import com.todoroo.astrid.alarms.AlarmJobServiceTest;
import com.todoroo.astrid.dao.MetadataDaoTests;
import com.todoroo.astrid.dao.TaskDaoTests;
import com.todoroo.astrid.gtasks.GtasksIndentActionTest;
@ -58,7 +58,7 @@ public interface TestComponent {
void inject(BackupServiceTests backupServiceTests);
void inject(AlarmServiceTest alarmServiceTest);
void inject(AlarmJobServiceTest alarmServiceTest);
void inject(RepeatTaskHelperTest repeatTaskHelperTest);
}

@ -20,7 +20,7 @@ import com.todoroo.astrid.data.Task;
import com.todoroo.astrid.service.SynchronizeMetadataCallback;
import org.tasks.injection.ApplicationScope;
import org.tasks.jobs.Alarm;
import org.tasks.jobs.AlarmJob;
import org.tasks.jobs.JobQueue;
import java.util.ArrayList;
@ -139,12 +139,12 @@ public class AlarmService {
return;
}
Alarm alarm = new Alarm(metadata);
long time = alarm.getTime();
AlarmJob alarmJob = new AlarmJob(metadata);
long time = alarmJob.getTime();
if(time == 0 || time == NO_ALARM) {
jobs.cancelAlarm(alarm.getId());
jobs.cancelAlarm(alarmJob.getId());
} else {
jobs.add(alarm);
jobs.add(alarmJob);
}
}

@ -8,16 +8,16 @@ import org.tasks.notifications.Notification;
import static org.tasks.time.DateTimeUtils.currentTimeMillis;
public class Alarm implements JobQueueEntry {
public class AlarmJob implements JobQueueEntry {
private final long alarmId;
private final long taskId;
private final long time;
public Alarm(Metadata metadata) {
public AlarmJob(Metadata metadata) {
this(metadata.getId(), metadata.getTask(), metadata.getValue(AlarmFields.TIME));
}
public Alarm(long alarmId, long taskId, Long time) {
public AlarmJob(long alarmId, long taskId, Long time) {
this.alarmId = alarmId;
this.taskId = taskId;
this.time = time;
@ -47,11 +47,11 @@ public class Alarm implements JobQueueEntry {
if (this == o) return true;
if (o == null || getClass() != o.getClass()) return false;
Alarm alarm = (Alarm) o;
AlarmJob alarmJob = (AlarmJob) o;
if (alarmId != alarm.alarmId) return false;
if (taskId != alarm.taskId) return false;
return time == alarm.time;
if (alarmId != alarmJob.alarmId) return false;
if (taskId != alarmJob.taskId) return false;
return time == alarmJob.time;
}
@Override
@ -64,7 +64,7 @@ public class Alarm implements JobQueueEntry {
@Override
public String toString() {
return "Alarm{" +
return "AlarmJob{" +
"alarmId=" + alarmId +
", taskId=" + taskId +
", time=" + time +

@ -42,7 +42,7 @@ public class JobQueue {
}
public synchronized void cancelAlarm(long alarmId) {
cancel(Alarm.class, alarmId);
cancel(AlarmJob.class, alarmId);
}
public synchronized void cancelReminder(long taskId) {

Loading…
Cancel
Save