add autoEjectTimerCounter to outputs

As per MISRA 15.7, All if, else if constructs shall be terminated with an else statement.

As per MISRA 15.7, All if, else if constructs shall be terminated with an else statement.

change to ///< Tubing set install state

Put it back

There should be a \b before the type of details (Inputs and Outputs here) for doxygen update for all

@details \b Outputs:
Please make sure to update all of them to Leahi format In this case, there should be a \b before the type of details (Inputs and Outputs here) so they will be bold when doxygen documentation is generated.

@details \b Inputs:

Can we please add more comments in this function to explain the rationale behind the checks and the actions?

add a space between message and )

Change // to /// for doxygen to catch it.

Can we remove the parameter and switch directly over currentPostTreatmentState?

The public function name signalGoToInstallState is not sufficiently descriptive. Would renaming it to requestTubeSetInstall() be appropriate?

I prefer you create a local BOOL result so you can have a simpler return statement.

Should go to your new complete state here.

The number looks incorrect. This was an issue in the staging branch, which is fixed now. Can you please pull latest staging and update the values of your enums?

check alarm status instead of switch state.

Uncommented

Remove extra blank line.

Why is this service prefixed with "Tx"?
Also, Praneeth and I agreed to put these state machine services into a sub-folder of Services. See how he did it and do the same.

DrySelfTests unit.

DrySelfTests unit initialized.

Remove extra blank line.

.. handles dry self-tests execution for pre-treatment and treatment modes.

Should be in "", not <>. And should be placed alphabetically down before TxParams.h.

Calling signal function here seems circular (that's how we got here in the first place).
Shouldn't we set state to the install state here instead?