not is testing for a normal exit with a non-zero value. It shouldn't