hd_test_script.py

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
RESOLVED

RESOLVED

DIALIN-DEN-4211_HD Treatment Parameters Mode
DIALIN-DEN-4211_HD Treatment Parameters Mode
DIALIN-DEN-4308_HD Valve Control (1 of 2)
DIALIN-DEN-4308_HD Valve Control (1 of 2)
RESOLVED.

RESOLVED.

I removed try and except and added has_value method.

I removed try and except and added has_value method.

RESOLVED

RESOLVED

RESOLVED.

RESOLVED.

Last ")" comes after vPWM)).

Last ")" comes after vPWM)).

This is not just to check the range of the valves. All the other enums that we have such as valves positions and valves states also need to be tried to make sure nothing out of range was selected.

This is not just to check the range of the valves. All the other enums that we have such as valves positions and valves states also need to be tried to make sure nothing out of range was selected.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

Can't we just range check vlv_ID instead of using try?

Can't we just range check vlv_ID instead of using try?

It has been removed.

It has been removed.

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED.

RESOLVED.

RESOLVED

RESOLVED

RESOLVED

RESOLVED

Done

Done

Done

Done

Done

Done

Done

Done

During testing, I have been commenting and uncommenting code to test different parts of code. Since this is in tests, does that really matter?

During testing, I have been commenting and uncommenting code to test different parts of code. Since this is in tests, does that really matter?

During testing, I have been commenting and uncommenting code to test different parts of code. Since this is in tests, does that really matter?

During testing, I have been commenting and uncommenting code to test different parts of code. Since this is in tests, does that really matter?

This handler will be removed.

This handler will be removed.

Any reason to keep this?

Any reason to keep this?