Add --testing and --testing-file options to post-review

Review Request #606 — Created Oct. 22, 2008 and submitted

Information

Review Board SVN (deprecated)

Reviewers

Added options to post-review to fill in the Testing field from the command line. 
  --testing=TESTING_DONE
                        details of testing done
  --testing-file=TESTING_FILE
                        text file containing details of testing done
Ran the following:
1) post-review --help to check usage message.
2) Ran with --testing="Ran ran all of the tests" to check specified text appeared in the Testing field of the generated review.
3) Ran with --testing="text" AND --testing-file="/tmp/file" to check mutually exclusive test.
4) Ran with --testing-file="/tmp/does_not_exist" to check exit on missing file.
5) Ran with --testing-file="/tmp/test.txt" to check file content loaded correctly.
chipx86
  1. Looks good, though for consistency I renamed "--testing" and "--testing-file" to "--testing-done" and "--testing-done-file" to be consistent with the web API and UI. It may be more wordy but since this will usually be used by automation scripts, that's probably not a big deal.
    
    Committed in r1555.
  2.