Merge pull request #1043 from jotomo/issue-646
[Bugfix] On bolus cancel request, cancel all boluses. Partially addresses #646.
This commit is contained in:
commit
e58149379a
3 changed files with 27 additions and 1 deletions
|
@ -112,7 +112,7 @@ public class BolusProgressDialog extends DialogFragment implements View.OnClickL
|
||||||
stopPressed = true;
|
stopPressed = true;
|
||||||
stopPressedView.setVisibility(View.VISIBLE);
|
stopPressedView.setVisibility(View.VISIBLE);
|
||||||
stopButton.setVisibility(View.INVISIBLE);
|
stopButton.setVisibility(View.INVISIBLE);
|
||||||
ConfigBuilderPlugin.getActivePump().stopBolusDelivering();
|
ConfigBuilderPlugin.getCommandQueue().cancelAllBoluses();
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -213,6 +213,12 @@ public class CommandQueue {
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public synchronized void cancelAllBoluses() {
|
||||||
|
removeAll(Command.CommandType.BOLUS);
|
||||||
|
removeAll(Command.CommandType.SMB_BOLUS);
|
||||||
|
ConfigBuilderPlugin.getActivePump().stopBolusDelivering();
|
||||||
|
}
|
||||||
|
|
||||||
// returns true if command is queued
|
// returns true if command is queued
|
||||||
public boolean tempBasalAbsolute(double absoluteRate, int durationInMinutes, boolean enforceNew, Profile profile, Callback callback) {
|
public boolean tempBasalAbsolute(double absoluteRate, int durationInMinutes, boolean enforceNew, Profile profile, Callback callback) {
|
||||||
if (!enforceNew && isRunning(Command.CommandType.TEMPBASAL)) {
|
if (!enforceNew && isRunning(Command.CommandType.TEMPBASAL)) {
|
||||||
|
|
|
@ -156,4 +156,24 @@ public class CommandQueueTest extends CommandQueue {
|
||||||
public boolean isThisProfileSet(Profile profile) {
|
public boolean isThisProfileSet(Profile profile) {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void callingCancelAllBolusesClearsQueue() throws Exception {
|
||||||
|
prepareMock(0d, 0);
|
||||||
|
|
||||||
|
// add normal and SMB-bolus to queue
|
||||||
|
Assert.assertEquals(0, size());
|
||||||
|
|
||||||
|
bolus(new DetailedBolusInfo(), null);
|
||||||
|
|
||||||
|
DetailedBolusInfo smb = new DetailedBolusInfo();
|
||||||
|
smb.isSMB = true;
|
||||||
|
bolus(smb, null);
|
||||||
|
|
||||||
|
Assert.assertEquals(2, size());
|
||||||
|
|
||||||
|
// cancelling all boluses clear all boluses from the queue
|
||||||
|
cancelAllBoluses();
|
||||||
|
Assert.assertEquals(0, size());
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue