tree dc8ea636c8c265fb783f93e84586b843a1cc79cf
parent 2136ce1397bab98e9fe09d351c683c07d513eb53
author Gilad Arnold <garnold@chromium.org> 1384457274 -0800
committer Mike Frysinger <vapier@chromium.org> 1384759485 +0000

devserver: fix exception related discrepancies

* DeltaPayloadsArtifact._Setup() neglected to (re-)raise and exception
  when an error occurred. This contradicts the docstring and a likely
  cause for inconsistencies surrounding missing artifact files.

* BuildArtifact.Process() caught and re-raised any exception, although
  it's only supposed to be raising BuildArtifactError. We now convert
  any unknown exception into the latter (albeit losing the stack trace).
  This necessitates that we adjust our handling of exception at call
  sites for build_artifact, too.

BUG=chromium:277839
TEST=Unit tests.

Change-Id: I30a87e9722460fdbfab54c3ff60e1d7a8013b3cb
Reviewed-on: https://chromium-review.googlesource.com/176932
Tested-by: Gilad Arnold <garnold@chromium.org>
Reviewed-by: Dan Shi <dshi@chromium.org>