simplify executeProgrammingCommand. Start implementing alerts

omnipod-dash/src/main/java/info/nightscout/androidaps/plugins/pump/omnipod/dash/OmnipodDashPumpPlugin.kt

@@ -3,7 +3,6 @@ package info.nightscout.androidaps.plugins.pump.omnipod.dash
 import dagger.android.HasAndroidInjector
 import info.nightscout.androidaps.data.DetailedBolusInfo
 import info.nightscout.androidaps.data.PumpEnactResult
-import info.nightscout.androidaps.events.EventPreferenceChange
 import info.nightscout.androidaps.events.EventProfileSwitchChanged
 import info.nightscout.androidaps.events.EventTempBasalChange
 import info.nightscout.androidaps.interfaces.*
@@ -19,7 +18,6 @@ import info.nightscout.androidaps.plugins.pump.omnipod.common.queue.command.*
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.OmnipodDashManager
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.event.PodEvent
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.pod.definition.ActivationProgress
-import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.pod.definition.BasalProgram
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.pod.definition.BeepType
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.pod.definition.DeliveryStatus
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.pod.definition.PodConstants
@@ -37,13 +35,10 @@ import info.nightscout.androidaps.queue.commands.CustomCommand
 import info.nightscout.androidaps.utils.T
 import info.nightscout.androidaps.utils.TimeChangeType
 import info.nightscout.androidaps.utils.resources.ResourceHelper
-import info.nightscout.androidaps.utils.rx.AapsSchedulers
 import info.nightscout.androidaps.utils.sharedPreferences.SP
 import io.reactivex.Completable
 import io.reactivex.Observable
 import io.reactivex.Single
-import io.reactivex.disposables.CompositeDisposable
-import io.reactivex.functions.Consumer
 import io.reactivex.rxkotlin.plusAssign
 import io.reactivex.rxkotlin.subscribeBy
 import org.json.JSONObject
