Raise an exception if a unittest target is missing.

Raise an exception if a unittest target is missing.

BUG=163653
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=170592

[Isaac (away), 2012-11-30 22:53:52]

https://codereview.chromium.org/11421187/diff/1/build/android/run_tests.py
File build/android/run_tests.py (right):

https://codereview.chromium.org/11421187/diff/1/build/android/run_tests.py#ne...
build/android/run_tests.py:120: sys.exit(1)
lets raise an exception instead of calling sys.exit.  Don't like calling
sys.exit in subfunctions.

[frankf, 2012-11-30 23:02:00]

PTAL

https://codereview.chromium.org/11421187/diff/1/build/android/run_tests.py
File build/android/run_tests.py (right):

https://codereview.chromium.org/11421187/diff/1/build/android/run_tests.py#ne...
build/android/run_tests.py:120: sys.exit(1)
Done.

On 2012/11/30 22:53:52, Isaac wrote:
> lets raise an exception instead of calling sys.exit.  Don't like calling
> sys.exit in subfunctions.

[Isaac (away), 2012-11-30 23:03:16]

lgtm

[Isaac (away), 2012-12-01 01:41:25]

+jamesr FYI, this fixes the issue where the bot silently fails the to run tests,
which let your original CL pass CQ.

[jamesr, 2012-12-01 01:42:03]

On 2012/12/01 01:41:25, Isaac wrote:
> +jamesr FYI, this fixes the issue where the bot silently fails the to run
tests,
> which let your original CL pass CQ.

OK great.  Protecting yourself from my clumsiness is definitely a good idea :)