Add --output option to Swarming client and pass it through to the bot

appengine/swarming/server/task_request_test.py

 #!/usr/bin/env python
 # Copyright 2014 The LUCI Authors. All rights reserved.
 # Use of this source code is governed under the Apache License, Version 2.0
 # that can be found in the LICENSE file.
 
 import datetime
 import logging
 import random
 import sys
 import unittest
@@ skipping 237 matching lines @@
       'extra_args': [],
       'execution_timeout_secs': 30,
       'grace_period_secs': 30,
       'idempotent': True,
       'inputs_ref': {
         'isolated': None,
         'isolatedserver': 'https://isolateserver.appspot.com',
         'namespace': 'default-gzip',
       },
       'io_timeout_secs': None,
+      'outputs': [],
     }
     expected_request = {
       'authenticated': auth_testing.DEFAULT_MOCKED_IDENTITY,
       'name': u'Request name',
       'parent_task_id': unicode(parent_id),
       'priority': 49,
       'properties': expected_properties,
       # Intentionally hard code the hash value since it has to be deterministic.
       # Other unit tests should use the calculated value.
-      'properties_hash': '2202337f592f7e31b407e38832c35e23f306c6c8',
+      'properties_hash': 'c6e0db8e858dd5b1bdae0e8eae7486f4a8a82967',
       'pubsub_topic': None,
       'pubsub_userdata': None,
       'service_account': u'none',
       'tags': [
         u'OS:Windows-3.1.1',
         u'hostname:localhost',
         u'pool:default',
         u'priority:49',
         u'service_account:none',
         u'tag:1',
@@ skipping 45 matching lines @@
       'extra_args': [],
       'execution_timeout_secs': 30,
       'grace_period_secs': 30,
       'idempotent': True,
       'inputs_ref': {
         'isolated': None,
         'isolatedserver': 'https://isolateserver.appspot.com',
         'namespace': 'default-gzip',
       },
       'io_timeout_secs': None,
+      'outputs': [],
     }
     expected_request = {
       'authenticated': auth_testing.DEFAULT_MOCKED_IDENTITY,
       'name': u'Request name',
       'parent_task_id': unicode(parent_id),
       'priority': 49,
       'properties': expected_properties,
       # Intentionally hard code the hash value since it has to be deterministic.
       # Other unit tests should use the calculated value.
-      'properties_hash': '2202337f592f7e31b407e38832c35e23f306c6c8',
+      'properties_hash': 'c6e0db8e858dd5b1bdae0e8eae7486f4a8a82967',
       'pubsub_topic': None,
       'pubsub_userdata': None,
       'service_account': u'none',
       'tags': [
         u'OS:Windows-3.1.1',
         u'hostname:localhost',
         u'pool:default',
         u'priority:49',
         u'service_account:none',
         u'tag:1',
@@ skipping 21 matching lines @@
       _gen_request(parent_task_id='1d69b9f088008810')
 
   def test_init_new_request_idempotent(self):
     request = mkreq(_gen_request(properties=dict(idempotent=True)))
     as_dict = request.to_dict()
     self.assertEqual(True, as_dict['properties']['idempotent'])
     # Intentionally hard code the hash value since it has to be deterministic.
     # Other unit tests should use the calculated value.
     # Ensure the algorithm is deterministic.
     self.assertEqual(
-        '2202337f592f7e31b407e38832c35e23f306c6c8', as_dict['properties_hash'])
+        'c6e0db8e858dd5b1bdae0e8eae7486f4a8a82967', as_dict['properties_hash'])
 
   def test_init_new_request_bot_service_account(self):
     request = mkreq(_gen_request(service_account_token='bot'))
     as_dict = request.to_dict()
     self.assertEqual('bot', as_dict['service_account'])
     self.assertIn(u'service_account:bot', as_dict['tags'])
 
   def test_duped(self):
     # Two TestRequest with the same properties.
     request_1 = mkreq(_gen_request(properties=dict(idempotent=True)))
@@ skipping 247 matching lines @@
       'execution_timeout_secs': 30,
       'extra_args': [],
       'grace_period_secs': 30,
       'idempotent': False,
       'inputs_ref': {
         'isolated': None,
         'isolatedserver': 'https://isolateserver.appspot.com',
         'namespace': 'default-gzip',
       },
       'io_timeout_secs': None,
+      'outputs': [],
     }
     # Differences from new_request() are:
     # - parent_task_id was reset to None.
     # - tag 'user:' was replaced
     # - user was replaced.
     expected_request = {
       'authenticated': auth_testing.DEFAULT_MOCKED_IDENTITY,
       'name': u'Request name (Retry #1)',
       'parent_task_id': None,
       'priority': 49,
@@ skipping 56 matching lines @@
         ndb.Key(task_request.TaskRequest, 0x7f14acec2fcfffff),
         task_request.request_id_to_key(0xeb5313d0300000))
 
 
 if __name__ == '__main__':
   if '-v' in sys.argv:
     unittest.TestCase.maxDiff = None
   logging.basicConfig(
       level=logging.DEBUG if '-v' in sys.argv else logging.ERROR)
   unittest.main()