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

Side by Side Diff: appengine/swarming/server/task_request_test.py

Issue 1906803003: Allow dots in dimension key name. (Closed) Base URL: git@github.com:luci/luci-py.git@master
Patch Set: Created 4 years, 8 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
« no previous file with comments | « appengine/swarming/server/task_request.py ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2014 The LUCI Authors. All rights reserved. 2 # Copyright 2014 The LUCI Authors. All rights reserved.
3 # Use of this source code is governed by the Apache v2.0 license that can be 3 # Use of this source code is governed by the Apache v2.0 license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 import datetime 6 import datetime
7 import logging 7 import logging
8 import os 8 import os
9 import random 9 import random
10 import sys 10 import sys
(...skipping 356 matching lines...) Expand 10 before | Expand all | Expand 10 after
367 task_request.make_request( 367 task_request.make_request(
368 _gen_request(properties=dict(dimensions=[])), True) 368 _gen_request(properties=dict(dimensions=[])), True)
369 with self.assertRaises(datastore_errors.BadValueError): 369 with self.assertRaises(datastore_errors.BadValueError):
370 task_request.make_request( 370 task_request.make_request(
371 _gen_request(properties=dict(dimensions={})), True) 371 _gen_request(properties=dict(dimensions={})), True)
372 with self.assertRaises(datastore_errors.BadValueError): 372 with self.assertRaises(datastore_errors.BadValueError):
373 task_request.make_request( 373 task_request.make_request(
374 _gen_request( 374 _gen_request(
375 properties=dict(dimensions={u'id': u'b', u'a:': u'b'})), True) 375 properties=dict(dimensions={u'id': u'b', u'a:': u'b'})), True)
376 task_request.make_request( 376 task_request.make_request(
377 _gen_request(properties=dict(dimensions={u'id': u'b'})), True) 377 _gen_request(
378 properties=dict(dimensions={u'id': u'b', u'a.': u'b'})), True)
378 379
379 with self.assertRaises(TypeError): 380 with self.assertRaises(TypeError):
380 task_request.make_request( 381 task_request.make_request(
381 _gen_request(properties=dict(env=[])), True) 382 _gen_request(properties=dict(env=[])), True)
382 with self.assertRaises(TypeError): 383 with self.assertRaises(TypeError):
383 task_request.make_request( 384 task_request.make_request(
384 _gen_request(properties=dict(env={u'a': 1})), True) 385 _gen_request(properties=dict(env={u'a': 1})), True)
385 task_request.make_request(_gen_request(properties=dict(env={})), True) 386 task_request.make_request(_gen_request(properties=dict(env={})), True)
386 387
387 with self.assertRaises(datastore_errors.BadValueError): 388 with self.assertRaises(datastore_errors.BadValueError):
(...skipping 146 matching lines...) Expand 10 before | Expand all | Expand 10 after
534 ndb.Key(task_request.TaskRequest, 0x7f14acec2fcfffff), 535 ndb.Key(task_request.TaskRequest, 0x7f14acec2fcfffff),
535 task_request.request_id_to_key(0xeb5313d0300000)) 536 task_request.request_id_to_key(0xeb5313d0300000))
536 537
537 538
538 if __name__ == '__main__': 539 if __name__ == '__main__':
539 if '-v' in sys.argv: 540 if '-v' in sys.argv:
540 unittest.TestCase.maxDiff = None 541 unittest.TestCase.maxDiff = None
541 logging.basicConfig( 542 logging.basicConfig(
542 level=logging.DEBUG if '-v' in sys.argv else logging.ERROR) 543 level=logging.DEBUG if '-v' in sys.argv else logging.ERROR)
543 unittest.main() 544 unittest.main()
OLDNEW
« no previous file with comments | « appengine/swarming/server/task_request.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698