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 |
| 44 | sys.path.insert(0, os.path.join(third_party, 'swarming.client')) |
| 45 | del third_party |
| 46 | |
Ralph Nathan | 446aee9 | 2015-03-23 14:44:56 -0700 | [diff] [blame] | 47 | from chromite.lib import cros_logging as logging |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 48 | from chromite.lib import cros_test_lib |
| 49 | from chromite.lib import osutils |
| 50 | from chromite.lib import parallel |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 51 | from chromite.lib import remote_access |
Mike Frysinger | 5e30a4b | 2014-02-12 20:23:04 -0500 | [diff] [blame] | 52 | from chromite.scripts import cros_generate_breakpad_symbols |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 53 | from chromite.scripts import upload_symbols |
| 54 | |
Mike Frysinger | bbd1f11 | 2016-09-08 18:25:11 -0400 | [diff] [blame] | 55 | # And our sys.path muckery confuses pylint. |
| 56 | import isolateserver # pylint: disable=import-error |
| 57 | |
Mike Frysinger | 0c0efa2 | 2014-02-09 23:32:23 -0500 | [diff] [blame] | 58 | |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 59 | class SymbolsTestBase(cros_test_lib.MockTempDirTestCase): |
| 60 | """Base class for most symbols tests.""" |
| 61 | |
| 62 | SLIM_CONTENT = """ |
| 63 | some junk |
| 64 | """ |
| 65 | |
| 66 | FAT_CONTENT = """ |
| 67 | STACK CFI 1234 |
| 68 | some junk |
| 69 | STACK CFI 1234 |
| 70 | """ |
| 71 | |
| 72 | def setUp(self): |
| 73 | # Make certain we don't use the network. |
| 74 | self.isolate_mock = self.PatchObject(isolateserver, 'get_storage_api') |
| 75 | self.urlopen_mock = self.PatchObject(urllib2, 'urlopen') |
| 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 | |
| 178 | # Just to be certain we don't really contact it. |
| 179 | self.PatchObject(isolateserver, 'get_storage_api') |
| 180 | |
Mike Frysinger | 0a2fd92 | 2014-09-12 20:23:42 -0700 | [diff] [blame] | 181 | def tearDown(self): |
| 182 | # Only kill the server if we forked one. |
| 183 | if self.httpd_pid: |
| 184 | os.kill(self.httpd_pid, signal.SIGUSR1) |
| 185 | |
| 186 | def testSuccess(self): |
| 187 | """The server returns success for all uploads""" |
| 188 | class Handler(SymbolServerRequestHandler): |
| 189 | """Always return 200""" |
| 190 | RESP_CODE = 200 |
| 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, |
| 196 | product_name='upload_symbols_test') |
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): |
| 202 | """Always return 500""" |
| 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, |
| 210 | product_name='upload_symbols_test') |
| 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, |
| 227 | product_name='upload_symbols_test') |
| 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 | |
| 380 | class DeduplicateTest(SymbolsTestBase): |
| 381 | """Test Isolate server Deduplication.""" |
| 382 | def setUp(self): |
| 383 | self.connection_mock = self.isolate_mock.return_value |
| 384 | self.contains_mock = self.connection_mock.contains |
| 385 | self.push_mock = self.connection_mock.push |
| 386 | |
| 387 | def testConnection(self): |
| 388 | result = upload_symbols.OpenDeduplicateConnection('namespace') |
| 389 | self.assertTrue(result is not None) |
| 390 | self.assertEqual( |
| 391 | self.isolate_mock.mock_calls, |
| 392 | [mock.call('https://isolateserver.appspot.com', 'namespace')]) |
| 393 | |
| 394 | def testConnectionFail(self): |
| 395 | self.isolate_mock.side_effect = Exception |
| 396 | result = upload_symbols.OpenDeduplicateConnection('namespace') |
| 397 | self.assertEqual(result, None) |
| 398 | |
| 399 | def testFindDuplicates(self): |
| 400 | # The first symbol will new new, the second a duplicate. |
| 401 | def containsOneNew(items): |
| 402 | return {items[0]: 'state'} |
| 403 | self.contains_mock.side_effect = containsOneNew |
| 404 | |
| 405 | sym1 = self.createSymbolFile('sym1.sym') |
| 406 | sym2 = self.createSymbolFile('sym2.sym') |
| 407 | |
| 408 | result = upload_symbols.FindDuplicates((sym1, sym2), 'namespace') |
| 409 | self.assertEqual(list(result), [sym1, sym2]) |
| 410 | |
| 411 | self.assertEqual(sym1.status, upload_symbols.SymbolFile.INITIAL) |
| 412 | self.assertIsInstance(sym1.dedupe_item, upload_symbols.DedupeItem) |
| 413 | self.assertEqual(sym1.dedupe_push_state, 'state') |
| 414 | |
| 415 | self.assertEqual(sym2.status, upload_symbols.SymbolFile.DUPLICATE) |
| 416 | self.assertIsNone(sym2.dedupe_item) |
| 417 | self.assertIsNone(sym2.dedupe_push_state) |
| 418 | |
| 419 | def testFindDuplicatesConnectFail(self): |
| 420 | self.isolate_mock.side_effect = Exception |
| 421 | |
| 422 | sym1 = self.createSymbolFile('sym1.sym') |
| 423 | sym2 = self.createSymbolFile('sym2.sym') |
| 424 | |
| 425 | result = upload_symbols.FindDuplicates((sym1, sym2), 'namespace') |
| 426 | self.assertEqual(list(result), [sym1, sym2]) |
| 427 | |
| 428 | self.assertFalse(self.contains_mock.called) |
| 429 | |
| 430 | self.assertEqual(sym1.status, upload_symbols.SymbolFile.INITIAL) |
| 431 | self.assertIsNone(sym1.dedupe_item) |
| 432 | self.assertIsNone(sym1.dedupe_push_state) |
| 433 | |
| 434 | self.assertEqual(sym2.status, upload_symbols.SymbolFile.INITIAL) |
| 435 | self.assertIsNone(sym2.dedupe_item) |
| 436 | self.assertIsNone(sym2.dedupe_push_state) |
| 437 | |
| 438 | def testFindDuplicatesContainsFail(self): |
| 439 | self.contains_mock.side_effect = Exception |
| 440 | |
| 441 | sym1 = self.createSymbolFile('sym1.sym') |
| 442 | sym2 = self.createSymbolFile('sym2.sym') |
| 443 | |
| 444 | result = upload_symbols.FindDuplicates((sym1, sym2), 'namespace') |
| 445 | self.assertEqual(list(result), [sym1, sym2]) |
| 446 | |
| 447 | self.assertEqual(sym1.status, upload_symbols.SymbolFile.INITIAL) |
| 448 | self.assertIsNone(sym1.dedupe_item) |
| 449 | self.assertIsNone(sym1.dedupe_push_state) |
| 450 | |
| 451 | self.assertEqual(sym2.status, upload_symbols.SymbolFile.INITIAL) |
| 452 | self.assertIsNone(sym2.dedupe_item) |
| 453 | self.assertIsNone(sym2.dedupe_push_state) |
| 454 | |
| 455 | def testPostForDeduplication(self): |
| 456 | symInitial = self.createSymbolFile( |
| 457 | 'initial.sym') |
| 458 | symDup = self.createSymbolFile( |
| 459 | 'dup.sym', status=upload_symbols.SymbolFile.DUPLICATE) |
| 460 | symUploaded = self.createSymbolFile( |
| 461 | 'uploaded.sym', status=upload_symbols.SymbolFile.UPLOADED) |
| 462 | |
| 463 | symInitialItem = self.createSymbolFile( |
| 464 | 'initialItem.sym', dedupe=True) |
| 465 | |
| 466 | symDupItem = self.createSymbolFile( |
| 467 | 'dupItem.sym', status=upload_symbols.SymbolFile.DUPLICATE, dedupe=True) |
| 468 | # This is the only symbol file which should be pushed to isolate. |
| 469 | symUploadedItem = self.createSymbolFile( |
| 470 | 'upItem.sym', status=upload_symbols.SymbolFile.UPLOADED, dedupe=True) |
| 471 | |
| 472 | symbols = (symInitial, symDup, symUploaded, |
| 473 | symInitialItem, symDupItem, symUploadedItem) |
| 474 | |
| 475 | result = upload_symbols.PostForDeduplication(symbols, 'namespace') |
| 476 | self.assertEqual(list(result), list(symbols)) |
| 477 | self.assertEqual( |
| 478 | self.push_mock.mock_calls, |
| 479 | [ |
| 480 | mock.call( |
| 481 | symUploadedItem.dedupe_item, 'push_state', |
| 482 | ["SymbolHeader(cpu='cpu', id='id', name='name', os='os')"]) |
| 483 | ]) |
| 484 | |
| 485 | def testPostForDeduplicationConnectFail(self): |
| 486 | self.isolate_mock.side_effect = Exception |
| 487 | |
| 488 | sym1 = self.createSymbolFile( |
| 489 | 'sym1.sym', status=upload_symbols.SymbolFile.UPLOADED, dedupe=True) |
| 490 | sym2 = self.createSymbolFile( |
| 491 | 'sym2.sym', status=upload_symbols.SymbolFile.UPLOADED, dedupe=True) |
| 492 | |
| 493 | result = upload_symbols.PostForDeduplication((sym1, sym2), 'namespace') |
| 494 | self.assertEqual(list(result), [sym1, sym2]) |
| 495 | self.assertEqual(self.push_mock.call_count, 0) |
| 496 | |
| 497 | def testPostForDeduplicationPushFail(self): |
| 498 | self.push_mock.side_effect = Exception |
| 499 | |
| 500 | sym1 = self.createSymbolFile( |
| 501 | 'sym1.sym', status=upload_symbols.SymbolFile.UPLOADED, dedupe=True) |
| 502 | sym2 = self.createSymbolFile( |
| 503 | 'sym2.sym', status=upload_symbols.SymbolFile.UPLOADED, dedupe=True) |
| 504 | |
| 505 | result = upload_symbols.PostForDeduplication((sym1, sym2), 'namespace') |
| 506 | self.assertEqual(list(result), [sym1, sym2]) |
| 507 | self.assertEqual(self.push_mock.call_count, 1) |
| 508 | |
| 509 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 510 | class PerformSymbolFilesUploadTest(SymbolsTestBase): |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 511 | """Test PerformSymbolFile, and it's helper methods.""" |
| 512 | def setUp(self): |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 513 | self.sym_initial = self.createSymbolFile( |
| 514 | 'initial.sym') |
| 515 | self.sym_error = self.createSymbolFile( |
| 516 | 'error.sym', status=upload_symbols.SymbolFile.ERROR) |
| 517 | self.sym_duplicate = self.createSymbolFile( |
| 518 | 'duplicate.sym', status=upload_symbols.SymbolFile.DUPLICATE) |
| 519 | self.sym_uploaded = self.createSymbolFile( |
| 520 | 'uploaded.sym', status=upload_symbols.SymbolFile.UPLOADED) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 521 | |
| 522 | def testGetUploadTimeout(self): |
| 523 | """Test GetUploadTimeout helper function.""" |
| 524 | # Timeout for small file. |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 525 | self.assertEqual(upload_symbols.GetUploadTimeout(self.sym_initial), |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 526 | upload_symbols.UPLOAD_MIN_TIMEOUT) |
| 527 | |
| 528 | # Timeout for 300M file. |
| 529 | large = self.createSymbolFile('large.sym', size=(300 * 1024 * 1024)) |
Ryo Hashimoto | f864c0e | 2017-02-14 12:46:08 +0900 | [diff] [blame] | 530 | self.assertEqual(upload_symbols.GetUploadTimeout(large), 771) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 531 | |
| 532 | def testUploadSymbolFile(self): |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 533 | upload_symbols.UploadSymbolFile('fake_url', self.sym_initial, 'product') |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 534 | # TODO: Examine mock in more detail to make sure request is correct. |
| 535 | self.assertEqual(self.urlopen_mock.call_count, 1) |
| 536 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 537 | def testPerformSymbolsFileUpload(self): |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 538 | """We upload on first try.""" |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 539 | symbols = [self.sym_initial] |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 540 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 541 | result = upload_symbols.PerformSymbolsFileUpload( |
| 542 | symbols, 'fake_url', product_name='product') |
| 543 | |
| 544 | self.assertEqual(list(result), symbols) |
| 545 | self.assertEqual(self.sym_initial.status, |
| 546 | upload_symbols.SymbolFile.UPLOADED) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 547 | self.assertEqual(self.urlopen_mock.call_count, 1) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 548 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 549 | def testPerformSymbolsFileUploadFailure(self): |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 550 | """All network requests fail.""" |
| 551 | self.urlopen_mock.side_effect = urllib2.URLError('network failure') |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 552 | symbols = [self.sym_initial] |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 553 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 554 | result = upload_symbols.PerformSymbolsFileUpload( |
| 555 | symbols, 'fake_url', product_name='product') |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 556 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 557 | self.assertEqual(list(result), symbols) |
| 558 | self.assertEqual(self.sym_initial.status, upload_symbols.SymbolFile.ERROR) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 559 | self.assertEqual(self.urlopen_mock.call_count, 7) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 560 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 561 | def testPerformSymbolsFileUploadTransisentFailure(self): |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 562 | """We fail once, then succeed.""" |
| 563 | self.urlopen_mock.side_effect = (urllib2.URLError('network failure'), None) |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 564 | symbols = [self.sym_initial] |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 565 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 566 | result = upload_symbols.PerformSymbolsFileUpload( |
| 567 | symbols, 'fake_url', product_name='product') |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 568 | |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 569 | self.assertEqual(list(result), symbols) |
| 570 | self.assertEqual(self.sym_initial.status, |
| 571 | upload_symbols.SymbolFile.UPLOADED) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 572 | self.assertEqual(self.urlopen_mock.call_count, 2) |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 573 | |
| 574 | def testPerformSymbolsFileUploadMixed(self): |
| 575 | """Upload symbols in mixed starting states. |
| 576 | |
| 577 | Demonstrate that INITIAL and ERROR are uploaded, but DUPLICATE/UPLOADED are |
| 578 | ignored. |
| 579 | """ |
| 580 | symbols = [self.sym_initial, self.sym_error, |
| 581 | self.sym_duplicate, self.sym_uploaded] |
| 582 | |
| 583 | result = upload_symbols.PerformSymbolsFileUpload( |
| 584 | symbols, 'fake_url', product_name='product') |
| 585 | |
| 586 | # |
| 587 | self.assertEqual(list(result), symbols) |
| 588 | self.assertEqual(self.sym_initial.status, |
| 589 | upload_symbols.SymbolFile.UPLOADED) |
| 590 | self.assertEqual(self.sym_error.status, |
| 591 | upload_symbols.SymbolFile.UPLOADED) |
| 592 | self.assertEqual(self.sym_duplicate.status, |
| 593 | upload_symbols.SymbolFile.DUPLICATE) |
| 594 | self.assertEqual(self.sym_uploaded.status, |
| 595 | upload_symbols.SymbolFile.UPLOADED) |
| 596 | self.assertEqual(self.urlopen_mock.call_count, 2) |
| 597 | |
| 598 | |
| 599 | def testPerformSymbolsFileUploadErrorOut(self): |
| 600 | """Demonstate we exit only after X errors.""" |
| 601 | |
| 602 | symbol_count = upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY + 10 |
| 603 | symbols = [] |
| 604 | fail_file = None |
| 605 | |
| 606 | # potentially twice as many errors as we should attempt. |
| 607 | for _ in xrange(symbol_count): |
| 608 | # Each loop will get unique SymbolFile instances that use the same files. |
| 609 | fail = self.createSymbolFile('fail.sym') |
| 610 | fail_file = fail.file_name |
| 611 | symbols.append(self.createSymbolFile('pass.sym')) |
| 612 | symbols.append(fail) |
| 613 | |
| 614 | # Mock out UploadSymbolFile and fail for fail.sym files. |
| 615 | def failSome(_url, symbol, _product): |
| 616 | if symbol.file_name == fail_file: |
| 617 | raise urllib2.URLError('network failure') |
| 618 | |
Luigi Semenzato | 5104f3c | 2016-10-12 12:37:42 -0700 | [diff] [blame] | 619 | upload_mock = self.PatchObject(upload_symbols, 'UploadSymbolFile', |
| 620 | side_effect=failSome) |
| 621 | upload_mock.__name__ = 'UploadSymbolFileMock2' |
Don Garrett | deb2e03 | 2016-07-06 16:44:14 -0700 | [diff] [blame] | 622 | |
| 623 | result = upload_symbols.PerformSymbolsFileUpload( |
| 624 | symbols, 'fake_url', product_name='product') |
| 625 | |
| 626 | self.assertEqual(list(result), symbols) |
| 627 | |
| 628 | passed = sum(s.status == upload_symbols.SymbolFile.UPLOADED |
| 629 | for s in symbols) |
| 630 | failed = sum(s.status == upload_symbols.SymbolFile.ERROR |
| 631 | for s in symbols) |
| 632 | skipped = sum(s.status == upload_symbols.SymbolFile.INITIAL |
| 633 | for s in symbols) |
| 634 | |
| 635 | # Shows we all pass.sym files worked until limit hit. |
| 636 | self.assertEqual(passed, upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY) |
| 637 | |
| 638 | # Shows we all fail.sym files failed until limit hit. |
| 639 | self.assertEqual(failed, upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY) |
| 640 | |
| 641 | # Shows both pass/fail were skipped after limit hit. |
| 642 | self.assertEqual(skipped, 10 * 2) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 643 | |
| 644 | |
| 645 | class UploadSymbolsTest(SymbolsTestBase): |
| 646 | """Test UploadSymbols, along with most helper methods.""" |
| 647 | def setUp(self): |
| 648 | # Results gathering. |
| 649 | self.failure_file = os.path.join(self.tempdir, 'failures.txt') |
| 650 | |
| 651 | def testUploadSymbolsEmpty(self): |
| 652 | """Upload dir is empty.""" |
| 653 | result = upload_symbols.UploadSymbols( |
| 654 | [self.data], 'fake_url', 'product') |
| 655 | |
| 656 | self.assertEquals(result, 0) |
| 657 | self.assertEqual(self.urlopen_mock.call_count, 0) |
| 658 | |
| 659 | def testUploadSymbols(self): |
| 660 | """Upload a few files.""" |
| 661 | self.createSymbolFile('slim.sym', self.SLIM_CONTENT) |
| 662 | self.createSymbolFile(os.path.join('nested', 'inner.sym')) |
| 663 | self.createSymbolFile('fat.sym', self.FAT_CONTENT) |
| 664 | |
| 665 | result = upload_symbols.UploadSymbols( |
| 666 | [self.data], 'fake_url', 'product', |
| 667 | failed_list=self.failure_file, strip_cfi=len(self.SLIM_CONTENT)+1) |
| 668 | |
| 669 | self.assertEquals(result, 0) |
| 670 | self.assertEqual(self.urlopen_mock.call_count, 3) |
| 671 | self.assertEquals(osutils.ReadFile(self.failure_file), '') |
| 672 | |
| 673 | def testUploadSymbolsLimited(self): |
| 674 | """Upload a few files.""" |
| 675 | self.createSymbolFile('slim.sym', self.SLIM_CONTENT) |
| 676 | self.createSymbolFile(os.path.join('nested', 'inner.sym')) |
| 677 | self.createSymbolFile('fat.sym', self.FAT_CONTENT) |
| 678 | |
| 679 | result = upload_symbols.UploadSymbols( |
| 680 | [self.data], 'fake_url', 'product', upload_limit=2) |
| 681 | |
| 682 | self.assertEquals(result, 0) |
| 683 | self.assertEqual(self.urlopen_mock.call_count, 2) |
Mike Frysinger | f2fa7d6 | 2017-12-14 18:33:59 -0500 | [diff] [blame] | 684 | self.assertNotExists(self.failure_file) |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 685 | |
| 686 | def testUploadSymbolsFailures(self): |
| 687 | """Upload a few files.""" |
| 688 | self.createSymbolFile('pass.sym') |
| 689 | fail = self.createSymbolFile('fail.sym') |
| 690 | |
| 691 | def failSome(_url, symbol, _product): |
| 692 | if symbol.file_name == fail.file_name: |
| 693 | raise urllib2.URLError('network failure') |
| 694 | |
| 695 | # Mock out UploadSymbolFile so it's easy to see which file to fail for. |
| 696 | upload_mock = self.PatchObject(upload_symbols, 'UploadSymbolFile', |
| 697 | side_effect=failSome) |
Luigi Semenzato | 5104f3c | 2016-10-12 12:37:42 -0700 | [diff] [blame] | 698 | # Mock __name__ for logging. |
| 699 | upload_mock.__name__ = 'UploadSymbolFileMock' |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 700 | |
| 701 | result = upload_symbols.UploadSymbols( |
| 702 | [self.data], 'fake_url', 'product', |
| 703 | failed_list=self.failure_file) |
| 704 | |
| 705 | self.assertEquals(result, 1) |
| 706 | self.assertEqual(upload_mock.call_count, 8) |
| 707 | self.assertEquals(osutils.ReadFile(self.failure_file), 'fail.sym\n') |
| 708 | |
Don Garrett | a28be6d | 2016-06-16 18:09:35 -0700 | [diff] [blame] | 709 | # TODO: We removed --network integration tests. |
Mike Frysinger | 0c0efa2 | 2014-02-09 23:32:23 -0500 | [diff] [blame] | 710 | |
| 711 | |
Mike Frysinger | ea838d1 | 2014-12-08 11:55:32 -0500 | [diff] [blame] | 712 | def main(_argv): |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 713 | # pylint: disable=W0212 |
| 714 | # Set timeouts small so that if the unit test hangs, it won't hang for long. |
| 715 | parallel._BackgroundTask.STARTUP_TIMEOUT = 5 |
| 716 | parallel._BackgroundTask.EXIT_TIMEOUT = 5 |
| 717 | |
Mike Frysinger | d5fcb3a | 2013-05-30 21:10:50 -0400 | [diff] [blame] | 718 | # Run the tests. |
Mike Frysinger | ba16737 | 2015-01-21 10:37:03 -0500 | [diff] [blame] | 719 | cros_test_lib.main(level='info', module=__name__) |