I think I understand how this works now. Kind of confusing how this is done and strange to see passed set here when it just failed. If I understand correctly, test must fail 3 out of the 5 tests to fail the whole test - otherwise it passes?
Do we need this timeout to be changed as task interval counter as we were seeing additional 100ms debounce time at times (hence adjusted the requirement)?
Sending event here is problematic now that we will likely defer zeroing until reservoir switch. It will send every 50ms until we finally zero. Move this to location where we are actually going to zero OR just remove the event as its maybe no longer applicable.
If a zero is requested, it is checked first and then this flag is reset. I also want this flag to be reset all the time so we do not zero on old reservoir.
Remove all POST tests that look like HD POST tests for now. Since we're not doing any POST tests initially, we don't need to worry too much about this enum yet.
Can these fields be a value that is not 0 or 1? Seems weird to have them be unpacked and casted as a bool only to have the override parameters have int instead of bool.
This function is a slot to debug the behavior of the power dialog for now The entire qml file may later be removed. We had some conversations to combine the power message into confirmation and remove the power qml, models, and so on.