commit 4d9c4d49a14ae60dc535e56f30f2e5b103ca327f
author Chris Sosa <sosa@chromium.org> Fri Jun 29 15:23:23 2012 -0700
committer Gerrit <chrome-bot@google.com> Mon Jul 02 12:24:45 2012 -0700
tree 39d0fc1b5bb7e00f2b28f8987f564089fe9ae422
parent f097564a77a6dac40c0e14c10fe087185e0e2828

Fix bug where we don't re-raise an exception in download.

This bug was introduced such that we could return 200 without
'Success'.

I also renamed the devserver unittests to be consistent with the others.

BUG=chromium-os:32072
TEST=devserver_unittest.py

Change-Id: Ibe1c4e1c8506ea919430297e018de98c6b824fb2
Reviewed-on: https://gerrit.chromium.org/gerrit/26497
Tested-by: Chris Sosa <sosa@chromium.org>
Reviewed-by: Chris Masone <cmasone@chromium.org>
Commit-Ready: Chris Sosa <sosa@chromium.org>

downloader.py

diff --git a/downloader.py b/downloader.py
index beb57c2..7f53ec3 100755
--- a/downloader.py
+++ b/downloader.py
@@ -255,6 +255,7 @@
       raise
     finally:
       self._Cleanup()
+
     return 'Success'
 
   def GatherArtifactDownloads(self, temp_download_dir, archive_url, short_build,