blob: 37e0a7af50c8133875730021b3c88c4513d839a6 [file] [log] [blame]
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -04001# 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 Frysingereb753bf2013-11-22 16:05:35 -05005"""Unittests for upload_symbols.py"""
6
Mike Frysinger06da5202014-09-26 17:30:33 -05007import errno
Don Garretta28be6d2016-06-16 18:09:35 -07008import itertools
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -04009import os
Mike Frysinger0a2fd922014-09-12 20:23:42 -070010import signal
Mike Frysinger06da5202014-09-26 17:30:33 -050011import socket
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040012import sys
Aviv Keshetd1f04632014-05-09 11:33:46 -070013import time
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040014
Mike Frysinger40ffb532021-02-12 07:36:08 -050015import pytest # pylint: disable=import-error
Mike Frysinger3d465162019-08-28 00:40:23 -040016from six.moves import BaseHTTPServer
17from six.moves import socketserver
Mike Frysinger3dcacee2019-08-23 17:09:11 -040018from six.moves import urllib
Mike Frysinger40ffb532021-02-12 07:36:08 -050019
20from chromite.third_party import mock
21
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
Mike Frysinger40ffb532021-02-12 07:36:08 -050033
Mike Frysingerbbd1f112016-09-08 18:25:11 -040034# The isolateserver includes a bunch of third_party python packages that clash
35# with chromite's bundled third_party python packages (like oauth2client).
36# Since upload_symbols is not imported in to other parts of chromite, and there
37# are no deps in third_party we care about, purge the chromite copy. This way
38# we can use isolateserver for deduping.
39# TODO: If we ever sort out third_party/ handling and make it per-script opt-in,
40# we can purge this logic.
41third_party = os.path.join(constants.CHROMITE_DIR, 'third_party')
42while True:
43 try:
44 sys.path.remove(third_party)
45 except ValueError:
46 break
Mike Frysingerbbd1f112016-09-08 18:25:11 -040047del third_party
48
Ralph Nathan446aee92015-03-23 14:44:56 -070049from chromite.lib import cros_logging as logging
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040050from chromite.lib import cros_test_lib
51from chromite.lib import osutils
52from chromite.lib import parallel
Mike Frysinger0a2fd922014-09-12 20:23:42 -070053from chromite.lib import remote_access
Mike Frysinger5e30a4b2014-02-12 20:23:04 -050054from chromite.scripts import cros_generate_breakpad_symbols
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040055from chromite.scripts import upload_symbols
56
Mike Frysinger0c0efa22014-02-09 23:32:23 -050057
Don Garretta28be6d2016-06-16 18:09:35 -070058class SymbolsTestBase(cros_test_lib.MockTempDirTestCase):
59 """Base class for most symbols tests."""
60
61 SLIM_CONTENT = """
62some junk
63"""
64
65 FAT_CONTENT = """
66STACK CFI 1234
67some junk
68STACK CFI 1234
69"""
70
71 def setUp(self):
72 # Make certain we don't use the network.
Mike Frysinger3dcacee2019-08-23 17:09:11 -040073 self.urlopen_mock = self.PatchObject(urllib.request, 'urlopen')
Mike Nichols90f7c152019-04-09 15:14:08 -060074 self.request_mock = self.PatchObject(upload_symbols, 'ExecRequest',
75 return_value={'uploadUrl':
76 'testurl',
77 'uploadKey':
78 'asdgasgas'})
Don Garretta28be6d2016-06-16 18:09:35 -070079
80 # Make 'uploads' go fast.
81 self.PatchObject(upload_symbols, 'SLEEP_DELAY', 0)
82 self.PatchObject(upload_symbols, 'INITIAL_RETRY_DELAY', 0)
83
84 # So our symbol file content doesn't have to be real.
85 self.PatchObject(cros_generate_breakpad_symbols, 'ReadSymsHeader',
86 return_value=cros_generate_breakpad_symbols.SymbolHeader(
87 os='os', cpu='cpu', id='id', name='name'))
88
89 self.working = os.path.join(self.tempdir, 'expand')
90 osutils.SafeMakedirs(self.working)
91
92 self.data = os.path.join(self.tempdir, 'data')
93 osutils.SafeMakedirs(self.data)
94
95 def createSymbolFile(self, filename, content=FAT_CONTENT, size=0,
96 status=None, dedupe=False):
97 fullname = os.path.join(self.data, filename)
98 osutils.SafeMakedirs(os.path.dirname(fullname))
99
100 # If a file size is given, force that to be the minimum file size. Create
101 # a sparse file so large files are practical.
102 with open(fullname, 'w+b') as f:
103 f.truncate(size)
104 f.seek(0)
Mike Frysingerac75f602019-11-14 23:18:51 -0500105 f.write(content.encode('utf-8'))
Don Garretta28be6d2016-06-16 18:09:35 -0700106
107 result = upload_symbols.SymbolFile(display_path=filename,
108 file_name=fullname)
109
110 if status:
111 result.status = status
112
113 if dedupe:
114 result.dedupe_item = upload_symbols.DedupeItem(result)
115 result.dedupe_push_state = 'push_state'
116
117 return result
118
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -0400119
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700120class SymbolServerRequestHandler(BaseHTTPServer.BaseHTTPRequestHandler):
121 """HTTP handler for symbol POSTs"""
122
123 RESP_CODE = None
124 RESP_MSG = None
125
126 def do_POST(self):
127 """Handle a POST request"""
128 # Drain the data from the client. If we don't, we might write the response
129 # and close the socket before the client finishes, so they die with EPIPE.
130 clen = int(self.headers.get('Content-Length', '0'))
131 self.rfile.read(clen)
132
133 self.send_response(self.RESP_CODE, self.RESP_MSG)
134 self.end_headers()
135
Mike Frysingera4418152019-09-22 04:00:24 -0400136 # pylint: disable=arguments-differ
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700137 def log_message(self, *args, **kwargs):
138 """Stub the logger as it writes to stderr"""
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700139
140
Mike Frysinger3d465162019-08-28 00:40:23 -0400141class SymbolServer(socketserver.ThreadingTCPServer, BaseHTTPServer.HTTPServer):
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700142 """Simple HTTP server that forks each request"""
143
144
Chris McDonalde705d8e2020-04-17 09:29:08 -0600145@pytest.mark.usefixtures('singleton_manager')
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700146class UploadSymbolsServerTest(cros_test_lib.MockTempDirTestCase):
147 """Tests for UploadSymbols() and a local HTTP server"""
148
149 SYM_CONTENTS = """MODULE Linux arm 123-456 blkid
150PUBLIC 1471 0 main"""
151
152 def SpawnServer(self, RequestHandler):
153 """Spawn a new http server"""
Mike Frysinger06da5202014-09-26 17:30:33 -0500154 while True:
155 try:
156 port = remote_access.GetUnusedPort()
157 address = ('', port)
158 self.httpd = SymbolServer(address, RequestHandler)
159 break
160 except socket.error as e:
161 if e.errno == errno.EADDRINUSE:
162 continue
163 raise
Don Garretta28be6d2016-06-16 18:09:35 -0700164 self.server_url = 'http://localhost:%i/post/path' % port
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700165 self.httpd_pid = os.fork()
166 if self.httpd_pid == 0:
167 self.httpd.serve_forever(poll_interval=0.1)
168 sys.exit(0)
Mike Frysingerac75f602019-11-14 23:18:51 -0500169 # The child runs the server, so close the socket in the parent.
170 self.httpd.server_close()
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700171
172 def setUp(self):
173 self.httpd_pid = None
174 self.httpd = None
Don Garretta28be6d2016-06-16 18:09:35 -0700175 self.server_url = None
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700176 self.sym_file = os.path.join(self.tempdir, 'test.sym')
177 osutils.WriteFile(self.sym_file, self.SYM_CONTENTS)
178
Don Garretta28be6d2016-06-16 18:09:35 -0700179 # Stop sleeps and retries for these tests.
180 self.PatchObject(upload_symbols, 'SLEEP_DELAY', 0)
181 self.PatchObject(upload_symbols, 'INITIAL_RETRY_DELAY', 0)
182 self.PatchObject(upload_symbols, 'MAX_RETRIES', 0)
183
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700184 def tearDown(self):
185 # Only kill the server if we forked one.
186 if self.httpd_pid:
187 os.kill(self.httpd_pid, signal.SIGUSR1)
188
189 def testSuccess(self):
190 """The server returns success for all uploads"""
191 class Handler(SymbolServerRequestHandler):
192 """Always return 200"""
193 RESP_CODE = 200
Mike Nichols90f7c152019-04-09 15:14:08 -0600194 self.PatchObject(upload_symbols, 'ExecRequest',
195 return_value={'uploadUrl': 'testurl',
196 'uploadKey': 'testSuccess'})
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700197 self.SpawnServer(Handler)
Don Garretta28be6d2016-06-16 18:09:35 -0700198 ret = upload_symbols.UploadSymbols(
199 sym_paths=[self.sym_file] * 10,
200 upload_url=self.server_url,
Mike Nichols90f7c152019-04-09 15:14:08 -0600201 api_key='testSuccess')
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700202 self.assertEqual(ret, 0)
203
204 def testError(self):
205 """The server returns errors for all uploads"""
206 class Handler(SymbolServerRequestHandler):
Mike Nichols90f7c152019-04-09 15:14:08 -0600207 """All connections error"""
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700208 RESP_CODE = 500
209 RESP_MSG = 'Internal Server Error'
210
211 self.SpawnServer(Handler)
Don Garretta28be6d2016-06-16 18:09:35 -0700212 ret = upload_symbols.UploadSymbols(
213 sym_paths=[self.sym_file] * 10,
214 upload_url=self.server_url,
Mike Nichols90f7c152019-04-09 15:14:08 -0600215 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700216 self.assertEqual(ret, 10)
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700217
218 def testHungServer(self):
219 """The server chokes, but we recover"""
220 class Handler(SymbolServerRequestHandler):
221 """All connections choke forever"""
Mike Nichols137e82d2019-05-15 18:40:34 -0600222 self.PatchObject(upload_symbols, 'ExecRequest',
223 return_value={'pairs': []})
224
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700225 def do_POST(self):
226 while True:
227 time.sleep(1000)
228
229 self.SpawnServer(Handler)
230 with mock.patch.object(upload_symbols, 'GetUploadTimeout') as m:
Don Garretta28be6d2016-06-16 18:09:35 -0700231 m.return_value = 0.01
Mike Frysinger58312e92014-03-18 04:18:36 -0400232 ret = upload_symbols.UploadSymbols(
Don Garretta28be6d2016-06-16 18:09:35 -0700233 sym_paths=[self.sym_file] * 10,
234 upload_url=self.server_url,
Mike Nichols137e82d2019-05-15 18:40:34 -0600235 timeout=m.return_value,
Mike Nichols90f7c152019-04-09 15:14:08 -0600236 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700237 self.assertEqual(ret, 10)
Aviv Keshetd1f04632014-05-09 11:33:46 -0700238
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -0400239
Don Garretta28be6d2016-06-16 18:09:35 -0700240class UploadSymbolsHelpersTest(cros_test_lib.TestCase):
241 """Test assorted helper functions and classes."""
Don Garretta28be6d2016-06-16 18:09:35 -0700242 def testIsTarball(self):
243 notTar = [
244 '/foo/bar/test.bin',
245 '/foo/bar/test.tar.bin',
246 '/foo/bar/test.faketar.gz',
247 '/foo/bar/test.nottgz',
Mike Frysinger0c0efa22014-02-09 23:32:23 -0500248 ]
Mike Frysinger0c0efa22014-02-09 23:32:23 -0500249
Don Garretta28be6d2016-06-16 18:09:35 -0700250 isTar = [
251 '/foo/bar/test.tar',
252 '/foo/bar/test.bin.tar',
253 '/foo/bar/test.bin.tar.bz2',
254 '/foo/bar/test.bin.tar.gz',
255 '/foo/bar/test.bin.tar.xz',
256 '/foo/bar/test.tbz2',
257 '/foo/bar/test.tbz',
258 '/foo/bar/test.tgz',
259 '/foo/bar/test.txz',
260 ]
Mike Frysinger0c0efa22014-02-09 23:32:23 -0500261
Don Garretta28be6d2016-06-16 18:09:35 -0700262 for p in notTar:
263 self.assertFalse(upload_symbols.IsTarball(p))
264
265 for p in isTar:
266 self.assertTrue(upload_symbols.IsTarball(p))
267
268 def testBatchGenerator(self):
269 result = upload_symbols.BatchGenerator([], 2)
270 self.assertEqual(list(result), [])
271
Mike Frysinger66848312019-07-03 02:12:39 -0400272 # BatchGenerator accepts iterators, so passing it here is safe.
273 # pylint: disable=range-builtin-not-iterating
Mike Frysinger79cca962019-06-13 15:26:53 -0400274 result = upload_symbols.BatchGenerator(range(6), 2)
Don Garretta28be6d2016-06-16 18:09:35 -0700275 self.assertEqual(list(result), [[0, 1], [2, 3], [4, 5]])
276
Mike Frysinger79cca962019-06-13 15:26:53 -0400277 result = upload_symbols.BatchGenerator(range(7), 2)
Don Garretta28be6d2016-06-16 18:09:35 -0700278 self.assertEqual(list(result), [[0, 1], [2, 3], [4, 5], [6]])
279
280 # Prove that we are streaming the results, not generating them all at once.
281 result = upload_symbols.BatchGenerator(itertools.repeat(0), 2)
Mike Frysinger67d90242019-07-03 19:03:58 -0400282 self.assertEqual(next(result), [0, 0])
Don Garretta28be6d2016-06-16 18:09:35 -0700283
284
285class FindSymbolFilesTest(SymbolsTestBase):
286 """Test FindSymbolFiles."""
287 def setUp(self):
288 self.symfile = self.createSymbolFile('root.sym').file_name
289 self.innerfile = self.createSymbolFile(
290 os.path.join('nested', 'inner.sym')).file_name
291
292 # CreateTarball is having issues outside the chroot from open file tests.
293 #
294 # self.tarball = os.path.join(self.tempdir, 'syms.tar.gz')
295 # cros_build_lib.CreateTarball(
296 # 'syms.tar.gz', self.tempdir, inputs=(self.data))
297
298 def testEmpty(self):
299 symbols = list(upload_symbols.FindSymbolFiles(
300 self.working, []))
301 self.assertEqual(symbols, [])
302
303 def testFile(self):
304 symbols = list(upload_symbols.FindSymbolFiles(
305 self.working, [self.symfile]))
306
307 self.assertEqual(len(symbols), 1)
308 sf = symbols[0]
309
310 self.assertEqual(sf.display_name, 'root.sym')
311 self.assertEqual(sf.display_path, self.symfile)
312 self.assertEqual(sf.file_name, self.symfile)
313 self.assertEqual(sf.status, upload_symbols.SymbolFile.INITIAL)
314 self.assertEqual(sf.FileSize(), len(self.FAT_CONTENT))
315
316 def testDir(self):
317 symbols = list(upload_symbols.FindSymbolFiles(
318 self.working, [self.data]))
319
320 self.assertEqual(len(symbols), 2)
321 root = symbols[0]
322 nested = symbols[1]
323
324 self.assertEqual(root.display_name, 'root.sym')
325 self.assertEqual(root.display_path, 'root.sym')
326 self.assertEqual(root.file_name, self.symfile)
327 self.assertEqual(root.status, upload_symbols.SymbolFile.INITIAL)
328 self.assertEqual(root.FileSize(), len(self.FAT_CONTENT))
329
330 self.assertEqual(nested.display_name, 'inner.sym')
331 self.assertEqual(nested.display_path, 'nested/inner.sym')
332 self.assertEqual(nested.file_name, self.innerfile)
333 self.assertEqual(nested.status, upload_symbols.SymbolFile.INITIAL)
334 self.assertEqual(nested.FileSize(), len(self.FAT_CONTENT))
335
336
337class AdjustSymbolFileSizeTest(SymbolsTestBase):
338 """Test AdjustSymbolFileSize."""
339 def setUp(self):
340 self.slim = self.createSymbolFile('slim.sym', self.SLIM_CONTENT)
341 self.fat = self.createSymbolFile('fat.sym', self.FAT_CONTENT)
342
343 self.warn_mock = self.PatchObject(logging, 'PrintBuildbotStepWarnings')
344
345 def _testNotStripped(self, symbol, size=None, content=None):
346 start_file = symbol.file_name
347 after = upload_symbols.AdjustSymbolFileSize(
348 symbol, self.working, size)
349 self.assertIs(after, symbol)
350 self.assertEqual(after.file_name, start_file)
351 if content is not None:
352 self.assertEqual(osutils.ReadFile(after.file_name), content)
353
354 def _testStripped(self, symbol, size=None, content=None):
355 after = upload_symbols.AdjustSymbolFileSize(
356 symbol, self.working, size)
357 self.assertIs(after, symbol)
358 self.assertTrue(after.file_name.startswith(self.working))
359 if content is not None:
360 self.assertEqual(osutils.ReadFile(after.file_name), content)
361
362 def testSmall(self):
363 """Ensure that files smaller than the limit are not modified."""
364 self._testNotStripped(self.slim, 1024, self.SLIM_CONTENT)
365 self._testNotStripped(self.fat, 1024, self.FAT_CONTENT)
366
367 def testLarge(self):
368 """Ensure that files larger than the limit are modified."""
369 self._testStripped(self.slim, 1, self.SLIM_CONTENT)
370 self._testStripped(self.fat, 1, self.SLIM_CONTENT)
371
372 def testMixed(self):
373 """Test mix of large and small."""
374 strip_size = len(self.SLIM_CONTENT) + 1
375
376 self._testNotStripped(self.slim, strip_size, self.SLIM_CONTENT)
377 self._testStripped(self.fat, strip_size, self.SLIM_CONTENT)
378
379 def testSizeWarnings(self):
380 large = self.createSymbolFile(
381 'large.sym', content=self.SLIM_CONTENT,
382 size=upload_symbols.CRASH_SERVER_FILE_LIMIT*2)
383
384 # Would like to Strip as part of this test, but that really copies all
385 # of the sparse file content, which is too expensive for a unittest.
386 self._testNotStripped(large, None, None)
387
388 self.assertEqual(self.warn_mock.call_count, 1)
389
390
Mike Nichols137e82d2019-05-15 18:40:34 -0600391class DeduplicateTest(SymbolsTestBase):
392 """Test server Deduplication."""
393 def setUp(self):
394 self.PatchObject(upload_symbols, 'ExecRequest',
395 return_value={'pairs': [
396 {'status': 'FOUND',
397 'symbolId':
398 {'debugFile': 'sym1_sym',
399 'debugId': 'BEAA9BE'}},
400 {'status': 'FOUND',
401 'symbolId':
402 {'debugFile': 'sym2_sym',
403 'debugId': 'B6B1A36'}},
404 {'status': 'MISSING',
405 'symbolId':
406 {'debugFile': 'sym3_sym',
407 'debugId': 'D4FC0FC'}}]})
408
409 def testFindDuplicates(self):
410 # The first two symbols will be duplicate, the third new.
411 sym1 = self.createSymbolFile('sym1.sym')
412 sym1.header = cros_generate_breakpad_symbols.SymbolHeader('cpu', 'BEAA9BE',
413 'sym1_sym', 'os')
414 sym2 = self.createSymbolFile('sym2.sym')
415 sym2.header = cros_generate_breakpad_symbols.SymbolHeader('cpu', 'B6B1A36',
416 'sym2_sym', 'os')
417 sym3 = self.createSymbolFile('sym3.sym')
418 sym3.header = cros_generate_breakpad_symbols.SymbolHeader('cpu', 'D4FC0FC',
419 'sym3_sym', 'os')
420
421 result = upload_symbols.FindDuplicates((sym1, sym2, sym3), 'fake_url',
422 api_key='testkey')
423 self.assertEqual(list(result), [sym1, sym2, sym3])
424
425 self.assertEqual(sym1.status, upload_symbols.SymbolFile.DUPLICATE)
426 self.assertEqual(sym2.status, upload_symbols.SymbolFile.DUPLICATE)
427 self.assertEqual(sym3.status, upload_symbols.SymbolFile.INITIAL)
428
429
Don Garrettdeb2e032016-07-06 16:44:14 -0700430class PerformSymbolFilesUploadTest(SymbolsTestBase):
Don Garretta28be6d2016-06-16 18:09:35 -0700431 """Test PerformSymbolFile, and it's helper methods."""
432 def setUp(self):
Don Garrettdeb2e032016-07-06 16:44:14 -0700433 self.sym_initial = self.createSymbolFile(
434 'initial.sym')
435 self.sym_error = self.createSymbolFile(
436 'error.sym', status=upload_symbols.SymbolFile.ERROR)
437 self.sym_duplicate = self.createSymbolFile(
438 'duplicate.sym', status=upload_symbols.SymbolFile.DUPLICATE)
439 self.sym_uploaded = self.createSymbolFile(
440 'uploaded.sym', status=upload_symbols.SymbolFile.UPLOADED)
Don Garretta28be6d2016-06-16 18:09:35 -0700441
442 def testGetUploadTimeout(self):
443 """Test GetUploadTimeout helper function."""
444 # Timeout for small file.
Don Garrettdeb2e032016-07-06 16:44:14 -0700445 self.assertEqual(upload_symbols.GetUploadTimeout(self.sym_initial),
Don Garretta28be6d2016-06-16 18:09:35 -0700446 upload_symbols.UPLOAD_MIN_TIMEOUT)
447
Ian Barkley-Yeung22ba8122020-02-05 15:39:02 -0800448 # Timeout for 512M file.
449 large = self.createSymbolFile('large.sym', size=(512 * 1024 * 1024))
450 self.assertEqual(upload_symbols.GetUploadTimeout(large), 15 * 60)
Don Garretta28be6d2016-06-16 18:09:35 -0700451
452 def testUploadSymbolFile(self):
Mike Nichols90f7c152019-04-09 15:14:08 -0600453 upload_symbols.UploadSymbolFile('fake_url', self.sym_initial,
454 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700455 # TODO: Examine mock in more detail to make sure request is correct.
Mike Nichols137e82d2019-05-15 18:40:34 -0600456 self.assertEqual(self.request_mock.call_count, 3)
Don Garretta28be6d2016-06-16 18:09:35 -0700457
Don Garrettdeb2e032016-07-06 16:44:14 -0700458 def testPerformSymbolsFileUpload(self):
Don Garretta28be6d2016-06-16 18:09:35 -0700459 """We upload on first try."""
Don Garrettdeb2e032016-07-06 16:44:14 -0700460 symbols = [self.sym_initial]
Don Garretta28be6d2016-06-16 18:09:35 -0700461
Don Garrettdeb2e032016-07-06 16:44:14 -0700462 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600463 symbols, 'fake_url', api_key='testkey')
Don Garrettdeb2e032016-07-06 16:44:14 -0700464
465 self.assertEqual(list(result), symbols)
466 self.assertEqual(self.sym_initial.status,
467 upload_symbols.SymbolFile.UPLOADED)
Mike Nichols137e82d2019-05-15 18:40:34 -0600468 self.assertEqual(self.request_mock.call_count, 3)
Don Garretta28be6d2016-06-16 18:09:35 -0700469
Don Garrettdeb2e032016-07-06 16:44:14 -0700470 def testPerformSymbolsFileUploadFailure(self):
Don Garretta28be6d2016-06-16 18:09:35 -0700471 """All network requests fail."""
Mike Frysinger3dcacee2019-08-23 17:09:11 -0400472 self.request_mock.side_effect = IOError('network failure')
Don Garrettdeb2e032016-07-06 16:44:14 -0700473 symbols = [self.sym_initial]
Don Garretta28be6d2016-06-16 18:09:35 -0700474
Don Garrettdeb2e032016-07-06 16:44:14 -0700475 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600476 symbols, 'fake_url', api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700477
Don Garrettdeb2e032016-07-06 16:44:14 -0700478 self.assertEqual(list(result), symbols)
479 self.assertEqual(self.sym_initial.status, upload_symbols.SymbolFile.ERROR)
Mike Nichols2a6f86f2020-08-18 15:56:07 -0600480 self.assertEqual(self.request_mock.call_count, 6)
Don Garretta28be6d2016-06-16 18:09:35 -0700481
Don Garrettdeb2e032016-07-06 16:44:14 -0700482 def testPerformSymbolsFileUploadTransisentFailure(self):
Don Garretta28be6d2016-06-16 18:09:35 -0700483 """We fail once, then succeed."""
Mike Frysinger3dcacee2019-08-23 17:09:11 -0400484 self.urlopen_mock.side_effect = (IOError('network failure'), None)
Don Garrettdeb2e032016-07-06 16:44:14 -0700485 symbols = [self.sym_initial]
Don Garretta28be6d2016-06-16 18:09:35 -0700486
Don Garrettdeb2e032016-07-06 16:44:14 -0700487 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600488 symbols, 'fake_url', api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700489
Don Garrettdeb2e032016-07-06 16:44:14 -0700490 self.assertEqual(list(result), symbols)
491 self.assertEqual(self.sym_initial.status,
492 upload_symbols.SymbolFile.UPLOADED)
Mike Nichols137e82d2019-05-15 18:40:34 -0600493 self.assertEqual(self.request_mock.call_count, 3)
Don Garrettdeb2e032016-07-06 16:44:14 -0700494
495 def testPerformSymbolsFileUploadMixed(self):
496 """Upload symbols in mixed starting states.
497
498 Demonstrate that INITIAL and ERROR are uploaded, but DUPLICATE/UPLOADED are
499 ignored.
500 """
501 symbols = [self.sym_initial, self.sym_error,
502 self.sym_duplicate, self.sym_uploaded]
503
504 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600505 symbols, 'fake_url', api_key='testkey')
Don Garrettdeb2e032016-07-06 16:44:14 -0700506
507 #
508 self.assertEqual(list(result), symbols)
509 self.assertEqual(self.sym_initial.status,
510 upload_symbols.SymbolFile.UPLOADED)
511 self.assertEqual(self.sym_error.status,
512 upload_symbols.SymbolFile.UPLOADED)
513 self.assertEqual(self.sym_duplicate.status,
514 upload_symbols.SymbolFile.DUPLICATE)
515 self.assertEqual(self.sym_uploaded.status,
516 upload_symbols.SymbolFile.UPLOADED)
Mike Nichols137e82d2019-05-15 18:40:34 -0600517 self.assertEqual(self.request_mock.call_count, 6)
Don Garrettdeb2e032016-07-06 16:44:14 -0700518
519
520 def testPerformSymbolsFileUploadErrorOut(self):
521 """Demonstate we exit only after X errors."""
522
523 symbol_count = upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY + 10
524 symbols = []
525 fail_file = None
526
527 # potentially twice as many errors as we should attempt.
Mike Frysinger79cca962019-06-13 15:26:53 -0400528 for _ in range(symbol_count):
Don Garrettdeb2e032016-07-06 16:44:14 -0700529 # Each loop will get unique SymbolFile instances that use the same files.
530 fail = self.createSymbolFile('fail.sym')
531 fail_file = fail.file_name
532 symbols.append(self.createSymbolFile('pass.sym'))
533 symbols.append(fail)
534
535 # Mock out UploadSymbolFile and fail for fail.sym files.
Mike Nichols90f7c152019-04-09 15:14:08 -0600536 def failSome(_url, symbol, _api_key):
Don Garrettdeb2e032016-07-06 16:44:14 -0700537 if symbol.file_name == fail_file:
Mike Frysinger3dcacee2019-08-23 17:09:11 -0400538 raise IOError('network failure')
Don Garrettdeb2e032016-07-06 16:44:14 -0700539
Luigi Semenzato5104f3c2016-10-12 12:37:42 -0700540 upload_mock = self.PatchObject(upload_symbols, 'UploadSymbolFile',
541 side_effect=failSome)
542 upload_mock.__name__ = 'UploadSymbolFileMock2'
Don Garrettdeb2e032016-07-06 16:44:14 -0700543
544 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600545 symbols, 'fake_url', api_key='testkey')
Don Garrettdeb2e032016-07-06 16:44:14 -0700546
547 self.assertEqual(list(result), symbols)
548
549 passed = sum(s.status == upload_symbols.SymbolFile.UPLOADED
550 for s in symbols)
551 failed = sum(s.status == upload_symbols.SymbolFile.ERROR
552 for s in symbols)
553 skipped = sum(s.status == upload_symbols.SymbolFile.INITIAL
554 for s in symbols)
555
556 # Shows we all pass.sym files worked until limit hit.
557 self.assertEqual(passed, upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY)
558
559 # Shows we all fail.sym files failed until limit hit.
560 self.assertEqual(failed, upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY)
561
562 # Shows both pass/fail were skipped after limit hit.
563 self.assertEqual(skipped, 10 * 2)
Don Garretta28be6d2016-06-16 18:09:35 -0700564
565
Chris McDonalde705d8e2020-04-17 09:29:08 -0600566@pytest.mark.usefixtures('singleton_manager')
Don Garretta28be6d2016-06-16 18:09:35 -0700567class UploadSymbolsTest(SymbolsTestBase):
568 """Test UploadSymbols, along with most helper methods."""
569 def setUp(self):
570 # Results gathering.
571 self.failure_file = os.path.join(self.tempdir, 'failures.txt')
572
573 def testUploadSymbolsEmpty(self):
574 """Upload dir is empty."""
Mike Nichols137e82d2019-05-15 18:40:34 -0600575 result = upload_symbols.UploadSymbols([self.data], 'fake_url')
Don Garretta28be6d2016-06-16 18:09:35 -0700576
Mike Frysinger2d589a12019-08-25 14:15:12 -0400577 self.assertEqual(result, 0)
Don Garretta28be6d2016-06-16 18:09:35 -0700578 self.assertEqual(self.urlopen_mock.call_count, 0)
579
580 def testUploadSymbols(self):
581 """Upload a few files."""
582 self.createSymbolFile('slim.sym', self.SLIM_CONTENT)
583 self.createSymbolFile(os.path.join('nested', 'inner.sym'))
584 self.createSymbolFile('fat.sym', self.FAT_CONTENT)
585
586 result = upload_symbols.UploadSymbols(
Mike Nichols90f7c152019-04-09 15:14:08 -0600587 [self.data], 'fake_url',
588 failed_list=self.failure_file, strip_cfi=len(self.SLIM_CONTENT)+1,
589 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700590
Mike Frysinger2d589a12019-08-25 14:15:12 -0400591 self.assertEqual(result, 0)
Mike Nichols137e82d2019-05-15 18:40:34 -0600592 self.assertEqual(self.request_mock.call_count, 10)
Mike Frysinger2d589a12019-08-25 14:15:12 -0400593 self.assertEqual(osutils.ReadFile(self.failure_file), '')
Don Garretta28be6d2016-06-16 18:09:35 -0700594
595 def testUploadSymbolsLimited(self):
596 """Upload a few files."""
597 self.createSymbolFile('slim.sym', self.SLIM_CONTENT)
598 self.createSymbolFile(os.path.join('nested', 'inner.sym'))
599 self.createSymbolFile('fat.sym', self.FAT_CONTENT)
600
601 result = upload_symbols.UploadSymbols(
Mike Nichols90f7c152019-04-09 15:14:08 -0600602 [self.data], 'fake_url', upload_limit=2,
603 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700604
Mike Frysinger2d589a12019-08-25 14:15:12 -0400605 self.assertEqual(result, 0)
Mike Nichols137e82d2019-05-15 18:40:34 -0600606 self.assertEqual(self.request_mock.call_count, 7)
Mike Frysingerf2fa7d62017-12-14 18:33:59 -0500607 self.assertNotExists(self.failure_file)
Don Garretta28be6d2016-06-16 18:09:35 -0700608
609 def testUploadSymbolsFailures(self):
610 """Upload a few files."""
611 self.createSymbolFile('pass.sym')
612 fail = self.createSymbolFile('fail.sym')
613
Mike Nichols90f7c152019-04-09 15:14:08 -0600614 def failSome(_url, symbol, _api_key):
Don Garretta28be6d2016-06-16 18:09:35 -0700615 if symbol.file_name == fail.file_name:
Mike Frysinger3dcacee2019-08-23 17:09:11 -0400616 raise IOError('network failure')
Don Garretta28be6d2016-06-16 18:09:35 -0700617
618 # Mock out UploadSymbolFile so it's easy to see which file to fail for.
619 upload_mock = self.PatchObject(upload_symbols, 'UploadSymbolFile',
620 side_effect=failSome)
Luigi Semenzato5104f3c2016-10-12 12:37:42 -0700621 # Mock __name__ for logging.
622 upload_mock.__name__ = 'UploadSymbolFileMock'
Don Garretta28be6d2016-06-16 18:09:35 -0700623
624 result = upload_symbols.UploadSymbols(
Mike Nichols90f7c152019-04-09 15:14:08 -0600625 [self.data], 'fake_url',
626 failed_list=self.failure_file, api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700627
Mike Frysinger2d589a12019-08-25 14:15:12 -0400628 self.assertEqual(result, 1)
Mike Nichols2a6f86f2020-08-18 15:56:07 -0600629 self.assertEqual(upload_mock.call_count, 7)
Mike Frysinger2d589a12019-08-25 14:15:12 -0400630 self.assertEqual(osutils.ReadFile(self.failure_file), 'fail.sym\n')
Don Garretta28be6d2016-06-16 18:09:35 -0700631
Don Garretta28be6d2016-06-16 18:09:35 -0700632# TODO: We removed --network integration tests.
Mike Frysinger0c0efa22014-02-09 23:32:23 -0500633
634
Mike Frysingerea838d12014-12-08 11:55:32 -0500635def main(_argv):
Mike Frysinger27e21b72018-07-12 14:20:21 -0400636 # pylint: disable=protected-access
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -0400637 # Set timeouts small so that if the unit test hangs, it won't hang for long.
638 parallel._BackgroundTask.STARTUP_TIMEOUT = 5
639 parallel._BackgroundTask.EXIT_TIMEOUT = 5
640
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -0400641 # Run the tests.
Mike Frysingerba167372015-01-21 10:37:03 -0500642 cros_test_lib.main(level='info', module=__name__)