blob: ee3bb0e52858c6923d8e8ece4f1860884413d349 [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 Frysinger0a2fd922014-09-12 20:23:42 -070010import BaseHTTPServer
Mike Frysinger06da5202014-09-26 17:30:33 -050011import errno
Don Garretta28be6d2016-06-16 18:09:35 -070012import itertools
Mike Frysingerea838d12014-12-08 11:55:32 -050013import mock
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040014import os
Mike Frysinger0a2fd922014-09-12 20:23:42 -070015import signal
Mike Frysinger06da5202014-09-26 17:30:33 -050016import socket
Mike Frysinger0a2fd922014-09-12 20:23:42 -070017import SocketServer
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040018import sys
Aviv Keshetd1f04632014-05-09 11:33:46 -070019import time
Mike Frysinger094a2172013-08-14 12:54:35 -040020import urllib2
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040021
Mike Frysinger079863c2014-10-09 23:16:46 -040022# 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 :(.
27os.environ.pop('http_proxy', None)
28
Aviv Keshetb7519e12016-10-04 00:50:00 -070029from chromite.lib import constants
Mike Frysingerbbd1f112016-09-08 18:25:11 -040030
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.
38third_party = os.path.join(constants.CHROMITE_DIR, 'third_party')
39while True:
40 try:
41 sys.path.remove(third_party)
42 except ValueError:
43 break
Mike Frysingerbbd1f112016-09-08 18:25:11 -040044del third_party
45
Ralph Nathan446aee92015-03-23 14:44:56 -070046from chromite.lib import cros_logging as logging
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040047from chromite.lib import cros_test_lib
48from chromite.lib import osutils
49from chromite.lib import parallel
Mike Frysinger0a2fd922014-09-12 20:23:42 -070050from chromite.lib import remote_access
Mike Frysinger5e30a4b2014-02-12 20:23:04 -050051from chromite.scripts import cros_generate_breakpad_symbols
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -040052from chromite.scripts import upload_symbols
53
Mike Frysinger0c0efa22014-02-09 23:32:23 -050054
Don Garretta28be6d2016-06-16 18:09:35 -070055class SymbolsTestBase(cros_test_lib.MockTempDirTestCase):
56 """Base class for most symbols tests."""
57
58 SLIM_CONTENT = """
59some junk
60"""
61
62 FAT_CONTENT = """
63STACK CFI 1234
64some junk
65STACK CFI 1234
66"""
67
68 def setUp(self):
69 # Make certain we don't use the network.
Don Garretta28be6d2016-06-16 18:09:35 -070070 self.urlopen_mock = self.PatchObject(urllib2, 'urlopen')
Mike Nichols90f7c152019-04-09 15:14:08 -060071 self.request_mock = self.PatchObject(upload_symbols, 'ExecRequest',
72 return_value={'uploadUrl':
73 'testurl',
74 'uploadKey':
75 'asdgasgas'})
Don Garretta28be6d2016-06-16 18:09:35 -070076
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 Frysingerd5fcb3a2013-05-30 21:10:50 -0400116
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700117class 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
138class SymbolServer(SocketServer.ThreadingTCPServer, BaseHTTPServer.HTTPServer):
139 """Simple HTTP server that forks each request"""
140
141
142class UploadSymbolsServerTest(cros_test_lib.MockTempDirTestCase):
143 """Tests for UploadSymbols() and a local HTTP server"""
144
145 SYM_CONTENTS = """MODULE Linux arm 123-456 blkid
146PUBLIC 1471 0 main"""
147
148 def SpawnServer(self, RequestHandler):
149 """Spawn a new http server"""
Mike Frysinger06da5202014-09-26 17:30:33 -0500150 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 Garretta28be6d2016-06-16 18:09:35 -0700160 self.server_url = 'http://localhost:%i/post/path' % port
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700161 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 Garretta28be6d2016-06-16 18:09:35 -0700169 self.server_url = None
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700170 self.sym_file = os.path.join(self.tempdir, 'test.sym')
171 osutils.WriteFile(self.sym_file, self.SYM_CONTENTS)
172
Don Garretta28be6d2016-06-16 18:09:35 -0700173 # Stop sleeps and retries for these tests.
174 self.PatchObject(upload_symbols, 'SLEEP_DELAY', 0)
175 self.PatchObject(upload_symbols, 'INITIAL_RETRY_DELAY', 0)
176 self.PatchObject(upload_symbols, 'MAX_RETRIES', 0)
177
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700178 def tearDown(self):
179 # Only kill the server if we forked one.
180 if self.httpd_pid:
181 os.kill(self.httpd_pid, signal.SIGUSR1)
182
183 def testSuccess(self):
184 """The server returns success for all uploads"""
185 class Handler(SymbolServerRequestHandler):
186 """Always return 200"""
187 RESP_CODE = 200
Mike Nichols90f7c152019-04-09 15:14:08 -0600188 self.PatchObject(upload_symbols, 'ExecRequest',
189 return_value={'uploadUrl': 'testurl',
190 'uploadKey': 'testSuccess'})
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700191 self.SpawnServer(Handler)
Don Garretta28be6d2016-06-16 18:09:35 -0700192 ret = upload_symbols.UploadSymbols(
193 sym_paths=[self.sym_file] * 10,
194 upload_url=self.server_url,
Mike Nichols90f7c152019-04-09 15:14:08 -0600195 api_key='testSuccess')
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700196 self.assertEqual(ret, 0)
197
198 def testError(self):
199 """The server returns errors for all uploads"""
200 class Handler(SymbolServerRequestHandler):
Mike Nichols90f7c152019-04-09 15:14:08 -0600201 """All connections error"""
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700202 RESP_CODE = 500
203 RESP_MSG = 'Internal Server Error'
204
205 self.SpawnServer(Handler)
Don Garretta28be6d2016-06-16 18:09:35 -0700206 ret = upload_symbols.UploadSymbols(
207 sym_paths=[self.sym_file] * 10,
208 upload_url=self.server_url,
Mike Nichols90f7c152019-04-09 15:14:08 -0600209 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700210 self.assertEqual(ret, 10)
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700211
212 def testHungServer(self):
213 """The server chokes, but we recover"""
214 class Handler(SymbolServerRequestHandler):
215 """All connections choke forever"""
Mike Nichols137e82d2019-05-15 18:40:34 -0600216 self.PatchObject(upload_symbols, 'ExecRequest',
217 return_value={'pairs': []})
218
Mike Frysinger0a2fd922014-09-12 20:23:42 -0700219 def do_POST(self):
220 while True:
221 time.sleep(1000)
222
223 self.SpawnServer(Handler)
224 with mock.patch.object(upload_symbols, 'GetUploadTimeout') as m:
Don Garretta28be6d2016-06-16 18:09:35 -0700225 m.return_value = 0.01
Mike Frysinger58312e92014-03-18 04:18:36 -0400226 ret = upload_symbols.UploadSymbols(
Don Garretta28be6d2016-06-16 18:09:35 -0700227 sym_paths=[self.sym_file] * 10,
228 upload_url=self.server_url,
Mike Nichols137e82d2019-05-15 18:40:34 -0600229 timeout=m.return_value,
Mike Nichols90f7c152019-04-09 15:14:08 -0600230 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700231 self.assertEqual(ret, 10)
Aviv Keshetd1f04632014-05-09 11:33:46 -0700232
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -0400233
Don Garretta28be6d2016-06-16 18:09:35 -0700234class UploadSymbolsHelpersTest(cros_test_lib.TestCase):
235 """Test assorted helper functions and classes."""
Don Garretta28be6d2016-06-16 18:09:35 -0700236 def testIsTarball(self):
237 notTar = [
238 '/foo/bar/test.bin',
239 '/foo/bar/test.tar.bin',
240 '/foo/bar/test.faketar.gz',
241 '/foo/bar/test.nottgz',
Mike Frysinger0c0efa22014-02-09 23:32:23 -0500242 ]
Mike Frysinger0c0efa22014-02-09 23:32:23 -0500243
Don Garretta28be6d2016-06-16 18:09:35 -0700244 isTar = [
245 '/foo/bar/test.tar',
246 '/foo/bar/test.bin.tar',
247 '/foo/bar/test.bin.tar.bz2',
248 '/foo/bar/test.bin.tar.gz',
249 '/foo/bar/test.bin.tar.xz',
250 '/foo/bar/test.tbz2',
251 '/foo/bar/test.tbz',
252 '/foo/bar/test.tgz',
253 '/foo/bar/test.txz',
254 ]
Mike Frysinger0c0efa22014-02-09 23:32:23 -0500255
Don Garretta28be6d2016-06-16 18:09:35 -0700256 for p in notTar:
257 self.assertFalse(upload_symbols.IsTarball(p))
258
259 for p in isTar:
260 self.assertTrue(upload_symbols.IsTarball(p))
261
262 def testBatchGenerator(self):
263 result = upload_symbols.BatchGenerator([], 2)
264 self.assertEqual(list(result), [])
265
Mike Frysinger79cca962019-06-13 15:26:53 -0400266 result = upload_symbols.BatchGenerator(range(6), 2)
Don Garretta28be6d2016-06-16 18:09:35 -0700267 self.assertEqual(list(result), [[0, 1], [2, 3], [4, 5]])
268
Mike Frysinger79cca962019-06-13 15:26:53 -0400269 result = upload_symbols.BatchGenerator(range(7), 2)
Don Garretta28be6d2016-06-16 18:09:35 -0700270 self.assertEqual(list(result), [[0, 1], [2, 3], [4, 5], [6]])
271
272 # Prove that we are streaming the results, not generating them all at once.
273 result = upload_symbols.BatchGenerator(itertools.repeat(0), 2)
274 self.assertEqual(result.next(), [0, 0])
275
276
277class FindSymbolFilesTest(SymbolsTestBase):
278 """Test FindSymbolFiles."""
279 def setUp(self):
280 self.symfile = self.createSymbolFile('root.sym').file_name
281 self.innerfile = self.createSymbolFile(
282 os.path.join('nested', 'inner.sym')).file_name
283
284 # CreateTarball is having issues outside the chroot from open file tests.
285 #
286 # self.tarball = os.path.join(self.tempdir, 'syms.tar.gz')
287 # cros_build_lib.CreateTarball(
288 # 'syms.tar.gz', self.tempdir, inputs=(self.data))
289
290 def testEmpty(self):
291 symbols = list(upload_symbols.FindSymbolFiles(
292 self.working, []))
293 self.assertEqual(symbols, [])
294
295 def testFile(self):
296 symbols = list(upload_symbols.FindSymbolFiles(
297 self.working, [self.symfile]))
298
299 self.assertEqual(len(symbols), 1)
300 sf = symbols[0]
301
302 self.assertEqual(sf.display_name, 'root.sym')
303 self.assertEqual(sf.display_path, self.symfile)
304 self.assertEqual(sf.file_name, self.symfile)
305 self.assertEqual(sf.status, upload_symbols.SymbolFile.INITIAL)
306 self.assertEqual(sf.FileSize(), len(self.FAT_CONTENT))
307
308 def testDir(self):
309 symbols = list(upload_symbols.FindSymbolFiles(
310 self.working, [self.data]))
311
312 self.assertEqual(len(symbols), 2)
313 root = symbols[0]
314 nested = symbols[1]
315
316 self.assertEqual(root.display_name, 'root.sym')
317 self.assertEqual(root.display_path, 'root.sym')
318 self.assertEqual(root.file_name, self.symfile)
319 self.assertEqual(root.status, upload_symbols.SymbolFile.INITIAL)
320 self.assertEqual(root.FileSize(), len(self.FAT_CONTENT))
321
322 self.assertEqual(nested.display_name, 'inner.sym')
323 self.assertEqual(nested.display_path, 'nested/inner.sym')
324 self.assertEqual(nested.file_name, self.innerfile)
325 self.assertEqual(nested.status, upload_symbols.SymbolFile.INITIAL)
326 self.assertEqual(nested.FileSize(), len(self.FAT_CONTENT))
327
328
329class AdjustSymbolFileSizeTest(SymbolsTestBase):
330 """Test AdjustSymbolFileSize."""
331 def setUp(self):
332 self.slim = self.createSymbolFile('slim.sym', self.SLIM_CONTENT)
333 self.fat = self.createSymbolFile('fat.sym', self.FAT_CONTENT)
334
335 self.warn_mock = self.PatchObject(logging, 'PrintBuildbotStepWarnings')
336
337 def _testNotStripped(self, symbol, size=None, content=None):
338 start_file = symbol.file_name
339 after = upload_symbols.AdjustSymbolFileSize(
340 symbol, self.working, size)
341 self.assertIs(after, symbol)
342 self.assertEqual(after.file_name, start_file)
343 if content is not None:
344 self.assertEqual(osutils.ReadFile(after.file_name), content)
345
346 def _testStripped(self, symbol, size=None, content=None):
347 after = upload_symbols.AdjustSymbolFileSize(
348 symbol, self.working, size)
349 self.assertIs(after, symbol)
350 self.assertTrue(after.file_name.startswith(self.working))
351 if content is not None:
352 self.assertEqual(osutils.ReadFile(after.file_name), content)
353
354 def testSmall(self):
355 """Ensure that files smaller than the limit are not modified."""
356 self._testNotStripped(self.slim, 1024, self.SLIM_CONTENT)
357 self._testNotStripped(self.fat, 1024, self.FAT_CONTENT)
358
359 def testLarge(self):
360 """Ensure that files larger than the limit are modified."""
361 self._testStripped(self.slim, 1, self.SLIM_CONTENT)
362 self._testStripped(self.fat, 1, self.SLIM_CONTENT)
363
364 def testMixed(self):
365 """Test mix of large and small."""
366 strip_size = len(self.SLIM_CONTENT) + 1
367
368 self._testNotStripped(self.slim, strip_size, self.SLIM_CONTENT)
369 self._testStripped(self.fat, strip_size, self.SLIM_CONTENT)
370
371 def testSizeWarnings(self):
372 large = self.createSymbolFile(
373 'large.sym', content=self.SLIM_CONTENT,
374 size=upload_symbols.CRASH_SERVER_FILE_LIMIT*2)
375
376 # Would like to Strip as part of this test, but that really copies all
377 # of the sparse file content, which is too expensive for a unittest.
378 self._testNotStripped(large, None, None)
379
380 self.assertEqual(self.warn_mock.call_count, 1)
381
382
Mike Nichols137e82d2019-05-15 18:40:34 -0600383class DeduplicateTest(SymbolsTestBase):
384 """Test server Deduplication."""
385 def setUp(self):
386 self.PatchObject(upload_symbols, 'ExecRequest',
387 return_value={'pairs': [
388 {'status': 'FOUND',
389 'symbolId':
390 {'debugFile': 'sym1_sym',
391 'debugId': 'BEAA9BE'}},
392 {'status': 'FOUND',
393 'symbolId':
394 {'debugFile': 'sym2_sym',
395 'debugId': 'B6B1A36'}},
396 {'status': 'MISSING',
397 'symbolId':
398 {'debugFile': 'sym3_sym',
399 'debugId': 'D4FC0FC'}}]})
400
401 def testFindDuplicates(self):
402 # The first two symbols will be duplicate, the third new.
403 sym1 = self.createSymbolFile('sym1.sym')
404 sym1.header = cros_generate_breakpad_symbols.SymbolHeader('cpu', 'BEAA9BE',
405 'sym1_sym', 'os')
406 sym2 = self.createSymbolFile('sym2.sym')
407 sym2.header = cros_generate_breakpad_symbols.SymbolHeader('cpu', 'B6B1A36',
408 'sym2_sym', 'os')
409 sym3 = self.createSymbolFile('sym3.sym')
410 sym3.header = cros_generate_breakpad_symbols.SymbolHeader('cpu', 'D4FC0FC',
411 'sym3_sym', 'os')
412
413 result = upload_symbols.FindDuplicates((sym1, sym2, sym3), 'fake_url',
414 api_key='testkey')
415 self.assertEqual(list(result), [sym1, sym2, sym3])
416
417 self.assertEqual(sym1.status, upload_symbols.SymbolFile.DUPLICATE)
418 self.assertEqual(sym2.status, upload_symbols.SymbolFile.DUPLICATE)
419 self.assertEqual(sym3.status, upload_symbols.SymbolFile.INITIAL)
420
421
Don Garrettdeb2e032016-07-06 16:44:14 -0700422class PerformSymbolFilesUploadTest(SymbolsTestBase):
Don Garretta28be6d2016-06-16 18:09:35 -0700423 """Test PerformSymbolFile, and it's helper methods."""
424 def setUp(self):
Don Garrettdeb2e032016-07-06 16:44:14 -0700425 self.sym_initial = self.createSymbolFile(
426 'initial.sym')
427 self.sym_error = self.createSymbolFile(
428 'error.sym', status=upload_symbols.SymbolFile.ERROR)
429 self.sym_duplicate = self.createSymbolFile(
430 'duplicate.sym', status=upload_symbols.SymbolFile.DUPLICATE)
431 self.sym_uploaded = self.createSymbolFile(
432 'uploaded.sym', status=upload_symbols.SymbolFile.UPLOADED)
Don Garretta28be6d2016-06-16 18:09:35 -0700433
434 def testGetUploadTimeout(self):
435 """Test GetUploadTimeout helper function."""
436 # Timeout for small file.
Don Garrettdeb2e032016-07-06 16:44:14 -0700437 self.assertEqual(upload_symbols.GetUploadTimeout(self.sym_initial),
Don Garretta28be6d2016-06-16 18:09:35 -0700438 upload_symbols.UPLOAD_MIN_TIMEOUT)
439
440 # Timeout for 300M file.
441 large = self.createSymbolFile('large.sym', size=(300 * 1024 * 1024))
Ryo Hashimotof864c0e2017-02-14 12:46:08 +0900442 self.assertEqual(upload_symbols.GetUploadTimeout(large), 771)
Don Garretta28be6d2016-06-16 18:09:35 -0700443
444 def testUploadSymbolFile(self):
Mike Nichols90f7c152019-04-09 15:14:08 -0600445 upload_symbols.UploadSymbolFile('fake_url', self.sym_initial,
446 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700447 # TODO: Examine mock in more detail to make sure request is correct.
Mike Nichols137e82d2019-05-15 18:40:34 -0600448 self.assertEqual(self.request_mock.call_count, 3)
Don Garretta28be6d2016-06-16 18:09:35 -0700449
Don Garrettdeb2e032016-07-06 16:44:14 -0700450 def testPerformSymbolsFileUpload(self):
Don Garretta28be6d2016-06-16 18:09:35 -0700451 """We upload on first try."""
Don Garrettdeb2e032016-07-06 16:44:14 -0700452 symbols = [self.sym_initial]
Don Garretta28be6d2016-06-16 18:09:35 -0700453
Don Garrettdeb2e032016-07-06 16:44:14 -0700454 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600455 symbols, 'fake_url', api_key='testkey')
Don Garrettdeb2e032016-07-06 16:44:14 -0700456
457 self.assertEqual(list(result), symbols)
458 self.assertEqual(self.sym_initial.status,
459 upload_symbols.SymbolFile.UPLOADED)
Mike Nichols137e82d2019-05-15 18:40:34 -0600460 self.assertEqual(self.request_mock.call_count, 3)
Don Garretta28be6d2016-06-16 18:09:35 -0700461
Don Garrettdeb2e032016-07-06 16:44:14 -0700462 def testPerformSymbolsFileUploadFailure(self):
Don Garretta28be6d2016-06-16 18:09:35 -0700463 """All network requests fail."""
Mike Nichols90f7c152019-04-09 15:14:08 -0600464 self.request_mock.side_effect = urllib2.URLError('network failure')
Don Garrettdeb2e032016-07-06 16:44:14 -0700465 symbols = [self.sym_initial]
Don Garretta28be6d2016-06-16 18:09:35 -0700466
Don Garrettdeb2e032016-07-06 16:44:14 -0700467 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600468 symbols, 'fake_url', api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700469
Don Garrettdeb2e032016-07-06 16:44:14 -0700470 self.assertEqual(list(result), symbols)
471 self.assertEqual(self.sym_initial.status, upload_symbols.SymbolFile.ERROR)
Mike Nichols90f7c152019-04-09 15:14:08 -0600472 self.assertEqual(self.request_mock.call_count, 7)
Don Garretta28be6d2016-06-16 18:09:35 -0700473
Don Garrettdeb2e032016-07-06 16:44:14 -0700474 def testPerformSymbolsFileUploadTransisentFailure(self):
Don Garretta28be6d2016-06-16 18:09:35 -0700475 """We fail once, then succeed."""
476 self.urlopen_mock.side_effect = (urllib2.URLError('network failure'), None)
Don Garrettdeb2e032016-07-06 16:44:14 -0700477 symbols = [self.sym_initial]
Don Garretta28be6d2016-06-16 18:09:35 -0700478
Don Garrettdeb2e032016-07-06 16:44:14 -0700479 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600480 symbols, 'fake_url', api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700481
Don Garrettdeb2e032016-07-06 16:44:14 -0700482 self.assertEqual(list(result), symbols)
483 self.assertEqual(self.sym_initial.status,
484 upload_symbols.SymbolFile.UPLOADED)
Mike Nichols137e82d2019-05-15 18:40:34 -0600485 self.assertEqual(self.request_mock.call_count, 3)
Don Garrettdeb2e032016-07-06 16:44:14 -0700486
487 def testPerformSymbolsFileUploadMixed(self):
488 """Upload symbols in mixed starting states.
489
490 Demonstrate that INITIAL and ERROR are uploaded, but DUPLICATE/UPLOADED are
491 ignored.
492 """
493 symbols = [self.sym_initial, self.sym_error,
494 self.sym_duplicate, self.sym_uploaded]
495
496 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600497 symbols, 'fake_url', api_key='testkey')
Don Garrettdeb2e032016-07-06 16:44:14 -0700498
499 #
500 self.assertEqual(list(result), symbols)
501 self.assertEqual(self.sym_initial.status,
502 upload_symbols.SymbolFile.UPLOADED)
503 self.assertEqual(self.sym_error.status,
504 upload_symbols.SymbolFile.UPLOADED)
505 self.assertEqual(self.sym_duplicate.status,
506 upload_symbols.SymbolFile.DUPLICATE)
507 self.assertEqual(self.sym_uploaded.status,
508 upload_symbols.SymbolFile.UPLOADED)
Mike Nichols137e82d2019-05-15 18:40:34 -0600509 self.assertEqual(self.request_mock.call_count, 6)
Don Garrettdeb2e032016-07-06 16:44:14 -0700510
511
512 def testPerformSymbolsFileUploadErrorOut(self):
513 """Demonstate we exit only after X errors."""
514
515 symbol_count = upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY + 10
516 symbols = []
517 fail_file = None
518
519 # potentially twice as many errors as we should attempt.
Mike Frysinger79cca962019-06-13 15:26:53 -0400520 for _ in range(symbol_count):
Don Garrettdeb2e032016-07-06 16:44:14 -0700521 # Each loop will get unique SymbolFile instances that use the same files.
522 fail = self.createSymbolFile('fail.sym')
523 fail_file = fail.file_name
524 symbols.append(self.createSymbolFile('pass.sym'))
525 symbols.append(fail)
526
527 # Mock out UploadSymbolFile and fail for fail.sym files.
Mike Nichols90f7c152019-04-09 15:14:08 -0600528 def failSome(_url, symbol, _api_key):
Don Garrettdeb2e032016-07-06 16:44:14 -0700529 if symbol.file_name == fail_file:
530 raise urllib2.URLError('network failure')
531
Luigi Semenzato5104f3c2016-10-12 12:37:42 -0700532 upload_mock = self.PatchObject(upload_symbols, 'UploadSymbolFile',
533 side_effect=failSome)
534 upload_mock.__name__ = 'UploadSymbolFileMock2'
Don Garrettdeb2e032016-07-06 16:44:14 -0700535
536 result = upload_symbols.PerformSymbolsFileUpload(
Mike Nichols90f7c152019-04-09 15:14:08 -0600537 symbols, 'fake_url', api_key='testkey')
Don Garrettdeb2e032016-07-06 16:44:14 -0700538
539 self.assertEqual(list(result), symbols)
540
541 passed = sum(s.status == upload_symbols.SymbolFile.UPLOADED
542 for s in symbols)
543 failed = sum(s.status == upload_symbols.SymbolFile.ERROR
544 for s in symbols)
545 skipped = sum(s.status == upload_symbols.SymbolFile.INITIAL
546 for s in symbols)
547
548 # Shows we all pass.sym files worked until limit hit.
549 self.assertEqual(passed, upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY)
550
551 # Shows we all fail.sym files failed until limit hit.
552 self.assertEqual(failed, upload_symbols.MAX_TOTAL_ERRORS_FOR_RETRY)
553
554 # Shows both pass/fail were skipped after limit hit.
555 self.assertEqual(skipped, 10 * 2)
Don Garretta28be6d2016-06-16 18:09:35 -0700556
557
558class UploadSymbolsTest(SymbolsTestBase):
559 """Test UploadSymbols, along with most helper methods."""
560 def setUp(self):
561 # Results gathering.
562 self.failure_file = os.path.join(self.tempdir, 'failures.txt')
563
564 def testUploadSymbolsEmpty(self):
565 """Upload dir is empty."""
Mike Nichols137e82d2019-05-15 18:40:34 -0600566 result = upload_symbols.UploadSymbols([self.data], 'fake_url')
Don Garretta28be6d2016-06-16 18:09:35 -0700567
568 self.assertEquals(result, 0)
569 self.assertEqual(self.urlopen_mock.call_count, 0)
570
571 def testUploadSymbols(self):
572 """Upload a few files."""
573 self.createSymbolFile('slim.sym', self.SLIM_CONTENT)
574 self.createSymbolFile(os.path.join('nested', 'inner.sym'))
575 self.createSymbolFile('fat.sym', self.FAT_CONTENT)
576
577 result = upload_symbols.UploadSymbols(
Mike Nichols90f7c152019-04-09 15:14:08 -0600578 [self.data], 'fake_url',
579 failed_list=self.failure_file, strip_cfi=len(self.SLIM_CONTENT)+1,
580 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700581
582 self.assertEquals(result, 0)
Mike Nichols137e82d2019-05-15 18:40:34 -0600583 self.assertEqual(self.request_mock.call_count, 10)
Don Garretta28be6d2016-06-16 18:09:35 -0700584 self.assertEquals(osutils.ReadFile(self.failure_file), '')
585
586 def testUploadSymbolsLimited(self):
587 """Upload a few files."""
588 self.createSymbolFile('slim.sym', self.SLIM_CONTENT)
589 self.createSymbolFile(os.path.join('nested', 'inner.sym'))
590 self.createSymbolFile('fat.sym', self.FAT_CONTENT)
591
592 result = upload_symbols.UploadSymbols(
Mike Nichols90f7c152019-04-09 15:14:08 -0600593 [self.data], 'fake_url', upload_limit=2,
594 api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700595
596 self.assertEquals(result, 0)
Mike Nichols137e82d2019-05-15 18:40:34 -0600597 self.assertEqual(self.request_mock.call_count, 7)
Mike Frysingerf2fa7d62017-12-14 18:33:59 -0500598 self.assertNotExists(self.failure_file)
Don Garretta28be6d2016-06-16 18:09:35 -0700599
600 def testUploadSymbolsFailures(self):
601 """Upload a few files."""
602 self.createSymbolFile('pass.sym')
603 fail = self.createSymbolFile('fail.sym')
604
Mike Nichols90f7c152019-04-09 15:14:08 -0600605 def failSome(_url, symbol, _api_key):
Don Garretta28be6d2016-06-16 18:09:35 -0700606 if symbol.file_name == fail.file_name:
607 raise urllib2.URLError('network failure')
608
609 # Mock out UploadSymbolFile so it's easy to see which file to fail for.
610 upload_mock = self.PatchObject(upload_symbols, 'UploadSymbolFile',
611 side_effect=failSome)
Luigi Semenzato5104f3c2016-10-12 12:37:42 -0700612 # Mock __name__ for logging.
613 upload_mock.__name__ = 'UploadSymbolFileMock'
Don Garretta28be6d2016-06-16 18:09:35 -0700614
615 result = upload_symbols.UploadSymbols(
Mike Nichols90f7c152019-04-09 15:14:08 -0600616 [self.data], 'fake_url',
617 failed_list=self.failure_file, api_key='testkey')
Don Garretta28be6d2016-06-16 18:09:35 -0700618
619 self.assertEquals(result, 1)
620 self.assertEqual(upload_mock.call_count, 8)
621 self.assertEquals(osutils.ReadFile(self.failure_file), 'fail.sym\n')
622
Don Garretta28be6d2016-06-16 18:09:35 -0700623# TODO: We removed --network integration tests.
Mike Frysinger0c0efa22014-02-09 23:32:23 -0500624
625
Mike Frysingerea838d12014-12-08 11:55:32 -0500626def main(_argv):
Mike Frysinger27e21b72018-07-12 14:20:21 -0400627 # pylint: disable=protected-access
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -0400628 # Set timeouts small so that if the unit test hangs, it won't hang for long.
629 parallel._BackgroundTask.STARTUP_TIMEOUT = 5
630 parallel._BackgroundTask.EXIT_TIMEOUT = 5
631
Mike Frysingerd5fcb3a2013-05-30 21:10:50 -0400632 # Run the tests.
Mike Frysingerba167372015-01-21 10:37:03 -0500633 cros_test_lib.main(level='info', module=__name__)