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

Unified Diff: appengine/swarming/server/task_request_test.py

Issue 2453873002: Add --output option to Swarming client and pass it through to the bot (Closed)
Patch Set: Extra help text for examples Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « appengine/swarming/server/task_request.py ('k') | appengine/swarming/swarming_rpcs.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/swarming/server/task_request_test.py
diff --git a/appengine/swarming/server/task_request_test.py b/appengine/swarming/server/task_request_test.py
index 360734bb013336c1656a63a963d498d046924daa..8776e203862e1f5c5b56b19d4f6739414b5c8797 100755
--- a/appengine/swarming/server/task_request_test.py
+++ b/appengine/swarming/server/task_request_test.py
@@ -255,6 +255,7 @@ class TaskRequestApiTest(TestCase):
'namespace': 'default-gzip',
},
'io_timeout_secs': None,
+ 'outputs': [],
}
expected_request = {
'authenticated': auth_testing.DEFAULT_MOCKED_IDENTITY,
@@ -264,7 +265,7 @@ class TaskRequestApiTest(TestCase):
'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',
@@ -330,6 +331,7 @@ class TaskRequestApiTest(TestCase):
'namespace': 'default-gzip',
},
'io_timeout_secs': None,
+ 'outputs': [],
}
expected_request = {
'authenticated': auth_testing.DEFAULT_MOCKED_IDENTITY,
@@ -339,7 +341,7 @@ class TaskRequestApiTest(TestCase):
'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',
@@ -381,7 +383,7 @@ class TaskRequestApiTest(TestCase):
# 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'))
@@ -649,6 +651,7 @@ class TaskRequestApiTest(TestCase):
'namespace': 'default-gzip',
},
'io_timeout_secs': None,
+ 'outputs': [],
}
# Differences from new_request() are:
# - parent_task_id was reset to None.
« no previous file with comments | « appengine/swarming/server/task_request.py ('k') | appengine/swarming/swarming_rpcs.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698