Index: unittests/tst_messaging.cpp
===================================================================
diff -u -r634824d6fce314971aa236e508024bd154562c9d -rb798668f16ad0967ab97e96f5f9a2cdd821e899f
--- unittests/tst_messaging.cpp	(.../tst_messaging.cpp)	(revision 634824d6fce314971aa236e508024bd154562c9d)
+++ unittests/tst_messaging.cpp	(.../tst_messaging.cpp)	(revision b798668f16ad0967ab97e96f5f9a2cdd821e899f)
@@ -1,14 +1,14 @@
 /*!
  *
  * Copyright (c) 2019-2020 Diality Inc. - All Rights Reserved.
- * copyright
+ * \copyright
  *          THIS CODE MAY NOT BE COPIED OR REPRODUCED IN ANY FORM,
  *          IN PART OR IN WHOLE,
  *          WITHOUT THE EXPLICIT PERMISSION OF THE COPYRIGHT OWNER.
  *
- * file    tst_messaging.cpp
- * date    4/20/2020
- * author  Behrouz NematiPour
+ * \file    tst_messaging.cpp
+ * \date    4/20/2020
+ * \author  Behrouz NematiPour
  *
  */
 #include "tst_messaging.h"
@@ -24,6 +24,10 @@
 
 tst_messaging::tst_messaging(QObject *parent) : QObject(parent) { }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_isType
