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
Actually, I should have removed that proc, because I no longer use it.  I went down the route of having the  the return_code method for exactly those reasons (not being able to count on in the tests having the CLI processing happening).    But with that method, the values of that Proc are no longer used.  Are you ok with me just removing it?