Index: unittests/tst_messaging.cpp
===================================================================
diff -u -rdf0c8b4fa51d4f5ef0216eb2c3f6f5dfc97bf12e -r911882aaf10ee02ef646013914a2ddd6225224ca
--- unittests/tst_messaging.cpp	(.../tst_messaging.cpp)	(revision df0c8b4fa51d4f5ef0216eb2c3f6f5dfc97bf12e)
+++ unittests/tst_messaging.cpp	(.../tst_messaging.cpp)	(revision 911882aaf10ee02ef646013914a2ddd6225224ca)
@@ -22,6 +22,7 @@
 #include "MessageBuilder.h"
 #include "MPowerOff.h"
 #include "MessageDispatcher.h"
+#include "AlarmGenerator.h"
 
 tst_messaging::tst_messaging(QObject *parent) : QObject(parent) { }
 
@@ -66,36 +67,6 @@
 }
 
 /*!
- * \brief   tst_messaging::tst_MessageInterpreter_getPowerOffData_isType
- * \details Tests MessageInterpreter getPowerOffData method Message Type checking
- */
-void tst_messaging::tst_MessageInterpreter_getPowerOffData_isType()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_Unknown;
-    msg.data = {};
-    Model::MPowerOff data;
-    data.fromByteArray(msg.data);
-    // QVERIFY( ! iMsg.interpretMessage(msg, data));
-}
-
-/*!
- * \brief   tst_messaging::tst_MessageInterpreter_getPowerOffData_payloadLen
- * \details Tests MessageInterpreter getPowerOffData method Message payload length check
- */
-void tst_messaging::tst_MessageInterpreter_getPowerOffData_payloadLen()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_PowerOff;
-    msg.data = {};
-    Model::MPowerOff data;
-    data.fromByteArray(msg.data);
-    // QVERIFY( ! iMsg.interpretMessage(msg, data));
-}
-
-/*!
  * \brief   tst_messaging::tst_MessageInterpreter_powerOffData
  * \details Tests MessageInterpreter powerOffData method Message payload length check
  */
@@ -110,36 +81,6 @@
 }
 
 /*!
- * \brief   tst_messaging::tst_MessageInterpreter_getBloodFlowData_isType
- * \details Tests MessageInterpreter getBloodFlowData method Message Type checking
- */
-void tst_messaging::tst_MessageInterpreter_getBloodFlowData_isType()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_Unknown;
-    msg.data = {};
-    Model::MBloodFlow data;
-    data.fromByteArray(msg.data);
-    //QVERIFY( ! iMsg.interpretMessage(msg, data));
-}
-
-/*!
- * \brief   tst_messaging::tst_MessageInterpreter_getBloodFlowData_payloadLen
- * \details Tests MessageInterpreter getBloodFlowData method Message payload length check
- */
-void tst_messaging::tst_MessageInterpreter_getBloodFlowData_payloadLen()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_BloodFlow;
-    msg.data = {};
-    Model::MBloodFlow data;
-    data.fromByteArray(msg.data);
-    //QVERIFY( ! iMsg.interpretMessage(msg, data));
-}
-
-/*!
  * \brief   tst_messaging::tst_MessageInterpreter_bloodFlowData
  * \details Tests MessageInterpreter bloodFlowData method Message payload length check
  */
@@ -155,36 +96,6 @@
 }
 
 /*!
- * \brief   tst_messaging::tst_MessageInterpreter_getDialysateInletFlowData_isType
- * \details Tests MessageInterpreter getDialysateInletFlowData method Message Type checking
- */
-void tst_messaging::tst_MessageInterpreter_getDialysateInletFlowData_isType()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_Unknown;
-    msg.data = {};
-    Model::MDialysateFlow data;
-    data.fromByteArray(msg.data);
-    //QVERIFY( ! iMsg.interpretMessage(msg, data));
-}
-
-/*!
- * \brief   tst_messaging::tst_MessageInterpreter_getDialysateInletFlowData_payloadLen
- * \details Tests MessageInterpreter getDialysateInletFlowData method Message payload length check
- */
-void tst_messaging::tst_MessageInterpreter_getDialysateInletFlowData_payloadLen()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_DialysateInletFlow;
-    msg.data = {};
-    Model::MDialysateFlow data;
-    data.fromByteArray(msg.data);
-    //QVERIFY( ! iMsg.interpretMessage(msg, data));
-}
-
-/*!
  * \brief   tst_messaging::tst_MessageInterpreter_dialysateInletFlowData
  * \details Tests MessageInterpreter dialysateInletFlowData method Message payload length check
  */
@@ -200,36 +111,6 @@
 }
 
 /*!
- * \brief   tst_messaging::tst_MessageInterpreter_getDialysateOutletFlowData_isType
- * \details Tests MessageInterpreter getDialysateOutletFlowData method Message Type checking
- */
-void tst_messaging::tst_MessageInterpreter_getDialysateOutletFlowData_isType()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_Unknown;
-    msg.data = {};
-    Model::MOutletFlow data;
-    data.fromByteArray(msg.data);
-    //QVERIFY( ! iMsg.interpretMessage(msg, data));
-}
-
-/*!
- * \brief   tst_messaging::tst_MessageInterpreter_getDialysateOutletFlowData_payloadLen
- * \details Tests MessageInterpreter getDialysateOutletFlowData method Message payload length check
- */
-void tst_messaging::tst_MessageInterpreter_getDialysateOutletFlowData_payloadLen()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_DialysateOutletFlow;
-    msg.data = {};
-    Model::MOutletFlow data;
-    data.fromByteArray(msg.data);
-    //QVERIFY( ! iMsg.interpretMessage(msg, data));
-}
-
-/*!
  * \brief   tst_messaging::tst_MessageInterpreter_dialysateOutletFlowData
  * \details Tests MessageInterpreter dialysateOutletFlowData method Message payload length check
  */
