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

Unified Diff: appengine/findit/waterfall/test/process_base_swarming_task_result_pipeline_test.py

Issue 2477343003: [Findit] Refactoring monitor swarming task pipelines (Closed)
Patch Set: Created 4 years, 1 month 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/findit/waterfall/test/process_base_swarming_task_result_pipeline_test.py
diff --git a/appengine/findit/waterfall/test/process_base_swarming_task_result_pipeline_test.py b/appengine/findit/waterfall/test/process_base_swarming_task_result_pipeline_test.py
index 5435ec42c75c422fafbe45add85a15f219264ce3..6c959b6c2e665a1b6323486f1f58499d38345968 100644
--- a/appengine/findit/waterfall/test/process_base_swarming_task_result_pipeline_test.py
+++ b/appengine/findit/waterfall/test/process_base_swarming_task_result_pipeline_test.py
@@ -1,10 +1,50 @@
import datetime
-from waterfall.test import wf_testcase
+from model import analysis_status
+from model.flake.flake_swarming_task import FlakeSwarmingTask
+from model.wf_swarming_task import WfSwarmingTask
+from waterfall import swarming_util
from waterfall.process_base_swarming_task_result_pipeline import (
ProcessBaseSwarmingTaskResultPipeline)
+from waterfall.process_flake_swarming_task_result_pipeline import (
+ ProcessFlakeSwarmingTaskResultPipeline)
+from waterfall.process_swarming_task_result_pipeline import (
+ ProcessSwarmingTaskResultPipeline)
+from waterfall.test import wf_testcase
+
+
+_ISOLATED_SERVER = 'https://isolateserver.appspot.com'
+
class ProcessBaseSwarmingTaskResultPipelineTest(wf_testcase.WaterfallTestCase):
+
+ def _MockedGetSwarmingTaskResultById(self, task_id, _):
+ swarming_task_results = {
+ 'task_id1': {
+ 'state': 'COMPLETED',
+ 'exit_code': '1',
+ 'tags': [
+ 'priority:25',
+ 'ref_name:abc_tests'
+ ],
+ 'outputs_ref': {
+ 'isolatedserver': _ISOLATED_SERVER,
+ 'namespace': 'default-gzip',
+ 'isolated': 'shard1_isolated'
+ },
+ 'created_ts': '2016-02-10T18:32:06.538220',
+ 'started_ts': '2016-02-10T18:32:09.090550',
+ 'completed_ts': '2016-02-10T18:33:09'
+ },
+ 'task_id2': {
+ 'state': 'TIMED_OUT',
+ 'outputs_ref': None
+ }
+ }
+
+ mocked_result = swarming_task_results[task_id]
+ return mocked_result
+
def setUp(self):
super(ProcessBaseSwarmingTaskResultPipelineTest, self).setUp()
self.pipeline = ProcessBaseSwarmingTaskResultPipeline()
@@ -13,6 +53,8 @@ class ProcessBaseSwarmingTaskResultPipelineTest(wf_testcase.WaterfallTestCase):
self.build_number = 121
self.step_name = 'abc_tests on platform'
self.test_name = 'test'
+ self.mock(swarming_util, 'GetSwarmingTaskResultById',
+ self._MockedGetSwarmingTaskResultById)
def testConvertDateTime(self):
fmt = '%Y-%m-%dT%H:%M:%S.%f'
@@ -29,3 +71,53 @@ class ProcessBaseSwarmingTaskResultPipelineTest(wf_testcase.WaterfallTestCase):
def testConvertDateTimefailure(self):
with self.assertRaises(ValueError):
self.pipeline._ConvertDateTime('abc')
+
+ def testMonitorSwarmingTaskTimeOut(self):
+ # Override swarming config settings to force a timeout.
+ override_swarming_settings = {
+ 'task_timeout_hours': -1
+ }
+ self.UpdateUnitTestConfigSettings(
+ 'swarming_settings', override_swarming_settings)
+
+ task = FlakeSwarmingTask.Create(
+ self.master_name, self.builder_name, self.build_number, self.step_name,
+ self.test_name)
+ task.task_id = 'task_id1'
+ task.put()
+
+ pipeline = ProcessFlakeSwarmingTaskResultPipeline()
+ step_name, task_info = pipeline.run(
+ self.master_name, self.builder_name, self.build_number, self.step_name,
+ 'task_id1', self.build_number, self.test_name, 1)
+ self.assertEqual('abc_tests', task_info)
+ self.assertEqual(self.step_name, step_name)
+
+ task = FlakeSwarmingTask.Get(
+ self.master_name, self.builder_name, self.build_number, self.step_name,
+ self.test_name)
+ self.assertEqual(analysis_status.ERROR, task.status)
+ self.assertEqual({}, task.tests_statuses)
+
+ def testMonitorSwarmingTaskNotRunning(self):
+ task = WfSwarmingTask.Create(
+ self.master_name, self.builder_name,
+ self.build_number, self.step_name)
+ task.task_id = 'task_id2'
+ task.put()
+
+ pipeline = ProcessSwarmingTaskResultPipeline()
+ step_name, task_info = pipeline.run(
+ self.master_name, self.builder_name,
+ self.build_number, self.step_name)
+
+ self.assertEqual(self.step_name, step_name)
+ self.assertIsNone(task_info[0])
+ self.assertEqual([], task_info[1])
+
+ task = WfSwarmingTask.Get(
+ self.master_name, self.builder_name, self.build_number, self.step_name)
+
+ self.assertEqual(analysis_status.ERROR, task.status)
+ self.assertEqual({}, task.tests_statuses)
+ self.assertEqual({}, task.classified_tests)

Powered by Google App Engine
This is Rietveld 408576698