blob: e7f9fba11321e7978047e02a443e13df27550c2d [file] [log] [blame]
# Copyright (c) 2012 The Chromium OS Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
"""Unittests for config. Needs to be run inside of chroot for mox."""
import json
import mock
import os
import subprocess
import sys
import constants
sys.path.insert(0, constants.SOURCE_ROOT)
from chromite.buildbot import builderstage
from chromite.buildbot import cbuildbot_config
from chromite.lib import cros_test_lib
from chromite.lib import git
from chromite.lib import parallel
# pylint: disable=W0212,R0904
class CBuildBotTest(cros_test_lib.MoxTestCase):
def testConfigsKeysMismatch(self):
"""Verify that all configs contain exactly the default keys.
This checks for mispelled keys, or keys that are somehow removed.
expected_keys = set(cbuildbot_config._default.keys())
for build_name, config in cbuildbot_config.config.iteritems():
config_keys = set(config.keys())
extra_keys = config_keys.difference(expected_keys)
self.assertFalse(extra_keys, ('Config %s has extra values %s' %
(build_name, list(extra_keys))))
missing_keys = expected_keys.difference(config_keys)
self.assertFalse(missing_keys, ('Config %s is missing values %s' %
(build_name, list(missing_keys))))
def testConfigsHaveName(self):
""" Configs must have names set."""
for build_name, config in cbuildbot_config.config.iteritems():
self.assertTrue(build_name == config['name'])
def testConfigUseflags(self):
""" Useflags must be lists.
Strings are interpreted as arrays of characters for this, which is not
for build_name, config in cbuildbot_config.config.iteritems():
useflags = config.get('useflags')
if not useflags is None:
isinstance(useflags, list),
'Config %s: useflags should be a list.' % build_name)
def testBoards(self):
"""Verify 'boards' is explicitly set for every config."""
for build_name, config in cbuildbot_config.config.iteritems():
self.assertTrue(isinstance(config['boards'], (tuple, list)),
"Config %s doesn't have a list of boards." % build_name)
self.assertEqual(len(set(config['boards'])), len(config['boards']),
'Config %s has duplicate boards.' % build_name)
self.assertTrue(config['boards'] is not None,
'Config %s defines a list of boards.' % build_name)
def testOverlaySettings(self):
"""Verify overlays and push_overlays have legal values."""
for build_name, config in cbuildbot_config.config.iteritems():
overlays = config['overlays']
push_overlays = config['push_overlays']
self.assertTrue(overlays in [None, 'public', 'private', 'both'],
'Config %s: has unexpected overlays value.' % build_name)
push_overlays in [None, 'public', 'private', 'both'],
'Config %s: has unexpected push_overlays value.' % build_name)
if overlays == None:
subset = [None]
elif overlays == 'public':
subset = [None, 'public']
elif overlays == 'private':
subset = [None, 'private']
elif overlays == 'both':
subset = [None, 'public', 'private', 'both']
push_overlays in subset,
'Config %s: push_overlays should be a subset of overlays.' %
def testOverlayMaster(self):
"""Verify that only one master is pushing uprevs for each overlay."""
masters = {}
for build_name, config in cbuildbot_config.config.iteritems():
overlays = config['overlays']
push_overlays = config['push_overlays']
if (overlays and push_overlays and config['uprev'] and config['master']
and not config['branch']):
other_master = masters.get(push_overlays)
err_msg = 'Found two masters for push_overlays=%s: %s and %s'
err_msg % (push_overlays, build_name, other_master))
masters[push_overlays] = build_name
if 'both' in masters:
self.assertEquals(len(masters), 1, 'Found too many masters.')
def testChromeRev(self):
"""Verify chrome_rev has an expected value"""
for build_name, config in cbuildbot_config.config.iteritems():
config['chrome_rev'] in constants.VALID_CHROME_REVISIONS + [None],
'Config %s: has unexpected chrome_rev value.' % build_name)
config['chrome_rev'] == constants.CHROME_REV_LOCAL,
'Config %s: has unexpected chrome_rev_local value.' % build_name)
if config['chrome_rev']:
'Config %s: has chrome_rev but is not a PFQ.' % build_name)
def testValidVMTestType(self):
"""Verify vm_tests has an expected value"""
for build_name, config in cbuildbot_config.config.iteritems():
config['vm_tests'] in constants.VALID_AU_TEST_TYPES + [None],
'Config %s: has unexpected vm test type value.' % build_name)
def testBuildType(self):
"""Verifies that all configs use valid build types."""
for build_name, config in cbuildbot_config.config.iteritems():
config['build_type'] in constants.VALID_BUILD_TYPES,
'Config %s: has unexpected build_type value.' % build_name)
def testGCCGitHash(self):
"""Verifies that gcc_githash is not set without setting latest_toolchain."""
for build_name, config in cbuildbot_config.config.iteritems():
if config['gcc_githash']:
'Config %s: has gcc_githash but not latest_toolchain.' % build_name)
def testBuildToRun(self):
"""Verify we don't try to run tests without building them."""
for build_name, config in cbuildbot_config.config.iteritems():
isinstance(config['useflags'], list) and
'-build_tests' in config['useflags'] and config['vm_tests'],
'Config %s: has vm_tests and use -build_tests.' % build_name)
def testARMNoVMTest(self):
"""Verify ARM builds don't get VMTests turned on by accident"""
for build_name, config in cbuildbot_config.config.iteritems():
if build_name.startswith('arm-') or config['arm']:
self.assertTrue(config['vm_tests'] is None,
"ARM builder %s can't run vm tests!" % build_name)
#TODO: Add test for compare functionality
def testJSONDumpLoadable(self):
"""Make sure config export functionality works."""
cwd = os.path.dirname(os.path.abspath(__file__))
output = subprocess.Popen(['./cbuildbot_config', '--dump'],
stdout=subprocess.PIPE, cwd=cwd).communicate()[0]
configs = json.loads(output)
self.assertFalse(not configs)
def testJSONBuildbotDumpHasOrder(self):
"""Make sure config export functionality works."""
cwd = os.path.dirname(os.path.abspath(__file__))
output = subprocess.Popen(['./cbuildbot_config', '--dump',
stdout=subprocess.PIPE, cwd=cwd).communicate()[0]
configs = json.loads(output)
for cfg in configs.itervalues():
self.assertTrue(cfg['display_position'] is not None)
self.assertFalse(not configs)
def testHWTestsIFFArchivingHWTestArtifacts(self):
"""Make sure all configs upload artifacts that need them for hw testing."""
for build_name, config in cbuildbot_config.config.iteritems():
if config['hw_tests']:
"%s is trying to run hw tests without uploading payloads." %
def testHWTestTimeout(self):
"""Verify that hw test timeout is in a reasonable range."""
# The parallel library will kill the process if it's silent for longer
# than the silent timeout.
max_timeout = parallel._BackgroundTask.MINIMUM_SILENT_TIMEOUT
for build_name, config in cbuildbot_config.config.iteritems():
for test_config in config['hw_tests']:
self.assertTrue(test_config.timeout < max_timeout,
'%s has a hw_tests_timeout that is too large.' % build_name)
def testValidUnifiedMasterConfig(self):
"""Make sure any unified master configurations are valid."""
for build_name, config in cbuildbot_config.config.iteritems():
error = 'Unified config for %s has invalid values' % build_name
# Unified masters must be internal and must rev both overlays.
if config['master']:
config['internal'] and config['manifest_version'], error)
elif not config['master'] and config['manifest_version']:
# Unified slaves can rev either public or both depending on whether
# they are internal or not.
if not config['internal']:
self.assertEqual(config['overlays'], constants.PUBLIC_OVERLAYS, error)
elif cbuildbot_config.IsCQType(config['build_type']):
self.assertEqual(config['overlays'], constants.BOTH_OVERLAYS, error)
def testGetSlaves(self):
"""Make sure every master has a sane list of slaves"""
for build_name, config in cbuildbot_config.config.iteritems():
if config['master']:
configs = builderstage.BuilderStage._GetSlavesForMaster(config)
len(map(repr, configs)), len(set(map(repr, configs))),
'Duplicate board in slaves of %s will cause upload prebuilts'
' failures' % build_name)
def testFactoryFirmwareValidity(self):
"""Ensures that firmware/factory branches have at least 1 valid name."""
tracking_branch = git.GetChromiteTrackingBranch()
for branch in ['firmware', 'factory']:
if tracking_branch.startswith(branch):
saw_config_for_branch = False
for build_name in cbuildbot_config.config:
if build_name.endswith('-%s' % branch):
self.assertFalse('release' in build_name,
'Factory|Firmware release builders should not '
'contain release in their name.')
saw_config_for_branch = True
saw_config_for_branch, 'No config found for %s branch. '
'As this is the %s branch, all release configs that are being used '
'must end in %s.' % (branch, tracking_branch, branch))
def testBuildTests(self):
"""Verify that we don't try to use tests without building them."""
for build_name, config in cbuildbot_config.config.iteritems():
if not config['build_tests']:
self.assertFalse('factory_test' in config['images'],
'Config %s builds factory_test without build_tests.' % build_name)
self.assertFalse('test' in config['images'],
'Config %s builds test image without build_tests.' % build_name)
for flag in ('vm_tests', 'hw_tests', 'upload_hw_test_artifacts'):
'Config %s set %s without build_tests.' % (build_name, flag))
def testUseChromeLKGMImpliesInternal(self):
"""Currently use_chrome_lkgm refers only to internal manifests."""
for build_name, config in cbuildbot_config.config.iteritems():
if config['use_chrome_lkgm']:
'Chrome lkgm currently only works with an internal manifest: %s' % (
def testChromePFQsNeedChromeOSPFQs(self):
"""Make sure every Chrome PFQ has a matching ChromeOS PFQ for prebuilts.
See for details on why.
Note: This check isn't perfect (as we can't check to see if we have
any ChromeOS PFQs actually running), but it at least makes sure people
realize we need to have the configs in sync."""
# Figure out all the boards our Chrome and ChromeOS PFQs are using.
cr_pfqs = set()
cros_pfqs = set()
for config in cbuildbot_config.config.itervalues():
if config['build_type'] == constants.CHROME_PFQ_TYPE:
elif config['build_type'] == constants.PALADIN_TYPE:
# Then make sure the ChromeOS PFQ set is a superset of the Chrome PFQ set.
missing_pfqs = cr_pfqs.difference(cros_pfqs)
if missing_pfqs:'Chrome PFQs are using boards that are missing ChromeOS PFQs:'
'\n\t' + ' '.join(missing_pfqs))
def testNonOverlappingConfigTypes(self):
"""Test that a config can only match one build suffix."""
for config_type in cbuildbot_config.CONFIG_TYPE_DUMP_ORDER:
# A longer config_type should never end with a shorter suffix.
my_list = list(cbuildbot_config.CONFIG_TYPE_DUMP_ORDER)
config_type, type_order=my_list),
def testCorrectConfigTypeIndex(self):
"""Test that the correct build suffix index is returned."""
type_order = (
for index, config_type in enumerate(type_order):
config = '-'.join(['pre-fix', config_type])
config, type_order=type_order),
# Verify suffix needs to match up to a '-'.
'pre-fix-sometype1', type_order=type_order),
def testConfigTypesComplete(self):
"""Verify CONFIG_TYPE_DUMP_ORDER contains all valid config types."""
for config_name in cbuildbot_config.config:
'%s did not match any types in %s' %
(config_name, 'cbuildbot_config.CONFIG_TYPE_DUMP_ORDER'))
def testCantBeBothTypesOfLKGM(self):
"""Using lkgm and chrome_lkgm doesn't make sense."""
for config in cbuildbot_config.config.values():
self.assertFalse(config['use_lkgm'] and config['use_chrome_lkgm'])
def testQuickPaladin(self):
"""Test that no paladin builder has both quick_unit=False and vm_tests."""
for build_name, config in cbuildbot_config.config.iteritems():
self.assertFalse(config['build_type'] == constants.PALADIN_TYPE and
config['important'] and not config['quick_unit'] and
config['vm_tests'], '%s has both quick_unit=False and '
'vm_tests' % build_name)
def testCantBeBothTypesOfPGO(self):
"""Using pgo_generate and pgo_use together doesn't work."""
for config in cbuildbot_config.config.values():
self.assertFalse(config['pgo_use'] and config['pgo_generate'])
def testValidPrebuilts(self):
"""Verify all builders have valid prebuilt values."""
for build_name, config in cbuildbot_config.config.iteritems():
msg = 'Config %s: has unexpected prebuilts value.' % build_name
valid_values = (False, constants.PRIVATE, constants.PUBLIC)
self.assertTrue(config['prebuilts'] in valid_values, msg)
def testInternalPrebuilts(self):
for build_name, config in cbuildbot_config.config.iteritems():
if (config['internal'] and
config['build_type'] != constants.CHROME_PFQ_TYPE):
msg = 'Config %s is internal but has public prebuilts.' % build_name
self.assertNotEqual(config['prebuilts'], constants.PUBLIC, msg)
class FindFullTest(cros_test_lib.TestCase):
"""Test locating of official build for a board."""
def _RunTest(self, board, external_expected=None, internal_expected=None):
def check_expected(l, expected):
if expected is not None:
self.assertTrue(expected in [v['name'] for v in l])
external, internal = cbuildbot_config.FindFullConfigsForBoard(board)
all(v is None for v in [external_expected, internal_expected]))
check_expected(external, external_expected)
check_expected(internal, internal_expected)
def _CheckCanonicalConfig(self, board, ending):
'-'.join((board, ending)),
def testExternal(self):
"""Test finding of a full builder."""
self._RunTest('amd64-generic', external_expected='amd64-generic-full')
def testInternal(self):
"""Test finding of a release builder."""
self._RunTest('lumpy', internal_expected='lumpy-release')
def testBoth(self):
"""Both an external and internal config exist for board."""
self._RunTest('daisy', external_expected='daisy-full',
def testExternalCanonicalResolution(self):
"""Test an external canonical config."""
self._CheckCanonicalConfig('x86-generic', 'full')
def testInternalCanonicalResolution(self):
"""Test prefer internal over external when both exist."""
self._CheckCanonicalConfig('daisy', 'release')
def testPGOCanonicalResolution(self):
"""Test prefer non-PGO over PGO builder."""
self._CheckCanonicalConfig('lumpy', 'release')
def testOneFullConfigPerBoard(self):
"""There is at most one 'full' config for a board."""
# Verifies that there is one external 'full' and one internal 'release'
# build per board. This is to ensure that we fail any new configs that
# wrongly have names like *-bla-release or *-bla-full. This case can also
# be caught if the new suffix was added to
# cbuildbot_config.CONFIG_TYPE_DUMP_ORDER
# (see testNonOverlappingConfigTypes), but that's not guaranteed to happen.
def AtMostOneConfig(board, label, configs):
if len(configs) > 1:
'Found more than one %s config for %s: %r'
% (label, board, [c['name'] for c in configs]))
boards = set()
for config in cbuildbot_config.config.itervalues():
# Sanity check of the boards.
assert boards
for b in boards:
external, internal = cbuildbot_config.FindFullConfigsForBoard(b)
AtMostOneConfig(b, 'external', external)
AtMostOneConfig(b, 'internal', internal)
class OverrideForTrybotTest(cros_test_lib.TestCase):
"""Test config override functionality."""
def _testWithOptions(self, **kwargs):
mock_options = mock.Mock()
for k, v in kwargs.iteritems():
mock_options.setattr(k, v)
for config in cbuildbot_config.config.itervalues():
cbuildbot_config.OverrideConfigForTrybot(config, mock_options)
def testLocalTrybot(self):
"""Override each config for local trybot."""
self._testWithOptions(remote_trybot=False, hw_test=False)
def testRemoteTrybot(self):
"""Override each config for remote trybot."""
self._testWithOptions(remote_trybot=True, hw_test=False)
def testRemoteHWTest(self):
"""Override each config for remote trybot + hwtests."""
self._testWithOptions(remote_trybot=True, hw_test=True)
def testChromeInternalOverride(self):
"""Verify that we are not using official Chrome for local trybots."""
mock_options = mock.Mock()
mock_options.remote_trybot = False
mock_options.hw_test = False
old = cbuildbot_config.config['mario-paladin']
new = cbuildbot_config.OverrideConfigForTrybot(old, mock_options)
self.assertTrue(constants.USE_CHROME_INTERNAL in old['useflags'])
self.assertTrue(constants.USE_CHROME_PDF in old['useflags'])
self.assertFalse(constants.USE_CHROME_INTERNAL in new['useflags'])
self.assertFalse(constants.USE_CHROME_PDF in new['useflags'])
if __name__ == '__main__':