commit 5117e3eb723645e48e004589123af8fb5e0787cf
author Chris Masone <cmasone@chromium.org> Wed May 09 17:08:20 2012 -0700
committer Gerrit <chrome-bot@google.com> Fri May 11 16:00:12 2012 -0700
tree 70c05c013a5e97131016096040f1f0b5df712692
parent ae4d278eaab0c998c11532332ba062729db95b53

[autotest] Put dev server error strings in exceptions

We've been ignoring error messages coming back from the dev server.
Strip out the HTML that comes in (some of) them and wrap them
in exceptions.

BUG=chromium-os:30279
TEST=unit
TEST=use atest suite create to run a suite that doesn't exist on build that doesn't exist.

Change-Id: I86b149610bf54334cdca12836a078ab874c818b2
Reviewed-on: https://gerrit.chromium.org/gerrit/22315
Commit-Ready: Chris Masone <cmasone@chromium.org>
Reviewed-by: Chris Masone <cmasone@chromium.org>
Tested-by: Chris Masone <cmasone@chromium.org>

frontend/afe/site_rpc_interface.py

diff --git a/frontend/afe/site_rpc_interface.py b/frontend/afe/site_rpc_interface.py
index 818f3d1..f82731b 100644
--- a/frontend/afe/site_rpc_interface.py
+++ b/frontend/afe/site_rpc_interface.py
@@ -71,8 +71,10 @@
     ds = dev_server.DevServer.create()
     timings['download_started_time'] = datetime.datetime.now().strftime(
         time_fmt)
-    if not ds.trigger_download(build, synchronous=False):
-        raise error.StageBuildFailure("Server error while staging " + build)
+    try:
+        ds.trigger_download(build, synchronous=False)
+    except dev_server.DevServerException as e:
+        raise error.StageBuildFailure(e)
     timings['payload_finished_time'] = datetime.datetime.now().strftime(
         time_fmt)