@@ -245,36 +126,6 @@
 }
 
 /*!
- * \brief   tst_messaging::tst_MessageInterpreter_getTreatmentTime_isType
- * \details Tests MessageInterpreter getTreatmentTime method Message Type checking
- */
-void tst_messaging::tst_MessageInterpreter_getTreatmentTime_isType()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_Unknown;
-    msg.data = {};
-    Model::MTreatmentTime data;
-    data.fromByteArray(msg.data);
-    //QVERIFY( ! iMsg.interpretMessage(msg, data));
-}
-
-/*!
- * \brief   tst_messaging::tst_MessageInterpreter_getTreatmentTime_payloadLen
- * \details Tests MessageInterpreter getTreatmentTime method Message payload length check
- */
-void tst_messaging::tst_MessageInterpreter_getTreatmentTime_payloadLen()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_TreatmentTime;
-    msg.data = {};
-    Model::MTreatmentTime data;
-    data.fromByteArray(msg.data);
-    //QVERIFY( ! iMsg.getTreatmentTime(msg, data));
-}
-
-/*!
  * \brief   tst_messaging::tst_MessageInterpreter_treatmentTime
  * \details Tests MessageInterpreter treatmentTime method Message payload length check
  */
@@ -290,81 +141,6 @@
 }
 
 /*!
- * \brief   tst_messaging::tst_MessageInterpreter_getAlarmStatus_isType
- * \details Tests MessageInterpreter getAlarmStatus method Message Type check
- */
-void tst_messaging::tst_MessageInterpreter_getAlarmStatus_isType()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_Unknown;
-    msg.data = {};
-    Model::MAlarmStatus data;
-    data.fromByteArray(msg.data);
-    //QVERIFY( ! iMsg.getAlarmStatus(msg, data));
-}
-
-/*!
- * \brief   tst_messaging::tst_MessageInterpreter_getAlarmStatus_payloadLen
- * \details Test MessageInterpreter getAlarmStatus method Message payload length check
- */
-void tst_messaging::tst_MessageInterpreter_getAlarmStatus_payloadLen()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_AlarmStatus;
-    msg.data = {};
-    Model::MAlarmStatus data;
-    data.fromByteArray(msg.data);
-    //QVERIFY( ! iMsg.getAlarmStatus(msg, data));
-}
-
-/*!
- * \brief   tst_messaging::tst_MessageInterpreter_alarmStatus
- * \details Tests MessageInterpreter alarmStatus method Message payload length check
- */
-void tst_messaging::tst_MessageInterpreter_alarmStatus()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_AlarmStatus;
-    msg.data = {};
-    QVariantList data;
-    QVERIFY( ! iMsg.alarmStatus(msg, data));
-    QVERIFY(   data.isEmpty());
-}
-
-/*!
- * \brief   tst_messaging::tst_MessageInterpreter_getPressureOcclusionData_isType
- * \details Tests MessageInterpreter getPressureOcclusionData method Message Type check
- */
-void tst_messaging::tst_MessageInterpreter_getPressureOcclusionData_isType()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_Unknown;
-    msg.data = {};
-    Model::MPressureOcclusion data;
-    data.fromByteArray(msg.data);
-    //QVERIFY( ! iMsg.getPressureOcclusionData(msg, data));
-}
-
-/*!
- * \brief   tst_messaging::tst_MessageInterpreter_getPressureOcclusionData_payloadLen
- * \details Tests MessageInterpreter getPressureOcclusionData method Message payload length check
- */
-void tst_messaging::tst_MessageInterpreter_getPressureOcclusionData_payloadLen()
-{
-    Can::MessageInterpreter iMsg;
-    Can::Message msg;
-    msg.actionId = Gui::GuiActionType::ID_PressureOcclusion;
-    msg.data = {};
-    Model::MPressureOcclusion data;
-    data.fromByteArray(msg.data);
-    //QVERIFY( ! iMsg.getPressureOcclusionData(msg, data));
-}
-
-/*!
  * \brief   tst_messaging::tst_MessageInterpreter_pressureOcclusionData
  * \details Tests MessageInterpreter pressureOcclusionData method Message payload length check
  */
@@ -941,3 +717,19 @@
     Can::Sequence seq = 1;
     dMsg.actionTransmit(Gui::GuiActionType::ID_KeepAlive, data, seq);
 }
+
+/*!
+ * \brief   tst_messaging::tst_AlarmGenerator_setBits_OutOfRange
+ * \details Tests the longer list of the bits sent to AlarmGenerator::setBits method.
+ */
+void tst_messaging::tst_AlarmGenerator_setBits_OutOfRange()
+{
+    quint8 len = sizeof(AlarmGenerator::TAlarms) * 8;
+    AlarmGenerator ag;
+    AlarmGenerator::TBits bits {0, 15, len}; // 16 is out of range
+    QBitArray ba_actual = ag.setBits(bits);
+    QBitArray ba_expected(len,false);
+    ba_expected.setBit(0);
+    ba_expected.setBit(15);
+    QCOMPARE(ba_actual, ba_expected);
+}