commit 5f451a4676da1173a6b8de5d1a5c76c4dd039bdf
author Yo Chiang <yochiang@google.com> Tue Apr 21 13:36:18 2020 +0800
committer Yo Chiang <yochiang@google.com> Mon May 04 03:23:15 2020 +0000
tree 6411d8c16500bd93dccd8cfba538e4045ee73079
parent 13954964bc9aa0db09138d6f6441981c3a8a3132

repo_pull: Refactor error handling

* repo_review.py requests all go through _make_json_post_request()
* _make_json_post_request() gracefully handles exceptions
* _print_error() prints json or plain text response
* Add some docstrings

Test: ./repo_review.py --add-reviewer [invalid username] [query]
Test: ./repo_review.py --add-reviewer [ambiguous username] [query]
Test: ./repo_review.py --delete-reviewer [invalid username] [query]
Test: ./repo_review.py --delete-topic [query]
Change-Id: I900ef416bb4195d6fb4d72b0c4f53f3fd78a6e7e