+ * \details Tests MessageInterpreter isType method
+ */
 void tst_messaging::tst_MessageInterpreter_isType()
 {
     Can::MessageInterpreter iMsg;
@@ -34,6 +38,10 @@
     QVERIFY(! iMsg.isType(msg, Gui::GuiActionType::BloodFlow));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_isPayloadLenValid
+ * \details Tests MessageInterpreter isPayloadLenValid method
+ */
 void tst_messaging::tst_MessageInterpreter_isPayloadLenValid()
 {
     Can::MessageInterpreter iMsg;
@@ -43,6 +51,10 @@
     QVERIFY( ! iMsg.isPayloadLenValid(msg, Gui::GuiActionType::AdjustBloodDialysateReq));
 }
 
+/*!
+ * \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;
@@ -54,6 +66,10 @@
     QVERIFY( ! iMsg.getPowerOffData(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;
@@ -65,6 +81,10 @@
     QVERIFY( ! iMsg.getPowerOffData(msg, data));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_powerOffData
+ * \details Tests MessageInterpreter powerOffData method Message payload length check
+ */
 void tst_messaging::tst_MessageInterpreter_powerOffData()
 {
     Can::MessageInterpreter iMsg;
@@ -75,6 +95,10 @@
     QVERIFY( ! iMsg.powerOffData(msg, data));
 }
 
+/*!
+ * \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;
@@ -86,6 +110,10 @@
     QVERIFY( ! iMsg.getBloodFlowData(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;
@@ -97,6 +125,10 @@
     QVERIFY( ! iMsg.getBloodFlowData(msg, data));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_bloodFlowData
+ * \details Tests MessageInterpreter bloodFlowData method Message payload length check
+ */
 void tst_messaging::tst_MessageInterpreter_bloodFlowData()
 {
     Can::MessageInterpreter iMsg;
@@ -108,6 +140,10 @@
     QVERIFY(   data.isEmpty());
 }
 
+/*!
+ * \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;
@@ -119,6 +155,10 @@
     QVERIFY( ! iMsg.getDialysateInletFlowData(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;
@@ -130,6 +170,10 @@
     QVERIFY( ! iMsg.getDialysateInletFlowData(msg, data));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_dialysateInletFlowData
+ * \details Tests MessageInterpreter dialysateInletFlowData method Message payload length check
+ */
 void tst_messaging::tst_MessageInterpreter_dialysateInletFlowData()
 {
     Can::MessageInterpreter iMsg;
@@ -141,6 +185,10 @@
     QVERIFY(   data.isEmpty());
 }
 
+/*!
+ * \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;
@@ -152,6 +200,10 @@
     QVERIFY( ! iMsg.getDialysateOutletFlowData(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;
@@ -163,6 +215,10 @@
     QVERIFY( ! iMsg.getDialysateOutletFlowData(msg, data));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_dialysateOutletFlowData
+ * \details Tests MessageInterpreter dialysateOutletFlowData method Message payload length check
+ */
 void tst_messaging::tst_MessageInterpreter_dialysateOutletFlowData()
 {
     Can::MessageInterpreter iMsg;
@@ -174,6 +230,10 @@
     QVERIFY(   data.isEmpty());
 }
 
+/*!
+ * \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;
@@ -185,6 +245,10 @@
     QVERIFY( ! iMsg.getTreatmentTime(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;
@@ -196,6 +260,10 @@
     QVERIFY( ! iMsg.getTreatmentTime(msg, data));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_treatmentTime
+ * \details Tests MessageInterpreter treatmentTime method Message payload length check
+ */
 void tst_messaging::tst_MessageInterpreter_treatmentTime()
 {
     Can::MessageInterpreter iMsg;
@@ -207,6 +275,10 @@
     QVERIFY(   data.isEmpty());
 }
 
+/*!
+ * \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;
@@ -218,6 +290,10 @@
     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;
@@ -229,6 +305,10 @@
     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;
@@ -240,6 +320,10 @@
     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;
@@ -251,6 +335,10 @@
     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;
@@ -262,6 +350,10 @@
     QVERIFY( ! iMsg.getPressureOcclusionData(msg, data));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_pressureOcclusionData
+ * \details Tests MessageInterpreter pressureOcclusionData method Message payload length check
+ */
 void tst_messaging::tst_MessageInterpreter_pressureOcclusionData()
 {
     Can::MessageInterpreter iMsg;
@@ -273,6 +365,10 @@
     QVERIFY(   data.isEmpty());
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_adjustBloodDialysateData_isType
+ * \details Tests MessageInterpreter adjustBloodDialysateData method Message Type check
+ */
 void tst_messaging::tst_MessageInterpreter_adjustBloodDialysateData_isType()
 {
     Can::MessageInterpreter iMsg;
@@ -283,6 +379,10 @@
     QVERIFY( ! iMsg.adjustBloodDialysateData(msg, list));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_adjustBloodDialysateData_payloadLen
+ * \details Tests MessageInterpreter adjustBloodDialysateData method Message payload length check
+ */
 void tst_messaging::tst_MessageInterpreter_adjustBloodDialysateData_payloadLen()
 {
     Can::MessageInterpreter iMsg;
@@ -293,6 +393,10 @@
     QVERIFY( ! iMsg.adjustBloodDialysateData(msg, list));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_adjustDurationData_isType
+ * \details Tests MessageInterpreter adjustDurationData method Message Type check
+ */
 void tst_messaging::tst_MessageInterpreter_adjustDurationData_isType()
 {
     Can::MessageInterpreter iMsg;
@@ -303,6 +407,10 @@
     QVERIFY( ! iMsg.adjustDurationData(msg, list));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_adjustDurationData_payloadLen
+ * \details Tests MessageInterpreter adjustDurationData method Message payload length check
+ */
 void tst_messaging::tst_MessageInterpreter_adjustDurationData_payloadLen()
 {
     Can::MessageInterpreter iMsg;
@@ -313,6 +421,10 @@
     QVERIFY( ! iMsg.adjustDurationData(msg, list));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_treatmentRangesData_isType
+ * \details Tests MessageInterpreter treatmentRangesData method Message Type check
+ */
 void tst_messaging::tst_MessageInterpreter_treatmentRangesData_isType()
 {
     Can::MessageInterpreter iMsg;
@@ -323,6 +435,10 @@
     QVERIFY( ! iMsg.treatmentRangesData(msg, list));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_treatmentRangesData_payloadLen
+ * \details Tests MessageInterpreter treatmentRangesData method Message payload length check
+ */
 void tst_messaging::tst_MessageInterpreter_treatmentRangesData_payloadLen()
 {
     Can::MessageInterpreter iMsg;
@@ -333,6 +449,11 @@
     QVERIFY( ! iMsg.treatmentRangesData(msg, list));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_interpretMessage_String
+ * \details Tests MessageInterpreter interpretMessage to handle String message type
+ *          if data is empty (count==0)
+ */
 void tst_messaging::tst_MessageInterpreter_interpretMessage_String()
 {
     Can::MessageInterpreter iMsg;
@@ -341,6 +462,11 @@
     QVERIFY(payload.isEmpty());
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_interpretMessage_AdjustBloodDialysateReq
+ * \details Tests MessageInterpreter interpretMessage to handle AdjustBloodDialysateReq message type
+ *          if data is empty (count==0)
+ */
 void tst_messaging::tst_MessageInterpreter_interpretMessage_AdjustBloodDialysateReq()
 {
     Can::MessageInterpreter iMsg;
@@ -349,6 +475,11 @@
     QVERIFY(payload.isEmpty());
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageInterpreter_interpretMessage_AdjustDurationReq
+ * \details Tests MessageInterpreter interpretMessage to handle AdjustDurationReq message type
+ *          if data is empty (count==0)
+ */
 void tst_messaging::tst_MessageInterpreter_interpretMessage_AdjustDurationReq()
 {
     Can::MessageInterpreter iMsg;
@@ -357,6 +488,10 @@
     QVERIFY(payload.isEmpty());
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageBuilder_addActionId
+ * \details Tests addActionId() method usage of MessageBuilder
+ */
 void tst_messaging::tst_MessageBuilder_addActionId()
 {
     Can::MessageBuilder bMsg;
@@ -365,6 +500,12 @@
     QVERIFY(   payload.isEmpty());
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageBuilder_addData_shorterLen
+ * \details Tests Data length of shorter than expected
+ *          in addData() method of MessageBuilder
+
+ */
 void tst_messaging::tst_MessageBuilder_addData_shorterLen()
 {
     Can::MessageBuilder bMsg;
@@ -373,6 +514,11 @@
     QVERIFY(   payload.isEmpty());
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageBuilder_addData_LongerThanMax
+ * \details Tests Data length of longer than maximum accepted
+ *          in addData() method of MessageBuilder
+ */
 void tst_messaging::tst_MessageBuilder_addData_LongerThanMax()
 {
     Can::MessageBuilder bMsg;
@@ -385,6 +531,10 @@
     QVERIFY( ! payload.isEmpty());
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageBuilder_checkCRC
+ * \details Tests checkCRC() method usage of MessageBuilder
+ */
 void tst_messaging::tst_MessageBuilder_checkCRC()
 {
     Can::MessageBuilder bMsg;
@@ -393,13 +543,21 @@
     QVERIFY( ! bMsg.checkCRC(data, vExpected, vActual));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageBuilder_getHeader
+ * \details Tests getHeader() method usage of MessageBuilder
+ */
 void tst_messaging::tst_MessageBuilder_getHeader()
 {
     Can::MessageBuilder bMsg;
     QByteArray payload;
     QVERIFY(bMsg.getHeader(payload).isEmpty());
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageBuilder_buildFrames_addActionId
+ * \details Tests addActionId() method usage of MessageBuilder in buildFrames method
+ */
 void tst_messaging::tst_MessageBuilder_buildFrames_addActionId()
 {
     Can::MessageBuilder bMsg;
@@ -409,6 +567,10 @@
     QVERIFY( ! bMsg.buildFrames(Gui::GuiActionType::Unknown, data, framelist, seq));
 }
 
+/*!
+ * \brief   tst_messaging::tst_MessageBuilder_buildFrames_addData
+ * \details Tests addData() method usage of MessageBuilder in buildFrames method
+ */
 void tst_messaging::tst_MessageBuilder_buildFrames_addData()
 {
     Can::MessageBuilder bMsg;
@@ -418,6 +580,10 @@
     QVERIFY( ! bMsg.buildFrames(Gui::GuiActionType::PowerOff, data, framelist, seq));
 }
 
+/*!
+ * \brief tst_messaging::tst_MessageBuilder_buildFrames_eLenCanFrame
+ * \details Tests canFrameLength check in buildFrames method of MessageBuilder
+ */
 void tst_messaging::tst_MessageBuilder_buildFrames_eLenCanFrame()
 {
     Can::MessageBuilder bMsg;
@@ -437,7 +603,11 @@
     QVERIFY( bMsg.buildFrames(Gui::GuiActionType::String, data, framelist, seq));
 }
 
-void tst_messaging::tst_MessageBuilder_actionTransmit()
+/*!
+ * \brief   tst_messaging::tst_MessageDispatcher_actionTransmit
+ * \details Tests MessageDispatcher actionTransmit()
+ */
+void tst_messaging::tst_MessageDispatcher_actionTransmit()
 {
     Can::MessageDispatcher dMsg;
     QVariantList data;