Mike Frysinger | e58c0e2 | 2017-10-04 15:43:30 -0400 | [diff] [blame] | 1 | # -*- coding: utf-8 -*- |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 2 | # Copyright (c) 2013 The Chromium OS Authors. All rights reserved. |
| 3 | # Use of this source code is governed by a BSD-style license that can be |
| 4 | # found in the LICENSE file. |
| 5 | |
Mike Frysinger | eb753bf | 2013-11-22 16:05:35 -0500 | [diff] [blame] | 6 | """Unittests for upload_symbols.py""" |
| 7 | |
Mike Frysinger | 7f9be14 | 2014-01-15 02:16:42 -0500 | [diff] [blame] | 8 | from __future__ import print_function |
| 9 | |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 10 | import BaseHTTPServer |
Mike Frysinger | 06da520 | 2014-09-26 17:30:33 -0500 | [diff] [blame] | 11 | import errno |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 12 | import itertools |
Mike Frysinger | ea838d1 | 2014-12-08 11:55:32 -0500 | [diff] [blame] | 13 | import mock |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 14 | import os |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 15 | import signal |
Mike Frysinger | 06da520 | 2014-09-26 17:30:33 -0500 | [diff] [blame] | 16 | import socket |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 17 | import SocketServer |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 18 | import sys |
Aviv Keshet | d1f0463 | 2014-05-09 11:33:46 -0700 | [diff] [blame] | 19 | import time |
Mike Frysinger | 094a217 | 2013-08-14 12:54:35 -0400 | [diff] [blame] | 20 | import urllib2 |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 21 | |
Mike Frysinger | 079863c | 2014-10-09 23:16:46 -0400 | [diff] [blame] | 22 | # We specifically set up a local server to connect to, so make sure we |
| 23 | # delete any proxy settings that might screw that up. We also need to |
| 24 | # do it here because modules that are imported below will implicitly |
| 25 | # initialize with this proxy setting rather than dynamically pull it |
| 26 | # on the fly :(. |
| 27 | os.environ.pop('http_proxy', None) |
| 28 | |
Aviv Keshet | b7519e1 | 2016-10-04 00:50:00 -0700 | [diff] [blame] | 29 | from chromite.lib import constants |
Mike Frysinger | bbd1f11 | 2016-09-08 18:25:11 -0400 | [diff] [blame] | 30 | |
| 31 | # The isolateserver includes a bunch of third_party python packages that clash |
| 32 | # with chromite's bundled third_party python packages (like oauth2client). |
| 33 | # Since upload_symbols is not imported in to other parts of chromite, and there |
| 34 | # are no deps in third_party we care about, purge the chromite copy. This way |
| 35 | # we can use isolateserver for deduping. |
| 36 | # TODO: If we ever sort out third_party/ handling and make it per-script opt-in, |
| 37 | # we can purge this logic. |
| 38 | third_party = os.path.join(constants.CHROMITE_DIR, 'third_party') |
| 39 | while True: |
| 40 | try: |
| 41 | sys.path.remove(third_party) |
| 42 | except ValueError: |
| 43 | break |
Mike Frysinger | bbd1f11 | 2016-09-08 18:25:11 -0400 | [diff] [blame] | 44 | del third_party |
| 45 | |
Ralph Nathan | 446aee9 | 2015-03-23 14:44:56 -0700 | [diff] [blame] | 46 | from chromite.lib import cros_logging as logging |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 47 | from chromite.lib import cros_test_lib |
| 48 | from chromite.lib import osutils |
| 49 | from chromite.lib import parallel |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 50 | from chromite.lib import remote_access |
Mike Frysinger | 5e30a4b | 2014-02-12 20:23:04 -0500 | [diff] [blame] | 51 | from chromite.scripts import cros_generate_breakpad_symbols |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 52 | from chromite.scripts import upload_symbols |
| 53 | |
Mike Frysinger | 0c0efa2 | 2014-02-09 23:32:23 -0500 | [diff] [blame] | 54 | |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 55 | class SymbolsTestBase(cros_test_lib.MockTempDirTestCase): |
| 56 | """Base class for most symbols tests.""" |
| 57 | |
| 58 | SLIM_CONTENT = """ |
| 59 | some junk |
| 60 | """ |
| 61 | |
| 62 | FAT_CONTENT = """ |
| 63 | STACK CFI 1234 |
| 64 | some junk |
| 65 | STACK CFI 1234 |
| 66 | """ |
| 67 | |
| 68 | def setUp(self): |
| 69 | # Make certain we don't use the network. |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 70 | self.urlopen_mock = self.PatchObject(urllib2, 'urlopen') |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 71 | self.request_mock = self.PatchObject(upload_symbols, 'ExecRequest', |
| 72 | return_value={'uploadUrl': |
| 73 | 'testurl', |
| 74 | 'uploadKey': |
| 75 | 'asdgasgas'}) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 76 | |
| 77 | # Make 'uploads' go fast. |
| 78 | self.PatchObject(upload_symbols, 'SLEEP_DELAY', 0) |
| 79 | self.PatchObject(upload_symbols, 'INITIAL_RETRY_DELAY', 0) |
| 80 | |
| 81 | # So our symbol file content doesn't have to be real. |
| 82 | self.PatchObject(cros_generate_breakpad_symbols, 'ReadSymsHeader', |
| 83 | return_value=cros_generate_breakpad_symbols.SymbolHeader( |
| 84 | os='os', cpu='cpu', id='id', name='name')) |
| 85 | |
| 86 | self.working = os.path.join(self.tempdir, 'expand') |
| 87 | osutils.SafeMakedirs(self.working) |
| 88 | |
| 89 | self.data = os.path.join(self.tempdir, 'data') |
| 90 | osutils.SafeMakedirs(self.data) |
| 91 | |
| 92 | def createSymbolFile(self, filename, content=FAT_CONTENT, size=0, |
| 93 | status=None, dedupe=False): |
| 94 | fullname = os.path.join(self.data, filename) |
| 95 | osutils.SafeMakedirs(os.path.dirname(fullname)) |
| 96 | |
| 97 | # If a file size is given, force that to be the minimum file size. Create |
| 98 | # a sparse file so large files are practical. |
| 99 | with open(fullname, 'w+b') as f: |
| 100 | f.truncate(size) |
| 101 | f.seek(0) |
| 102 | f.write(content) |
| 103 | |
| 104 | result = upload_symbols.SymbolFile(display_path=filename, |
| 105 | file_name=fullname) |
| 106 | |
| 107 | if status: |
| 108 | result.status = status |
| 109 | |
| 110 | if dedupe: |
| 111 | result.dedupe_item = upload_symbols.DedupeItem(result) |
| 112 | result.dedupe_push_state = 'push_state' |
| 113 | |
| 114 | return result |
| 115 | |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 116 | |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 117 | class SymbolServerRequestHandler(BaseHTTPServer.BaseHTTPRequestHandler): |
| 118 | """HTTP handler for symbol POSTs""" |
| 119 | |
| 120 | RESP_CODE = None |
| 121 | RESP_MSG = None |
| 122 | |
| 123 | def do_POST(self): |
| 124 | """Handle a POST request""" |
| 125 | # Drain the data from the client. If we don't, we might write the response |
| 126 | # and close the socket before the client finishes, so they die with EPIPE. |
| 127 | clen = int(self.headers.get('Content-Length', '0')) |
| 128 | self.rfile.read(clen) |
| 129 | |
| 130 | self.send_response(self.RESP_CODE, self.RESP_MSG) |
| 131 | self.end_headers() |
| 132 | |
| 133 | def log_message(self, *args, **kwargs): |
| 134 | """Stub the logger as it writes to stderr""" |
| 135 | pass |
| 136 | |
| 137 | |
| 138 | class SymbolServer(SocketServer.ThreadingTCPServer, BaseHTTPServer.HTTPServer): |
| 139 | """Simple HTTP server that forks each request""" |
| 140 | |
| 141 | |
| 142 | class UploadSymbolsServerTest(cros_test_lib.MockTempDirTestCase): |
| 143 | """Tests for UploadSymbols() and a local HTTP server""" |
| 144 | |
| 145 | SYM_CONTENTS = """MODULE Linux arm 123-456 blkid |
| 146 | PUBLIC 1471 0 main""" |
| 147 | |
| 148 | def SpawnServer(self, RequestHandler): |
| 149 | """Spawn a new http server""" |
Mike Frysinger | 06da520 | 2014-09-26 17:30:33 -0500 | [diff] [blame] | 150 | while True: |
| 151 | try: |
| 152 | port = remote_access.GetUnusedPort() |
| 153 | address = ('', port) |
| 154 | self.httpd = SymbolServer(address, RequestHandler) |
| 155 | break |
| 156 | except socket.error as e: |
| 157 | if e.errno == errno.EADDRINUSE: |
| 158 | continue |
| 159 | raise |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 160 | self.server_url = 'http://localhost:%i/post/path' % port |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 161 | self.httpd_pid = os.fork() |
| 162 | if self.httpd_pid == 0: |
| 163 | self.httpd.serve_forever(poll_interval=0.1) |
| 164 | sys.exit(0) |
| 165 | |
| 166 | def setUp(self): |
| 167 | self.httpd_pid = None |
| 168 | self.httpd = None |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 169 | self.server_url = None |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 170 | self.sym_file = os.path.join(self.tempdir, 'test.sym') |
| 171 | osutils.WriteFile(self.sym_file, self.SYM_CONTENTS) |
| 172 | |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 173 | # Stop sleeps and retries for these tests. |
| 174 | self.PatchObject(upload_symbols, 'SLEEP_DELAY', 0) |
| 175 | self.PatchObject(upload_symbols, 'INITIAL_RETRY_DELAY', 0) |
| 176 | self.PatchObject(upload_symbols, 'MAX_RETRIES', 0) |
| 177 | |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 178 | def tearDown(self): |
| 179 | # Only kill the server if we forked one. |
| 180 | if self.httpd_pid: |
| 181 | os.kill(self.httpd_pid, signal.SIGUSR1) |
| 182 | |
| 183 | def testSuccess(self): |
| 184 | """The server returns success for all uploads""" |
| 185 | class Handler(SymbolServerRequestHandler): |
| 186 | """Always return 200""" |
| 187 | RESP_CODE = 200 |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 188 | self.PatchObject(upload_symbols, 'ExecRequest', |
| 189 | return_value={'uploadUrl': 'testurl', |
| 190 | 'uploadKey': 'testSuccess'}) |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 191 | |
| 192 | self.SpawnServer(Handler) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 193 | ret = upload_symbols.UploadSymbols( |
| 194 | sym_paths=[self.sym_file] * 10, |
| 195 | upload_url=self.server_url, |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 196 | api_key='testSuccess') |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 197 | self.assertEqual(ret, 0) |
| 198 | |
| 199 | def testError(self): |
| 200 | """The server returns errors for all uploads""" |
| 201 | class Handler(SymbolServerRequestHandler): |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 202 | """All connections error""" |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 203 | RESP_CODE = 500 |
| 204 | RESP_MSG = 'Internal Server Error' |
| 205 | |
| 206 | self.SpawnServer(Handler) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 207 | ret = upload_symbols.UploadSymbols( |
| 208 | sym_paths=[self.sym_file] * 10, |
| 209 | upload_url=self.server_url, |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 210 | api_key='testkey') |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 211 | self.assertEqual(ret, 10) |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 212 | |
| 213 | def testHungServer(self): |
| 214 | """The server chokes, but we recover""" |
| 215 | class Handler(SymbolServerRequestHandler): |
| 216 | """All connections choke forever""" |
| 217 | def do_POST(self): |
| 218 | while True: |
| 219 | time.sleep(1000) |
| 220 | |
| 221 | self.SpawnServer(Handler) |
| 222 | with mock.patch.object(upload_symbols, 'GetUploadTimeout') as m: |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 223 | m.return_value = 0.01 |
Mike Frysinger | 58312e9 | 2014-03-18 04:18:36 -0400 | [diff] [blame] | 224 | ret = upload_symbols.UploadSymbols( |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 225 | sym_paths=[self.sym_file] * 10, |
| 226 | upload_url=self.server_url, |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 227 | api_key='testkey') |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 228 | self.assertEqual(ret, 10) |
Aviv Keshet | d1f0463 | 2014-05-09 11:33:46 -0700 | [diff] [blame] | 229 | |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 230 | |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 231 | class UploadSymbolsHelpersTest(cros_test_lib.TestCase): |
| 232 | """Test assorted helper functions and classes.""" |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 233 | def testIsTarball(self): |
| 234 | notTar = [ |
| 235 | '/foo/bar/test.bin', |
| 236 | '/foo/bar/test.tar.bin', |
| 237 | '/foo/bar/test.faketar.gz', |
| 238 | '/foo/bar/test.nottgz', |
Mike Frysinger | 0c0efa2 | 2014-02-09 23:32:23 -0500 | [diff] [blame] | 239 | ] |
Mike Frysinger | 0c0efa2 | 2014-02-09 23:32:23 -0500 | [diff] [blame] | 240 | |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 241 | isTar = [ |
| 242 | '/foo/bar/test.tar', |
| 243 | '/foo/bar/test.bin.tar', |
| 244 | '/foo/bar/test.bin.tar.bz2', |
| 245 | '/foo/bar/test.bin.tar.gz', |
| 246 | '/foo/bar/test.bin.tar.xz', |
| 247 | '/foo/bar/test.tbz2', |
| 248 | '/foo/bar/test.tbz', |
| 249 | '/foo/bar/test.tgz', |
| 250 | '/foo/bar/test.txz', |
| 251 | ] |
Mike Frysinger | 0c0efa2 | 2014-02-09 23:32:23 -0500 | [diff] [blame] | 252 | |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 253 | for p in notTar: |
| 254 | self.assertFalse(upload_symbols.IsTarball(p)) |
| 255 | |
| 256 | for p in isTar: |
| 257 | self.assertTrue(upload_symbols.IsTarball(p)) |
| 258 | |
| 259 | def testBatchGenerator(self): |
| 260 | result = upload_symbols.BatchGenerator([], 2) |
| 261 | self.assertEqual(list(result), []) |
| 262 | |
| 263 | result = upload_symbols.BatchGenerator(xrange(6), 2) |
| 264 | self.assertEqual(list(result), [[0, 1], [2, 3], [4, 5]]) |
| 265 | |
| 266 | result = upload_symbols.BatchGenerator(xrange(7), 2) |
| 267 | self.assertEqual(list(result), [[0, 1], [2, 3], [4, 5], [6]]) |
| 268 | |
| 269 | # Prove that we are streaming the results, not generating them all at once. |
| 270 | result = upload_symbols.BatchGenerator(itertools.repeat(0), 2) |
| 271 | self.assertEqual(result.next(), [0, 0]) |
| 272 | |
| 273 | |
| 274 | class FindSymbolFilesTest(SymbolsTestBase): |
| 275 | """Test FindSymbolFiles.""" |
| 276 | def setUp(self): |
| 277 | self.symfile = self.createSymbolFile('root.sym').file_name |
| 278 | self.innerfile = self.createSymbolFile( |
| 279 | os.path.join('nested', 'inner.sym')).file_name |
| 280 | |
| 281 | # CreateTarball is having issues outside the chroot from open file tests. |
| 282 | # |
| 283 | # self.tarball = os.path.join(self.tempdir, 'syms.tar.gz') |
| 284 | # cros_build_lib.CreateTarball( |
| 285 | # 'syms.tar.gz', self.tempdir, inputs=(self.data)) |
| 286 | |
| 287 | def testEmpty(self): |
| 288 | symbols = list(upload_symbols.FindSymbolFiles( |
| 289 | self.working, [])) |
| 290 | self.assertEqual(symbols, []) |
| 291 | |
| 292 | def testFile(self): |
| 293 | symbols = list(upload_symbols.FindSymbolFiles( |
| 294 | self.working, [self.symfile])) |
| 295 | |
| 296 | self.assertEqual(len(symbols), 1) |
| 297 | sf = symbols[0] |
| 298 | |
| 299 | self.assertEqual(sf.display_name, 'root.sym') |
| 300 | self.assertEqual(sf.display_path, self.symfile) |
| 301 | self.assertEqual(sf.file_name, self.symfile) |
| 302 | self.assertEqual(sf.status, upload_symbols.SymbolFile.INITIAL) |
| 303 | self.assertEqual(sf.FileSize(), len(self.FAT_CONTENT)) |
| 304 | |
| 305 | def testDir(self): |
| 306 | symbols = list(upload_symbols.FindSymbolFiles( |
| 307 | self.working, [self.data])) |
| 308 | |
| 309 | self.assertEqual(len(symbols), 2) |
| 310 | root = symbols[0] |
| 311 | nested = symbols[1] |
| 312 | |
| 313 | self.assertEqual(root.display_name, 'root.sym') |
| 314 | self.assertEqual(root.display_path, 'root.sym') |
| 315 | self.assertEqual(root.file_name, self.symfile) |
| 316 | self.assertEqual(root.status, upload_symbols.SymbolFile.INITIAL) |
| 317 | self.assertEqual(root.FileSize(), len(self.FAT_CONTENT)) |
| 318 | |
| 319 | self.assertEqual(nested.display_name, 'inner.sym') |
| 320 | self.assertEqual(nested.display_path, 'nested/inner.sym') |
| 321 | self.assertEqual(nested.file_name, self.innerfile) |
| 322 | self.assertEqual(nested.status, upload_symbols.SymbolFile.INITIAL) |
| 323 | self.assertEqual(nested.FileSize(), len(self.FAT_CONTENT)) |
| 324 | |
| 325 | |
| 326 | class AdjustSymbolFileSizeTest(SymbolsTestBase): |
| 327 | """Test AdjustSymbolFileSize.""" |
| 328 | def setUp(self): |
| 329 | self.slim = self.createSymbolFile('slim.sym', self.SLIM_CONTENT) |
| 330 | self.fat = self.createSymbolFile('fat.sym', self.FAT_CONTENT) |
| 331 | |
| 332 | self.warn_mock = self.PatchObject(logging, 'PrintBuildbotStepWarnings') |
| 333 | |
| 334 | def _testNotStripped(self, symbol, size=None, content=None): |
| 335 | start_file = symbol.file_name |
| 336 | after = upload_symbols.AdjustSymbolFileSize( |
| 337 | symbol, self.working, size) |
| 338 | self.assertIs(after, symbol) |
| 339 | self.assertEqual(after.file_name, start_file) |
| 340 | if content is not None: |
| 341 | self.assertEqual(osutils.ReadFile(after.file_name), content) |
| 342 | |
| 343 | def _testStripped(self, symbol, size=None, content=None): |
| 344 | after = upload_symbols.AdjustSymbolFileSize( |
| 345 | symbol, self.working, size) |
| 346 | self.assertIs(after, symbol) |
| 347 | self.assertTrue(after.file_name.startswith(self.working)) |
| 348 | if content is not None: |
| 349 | self.assertEqual(osutils.ReadFile(after.file_name), content) |
| 350 | |
| 351 | def testSmall(self): |
| 352 | """Ensure that files smaller than the limit are not modified.""" |
| 353 | self._testNotStripped(self.slim, 1024, self.SLIM_CONTENT) |
| 354 | self._testNotStripped(self.fat, 1024, self.FAT_CONTENT) |
| 355 | |
| 356 | def testLarge(self): |
| 357 | """Ensure that files larger than the limit are modified.""" |
| 358 | self._testStripped(self.slim, 1, self.SLIM_CONTENT) |
| 359 | self._testStripped(self.fat, 1, self.SLIM_CONTENT) |
| 360 | |
| 361 | def testMixed(self): |
| 362 | """Test mix of large and small.""" |
| 363 | strip_size = len(self.SLIM_CONTENT) + 1 |
| 364 | |
| 365 | self._testNotStripped(self.slim, strip_size, self.SLIM_CONTENT) |
| 366 | self._testStripped(self.fat, strip_size, self.SLIM_CONTENT) |
| 367 | |
| 368 | def testSizeWarnings(self): |
| 369 | large = self.createSymbolFile( |
| 370 | 'large.sym', content=self.SLIM_CONTENT, |
| 371 | size=upload_symbols.CRASH_SERVER_FILE_LIMIT*2) |
| 372 | |
| 373 | # Would like to Strip as part of this test, but that really copies all |
| 374 | # of the sparse file content, which is too expensive for a unittest. |
| 375 | self._testNotStripped(large, None, None) |
| 376 | |
| 377 | self.assertEqual(self.warn_mock.call_count, 1) |
| 378 | |
| 379 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 380 | class PerformSymbolFilesUploadTest(SymbolsTestBase): |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 381 | """Test PerformSymbolFile, and it's helper methods.""" |
| 382 | def setUp(self): |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 383 | self.sym_initial = self.createSymbolFile( |
| 384 | 'initial.sym') |
| 385 | self.sym_error = self.createSymbolFile( |
| 386 | 'error.sym', status=upload_symbols.SymbolFile.ERROR) |
| 387 | self.sym_duplicate = self.createSymbolFile( |
| 388 | 'duplicate.sym', status=upload_symbols.SymbolFile.DUPLICATE) |
| 389 | self.sym_uploaded = self.createSymbolFile( |
| 390 | 'uploaded.sym', status=upload_symbols.SymbolFile.UPLOADED) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 391 | |
| 392 | def testGetUploadTimeout(self): |
| 393 | """Test GetUploadTimeout helper function.""" |
| 394 | # Timeout for small file. |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 395 | self.assertEqual(upload_symbols.GetUploadTimeout(self.sym_initial), |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 396 | upload_symbols.UPLOAD_MIN_TIMEOUT) |
| 397 | |
| 398 | # Timeout for 300M file. |
| 399 | large = self.createSymbolFile('large.sym', size=(300 * 1024 * 1024)) |
Ryo Hashimoto | f864c0e | 2017-02-14 12:46:08 +0900 | [diff] [blame] | 400 | self.assertEqual(upload_symbols.GetUploadTimeout(large), 771) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 401 | |
Mike Nichols | 649e6a8 | 2019-04-23 11:44:48 -0600 | [diff] [blame] | 402 | def testUploadExists(self): |
| 403 | """Test IsFound helper function.""" |
| 404 | self.PatchObject(upload_symbols, 'UploadExists', |
| 405 | return_value=True) |
| 406 | symbols = [self.sym_initial] |
| 407 | result = upload_symbols.PerformSymbolsFileUpload( |
| 408 | symbols, 'fake_url', api_key='testkey') |
| 409 | self.assertEqual(list(result), symbols) |
| 410 | self.assertEqual(self.sym_initial.status, |
| 411 | upload_symbols.SymbolFile.DUPLICATE) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 412 | def testUploadSymbolFile(self): |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 413 | upload_symbols.UploadSymbolFile('fake_url', self.sym_initial, |
| 414 | api_key='testkey') |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 415 | # TODO: Examine mock in more detail to make sure request is correct. |
Mike Nichols | 649e6a8 | 2019-04-23 11:44:48 -0600 | [diff] [blame] | 416 | self.assertEqual(self.request_mock.call_count, 4) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 417 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 418 | def testPerformSymbolsFileUpload(self): |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 419 | """We upload on first try.""" |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 420 | symbols = [self.sym_initial] |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 421 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 422 | result = upload_symbols.PerformSymbolsFileUpload( |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 423 | symbols, 'fake_url', api_key='testkey') |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 424 | |
| 425 | self.assertEqual(list(result), symbols) |
| 426 | self.assertEqual(self.sym_initial.status, |
| 427 | upload_symbols.SymbolFile.UPLOADED) |
Mike Nichols | 649e6a8 | 2019-04-23 11:44:48 -0600 | [diff] [blame] | 428 | self.assertEqual(self.request_mock.call_count, 4) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 429 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 430 | def testPerformSymbolsFileUploadFailure(self): |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 431 | """All network requests fail.""" |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 432 | self.request_mock.side_effect = urllib2.URLError('network failure') |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 433 | symbols = [self.sym_initial] |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 434 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 435 | result = upload_symbols.PerformSymbolsFileUpload( |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 436 | symbols, 'fake_url', api_key='testkey') |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 437 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 438 | self.assertEqual(list(result), symbols) |
| 439 | self.assertEqual(self.sym_initial.status, upload_symbols.SymbolFile.ERROR) |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 440 | self.assertEqual(self.request_mock.call_count, 7) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 441 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 442 | def testPerformSymbolsFileUploadTransisentFailure(self): |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 443 | """We fail once, then succeed.""" |
| 444 | self.urlopen_mock.side_effect = (urllib2.URLError('network failure'), None) |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 445 | symbols = [self.sym_initial] |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 446 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 447 | result = upload_symbols.PerformSymbolsFileUpload( |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 448 | symbols, 'fake_url', api_key='testkey') |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 449 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 450 | self.assertEqual(list(result), symbols) |
| 451 | self.assertEqual(self.sym_initial.status, |
| 452 | upload_symbols.SymbolFile.UPLOADED) |
Mike Nichols | 649e6a8 | 2019-04-23 11:44:48 -0600 | [diff] [blame] | 453 | self.assertEqual(self.request_mock.call_count, 4) |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 454 | |
| 455 | def testPerformSymbolsFileUploadMixed(self): |
| 456 | """Upload symbols in mixed starting states. |
| 457 | |
| 458 | Demonstrate that INITIAL and ERROR are uploaded, but DUPLICATE/UPLOADED are |
| 459 | ignored. |
| 460 | """ |
| 461 | symbols = [self.sym_initial, self.sym_error, |
| 462 | self.sym_duplicate, self.sym_uploaded] |
| 463 | |
| 464 | result = upload_symbols.PerformSymbolsFileUpload( |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 465 | symbols, 'fake_url', api_key='testkey') |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 466 | |
| 467 | # |
| 468 | self.assertEqual(list(result), symbols) |
| 469 | self.assertEqual(self.sym_initial.status, |
| 470 | upload_symbols.SymbolFile.UPLOADED) |
| 471 | self.assertEqual(self.sym_error.status, |
| 472 | upload_symbols.SymbolFile.UPLOADED) |
| 473 | self.assertEqual(self.sym_duplicate.status, |
| 474 | upload_symbols.SymbolFile.DUPLICATE) |
| 475 | self.assertEqual(self.sym_uploaded.status, |
| 476 | upload_symbols.SymbolFile.UPLOADED) |
Mike Nichols | 649e6a8 | 2019-04-23 11:44:48 -0600 | [diff] [blame] | 477 | self.assertEqual(self.request_mock.call_count, 8) |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 478 | |
| 479 | |
| 480 | def testPerformSymbolsFileUploadErrorOut(self): |
| 481 | """Demonstate we exit only after X errors.""" |
| 482 | |
| 483 | symbol_count = upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY + 10 |
| 484 | symbols = [] |
| 485 | fail_file = None |
| 486 | |
| 487 | # potentially twice as many errors as we should attempt. |
| 488 | for _ in xrange(symbol_count): |
| 489 | # Each loop will get unique SymbolFile instances that use the same files. |
| 490 | fail = self.createSymbolFile('fail.sym') |
| 491 | fail_file = fail.file_name |
| 492 | symbols.append(self.createSymbolFile('pass.sym')) |
| 493 | symbols.append(fail) |
| 494 | |
| 495 | # Mock out UploadSymbolFile and fail for fail.sym files. |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 496 | def failSome(_url, symbol, _api_key): |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 497 | if symbol.file_name == fail_file: |
| 498 | raise urllib2.URLError('network failure') |
| 499 | |
Luigi Semenzato | 5104f3c | 2016-10-12 12:37:42 -0700 | [diff] [blame] | 500 | upload_mock = self.PatchObject(upload_symbols, 'UploadSymbolFile', |
| 501 | side_effect=failSome) |
| 502 | upload_mock.__name__ = 'UploadSymbolFileMock2' |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 503 | |
| 504 | result = upload_symbols.PerformSymbolsFileUpload( |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 505 | symbols, 'fake_url', api_key='testkey') |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 506 | |
| 507 | self.assertEqual(list(result), symbols) |
| 508 | |
| 509 | passed = sum(s.status == upload_symbols.SymbolFile.UPLOADED |
| 510 | for s in symbols) |
| 511 | failed = sum(s.status == upload_symbols.SymbolFile.ERROR |
| 512 | for s in symbols) |
| 513 | skipped = sum(s.status == upload_symbols.SymbolFile.INITIAL |
| 514 | for s in symbols) |
| 515 | |
| 516 | # Shows we all pass.sym files worked until limit hit. |
| 517 | self.assertEqual(passed, upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY) |
| 518 | |
| 519 | # Shows we all fail.sym files failed until limit hit. |
| 520 | self.assertEqual(failed, upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY) |
| 521 | |
| 522 | # Shows both pass/fail were skipped after limit hit. |
| 523 | self.assertEqual(skipped, 10 * 2) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 524 | |
| 525 | |
| 526 | class UploadSymbolsTest(SymbolsTestBase): |
| 527 | """Test UploadSymbols, along with most helper methods.""" |
| 528 | def setUp(self): |
| 529 | # Results gathering. |
| 530 | self.failure_file = os.path.join(self.tempdir, 'failures.txt') |
| 531 | |
| 532 | def testUploadSymbolsEmpty(self): |
| 533 | """Upload dir is empty.""" |
| 534 | result = upload_symbols.UploadSymbols( |
| 535 | [self.data], 'fake_url', 'product') |
| 536 | |
| 537 | self.assertEquals(result, 0) |
| 538 | self.assertEqual(self.urlopen_mock.call_count, 0) |
| 539 | |
| 540 | def testUploadSymbols(self): |
| 541 | """Upload a few files.""" |
| 542 | self.createSymbolFile('slim.sym', self.SLIM_CONTENT) |
| 543 | self.createSymbolFile(os.path.join('nested', 'inner.sym')) |
| 544 | self.createSymbolFile('fat.sym', self.FAT_CONTENT) |
| 545 | |
| 546 | result = upload_symbols.UploadSymbols( |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 547 | [self.data], 'fake_url', |
| 548 | failed_list=self.failure_file, strip_cfi=len(self.SLIM_CONTENT)+1, |
| 549 | api_key='testkey') |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 550 | |
| 551 | self.assertEquals(result, 0) |
Mike Nichols | 649e6a8 | 2019-04-23 11:44:48 -0600 | [diff] [blame] | 552 | self.assertEqual(self.request_mock.call_count, 12) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 553 | self.assertEquals(osutils.ReadFile(self.failure_file), '') |
| 554 | |
| 555 | def testUploadSymbolsLimited(self): |
| 556 | """Upload a few files.""" |
| 557 | self.createSymbolFile('slim.sym', self.SLIM_CONTENT) |
| 558 | self.createSymbolFile(os.path.join('nested', 'inner.sym')) |
| 559 | self.createSymbolFile('fat.sym', self.FAT_CONTENT) |
| 560 | |
| 561 | result = upload_symbols.UploadSymbols( |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 562 | [self.data], 'fake_url', upload_limit=2, |
| 563 | api_key='testkey') |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 564 | |
| 565 | self.assertEquals(result, 0) |
Mike Nichols | 649e6a8 | 2019-04-23 11:44:48 -0600 | [diff] [blame] | 566 | self.assertEqual(self.request_mock.call_count, 8) |
Mike Frysinger | f2fa7d6 | 2017-12-14 18:33:59 -0500 | [diff] [blame] | 567 | self.assertNotExists(self.failure_file) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 568 | |
| 569 | def testUploadSymbolsFailures(self): |
| 570 | """Upload a few files.""" |
| 571 | self.createSymbolFile('pass.sym') |
| 572 | fail = self.createSymbolFile('fail.sym') |
| 573 | |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 574 | def failSome(_url, symbol, _api_key): |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 575 | if symbol.file_name == fail.file_name: |
| 576 | raise urllib2.URLError('network failure') |
| 577 | |
| 578 | # Mock out UploadSymbolFile so it's easy to see which file to fail for. |
| 579 | upload_mock = self.PatchObject(upload_symbols, 'UploadSymbolFile', |
| 580 | side_effect=failSome) |
Luigi Semenzato | 5104f3c | 2016-10-12 12:37:42 -0700 | [diff] [blame] | 581 | # Mock __name__ for logging. |
| 582 | upload_mock.__name__ = 'UploadSymbolFileMock' |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 583 | |
| 584 | result = upload_symbols.UploadSymbols( |
Mike Nichols | 90f7c15 | 2019-04-09 15:14:08 -0600 | [diff] [blame] | 585 | [self.data], 'fake_url', |
| 586 | failed_list=self.failure_file, api_key='testkey') |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 587 | |
| 588 | self.assertEquals(result, 1) |
| 589 | self.assertEqual(upload_mock.call_count, 8) |
| 590 | self.assertEquals(osutils.ReadFile(self.failure_file), 'fail.sym\n') |
| 591 | |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 592 | # TODO: We removed --network integration tests. |
Mike Frysinger | 0c0efa2 | 2014-02-09 23:32:23 -0500 | [diff] [blame] | 593 | |
| 594 | |
Mike Frysinger | ea838d1 | 2014-12-08 11:55:32 -0500 | [diff] [blame] | 595 | def main(_argv): |
Mike Frysinger | 27e21b7 | 2018-07-12 14:20:21 -0400 | [diff] [blame] | 596 | # pylint: disable=protected-access |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 597 | # Set timeouts small so that if the unit test hangs, it won't hang for long. |
| 598 | parallel._BackgroundTask.STARTUP_TIMEOUT = 5 |
| 599 | parallel._BackgroundTask.EXIT_TIMEOUT = 5 |
| 600 | |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 601 | # Run the tests. |
Mike Frysinger | ba16737 | 2015-01-21 10:37:03 -0500 | [diff] [blame] | 602 | cros_test_lib.main(level='info', module=__name__) |