diff options
| author | Bertrand SIMONNET <bsimonnet@chromium.org> | 2014-09-23 11:08:39 -0700 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-09-30 01:53:25 +0000 |
| commit | 2d66e1bc9bf431023945dd73884b9b0bb4b4742b (patch) | |
| tree | 608406387ca4457c0d838ab8ea2b4b693f6dbaf3 /cbuildbot/cbuildbot_config_unittest.py | |
| parent | 240321725d0ed5aa96fab7996c1ecb1dccd109b7 (diff) | |
| download | chromite-2d66e1bc9bf431023945dd73884b9b0bb4b4742b.tar.gz | |
cbuildbot_config: allow vm_tests to be None
vm_tests is allowed to be None. Unit tests should not assume that it is an
array.
BUG=None
TEST=unittests pass with gizmo-paladin setting vm_tests to None.
Change-Id: I77684c3d299e2ac7e4d660b83e2f565d16e2e120
Reviewed-on: https://chromium-review.googlesource.com/219489
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Bertrand Simonnet <bsimonnet@chromium.org>
Commit-Queue: Bertrand Simonnet <bsimonnet@chromium.org>
Diffstat (limited to 'cbuildbot/cbuildbot_config_unittest.py')
| -rwxr-xr-x | cbuildbot/cbuildbot_config_unittest.py | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/cbuildbot/cbuildbot_config_unittest.py b/cbuildbot/cbuildbot_config_unittest.py index b82b86051..c46ba497a 100755 --- a/cbuildbot/cbuildbot_config_unittest.py +++ b/cbuildbot/cbuildbot_config_unittest.py @@ -172,6 +172,8 @@ class CBuildBotTest(cros_test_lib.MoxTestCase): def testValidVMTestType(self): """Verify vm_tests has an expected value""" for build_name, config in cbuildbot_config.config.iteritems(): + if config['vm_tests'] is None: + continue for test_type in config['vm_tests']: self.assertTrue( test_type in constants.VALID_VM_TEST_TYPES, |