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

Side by Side Diff: scripts/slave/recipe_modules/chromium_tests/steps.py

Issue 1104533002: Add recipe for split AMP/local CQ. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Parent builder Android Builder (dbg) and expectation changes. Created 5 years, 6 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 | Annotate | Revision Log
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import re 5 import re
6 import string 6 import string
7 7
8 8
9 class Test(object): 9 class Test(object):
10 """ 10 """
(...skipping 600 matching lines...) Expand 10 before | Expand all | Expand 10 after
611 if not gtest_results: 611 if not gtest_results:
612 return False, None # pragma: no cover 612 return False, None # pragma: no cover
613 613
614 global_tags = gtest_results.raw.get('global_tags', []) 614 global_tags = gtest_results.raw.get('global_tags', [])
615 if 'UNRELIABLE_RESULTS' in global_tags: 615 if 'UNRELIABLE_RESULTS' in global_tags:
616 return False, None # pragma: no cover 616 return False, None # pragma: no cover
617 617
618 return True, gtest_results.failures 618 return True, gtest_results.failures
619 619
620 620
621 class AMPGTestTest(Test):
622 AMP_INSTANCE_ADDRESS = '172.22.21.180'
623 AMP_INSTANCE_PORT = '80'
624 AMP_INSTANCE_PROTOCOL = 'http'
625 AMP_RESULTS_BUCKET = 'chrome-amp-results'
626 def __init__(self, name, args=None, target_name=None, device_name=['Nexus 5'],
627 device_os=['4.4.2'], android_isolate_path=None,
628 **runtest_kwargs):
629 self._name = name
630 self._args = args
631 self._target_name = target_name
632 self._android_isolate_path = android_isolate_path
633 # LocalGTestTest is used when AMP tests are not triggered successfully.
634 self._local_test = LocalGTestTest(name, args, target_name, **runtest_kwargs)
635 self._device_name = device_name
636 self._device_os = device_os
637 self._trigger_successful = None
638
639 @property
640 def name(self):
641 return self._name
642
643 def compile_targets(self, api):
navabi 2015/06/23 21:38:30 had to add this for recipe_simulation_tests.py.
644 return self._local_test.compile_targets(api)
645
646 def pre_run(self, api, suffix):
647 """Triggers an AMP test."""
648 amp_arguments = api.amp.amp_arguments(
649 api_address=AMPGTestTest.AMP_INSTANCE_ADDRESS,
650 api_port=AMPGTestTest.AMP_INSTANCE_PORT,
651 api_protocol=AMPGTestTest.AMP_INSTANCE_PROTOCOL,
652 device_name=self._device_name,
653 device_os=self._device_os)
654
655 isolate_file_path = (api.path['checkout'].join(self._android_isolate_path)
656 if self._android_isolate_path else None)
657 try:
658 api.amp.trigger_test_suite(
659 self._name, 'gtest',
660 api.amp.gtest_arguments(self._name,
661 isolate_file_path=isolate_file_path),
662 amp_arguments)
663 self._trigger_successful = True
664 except api.step.StepFailure:
665 self._trigger_successful = False
666
667 def run(self, api, suffix): # pylint: disable=R0201
668 # If we were unable to successfully trigger the AMP job, run locally;
669 # otherwise return no results as results will be collected in post_run.
670 if not self._trigger_successful:
671 return self._local_test.run(api, suffix)
672 else:
673 return []
674
675 def post_run(self, api, suffix):
676 if self._trigger_successful:
677 amp_arguments = api.amp.amp_arguments(
678 api_address=AMPGTestTest.AMP_INSTANCE_ADDRESS,
679 api_port=AMPGTestTest.AMP_INSTANCE_PORT,
680 api_protocol=AMPGTestTest.AMP_INSTANCE_PROTOCOL,
681 device_name=self._device_name,
682 device_os=self._device_os)
683
684 api.amp.collect_test_suite(
685 self._name, 'gtest', api.amp.gtest_arguments(self._name),
686 amp_arguments)
687
688
621 class GTestTest(Test): 689 class GTestTest(Test):
622 def __init__(self, name, args=None, target_name=None, enable_swarming=False, 690 def __init__(self, name, args=None, target_name=None, enable_swarming=False,
623 swarming_shards=1, swarming_dimensions=None, swarming_tags=None, 691 swarming_shards=1, swarming_dimensions=None, swarming_tags=None,
624 swarming_extra_suffix=None, **runtest_kwargs): 692 swarming_extra_suffix=None, **runtest_kwargs):
625 super(GTestTest, self).__init__() 693 super(GTestTest, self).__init__()
626 if enable_swarming: 694 if enable_swarming:
627 self._test = SwarmingGTestTest( 695 self._test = SwarmingGTestTest(
628 name, args, target_name, swarming_shards, swarming_dimensions, 696 name, args, target_name, swarming_shards, swarming_dimensions,
629 swarming_tags, swarming_extra_suffix) 697 swarming_tags, swarming_extra_suffix)
630 else: 698 else:
(...skipping 624 matching lines...) Expand 10 before | Expand all | Expand 10 after
1255 GTestTest('ui_base_unittests'), 1323 GTestTest('ui_base_unittests'),
1256 GTestTest('ui_ios_unittests'), 1324 GTestTest('ui_ios_unittests'),
1257 GTestTest('sync_unit_tests'), 1325 GTestTest('sync_unit_tests'),
1258 GTestTest('sql_unittests'), 1326 GTestTest('sql_unittests'),
1259 ] 1327 ]
1260 1328
1261 GOMA_TESTS = [ 1329 GOMA_TESTS = [
1262 GTestTest('base_unittests'), 1330 GTestTest('base_unittests'),
1263 GTestTest('content_unittests'), 1331 GTestTest('content_unittests'),
1264 ] 1332 ]
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698