Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(36)

Side by Side Diff: appengine/swarming/handlers_bot_test.py

Issue 2832243002: swarming: mechanical changes to unit tests (Closed)
Patch Set: Disable tx before calling tq Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # coding: utf-8 2 # coding: utf-8
3 # Copyright 2015 The LUCI Authors. All rights reserved. 3 # Copyright 2015 The LUCI Authors. All rights reserved.
4 # Use of this source code is governed under the Apache License, Version 2.0 4 # Use of this source code is governed under the Apache License, Version 2.0
5 # that can be found in the LICENSE file. 5 # that can be found in the LICENSE file.
6 6
7 import base64 7 import base64
8 import datetime 8 import datetime
9 import logging 9 import logging
10 import os 10 import os
11 import random 11 import random
12 import StringIO 12 import StringIO
13 import sys 13 import sys
14 import unittest 14 import unittest
15 import zipfile 15 import zipfile
16 16
17 # Setups environment. 17 # Setups environment.
18 import test_env_handlers 18 import test_env_handlers
19 19
20 from google.appengine.api import datastore_errors 20 from google.appengine.api import datastore_errors
21 from google.appengine.ext import ndb 21 from google.appengine.ext import ndb
22 22
23 import webapp2 23 import webapp2
24 import webtest 24 import webtest
25 25
26 import handlers_backend
26 import handlers_bot 27 import handlers_bot
27 from components import ereporter2 28 from components import ereporter2
28 from components import utils 29 from components import utils
29 from server import bot_archive 30 from server import bot_archive
30 from server import bot_auth 31 from server import bot_auth
31 from server import bot_code 32 from server import bot_code
32 from server import bot_groups_config 33 from server import bot_groups_config
33 from server import bot_management 34 from server import bot_management
34 from server import stats 35 from server import stats
35 36
(...skipping 11 matching lines...) Expand all
47 app, 48 app,
48 extra_environ={ 49 extra_environ={
49 'REMOTE_ADDR': self.source_ip, 50 'REMOTE_ADDR': self.source_ip,
50 'SERVER_SOFTWARE': os.environ['SERVER_SOFTWARE'], 51 'SERVER_SOFTWARE': os.environ['SERVER_SOFTWARE'],
51 }) 52 })
52 self.mock( 53 self.mock(
53 ereporter2, 'log_request', 54 ereporter2, 'log_request',
54 lambda *args, **kwargs: self.fail('%s, %s' % (args, kwargs))) 55 lambda *args, **kwargs: self.fail('%s, %s' % (args, kwargs)))
55 # Bot API test cases run by default as bot. 56 # Bot API test cases run by default as bot.
56 self.set_as_bot() 57 self.set_as_bot()
58 self._enqueue_task_orig = self.mock(
59 utils, 'enqueue_task', self._enqueue_task)
60
61 @ndb.non_transactional
62 def _enqueue_task(self, url, queue_name, **kwargs):
63 if queue_name == 'task-dimensions':
64 # Call directly into it, ignores any current transaction.
65 handlers_backend.TaskDimensionsHandler.tidy_stale(kwargs['payload'])
66 return True
67 if queue_name == 'pubsub':
68 return True
69 self.fail(url)
57 70
58 def test_handshake(self): 71 def test_handshake(self):
59 errors = [] 72 errors = []
60 def add_error(request, source, message): 73 def add_error(request, source, message):
61 self.assertTrue(request) 74 self.assertTrue(request)
62 self.assertEqual('bot', source) 75 self.assertEqual('bot', source)
63 errors.append(message) 76 errors.append(message)
64 self.mock(ereporter2, 'log_request', add_error) 77 self.mock(ereporter2, 'log_request', add_error)
65 params = { 78 params = {
66 'dimensions': { 79 'dimensions': {
(...skipping 1070 matching lines...) Expand 10 before | Expand all | Expand 10 after
1137 self.app.get('/bot_code?tok=%s' % tok, status=200) 1150 self.app.get('/bot_code?tok=%s' % tok, status=200)
1138 1151
1139 1152
1140 if __name__ == '__main__': 1153 if __name__ == '__main__':
1141 if '-v' in sys.argv: 1154 if '-v' in sys.argv:
1142 unittest.TestCase.maxDiff = None 1155 unittest.TestCase.maxDiff = None
1143 logging.basicConfig( 1156 logging.basicConfig(
1144 level=logging.DEBUG if '-v' in sys.argv else logging.CRITICAL, 1157 level=logging.DEBUG if '-v' in sys.argv else logging.CRITICAL,
1145 format='%(levelname)-7s %(filename)s:%(lineno)3d %(message)s') 1158 format='%(levelname)-7s %(filename)s:%(lineno)3d %(message)s')
1146 unittest.main() 1159 unittest.main()
OLDNEW
« no previous file with comments | « appengine/components/test_support/test_case.py ('k') | appengine/swarming/handlers_endpoints_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698