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

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: Add master.cfg/slave.cfg to fyi bot to use new recipe. 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 pre_run(self, api, suffix):
644 """Triggers an AMP test."""
645 amp_arguments = api.amp.amp_arguments(
646 api_address=AMPGTestTest.AMP_INSTANCE_ADDRESS,
647 api_port=AMPGTestTest.AMP_INSTANCE_PORT,
648 api_protocol=AMPGTestTest.AMP_INSTANCE_PROTOCOL,
649 device_name=self._device_name,
650 device_os=self._device_os)
651
652 isolate_file_path = (api.path['checkout'].join(self._android_isolate_path)
653 if self._android_isolate_path else None)
654 try:
655 api.amp.trigger_test_suite(
656 self._name, 'gtest',
657 api.amp.gtest_arguments(self._name,
658 isolate_file_path=isolate_file_path),
659 amp_arguments)
660 self._trigger_successful = True
661 except api.step.StepFailure:
662 self._trigger_successful = False
663
664 def run(self, api, suffix): # pylint: disable=R0201
665 # If we were unable to successfully trigger the AMP job, run locally;
666 # otherwise return no results as results will be collected in post_run.
667 if not self._trigger_successful:
668 return self._local_test.run(api, suffix)
669 else:
670 return []
671
672 def post_run(self, api, suffix):
673 if self._trigger_successful:
674 amp_arguments = api.amp.amp_arguments(
675 api_address=AMPGTestTest.AMP_INSTANCE_ADDRESS,
676 api_port=AMPGTestTest.AMP_INSTANCE_PORT,
677 api_protocol=AMPGTestTest.AMP_INSTANCE_PROTOCOL,
678 device_name=self._device_name,
679 device_os=self._device_os)
680
681 api.amp.collect_test_suite(
682 self._name, 'gtest', api.amp.gtest_arguments(self._name),
683 amp_arguments)
684
685
621 class GTestTest(Test): 686 class GTestTest(Test):
622 def __init__(self, name, args=None, target_name=None, enable_swarming=False, 687 def __init__(self, name, args=None, target_name=None, enable_swarming=False,
623 swarming_shards=1, swarming_dimensions=None, swarming_tags=None, 688 swarming_shards=1, swarming_dimensions=None, swarming_tags=None,
624 swarming_extra_suffix=None, **runtest_kwargs): 689 swarming_extra_suffix=None, **runtest_kwargs):
625 super(GTestTest, self).__init__() 690 super(GTestTest, self).__init__()
626 if enable_swarming: 691 if enable_swarming:
627 self._test = SwarmingGTestTest( 692 self._test = SwarmingGTestTest(
628 name, args, target_name, swarming_shards, swarming_dimensions, 693 name, args, target_name, swarming_shards, swarming_dimensions,
629 swarming_tags, swarming_extra_suffix) 694 swarming_tags, swarming_extra_suffix)
630 else: 695 else:
(...skipping 624 matching lines...) Expand 10 before | Expand all | Expand 10 after
1255 GTestTest('ui_base_unittests'), 1320 GTestTest('ui_base_unittests'),
1256 GTestTest('ui_ios_unittests'), 1321 GTestTest('ui_ios_unittests'),
1257 GTestTest('sync_unit_tests'), 1322 GTestTest('sync_unit_tests'),
1258 GTestTest('sql_unittests'), 1323 GTestTest('sql_unittests'),
1259 ] 1324 ]
1260 1325
1261 GOMA_TESTS = [ 1326 GOMA_TESTS = [
1262 GTestTest('base_unittests'), 1327 GTestTest('base_unittests'),
1263 GTestTest('content_unittests'), 1328 GTestTest('content_unittests'),
1264 ] 1329 ]
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698