test suite changes related to scancel

Modify all tests to use cancel_job() rather than "exec scancel ..."
  so that we can do better error handling all in one place.
There were several places in the globals module that printed "FAILURE"
  and set a local "exit_code" variable. I added "global exit_code"
  to those functions in hopes of those failures being reflected
  in the test's exit code.