https://cs.chromium.org/chromium/infra/appengine/findit/infra_api_clients/codereview/cl_info.py has a lot of "# pragma: no cover" which we should get rid of.
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/8fd92d8dd2bef0c68869e9d64a68428d644bc262 commit 8fd92d8dd2bef0c68869e9d64a68428d644bc262 Author: Roberto Carrillo <robertocn@google.com> Date: Sat Jun 10 05:37:13 2017 [Findit] Add unit tests for cl_info.py R=lijeffrey,chanli,stgao BUG= 731176 Change-Id: I502bf68448221992e5023a129710c70c70f04728 Reviewed-on: https://chromium-review.googlesource.com/528480 Commit-Queue: Roberto Carrillo <robertocn@chromium.org> Reviewed-by: Shuotao Gao <stgao@chromium.org> Reviewed-by: Jeffrey Li <lijeffrey@chromium.org> [add] https://crrev.com/8fd92d8dd2bef0c68869e9d64a68428d644bc262/appengine/findit/infra_api_clients/codereview/test/cl_info_test.py [modify] https://crrev.com/8fd92d8dd2bef0c68869e9d64a68428d644bc262/appengine/findit/infra_api_clients/codereview/cl_info.py
Comment 1 by robert...@chromium.org
, Jun 8 2017