Lines of code count not available
This was updated but not pushed.
This was a mistake. I fixed it.
RESOLVED in CODE WALKTHROUGH
Is this needed? Why would a worn cartridge fail pressure test?
Why isn't this just an else? And if else should be swapped so normal action (alarm) comes first with if and the else should be the exception (else).
I don't think a worn cartridge should fail here (should still be dry). I think only wet cartridge should be checked here.
RESOLVED in CODE WALKTHROUGH.
Recommend being more direct in naming: TEST_CONFIG_USE_WET_CARTRIDGE TEST_CONFIG_USE_WORN_CARTRIDGE
Done.
Remove extra blank line.
Please see changes.
Done
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.