Is this really a "test" function? It does not have the correct Dialin prototype (returns BOOL and takes a message as param) and it does not appear to handle a request from or respond to Dialin.
I don't think we want this enum. There is already a VALVE_T in RotaryValve.h that lists the 2 pinch valves in TD. These are the 4 pinch valves in Denali HD.
Ok (But feels weird to start the counting of existing items from 0 if that's the spare3 instead of 1. Calling the D79 the "zeroeth" spare valve instead of the first spare valve... Personally I think we should name them Spare1-8 instead of Spare 0-7)