blob: b52666825ae26a3d40f6077f5513fe3226b4db19 [file] [log] [blame]
Mike Frysingere58c0e22017-10-04 15:43:30 -04001# -*- coding: utf-8 -*-
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -04002# 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 Frysingereb753bf2013-11-22 16:05:35 -05006"""Unittests for upload_symbols.py"""
7
Mike Frysinger7f9be142014-01-15 02:16:42 -05008from __future__ import print_function
9
Mike Frysinger06da5202014-09-26 17:30:33 -050010import errno
Don Garretta28be6d2016-06-16 18:09:35 -070011import itertools
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040012import os
Mike Frysinger0a2fd922014-09-12 20:23:42 -070013import signal
Mike Frysinger06da5202014-09-26 17:30:33 -050014import socket
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040015import sys
Aviv Keshetd1f04632014-05-09 11:33:46 -070016import time
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040017
Mike Frysinger6db648e2018-07-24 19:57:58 -040018import mock
Mike Frysinger3d465162019-08-28 00:40:23 -040019from six.moves import BaseHTTPServer
20from six.moves import socketserver
Mike Frysinger3dcacee2019-08-23 17:09:11 -040021from six.moves import urllib
Mike Frysinger6db648e2018-07-24 19:57:58 -040022
Mike Frysinger079863c2014-10-09 23:16:46 -040023# We specifically set up a local server to connect to, so make sure we
24# delete any proxy settings that might screw that up. We also need to
25# do it here because modules that are imported below will implicitly
26# initialize with this proxy setting rather than dynamically pull it
27# on the fly :(.
Mike Frysinger92bdef52019-08-21 21:05:13 -040028# pylint: disable=wrong-import-position
Mike Frysinger079863c2014-10-09 23:16:46 -040029os.environ.pop('http_proxy', None)
30
Aviv Keshetb7519e12016-10-04 00:50:00 -070031from chromite.lib import constants
Mike Frysingerbbd1f112016-09-08 18:25:11 -040032
33# The isolateserver includes a bunch of third_party python packages that clash
34# with chromite's bundled third_party python packages (like oauth2client).
35# Since upload_symbols is not imported in to other parts of chromite, and there
36# are no deps in third_party we care about, purge the chromite copy. This way
37# we can use isolateserver for deduping.
38# TODO: If we ever sort out third_party/ handling and make it per-script opt-in,
39# we can purge this logic.
40third_party = os.path.join(constants.CHROMITE_DIR, 'third_party')
41while True:
42 try:
43 sys.path.remove(third_party)
44 except ValueError:
45 break
Mike Frysingerbbd1f112016-09-08 18:25:11 -040046del third_party
47
Ralph Nathan446aee92015-03-23 14:44:56 -070048from chromite.lib import cros_logging as logging
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040049from chromite.lib import cros_test_lib
50from chromite.lib import osutils
51from chromite.lib import parallel
Mike Frysinger0a2fd922014-09-12 20:23:42 -070052from chromite.lib import remote_access
Mike Frysinger5e30a4b2014-02-12 20:23:04 -050053from chromite.scripts import cros_generate_breakpad_symbols
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040054from chromite.scripts import upload_symbols
55
Mike Frysinger0c0efa22014-02-09 23:32:23 -050056
Don Garretta28be6d2016-06-16 18:09:35 -070057class SymbolsTestBase(cros_test_lib.MockTempDirTestCase):
58 """Base class for most symbols tests."""
59
60 SLIM_CONTENT = """
61some junk
62"""
63
64 FAT_CONTENT = """
65STACK CFI 1234
66some junk
67STACK CFI 1234
68"""
69
70 def setUp(self):
71 # Make certain we don't use the network.
Mike Frysinger3dcacee2019-08-23 17:09:11 -040072 self.urlopen_mock = self.PatchObject(urllib.request, 'urlopen')
Mike Nichols90f7c152019-04-09 15:14:08 -060073 self.request_mock = self.PatchObject(upload_symbols, 'ExecRequest',
74 return_value={'uploadUrl':
75 'testurl',
76 'uploadKey':
77 'asdgasgas'})
Don Garretta28be6d2016-06-16 18:09:35 -070078
79 # Make 'uploads' go fast.
80 self.PatchObject(upload_symbols, 'SLEEP_DELAY', 0)
81 self.PatchObject(upload_symbols, 'INITIAL_RETRY_DELAY', 0)
82
83 # So our symbol file content doesn't have to be real.
84 self.PatchObject(cros_generate_breakpad_symbols, 'ReadSymsHeader',
85 return_value=cros_generate_breakpad_symbols.SymbolHeader(
86 os='os', cpu='cpu', id='id', name='name'))
87
88 self.working = os.path.join(self.tempdir, 'expand')
89 osutils.SafeMakedirs(self.working)
90
91 self.data = os.path.join(self.tempdir, 'data')
92 osutils.SafeMakedirs(self.data)
93
94 def createSymbolFile(self, filename, content=FAT_CONTENT, size=0,
95 status=None, dedupe=False):
96 fullname = os.path.join(self.data, filename)
97 osutils.SafeMakedirs(os.path.dirname(fullname))
98
99 # If a file size is given, force that to be the minimum file size. Create
100 # a sparse file so large files are practical.
101 with open(fullname, 'w+b') as f:
102 f.truncate(size)
103 f.seek(0)
104 f.write(content)
105
106 result = upload_symbols.SymbolFile(display_path=filename,
107 file_name=fullname)
108
109 if status:
110 result.status = status
111
112 if dedupe:
113 result.dedupe_item = upload_symbols.DedupeItem(result)
114 result.dedupe_push_state = 'push_state'
115
116 return result
117
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -0400118
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700119class SymbolServerRequestHandler(BaseHTTPServer.BaseHTTPRequestHandler):
120 """HTTP handler for symbol POSTs"""
121
122 RESP_CODE = None
123 RESP_MSG = None
124
125 def do_POST(self):
126 """Handle a POST request"""
127 # Drain the data from the client. If we don't, we might write the response
128 # and close the socket before the client finishes, so they die with EPIPE.
129 clen = int(self.headers.get('Content-Length', '0'))
130 self.rfile.read(clen)
131
132 self.send_response(self.RESP_CODE, self.RESP_MSG)
133 self.end_headers()
134
Mike Frysingera4418152019-09-22 04:00:24 -0400135 # pylint: disable=arguments-differ
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700136 def log_message(self, *args, **kwargs):
137 """Stub the logger as it writes to stderr"""
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700138
139
Mike Frysinger3d465162019-08-28 00:40:23 -0400140class SymbolServer(socketserver.ThreadingTCPServer, BaseHTTPServer.HTTPServer):
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700141 """Simple HTTP server that forks each request"""
142
143
144class UploadSymbolsServerTest(cros_test_lib.MockTempDirTestCase):
145 """Tests for UploadSymbols() and a local HTTP server"""
146
147 SYM_CONTENTS = """MODULE Linux arm 123-456 blkid
148PUBLIC 1471 0 main"""
149
150 def SpawnServer(self, RequestHandler):
151 """Spawn a new http server"""
Mike Frysinger06da5202014-09-26 17:30:33 -0500152 while True:
153 try:
154 port = remote_access.GetUnusedPort()
155 address = ('', port)
156 self.httpd = SymbolServer(address, RequestHandler)
157 break
158 except socket.error as e:
159 if e.errno == errno.EADDRINUSE:
160 continue
161 raise
Don Garretta28be6d2016-06-16 18:09:35 -0700162 self.server_url = 'http://localhost:%i/post/path' % port
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700163 self.httpd_pid = os.fork()
164 if self.httpd_pid == 0:
165 self.httpd.serve_forever(poll_interval=0.1)
166 sys.exit(0)
167
168 def setUp(self):
169 self.httpd_pid = None
170 self.httpd = None
Don Garretta28be6d2016-06-16 18:09:35 -0700171 self.server_url = None
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700172 self.sym_file = os.path.join(self.tempdir, 'test.sym')
173 osutils.WriteFile(self.sym_file, self.SYM_CONTENTS)
174
Don Garretta28be6d2016-06-16 18:09:35 -0700175 # Stop sleeps and retries for these tests.
176 self.PatchObject(upload_symbols, 'SLEEP_DELAY', 0)
177 self.PatchObject(upload_symbols, 'INITIAL_RETRY_DELAY', 0)
178 self.PatchObject(upload_symbols, 'MAX_RETRIES', 0)
179
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700180 def tearDown(self):
181 # Only kill the server if we forked one.
182 if self.httpd_pid:
183 os.kill(self.httpd_pid, signal.SIGUSR1)
184
185 def testSuccess(self):
186 """The server returns success for all uploads"""
187 class Handler(SymbolServerRequestHandler):
188 """Always return 200"""
189 RESP_CODE = 200
Mike Nichols90f7c152019-04-09 15:14:08 -0600190 self.PatchObject(upload_symbols, 'ExecRequest',
191 return_value={'uploadUrl': 'testurl',
192 'uploadKey': 'testSuccess'})
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700193 self.SpawnServer(Handler)
Don Garretta28be6d2016-06-16 18:09:35 -0700194 ret = upload_symbols.UploadSymbols(
195 sym_paths=[self.sym_file] * 10,
196 upload_url=self.server_url,
Mike Nichols90f7c152019-04-09 15:14:08 -0600197 api_key='testSuccess')
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700198 self.assertEqual(ret, 0)
199
200 def testError(self):
201 """The server returns errors for all uploads"""
202 class Handler(SymbolServerRequestHandler):
Mike Nichols90f7c152019-04-09 15:14:08 -0600203 """All connections error"""
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700204 RESP_CODE = 500
205 RESP_MSG = 'Internal Server Error'
206
207 self.SpawnServer(Handler)
Don Garretta28be6d2016-06-16 18:09:35 -0700208 ret = upload_symbols.UploadSymbols(
209 sym_paths=[self.sym_file] * 10,
210 upload_url=self.server_url,
Mike Nichols90f7c152019-04-09 15:14:08 -0600211 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700212 self.assertEqual(ret, 10)
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700213
214 def testHungServer(self):
215 """The server chokes, but we recover"""
216 class Handler(SymbolServerRequestHandler):
217 """All connections choke forever"""
Mike Nichols137e82d2019-05-15 18:40:34 -0600218 self.PatchObject(upload_symbols, 'ExecRequest',
219 return_value={'pairs': []})
220
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700221 def do_POST(self):
222 while True:
223 time.sleep(1000)
224
225 self.SpawnServer(Handler)
226 with mock.patch.object(upload_symbols, 'GetUploadTimeout') as m:
Don Garretta28be6d2016-06-16 18:09:35 -0700227 m.return_value = 0.01
Mike Frysinger58312e92014-03-18 04:18:36 -0400228 ret = upload_symbols.UploadSymbols(
Don Garretta28be6d2016-06-16 18:09:35 -0700229 sym_paths=[self.sym_file] * 10,
230 upload_url=self.server_url,
Mike Nichols137e82d2019-05-15 18:40:34 -0600231 timeout=m.return_value,
Mike Nichols90f7c152019-04-09 15:14:08 -0600232 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700233 self.assertEqual(ret, 10)
Aviv Keshetd1f04632014-05-09 11:33:46 -0700234
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -0400235
Don Garretta28be6d2016-06-16 18:09:35 -0700236class UploadSymbolsHelpersTest(cros_test_lib.TestCase):
237 """Test assorted helper functions and classes."""
Don Garretta28be6d2016-06-16 18:09:35 -0700238 def testIsTarball(self):
239 notTar = [
240 '/foo/bar/test.bin',
241 '/foo/bar/test.tar.bin',
242 '/foo/bar/test.faketar.gz',
243 '/foo/bar/test.nottgz',
Mike Frysinger0c0efa22014-02-09 23:32:23 -0500244 ]
Mike Frysinger0c0efa22014-02-09 23:32:23 -0500245
Don Garretta28be6d2016-06-16 18:09:35 -0700246 isTar = [
247 '/foo/bar/test.tar',
248 '/foo/bar/test.bin.tar',
249 '/foo/bar/test.bin.tar.bz2',
250 '/foo/bar/test.bin.tar.gz',
251 '/foo/bar/test.bin.tar.xz',
252 '/foo/bar/test.tbz2',
253 '/foo/bar/test.tbz',
254 '/foo/bar/test.tgz',
255 '/foo/bar/test.txz',
256 ]
Mike Frysinger0c0efa22014-02-09 23:32:23 -0500257
Don Garretta28be6d2016-06-16 18:09:35 -0700258 for p in notTar:
259 self.assertFalse(upload_symbols.IsTarball(p))
260
261 for p in isTar:
262 self.assertTrue(upload_symbols.IsTarball(p))
263
264 def testBatchGenerator(self):
265 result = upload_symbols.BatchGenerator([], 2)
266 self.assertEqual(list(result), [])
267
Mike Frysinger66848312019-07-03 02:12:39 -0400268 # BatchGenerator accepts iterators, so passing it here is safe.
269 # pylint: disable=range-builtin-not-iterating
Mike Frysinger79cca962019-06-13 15:26:53 -0400270 result = upload_symbols.BatchGenerator(range(6), 2)
Don Garretta28be6d2016-06-16 18:09:35 -0700271 self.assertEqual(list(result), [[0, 1], [2, 3], [4, 5]])
272
Mike Frysinger79cca962019-06-13 15:26:53 -0400273 result = upload_symbols.BatchGenerator(range(7), 2)
Don Garretta28be6d2016-06-16 18:09:35 -0700274 self.assertEqual(list(result), [[0, 1], [2, 3], [4, 5], [6]])
275
276 # Prove that we are streaming the results, not generating them all at once.
277 result = upload_symbols.BatchGenerator(itertools.repeat(0), 2)
Mike Frysinger67d90242019-07-03 19:03:58 -0400278 self.assertEqual(next(result), [0, 0])
Don Garretta28be6d2016-06-16 18:09:35 -0700279
280
281class FindSymbolFilesTest(SymbolsTestBase):
282 """Test FindSymbolFiles."""
283 def setUp(self):
284 self.symfile = self.createSymbolFile('root.sym').file_name
285 self.innerfile = self.createSymbolFile(
286 os.path.join('nested', 'inner.sym')).file_name
287
288 # CreateTarball is having issues outside the chroot from open file tests.
289 #
290 # self.tarball = os.path.join(self.tempdir, 'syms.tar.gz')
291 # cros_build_lib.CreateTarball(
292 # 'syms.tar.gz', self.tempdir, inputs=(self.data))
293
294 def testEmpty(self):
295 symbols = list(upload_symbols.FindSymbolFiles(
296 self.working, []))
297 self.assertEqual(symbols, [])
298
299 def testFile(self):
300 symbols = list(upload_symbols.FindSymbolFiles(
301 self.working, [self.symfile]))
302
303 self.assertEqual(len(symbols), 1)
304 sf = symbols[0]
305
306 self.assertEqual(sf.display_name, 'root.sym')
307 self.assertEqual(sf.display_path, self.symfile)
308 self.assertEqual(sf.file_name, self.symfile)
309 self.assertEqual(sf.status, upload_symbols.SymbolFile.INITIAL)
310 self.assertEqual(sf.FileSize(), len(self.FAT_CONTENT))
311
312 def testDir(self):
313 symbols = list(upload_symbols.FindSymbolFiles(
314 self.working, [self.data]))
315
316 self.assertEqual(len(symbols), 2)
317 root = symbols[0]
318 nested = symbols[1]
319
320 self.assertEqual(root.display_name, 'root.sym')
321 self.assertEqual(root.display_path, 'root.sym')
322 self.assertEqual(root.file_name, self.symfile)
323 self.assertEqual(root.status, upload_symbols.SymbolFile.INITIAL)
324 self.assertEqual(root.FileSize(), len(self.FAT_CONTENT))
325
326 self.assertEqual(nested.display_name, 'inner.sym')
327 self.assertEqual(nested.display_path, 'nested/inner.sym')
328 self.assertEqual(nested.file_name, self.innerfile)
329 self.assertEqual(nested.status, upload_symbols.SymbolFile.INITIAL)
330 self.assertEqual(nested.FileSize(), len(self.FAT_CONTENT))
331
332
333class AdjustSymbolFileSizeTest(SymbolsTestBase):
334 """Test AdjustSymbolFileSize."""
335 def setUp(self):
336 self.slim = self.createSymbolFile('slim.sym', self.SLIM_CONTENT)
337 self.fat = self.createSymbolFile('fat.sym', self.FAT_CONTENT)
338
339 self.warn_mock = self.PatchObject(logging, 'PrintBuildbotStepWarnings')
340
341 def _testNotStripped(self, symbol, size=None, content=None):
342 start_file = symbol.file_name
343 after = upload_symbols.AdjustSymbolFileSize(
344 symbol, self.working, size)
345 self.assertIs(after, symbol)
346 self.assertEqual(after.file_name, start_file)
347 if content is not None:
348 self.assertEqual(osutils.ReadFile(after.file_name), content)
349
350 def _testStripped(self, symbol, size=None, content=None):
351 after = upload_symbols.AdjustSymbolFileSize(
352 symbol, self.working, size)
353 self.assertIs(after, symbol)
354 self.assertTrue(after.file_name.startswith(self.working))
355 if content is not None:
356 self.assertEqual(osutils.ReadFile(after.file_name), content)
357
358 def testSmall(self):
359 """Ensure that files smaller than the limit are not modified."""
360 self._testNotStripped(self.slim, 1024, self.SLIM_CONTENT)
361 self._testNotStripped(self.fat, 1024, self.FAT_CONTENT)
362
363 def testLarge(self):
364 """Ensure that files larger than the limit are modified."""
365 self._testStripped(self.slim, 1, self.SLIM_CONTENT)
366 self._testStripped(self.fat, 1, self.SLIM_CONTENT)
367
368 def testMixed(self):
369 """Test mix of large and small."""
370 strip_size = len(self.SLIM_CONTENT) + 1
371
372 self._testNotStripped(self.slim, strip_size, self.SLIM_CONTENT)
373 self._testStripped(self.fat, strip_size, self.SLIM_CONTENT)
374
375 def testSizeWarnings(self):
376 large = self.createSymbolFile(
377 'large.sym', content=self.SLIM_CONTENT,
378 size=upload_symbols.CRASH_SERVER_FILE_LIMIT*2)
379
380 # Would like to Strip as part of this test, but that really copies all
381 # of the sparse file content, which is too expensive for a unittest.
382 self._testNotStripped(large, None, None)
383
384 self.assertEqual(self.warn_mock.call_count, 1)
385
386
Mike Nichols137e82d2019-05-15 18:40:34 -0600387class DeduplicateTest(SymbolsTestBase):
388 """Test server Deduplication."""
389 def setUp(self):
390 self.PatchObject(upload_symbols, 'ExecRequest',
391 return_value={'pairs': [
392 {'status': 'FOUND',
393 'symbolId':
394 {'debugFile': 'sym1_sym',
395 'debugId': 'BEAA9BE'}},
396 {'status': 'FOUND',
397 'symbolId':
398 {'debugFile': 'sym2_sym',
399 'debugId': 'B6B1A36'}},
400 {'status': 'MISSING',
401 'symbolId':
402 {'debugFile': 'sym3_sym',
403 'debugId': 'D4FC0FC'}}]})
404
405 def testFindDuplicates(self):
406 # The first two symbols will be duplicate, the third new.
407 sym1 = self.createSymbolFile('sym1.sym')
408 sym1.header = cros_generate_breakpad_symbols.SymbolHeader('cpu', 'BEAA9BE',
409 'sym1_sym', 'os')
410 sym2 = self.createSymbolFile('sym2.sym')
411 sym2.header = cros_generate_breakpad_symbols.SymbolHeader('cpu', 'B6B1A36',
412 'sym2_sym', 'os')
413 sym3 = self.createSymbolFile('sym3.sym')
414 sym3.header = cros_generate_breakpad_symbols.SymbolHeader('cpu', 'D4FC0FC',
415 'sym3_sym', 'os')
416
417 result = upload_symbols.FindDuplicates((sym1, sym2, sym3), 'fake_url',
418 api_key='testkey')
419 self.assertEqual(list(result), [sym1, sym2, sym3])
420
421 self.assertEqual(sym1.status, upload_symbols.SymbolFile.DUPLICATE)
422 self.assertEqual(sym2.status, upload_symbols.SymbolFile.DUPLICATE)
423 self.assertEqual(sym3.status, upload_symbols.SymbolFile.INITIAL)
424
425
Don Garrettdeb2e032016-07-06 16:44:14 -0700426class PerformSymbolFilesUploadTest(SymbolsTestBase):
Don Garretta28be6d2016-06-16 18:09:35 -0700427 """Test PerformSymbolFile, and it's helper methods."""
428 def setUp(self):
Don Garrettdeb2e032016-07-06 16:44:14 -0700429 self.sym_initial = self.createSymbolFile(
430 'initial.sym')
431 self.sym_error = self.createSymbolFile(
432 'error.sym', status=upload_symbols.SymbolFile.ERROR)
433 self.sym_duplicate = self.createSymbolFile(
434 'duplicate.sym', status=upload_symbols.SymbolFile.DUPLICATE)
435 self.sym_uploaded = self.createSymbolFile(
436 'uploaded.sym', status=upload_symbols.SymbolFile.UPLOADED)
Don Garretta28be6d2016-06-16 18:09:35 -0700437
438 def testGetUploadTimeout(self):
439 """Test GetUploadTimeout helper function."""
440 # Timeout for small file.
Don Garrettdeb2e032016-07-06 16:44:14 -0700441 self.assertEqual(upload_symbols.GetUploadTimeout(self.sym_initial),
Don Garretta28be6d2016-06-16 18:09:35 -0700442 upload_symbols.UPLOAD_MIN_TIMEOUT)
443
444 # Timeout for 300M file.
445 large = self.createSymbolFile('large.sym', size=(300 * 1024 * 1024))
Ryo Hashimotof864c0e2017-02-14 12:46:08 +0900446 self.assertEqual(upload_symbols.GetUploadTimeout(large), 771)
Don Garretta28be6d2016-06-16 18:09:35 -0700447
448 def testUploadSymbolFile(self):
Mike Nichols90f7c152019-04-09 15:14:08 -0600449 upload_symbols.UploadSymbolFile('fake_url', self.sym_initial,
450 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700451 # TODO: Examine mock in more detail to make sure request is correct.
Mike Nichols137e82d2019-05-15 18:40:34 -0600452 self.assertEqual(self.request_mock.call_count, 3)
Don Garretta28be6d2016-06-16 18:09:35 -0700453
Don Garrettdeb2e032016-07-06 16:44:14 -0700454 def testPerformSymbolsFileUpload(self):
Don Garretta28be6d2016-06-16 18:09:35 -0700455 """We upload on first try."""
Don Garrettdeb2e032016-07-06 16:44:14 -0700456 symbols = [self.sym_initial]
Don Garretta28be6d2016-06-16 18:09:35 -0700457
Don Garrettdeb2e032016-07-06 16:44:14 -0700458 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600459 symbols, 'fake_url', api_key='testkey')
Don Garrettdeb2e032016-07-06 16:44:14 -0700460
461 self.assertEqual(list(result), symbols)
462 self.assertEqual(self.sym_initial.status,
463 upload_symbols.SymbolFile.UPLOADED)
Mike Nichols137e82d2019-05-15 18:40:34 -0600464 self.assertEqual(self.request_mock.call_count, 3)
Don Garretta28be6d2016-06-16 18:09:35 -0700465
Don Garrettdeb2e032016-07-06 16:44:14 -0700466 def testPerformSymbolsFileUploadFailure(self):
Don Garretta28be6d2016-06-16 18:09:35 -0700467 """All network requests fail."""
Mike Frysinger3dcacee2019-08-23 17:09:11 -0400468 self.request_mock.side_effect = IOError('network failure')
Don Garrettdeb2e032016-07-06 16:44:14 -0700469 symbols = [self.sym_initial]
Don Garretta28be6d2016-06-16 18:09:35 -0700470
Don Garrettdeb2e032016-07-06 16:44:14 -0700471 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600472 symbols, 'fake_url', api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700473
Don Garrettdeb2e032016-07-06 16:44:14 -0700474 self.assertEqual(list(result), symbols)
475 self.assertEqual(self.sym_initial.status, upload_symbols.SymbolFile.ERROR)
Mike Nichols90f7c152019-04-09 15:14:08 -0600476 self.assertEqual(self.request_mock.call_count, 7)
Don Garretta28be6d2016-06-16 18:09:35 -0700477
Don Garrettdeb2e032016-07-06 16:44:14 -0700478 def testPerformSymbolsFileUploadTransisentFailure(self):
Don Garretta28be6d2016-06-16 18:09:35 -0700479 """We fail once, then succeed."""
Mike Frysinger3dcacee2019-08-23 17:09:11 -0400480 self.urlopen_mock.side_effect = (IOError('network failure'), None)
Don Garrettdeb2e032016-07-06 16:44:14 -0700481 symbols = [self.sym_initial]
Don Garretta28be6d2016-06-16 18:09:35 -0700482
Don Garrettdeb2e032016-07-06 16:44:14 -0700483 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600484 symbols, 'fake_url', api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700485
Don Garrettdeb2e032016-07-06 16:44:14 -0700486 self.assertEqual(list(result), symbols)
487 self.assertEqual(self.sym_initial.status,
488 upload_symbols.SymbolFile.UPLOADED)
Mike Nichols137e82d2019-05-15 18:40:34 -0600489 self.assertEqual(self.request_mock.call_count, 3)
Don Garrettdeb2e032016-07-06 16:44:14 -0700490
491 def testPerformSymbolsFileUploadMixed(self):
492 """Upload symbols in mixed starting states.
493
494 Demonstrate that INITIAL and ERROR are uploaded, but DUPLICATE/UPLOADED are
495 ignored.
496 """
497 symbols = [self.sym_initial, self.sym_error,
498 self.sym_duplicate, self.sym_uploaded]
499
500 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600501 symbols, 'fake_url', api_key='testkey')
Don Garrettdeb2e032016-07-06 16:44:14 -0700502
503 #
504 self.assertEqual(list(result), symbols)
505 self.assertEqual(self.sym_initial.status,
506 upload_symbols.SymbolFile.UPLOADED)
507 self.assertEqual(self.sym_error.status,
508 upload_symbols.SymbolFile.UPLOADED)
509 self.assertEqual(self.sym_duplicate.status,
510 upload_symbols.SymbolFile.DUPLICATE)
511 self.assertEqual(self.sym_uploaded.status,
512 upload_symbols.SymbolFile.UPLOADED)
Mike Nichols137e82d2019-05-15 18:40:34 -0600513 self.assertEqual(self.request_mock.call_count, 6)
Don Garrettdeb2e032016-07-06 16:44:14 -0700514
515
516 def testPerformSymbolsFileUploadErrorOut(self):
517 """Demonstate we exit only after X errors."""
518
519 symbol_count = upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY + 10
520 symbols = []
521 fail_file = None
522
523 # potentially twice as many errors as we should attempt.
Mike Frysinger79cca962019-06-13 15:26:53 -0400524 for _ in range(symbol_count):
Don Garrettdeb2e032016-07-06 16:44:14 -0700525 # Each loop will get unique SymbolFile instances that use the same files.
526 fail = self.createSymbolFile('fail.sym')
527 fail_file = fail.file_name
528 symbols.append(self.createSymbolFile('pass.sym'))
529 symbols.append(fail)
530
531 # Mock out UploadSymbolFile and fail for fail.sym files.
Mike Nichols90f7c152019-04-09 15:14:08 -0600532 def failSome(_url, symbol, _api_key):
Don Garrettdeb2e032016-07-06 16:44:14 -0700533 if symbol.file_name == fail_file:
Mike Frysinger3dcacee2019-08-23 17:09:11 -0400534 raise IOError('network failure')
Don Garrettdeb2e032016-07-06 16:44:14 -0700535
Luigi Semenzato5104f3c2016-10-12 12:37:42 -0700536 upload_mock = self.PatchObject(upload_symbols, 'UploadSymbolFile',
537 side_effect=failSome)
538 upload_mock.__name__ = 'UploadSymbolFileMock2'
Don Garrettdeb2e032016-07-06 16:44:14 -0700539
540 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600541 symbols, 'fake_url', api_key='testkey')
Don Garrettdeb2e032016-07-06 16:44:14 -0700542
543 self.assertEqual(list(result), symbols)
544
545 passed = sum(s.status == upload_symbols.SymbolFile.UPLOADED
546 for s in symbols)
547 failed = sum(s.status == upload_symbols.SymbolFile.ERROR
548 for s in symbols)
549 skipped = sum(s.status == upload_symbols.SymbolFile.INITIAL
550 for s in symbols)
551
552 # Shows we all pass.sym files worked until limit hit.
553 self.assertEqual(passed, upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY)
554
555 # Shows we all fail.sym files failed until limit hit.
556 self.assertEqual(failed, upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY)
557
558 # Shows both pass/fail were skipped after limit hit.
559 self.assertEqual(skipped, 10 * 2)
Don Garretta28be6d2016-06-16 18:09:35 -0700560
561
562class UploadSymbolsTest(SymbolsTestBase):
563 """Test UploadSymbols, along with most helper methods."""
564 def setUp(self):
565 # Results gathering.
566 self.failure_file = os.path.join(self.tempdir, 'failures.txt')
567
568 def testUploadSymbolsEmpty(self):
569 """Upload dir is empty."""
Mike Nichols137e82d2019-05-15 18:40:34 -0600570 result = upload_symbols.UploadSymbols([self.data], 'fake_url')
Don Garretta28be6d2016-06-16 18:09:35 -0700571
Mike Frysinger2d589a12019-08-25 14:15:12 -0400572 self.assertEqual(result, 0)
Don Garretta28be6d2016-06-16 18:09:35 -0700573 self.assertEqual(self.urlopen_mock.call_count, 0)
574
575 def testUploadSymbols(self):
576 """Upload a few files."""
577 self.createSymbolFile('slim.sym', self.SLIM_CONTENT)
578 self.createSymbolFile(os.path.join('nested', 'inner.sym'))
579 self.createSymbolFile('fat.sym', self.FAT_CONTENT)
580
581 result = upload_symbols.UploadSymbols(
Mike Nichols90f7c152019-04-09 15:14:08 -0600582 [self.data], 'fake_url',
583 failed_list=self.failure_file, strip_cfi=len(self.SLIM_CONTENT)+1,
584 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700585
Mike Frysinger2d589a12019-08-25 14:15:12 -0400586 self.assertEqual(result, 0)
Mike Nichols137e82d2019-05-15 18:40:34 -0600587 self.assertEqual(self.request_mock.call_count, 10)
Mike Frysinger2d589a12019-08-25 14:15:12 -0400588 self.assertEqual(osutils.ReadFile(self.failure_file), '')
Don Garretta28be6d2016-06-16 18:09:35 -0700589
590 def testUploadSymbolsLimited(self):
591 """Upload a few files."""
592 self.createSymbolFile('slim.sym', self.SLIM_CONTENT)
593 self.createSymbolFile(os.path.join('nested', 'inner.sym'))
594 self.createSymbolFile('fat.sym', self.FAT_CONTENT)
595
596 result = upload_symbols.UploadSymbols(
Mike Nichols90f7c152019-04-09 15:14:08 -0600597 [self.data], 'fake_url', upload_limit=2,
598 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700599
Mike Frysinger2d589a12019-08-25 14:15:12 -0400600 self.assertEqual(result, 0)
Mike Nichols137e82d2019-05-15 18:40:34 -0600601 self.assertEqual(self.request_mock.call_count, 7)
Mike Frysingerf2fa7d62017-12-14 18:33:59 -0500602 self.assertNotExists(self.failure_file)
Don Garretta28be6d2016-06-16 18:09:35 -0700603
604 def testUploadSymbolsFailures(self):
605 """Upload a few files."""
606 self.createSymbolFile('pass.sym')
607 fail = self.createSymbolFile('fail.sym')
608
Mike Nichols90f7c152019-04-09 15:14:08 -0600609 def failSome(_url, symbol, _api_key):
Don Garretta28be6d2016-06-16 18:09:35 -0700610 if symbol.file_name == fail.file_name:
Mike Frysinger3dcacee2019-08-23 17:09:11 -0400611 raise IOError('network failure')
Don Garretta28be6d2016-06-16 18:09:35 -0700612
613 # Mock out UploadSymbolFile so it's easy to see which file to fail for.
614 upload_mock = self.PatchObject(upload_symbols, 'UploadSymbolFile',
615 side_effect=failSome)
Luigi Semenzato5104f3c2016-10-12 12:37:42 -0700616 # Mock __name__ for logging.
617 upload_mock.__name__ = 'UploadSymbolFileMock'
Don Garretta28be6d2016-06-16 18:09:35 -0700618
619 result = upload_symbols.UploadSymbols(
Mike Nichols90f7c152019-04-09 15:14:08 -0600620 [self.data], 'fake_url',
621 failed_list=self.failure_file, api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700622
Mike Frysinger2d589a12019-08-25 14:15:12 -0400623 self.assertEqual(result, 1)
Don Garretta28be6d2016-06-16 18:09:35 -0700624 self.assertEqual(upload_mock.call_count, 8)
Mike Frysinger2d589a12019-08-25 14:15:12 -0400625 self.assertEqual(osutils.ReadFile(self.failure_file), 'fail.sym\n')
Don Garretta28be6d2016-06-16 18:09:35 -0700626
Don Garretta28be6d2016-06-16 18:09:35 -0700627# TODO: We removed --network integration tests.
Mike Frysinger0c0efa22014-02-09 23:32:23 -0500628
629
Mike Frysingerea838d12014-12-08 11:55:32 -0500630def main(_argv):
Mike Frysinger27e21b72018-07-12 14:20:21 -0400631 # pylint: disable=protected-access
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -0400632 # Set timeouts small so that if the unit test hangs, it won't hang for long.
633 parallel._BackgroundTask.STARTUP_TIMEOUT = 5
634 parallel._BackgroundTask.EXIT_TIMEOUT = 5
635
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -0400636 # Run the tests.
Mike Frysingerba167372015-01-21 10:37:03 -0500637 cros_test_lib.main(level='info', module=__name__)