123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178 |
- # Copyright (c) 2012 The Chromium Authors. All rights reserved.
- # Use of this source code is governed by a BSD-style license that can be
- # found in the LICENSE file.
- """Top-level presubmit script for depot tools.
- See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts for
- details on the presubmit API built into depot_tools.
- """
- import fnmatch
- import os
- import sys
- # Whether to run the checks under Python2 or Python3.
- # TODO: Uncomment this to run the checks under Python3, and change the tests
- # in _CommonChecks in this file and the values and tests in
- # //tests/PRESUBMIT.py as well to keep the test coverage of all three cases
- # (true, false, and default/not specified).
- # USE_PYTHON3 = False
- # CIPD ensure manifest for checking CIPD client itself.
- CIPD_CLIENT_ENSURE_FILE_TEMPLATE = r'''
- # Full supported.
- $VerifiedPlatform linux-amd64 mac-amd64 windows-amd64 windows-386
- # Best effort support.
- $VerifiedPlatform linux-386 linux-ppc64 linux-ppc64le linux-s390x
- $VerifiedPlatform linux-arm64 linux-armv6l
- $VerifiedPlatform linux-mips64 linux-mips64le linux-mipsle
- %s %s
- '''
- # Timeout for a test to be executed.
- TEST_TIMEOUT_S = 330 # 5m 30s
- def DepotToolsPylint(input_api, output_api):
- """Gather all the pylint logic into one place to make it self-contained."""
- files_to_check = [
- r'^[^/]*\.py$',
- r'^testing_support/[^/]*\.py$',
- r'^tests/[^/]*\.py$',
- r'^recipe_modules/.*\.py$', # Allow recursive search in recipe modules.
- ]
- files_to_skip = list(input_api.DEFAULT_FILES_TO_SKIP)
- if os.path.exists('.gitignore'):
- with open('.gitignore') as fh:
- lines = [l.strip() for l in fh.readlines()]
- files_to_skip.extend([fnmatch.translate(l) for l in lines if
- l and not l.startswith('#')])
- if os.path.exists('.git/info/exclude'):
- with open('.git/info/exclude') as fh:
- lines = [l.strip() for l in fh.readlines()]
- files_to_skip.extend([fnmatch.translate(l) for l in lines if
- l and not l.startswith('#')])
- disabled_warnings = [
- 'R0401', # Cyclic import
- 'W0613', # Unused argument
- ]
- return input_api.canned_checks.GetPylint(
- input_api,
- output_api,
- files_to_check=files_to_check,
- files_to_skip=files_to_skip,
- disabled_warnings=disabled_warnings)
- def CommonChecks(input_api, output_api, tests_to_skip_list):
- input_api.SetTimeout(TEST_TIMEOUT_S)
- file_filter = lambda x: x.LocalPath() == 'infra/config/recipes.cfg'
- results = input_api.canned_checks.CheckJsonParses(input_api, output_api,
- file_filter=file_filter)
- # The tests here are assuming this is not defined, so raise an error
- # if it is.
- if 'USE_PYTHON3' in globals():
- results.append(output_api.PresubmitError(
- 'USE_PYTHON3 is defined; update the tests in //PRESUBMIT.py and '
- '//tests/PRESUBMIT.py.'))
- elif sys.version_info.major != 2:
- results.append(output_api.PresubmitError(
- 'Did not use Python2 for //PRESUBMIT.py by default.'))
- results.extend(input_api.canned_checks.CheckJsonParses(
- input_api, output_api))
- # Run only selected tests on Windows.
- test_to_run_list = [r'.*test\.py$']
- if input_api.platform.startswith(('cygwin', 'win32')):
- print('Warning: skipping most unit tests on Windows')
- tests_to_skip_list.extend([
- r'.*auth_test\.py$',
- r'.*git_common_test\.py$',
- r'.*git_hyper_blame_test\.py$',
- r'.*git_map_test\.py$',
- r'.*ninjalog_uploader_test\.py$',
- r'.*recipes_test\.py$',
- ])
- tests_to_skip_list.append(r'.*my_activity_test\.py')
- # TODO(maruel): Make sure at least one file is modified first.
- # TODO(maruel): If only tests are modified, only run them.
- tests = DepotToolsPylint(input_api, output_api)
- tests.extend(input_api.canned_checks.GetUnitTestsInDirectory(
- input_api,
- output_api,
- 'tests',
- files_to_check=test_to_run_list,
- files_to_skip=tests_to_skip_list,
- run_on_python3=False))
- tests.extend(input_api.canned_checks.GetUnitTestsInDirectory(
- input_api,
- output_api,
- 'tests',
- files_to_check=[r'.*my_activity_test\.py'],
- run_on_python3=True))
- # Validate CIPD manifests.
- root = input_api.os_path.normpath(
- input_api.os_path.abspath(input_api.PresubmitLocalPath()))
- rel_file = lambda rel: input_api.os_path.join(root, rel)
- cipd_manifests = set(rel_file(input_api.os_path.join(*x)) for x in (
- ('cipd_manifest.txt',),
- ('bootstrap', 'manifest.txt'),
- ('bootstrap', 'manifest_bleeding_edge.txt'),
- # Also generate a file for the cipd client itself.
- ('cipd_client_version',),
- ))
- affected_manifests = input_api.AffectedFiles(
- include_deletes=False,
- file_filter=lambda x:
- input_api.os_path.normpath(x.AbsoluteLocalPath()) in cipd_manifests)
- for path in affected_manifests:
- path = path.AbsoluteLocalPath()
- if path.endswith('.txt'):
- tests.append(input_api.canned_checks.CheckCIPDManifest(
- input_api, output_api, path=path))
- else:
- pkg = 'infra/tools/cipd/${platform}'
- ver = input_api.ReadFile(path)
- tests.append(input_api.canned_checks.CheckCIPDManifest(
- input_api, output_api,
- content=CIPD_CLIENT_ENSURE_FILE_TEMPLATE % (pkg, ver)))
- tests.append(input_api.canned_checks.CheckCIPDClientDigests(
- input_api, output_api, client_version_file=path))
- results.extend(input_api.RunTests(tests))
- return results
- def CheckChangeOnUpload(input_api, output_api):
- # Do not run integration tests on upload since they are way too slow.
- tests_to_skip_list = [
- r'^checkout_test\.py$',
- r'^cipd_bootstrap_test\.py$',
- r'^gclient_smoketest\.py$',
- ]
- results = []
- results.extend(input_api.canned_checks.CheckOwners(
- input_api, output_api, allow_tbr=False))
- results.extend(input_api.canned_checks.CheckOwnersFormat(
- input_api, output_api))
- results.extend(CommonChecks(input_api, output_api, tests_to_skip_list))
- return results
- def CheckChangeOnCommit(input_api, output_api):
- output = []
- output.extend(CommonChecks(input_api, output_api, []))
- output.extend(input_api.canned_checks.CheckDoNotSubmit(
- input_api,
- output_api))
- return output
|