commit 5104f3ccda700d3651e76ec6eded790b48e8edbb
author Luigi Semenzato <semenzato@chromium.org> Wed Oct 12 12:37:42 2016 -0700
committer chrome-bot <chrome-bot@chromium.org> Mon Oct 17 15:24:25 2016 -0700
tree 9395716e7519f885cdbdb628a279ffd0e5fa5c62
parent 7c32ccd704eeed9f505f82e80f19d86aaff71071

chromite: log retries of debug symbol file uploads

BUG=chromium:649791
TEST=none

Change-Id: I5bbc09d571aa145475a123293e625a93cfcbe2a1
Reviewed-on: https://chromium-review.googlesource.com/397638
Commit-Ready: Luigi Semenzato <semenzato@chromium.org>
Tested-by: Luigi Semenzato <semenzato@chromium.org>
Reviewed-by: Dan Shi <dshi@google.com>

scripts/upload_symbols_unittest.py

diff --git a/scripts/upload_symbols_unittest.py b/scripts/upload_symbols_unittest.py
index 4544a15..ac074be 100644
--- a/scripts/upload_symbols_unittest.py
+++ b/scripts/upload_symbols_unittest.py
@@ -615,7 +615,9 @@
       if symbol.file_name == fail_file:
         raise urllib2.URLError('network failure')
 
-    self.PatchObject(upload_symbols, 'UploadSymbolFile', side_effect=failSome)
+    upload_mock = self.PatchObject(upload_symbols, 'UploadSymbolFile',
+                                   side_effect=failSome)
+    upload_mock.__name__ = 'UploadSymbolFileMock2'
 
     result = upload_symbols.PerformSymbolsFileUpload(
         symbols, 'fake_url', product_name='product')
@@ -692,6 +694,8 @@
     # Mock out UploadSymbolFile so it's easy to see which file to fail for.
     upload_mock = self.PatchObject(upload_symbols, 'UploadSymbolFile',
                                    side_effect=failSome)
+    # Mock __name__ for logging.
+    upload_mock.__name__ = 'UploadSymbolFileMock'
 
     result = upload_symbols.UploadSymbols(
         [self.data], 'fake_url', 'product',