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