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

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: Fix help text for --output 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
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 0546c9f1f717c6621e4e2ba4be57d3ad92c38400..1eaffc267fba7b000ae6291a60b0dd79691f29c6 100755
--- a/appengine/swarming/server/task_request_test.py
+++ b/appengine/swarming/server/task_request_test.py
@@ -254,6 +254,7 @@ class TaskRequestApiTest(TestCase):
'namespace': 'default-gzip',
},
'io_timeout_secs': None,
+ 'outputs': [],
}
expected_request = {
'authenticated': auth_testing.DEFAULT_MOCKED_IDENTITY,
@@ -263,7 +264,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': '411f20b8cda819b5fe47bcf26f5566c37afce0ed',
+ 'properties_hash': 'ca5c7327e43431514472b081b0bbeb62dc1bd8e9',
'pubsub_topic': None,
'pubsub_userdata': None,
'service_account': u'none',
@@ -328,6 +329,7 @@ class TaskRequestApiTest(TestCase):
'namespace': 'default-gzip',
},
'io_timeout_secs': None,
+ 'outputs': [],
}
expected_request = {
'authenticated': auth_testing.DEFAULT_MOCKED_IDENTITY,
@@ -337,7 +339,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': '411f20b8cda819b5fe47bcf26f5566c37afce0ed',
+ 'properties_hash': 'ca5c7327e43431514472b081b0bbeb62dc1bd8e9',
'pubsub_topic': None,
'pubsub_userdata': None,
'service_account': u'none',
@@ -379,7 +381,7 @@ class TaskRequestApiTest(TestCase):
# Other unit tests should use the calculated value.
# Ensure the algorithm is deterministic.
self.assertEqual(
- '411f20b8cda819b5fe47bcf26f5566c37afce0ed', as_dict['properties_hash'])
+ 'ca5c7327e43431514472b081b0bbeb62dc1bd8e9', as_dict['properties_hash'])
def test_init_new_request_bot_service_account(self):
request = mkreq(_gen_request(service_account_token='bot'))
@@ -585,6 +587,7 @@ class TaskRequestApiTest(TestCase):
'namespace': 'default-gzip',
},
'io_timeout_secs': None,
+ 'outputs': [],
}
# Differences from new_request() are:
# - parent_task_id was reset to None.

Powered by Google App Engine
This is Rietveld 408576698