In the testsuite make functions run_and_verify_update(), _switch(),
_commit() and _patch verify passed output and status lists even though
an error is expected.

If the caller intended not to check the value it would/should have passed None
for those things not to check, or the caller should have used a different
function.

This allows using these functions to process results for verifying warning
cases like failed external checkouts that only produce a final error.

* subversion/tests/cmdline/basic_tests.py
  (basic_commit_corruption): Fix test expectations. No values were checked
    here, even though the code assumed that they were.

* subversion/tests/cmdline/svntest/actions.py
  (run_and_verify_update): Continue checking after expected error.
  (run_and_verify_patch): Fix documentation only.
  (process_output_for_commit): Don't verify a committed line on an error.
  (run_and_verify_commit): Don't return success early on an expected error.


git-svn-id: https://svn.apache.org/repos/asf/subversion/trunk@1434617 13f79535-47bb-0310-9956-ffa450edef68