@@ -128,7 +123,8 @@ class OmnipodDashPumpPlugin @Inject constructor(
     }
 
     override fun getPumpStatus(reason: String) {
-        val throwable = Completable.concat(listOf(
+        val throwable = Completable.concat(
+            listOf(
                 omnipodManager
                     .getStatus(ResponseType.StatusResponseType.DEFAULT_STATUS_RESPONSE)
                     .ignoreElements(),
@@ -136,12 +132,12 @@ class OmnipodDashPumpPlugin @Inject constructor(
                 podStateManager.updateActiveCommand()
                     .map { handleCommandConfirmation(it) }
                     .ignoreElement(),
-        )).blockingGet()
+            )
+        ).blockingGet()
         if (throwable != null) {
             aapsLogger.error(LTag.PUMP, "Error in getPumpStatus", throwable)
         } else {
             aapsLogger.info(LTag.PUMP, "getPumpStatus executed with success")
-
         }
     }
 
@@ -150,8 +146,10 @@ class OmnipodDashPumpPlugin @Inject constructor(
         return executeProgrammingCommand(
             pre = suspendDeliveryIfActive(),
             historyEntry = history.createRecord(commandType = OmnipodCommandType.SET_BASAL_PROFILE),
+            activeCommandEntry = { historyId ->
+                podStateManager.createActiveCommand(historyId, basalProgram = basalProgram)
+            },
             command = omnipodManager.setBasalProgram(basalProgram).ignoreElements(),
-            basalProgram = basalProgram,
             post = failWhenUnconfirmed(),
         ).toPumpEnactResult()
     }
@@ -170,8 +168,8 @@ class OmnipodDashPumpPlugin @Inject constructor(
             Completable.complete()
         else
             executeProgrammingCommand(
-                history.createRecord(OmnipodCommandType.SUSPEND_DELIVERY),
+                historyEntry = history.createRecord(OmnipodCommandType.SUSPEND_DELIVERY),
-                omnipodManager.suspendDelivery()
+                command = omnipodManager.suspendDelivery()
                     .filter { podEvent -> podEvent is PodEvent.CommandSent }
                     .map {
                         pumpSyncTempBasal(
@@ -296,8 +294,8 @@ class OmnipodDashPumpPlugin @Inject constructor(
     override fun stopBolusDelivering() {
         // TODO update Treatments (?)
         executeProgrammingCommand(
-            history.createRecord(OmnipodCommandType.CANCEL_BOLUS),
+            historyEntry = history.createRecord(OmnipodCommandType.CANCEL_BOLUS),
-            omnipodManager.stopBolus().ignoreElements()
+            command = omnipodManager.stopBolus().ignoreElements()
         ).toPumpEnactResult()
     }
 
@@ -310,7 +308,8 @@ class OmnipodDashPumpPlugin @Inject constructor(
     ): PumpEnactResult {
         val tempBasalBeeps = sp.getBoolean(R.string.key_omnipod_common_tbr_beeps_enabled, false)
         aapsLogger.info(
-            LTag.PUMP, "setTempBasalAbsolute: $durationInMinutes min :: $absoluteRate U/h :: " +
+            LTag.PUMP,
+            "setTempBasalAbsolute: $durationInMinutes min :: $absoluteRate U/h :: " +
                 "enforce: $enforceNew :: tbrType: $tbrType"
         )
         return executeProgrammingCommand(
@@ -318,6 +317,16 @@ class OmnipodDashPumpPlugin @Inject constructor(
                 commandType = OmnipodCommandType.SET_TEMPORARY_BASAL,
                 tempBasalRecord = TempBasalRecord(duration = durationInMinutes, rate = absoluteRate)
             ),
+            activeCommandEntry = { historyId ->
+                podStateManager.createActiveCommand(
+                    historyId,
+                    tempBasal = OmnipodDashPodStateManager.TempBasal(
+                        startTime = System.currentTimeMillis(),
+                        rate = absoluteRate,
+                        durationInMinutes = durationInMinutes.toShort(),
+                    )
+                )
+            },
             command = omnipodManager.setTempBasal(
                 absoluteRate,
                 durationInMinutes.toShort(),
@@ -327,11 +336,6 @@ class OmnipodDashPumpPlugin @Inject constructor(
                 .map { pumpSyncTempBasal(it, absoluteRate, durationInMinutes.toLong(), tbrType) }
                 .ignoreElements(),
             pre = observeNoActiveTempBasal(true),
-            tempBasal = OmnipodDashPodStateManager.TempBasal(
-                startTime = System.currentTimeMillis(),
-                rate = absoluteRate,
-                durationInMinutes = durationInMinutes.toShort(),
-            )
         ).toPumpEnactResult()
     }
 
@@ -386,8 +390,8 @@ class OmnipodDashPumpPlugin @Inject constructor(
                     // enforceNew == true
                     aapsLogger.info(LTag.PUMP, "Canceling existing temp basal")
                     executeProgrammingCommand(
-                        history.createRecord(OmnipodCommandType.CANCEL_TEMPORARY_BASAL),
+                        historyEntry = history.createRecord(OmnipodCommandType.CANCEL_TEMPORARY_BASAL),
-                        omnipodManager.stopTempBasal().ignoreElements()
+                        command = omnipodManager.stopTempBasal().ignoreElements()
                     )
                 }
             }
@@ -582,17 +586,17 @@ class OmnipodDashPumpPlugin @Inject constructor(
     private fun resumeDelivery(): PumpEnactResult {
         return profileFunction.getProfile()?.let {
             executeProgrammingCommand(
-                history.createRecord(OmnipodCommandType.RESUME_DELIVERY),
-                omnipodManager.setBasalProgram(mapProfileToBasalProgram(it)).ignoreElements(),
                 pre = observeDeliverySuspended(),
+                historyEntry = history.createRecord(OmnipodCommandType.RESUME_DELIVERY),
+                command = omnipodManager.setBasalProgram(mapProfileToBasalProgram(it)).ignoreElements()
             ).toPumpEnactResult()
         } ?: PumpEnactResult(injector).success(false).enacted(false).comment("No profile active") // TODO i18n
     }
 
     private fun deactivatePod(): PumpEnactResult {
         return executeProgrammingCommand(
-            history.createRecord(OmnipodCommandType.DEACTIVATE_POD),
+            historyEntry = history.createRecord(OmnipodCommandType.DEACTIVATE_POD),
-            omnipodManager.deactivatePod().ignoreElements()
+            command = omnipodManager.deactivatePod().ignoreElements()
         ).toPumpEnactResult()
     }
 
@@ -608,8 +612,8 @@ class OmnipodDashPumpPlugin @Inject constructor(
 
     private fun playTestBeep(): PumpEnactResult {
         return executeProgrammingCommand(
-            history.createRecord(OmnipodCommandType.PLAY_TEST_BEEP),
+            historyEntry = history.createRecord(OmnipodCommandType.PLAY_TEST_BEEP),
-            omnipodManager.playBeep(BeepType.LONG_SINGLE_BEEP).ignoreElements()
+            command = omnipodManager.playBeep(BeepType.LONG_SINGLE_BEEP).ignoreElements()
         ).toPumpEnactResult()
     }
 
@@ -635,19 +639,19 @@ class OmnipodDashPumpPlugin @Inject constructor(
     }
 
     private fun executeProgrammingCommand(
-        historyEntry: Single<String>,
-        command: Completable,
         pre: Completable = Completable.complete(),
+        historyEntry: Single<String>,
+        activeCommandEntry: (historyId: String) -> Single<OmnipodDashPodStateManager.ActiveCommand> =
+            { historyId -> podStateManager.createActiveCommand(historyId) },
+        command: Completable,
         post: Completable = Completable.complete(),
-        basalProgram: BasalProgram? = null,
-        tempBasal: OmnipodDashPodStateManager.TempBasal? = null,
     ): Completable {
         return Completable.concat(
             listOf(
                 pre,
                 podStateManager.observeNoActiveCommand().ignoreElements(),
                 historyEntry
-                    .flatMap { podStateManager.createActiveCommand(it, basalProgram, tempBasal) }
+                    .flatMap { activeCommandEntry(it) }
                     .ignoreElement(),
                 command.doOnError {
                     podStateManager.activeCommand?.sendError = it

omnipod-dash/src/main/java/info/nightscout/androidaps/plugins/pump/omnipod/dash/driver/comm/session/Session.kt

@@ -6,15 +6,10 @@ import info.nightscout.androidaps.logging.LTag
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.comm.Ids
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.comm.endecrypt.EnDecrypt
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.comm.exceptions.CouldNotParseResponseException
-import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.comm.exceptions.IllegalResponseException
-import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.comm.exceptions.NakResponseException
-import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.comm.exceptions.PodAlarmException
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.comm.message.*
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.comm.message.MessageType
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.comm.message.StringLengthPrefixEncoding.Companion.parseKeys
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.pod.command.base.Command
-import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.pod.response.AlarmStatusResponse
-import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.pod.response.NakResponse
 import info.nightscout.androidaps.plugins.pump.omnipod.dash.driver.pod.response.Response
 import kotlin.reflect.KClass
 
@@ -88,7 +83,7 @@ class Session(
 
         val response = parseResponse(decrypted)
 
-        if (!responseType.isInstance(response)) {
+        /*if (!responseType.isInstance(response)) {
             if (response is AlarmStatusResponse) {
                 throw PodAlarmException(response)
             }
@@ -98,6 +93,8 @@ class Session(
             throw IllegalResponseException(responseType, response)
         }
 
+         */
+
         sessionKeys.msgSequenceNumber++
         val ack = getAck(responseMsgPacket)
         aapsLogger.debug(LTag.PUMPBTCOMM, "Sending ACK: ${ack.payload.toHex()} in packet $ack")

omnipod-dash/src/main/java/info/nightscout/androidaps/plugins/pump/omnipod/dash/driver/pod/state/OmnipodDashPodStateManagerImpl.kt

@@ -388,7 +388,10 @@ class OmnipodDashPodStateManagerImpl @Inject constructor(
             LTag.PUMP,
             "Not implemented: OmnipodDashPodStateManagerImpl.updateFromAlarmStatusResponse(AlarmStatusResponse)"
         )
-
+        logger.info(
+            LTag.PUMP,
+            "Received AlarmStatusReponse: $response"
+        )
         store()
         rxBus.send(EventOmnipodDashPumpValuesChanged())
     }

omnipod-dash/src/main/java/info/nightscout/androidaps/plugins/pump/omnipod/dash/ui/OmnipodDashOverviewFragment.kt

@@ -42,7 +42,6 @@ import info.nightscout.androidaps.utils.ui.UIRunnable
 import io.reactivex.disposables.CompositeDisposable
 import io.reactivex.rxkotlin.plusAssign
 import org.apache.commons.lang3.StringUtils
-import org.joda.time.DateTime
 import org.joda.time.Duration
 import java.util.*
 import javax.inject.Inject