Make bootstrap with winrm have correct exit code with failure, and change default behavior of winrm command to fail on non zero exit code

Our integration tests correctly fail on non windows nodes when we bootstrap new nodes if there are recipe failures, but windows doesn't behave the same way.  This sets as default failure on the "workstation" if the bootstrap or other commands fail
It indeed is set always, although I fundamentally like those null checks just for future changes.  That being said, i'm okay with removing that clause if you want