blob: 553aa81192b3d06e04ffdf40cfeae7d94022d43c [file] [log] [blame]
chrisha@chromium.orgc920ad22012-02-02 18:26:04 +00001# Copyright (c) 2012 The Chromium Authors. All rights reserved.
maruel@chromium.org3d235242009-05-15 12:40:48 +00002# Use of this source code is governed by a BSD-style license that can be
3# found in the LICENSE file.
4
5"""Top-level presubmit script for depot tools.
6
7See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts for
maruel@chromium.org2a74d372011-03-29 19:05:50 +00008details on the presubmit API built into depot_tools.
maruel@chromium.org3d235242009-05-15 12:40:48 +00009"""
10
Josip Sokcevic57c928c2021-11-02 22:56:28 +000011PRESUBMIT_VERSION = '2.0.0'
12
szager@chromium.org34071032014-03-18 17:23:48 +000013import fnmatch
14import os
Edward Lemur595eb192020-03-12 21:53:52 +000015import sys
szager@chromium.org34071032014-03-18 17:23:48 +000016
Dirk Pranke61bf6e82021-04-23 00:50:21 +000017# Whether to run the checks under Python2 or Python3.
18# TODO: Uncomment this to run the checks under Python3, and change the tests
19# in _CommonChecks in this file and the values and tests in
20# //tests/PRESUBMIT.py as well to keep the test coverage of all three cases
21# (true, false, and default/not specified).
22# USE_PYTHON3 = False
dpranke@chromium.org627ea672011-03-11 23:29:03 +000023
Vadim Shtayura09098852018-06-21 22:50:30 +000024# CIPD ensure manifest for checking CIPD client itself.
25CIPD_CLIENT_ENSURE_FILE_TEMPLATE = r'''
26# Full supported.
27$VerifiedPlatform linux-amd64 mac-amd64 windows-amd64 windows-386
28# Best effort support.
29$VerifiedPlatform linux-386 linux-ppc64 linux-ppc64le linux-s390x
Vadim Shtayura333617b2018-08-07 20:46:38 +000030$VerifiedPlatform linux-arm64 linux-armv6l
31$VerifiedPlatform linux-mips64 linux-mips64le linux-mipsle
Robert Iannucci7d47eb52017-12-06 12:18:18 -080032
33%s %s
34'''
35
Edward Lemur595eb192020-03-12 21:53:52 +000036# Timeout for a test to be executed.
Edward Lesmesae3586b2020-03-23 21:21:14 +000037TEST_TIMEOUT_S = 330 # 5m 30s
Edward Lemur595eb192020-03-12 21:53:52 +000038
39
Josip Sokcevic57c928c2021-11-02 22:56:28 +000040def CheckPylint(input_api, output_api):
agablef39c3332016-09-26 09:35:42 -070041 """Gather all the pylint logic into one place to make it self-contained."""
Ayu Ishii81a90742020-07-07 19:53:00 +000042 files_to_check = [
agablef39c3332016-09-26 09:35:42 -070043 r'^[^/]*\.py$',
44 r'^testing_support/[^/]*\.py$',
45 r'^tests/[^/]*\.py$',
46 r'^recipe_modules/.*\.py$', # Allow recursive search in recipe modules.
tandrii@chromium.org55cb27e2016-05-16 17:54:40 +000047 ]
local_bot9af33fa2020-07-09 03:26:20 +000048 files_to_skip = list(input_api.DEFAULT_FILES_TO_SKIP)
szager@chromium.org34071032014-03-18 17:23:48 +000049 if os.path.exists('.gitignore'):
50 with open('.gitignore') as fh:
51 lines = [l.strip() for l in fh.readlines()]
Ayu Ishii81a90742020-07-07 19:53:00 +000052 files_to_skip.extend([fnmatch.translate(l) for l in lines if
szager@chromium.org34071032014-03-18 17:23:48 +000053 l and not l.startswith('#')])
54 if os.path.exists('.git/info/exclude'):
55 with open('.git/info/exclude') as fh:
56 lines = [l.strip() for l in fh.readlines()]
Ayu Ishii81a90742020-07-07 19:53:00 +000057 files_to_skip.extend([fnmatch.translate(l) for l in lines if
szager@chromium.org34071032014-03-18 17:23:48 +000058 l and not l.startswith('#')])
maruel@chromium.orgff9a2172012-04-24 16:55:32 +000059 disabled_warnings = [
Aravind Vasudevanc5f0cbb2022-01-24 23:56:57 +000060 'R0401', # Cyclic import
61 'W0613', # Unused argument
62 'C0415', # import-outside-toplevel
63 'R1710', # inconsistent-return-statements
64 'E1101', # no-member
65 'E1120', # no-value-for-parameter
66 'R1708', # stop-iteration-return
67 'W1510', # subprocess-run-check
68 # Checks which should be re-enabled after Python 2 support is removed.
69 'R0205', # useless-object-inheritance
70 'R1725', # super-with-arguments
71 'W0707', # raise-missing-from
72 'W1113', # keyword-arg-before-vararg
maruel@chromium.orgff9a2172012-04-24 16:55:32 +000073 ]
Josip Sokcevic57c928c2021-11-02 22:56:28 +000074 return input_api.RunTests(input_api.canned_checks.GetPylint(
scottmg@chromium.org69ae86a2014-01-17 03:55:45 +000075 input_api,
76 output_api,
local_bot9af33fa2020-07-09 03:26:20 +000077 files_to_check=files_to_check,
78 files_to_skip=files_to_skip,
Aravind Vasudevanc5f0cbb2022-01-24 23:56:57 +000079 disabled_warnings=disabled_warnings,
80 version='2.7'), parallel=False)
agablef39c3332016-09-26 09:35:42 -070081
82
Josip Sokcevic57c928c2021-11-02 22:56:28 +000083def CheckRecipes(input_api, output_api):
Josip Sokcevic9eee47a2021-07-13 17:38:53 +000084 file_filter = lambda x: x.LocalPath() == 'infra/config/recipes.cfg'
Josip Sokcevic57c928c2021-11-02 22:56:28 +000085 return input_api.canned_checks.CheckJsonParses(input_api, output_api,
86 file_filter=file_filter)
Dirk Pranke61bf6e82021-04-23 00:50:21 +000087
Josip Sokcevic57c928c2021-11-02 22:56:28 +000088def CheckPythonVersion(input_api, output_api):
Dirk Pranke61bf6e82021-04-23 00:50:21 +000089 # The tests here are assuming this is not defined, so raise an error
90 # if it is.
91 if 'USE_PYTHON3' in globals():
Josip Sokcevic57c928c2021-11-02 22:56:28 +000092 return [output_api.PresubmitError(
Dirk Pranke61bf6e82021-04-23 00:50:21 +000093 'USE_PYTHON3 is defined; update the tests in //PRESUBMIT.py and '
Josip Sokcevic57c928c2021-11-02 22:56:28 +000094 '//tests/PRESUBMIT.py.')]
95 if sys.version_info.major != 2:
96 return [output_api.PresubmitError(
97 'Did not use Python2 for //PRESUBMIT.py by default.')]
98 return []
Dirk Pranke61bf6e82021-04-23 00:50:21 +000099
Josip Sokcevic57c928c2021-11-02 22:56:28 +0000100
101def CheckJsonFiles(input_api, output_api):
102 return input_api.canned_checks.CheckJsonParses(
103 input_api, output_api)
104
105
106def CheckUnitTestsOnCommit(input_api, output_api):
107 # Do not run integration tests on upload since they are way too slow.
108 input_api.SetTimeout(TEST_TIMEOUT_S)
Vadim Shtayura7e50ee32018-07-26 17:43:51 +0000109
110 # Run only selected tests on Windows.
Ayu Ishii81a90742020-07-07 19:53:00 +0000111 test_to_run_list = [r'.*test\.py$']
Josip Sokcevic57c928c2021-11-02 22:56:28 +0000112 tests_to_skip_list = []
Vadim Shtayura7e50ee32018-07-26 17:43:51 +0000113 if input_api.platform.startswith(('cygwin', 'win32')):
114 print('Warning: skipping most unit tests on Windows')
Josip Sokcevic4940cc42021-10-05 23:55:34 +0000115 tests_to_skip_list.extend([
Edward Lemurbbd70fd2019-09-26 18:19:41 +0000116 r'.*auth_test\.py$',
Edward Lemurbbd70fd2019-09-26 18:19:41 +0000117 r'.*git_common_test\.py$',
Edward Lemurbbd70fd2019-09-26 18:19:41 +0000118 r'.*git_hyper_blame_test\.py$',
Edward Lemurb800fde2020-01-10 23:04:44 +0000119 r'.*git_map_test\.py$',
Edward Lemurbbd70fd2019-09-26 18:19:41 +0000120 r'.*ninjalog_uploader_test\.py$',
Edward Lemurbbd70fd2019-09-26 18:19:41 +0000121 r'.*recipes_test\.py$',
Josip Sokcevic4940cc42021-10-05 23:55:34 +0000122 ])
Josip Sokcevic11c32b52021-11-19 01:10:50 +0000123 py2_only_tests = [
Josip Sokcevic11c32b52021-11-19 01:10:50 +0000124 'recipes_test.py',
125 ]
Vadim Shtayura7e50ee32018-07-26 17:43:51 +0000126
Takuto Ikuta381db682022-04-27 23:54:02 +0000127 py3_only_tests = [
128 'autoninja_test.py',
129 'ninjalog_uploader_test.py',
130 ]
Takuto Ikutaa6573312022-01-20 00:24:57 +0000131
Josip Sokcevic57c928c2021-11-02 22:56:28 +0000132 tests = input_api.canned_checks.GetUnitTestsInDirectory(
scottmg@chromium.org69ae86a2014-01-17 03:55:45 +0000133 input_api,
134 output_api,
135 'tests',
Josip Sokcevicc6aa1512020-08-25 23:47:13 +0000136 files_to_check=test_to_run_list,
Takuto Ikutaa6573312022-01-20 00:24:57 +0000137 files_to_skip=tests_to_skip_list + py2_only_tests + py3_only_tests,
Josip Sokcevic11c32b52021-11-19 01:10:50 +0000138 run_on_python3=True)
Robert Iannucci7d47eb52017-12-06 12:18:18 -0800139
Josip Sokcevic11c32b52021-11-19 01:10:50 +0000140 # TODO: once py3 compatbile, remove those tests
Takuto Ikutaa6573312022-01-20 00:24:57 +0000141 tests.extend(
142 input_api.canned_checks.GetUnitTestsInDirectory(
143 input_api,
144 output_api,
145 'tests',
146 files_to_check=py2_only_tests,
147 files_to_skip=tests_to_skip_list,
148 run_on_python3=False))
149
150 # TODO: use this for all tests when py2 support is dropped.
151 tests.extend(
152 input_api.canned_checks.GetUnitTestsInDirectory(
153 input_api,
154 output_api,
155 'tests',
156 files_to_check=py3_only_tests,
157 files_to_skip=tests_to_skip_list,
158 run_on_python3=True,
159 run_on_python2=False))
Josip Sokcevic4940cc42021-10-05 23:55:34 +0000160
Josip Sokcevic57c928c2021-11-02 22:56:28 +0000161 return input_api.RunTests(tests)
162
163
164def CheckCIPDManifest(input_api, output_api):
Robert Iannucci7d47eb52017-12-06 12:18:18 -0800165 # Validate CIPD manifests.
166 root = input_api.os_path.normpath(
167 input_api.os_path.abspath(input_api.PresubmitLocalPath()))
168 rel_file = lambda rel: input_api.os_path.join(root, rel)
169 cipd_manifests = set(rel_file(input_api.os_path.join(*x)) for x in (
170 ('cipd_manifest.txt',),
Robert Iannucciee67b972019-12-10 22:46:56 +0000171 ('bootstrap', 'manifest.txt'),
172 ('bootstrap', 'manifest_bleeding_edge.txt'),
Robert Iannucci7d47eb52017-12-06 12:18:18 -0800173
174 # Also generate a file for the cipd client itself.
175 ('cipd_client_version',),
176 ))
177 affected_manifests = input_api.AffectedFiles(
178 include_deletes=False,
179 file_filter=lambda x:
180 input_api.os_path.normpath(x.AbsoluteLocalPath()) in cipd_manifests)
Josip Sokcevic57c928c2021-11-02 22:56:28 +0000181 tests = []
Robert Iannucci7d47eb52017-12-06 12:18:18 -0800182 for path in affected_manifests:
183 path = path.AbsoluteLocalPath()
184 if path.endswith('.txt'):
185 tests.append(input_api.canned_checks.CheckCIPDManifest(
186 input_api, output_api, path=path))
187 else:
188 pkg = 'infra/tools/cipd/${platform}'
189 ver = input_api.ReadFile(path)
190 tests.append(input_api.canned_checks.CheckCIPDManifest(
Vadim Shtayura09098852018-06-21 22:50:30 +0000191 input_api, output_api,
192 content=CIPD_CLIENT_ENSURE_FILE_TEMPLATE % (pkg, ver)))
Vadim Shtayuraf9b48452018-09-18 00:22:26 +0000193 tests.append(input_api.canned_checks.CheckCIPDClientDigests(
194 input_api, output_api, client_version_file=path))
Robert Iannucci7d47eb52017-12-06 12:18:18 -0800195
Josip Sokcevic57c928c2021-11-02 22:56:28 +0000196 return input_api.RunTests(tests)
maruel@chromium.org2a74d372011-03-29 19:05:50 +0000197
198
Josip Sokcevic57c928c2021-11-02 22:56:28 +0000199def CheckOwnersFormat(input_api, output_api):
200 return input_api.canned_checks.CheckOwnersFormat(input_api, output_api)
maruel@chromium.orge94aedc2010-12-13 21:11:30 +0000201
202
Josip Sokcevic57c928c2021-11-02 22:56:28 +0000203def CheckOwnersOnUpload(input_api, output_api):
204 return input_api.canned_checks.CheckOwners(input_api, output_api,
205 allow_tbr=False)
206
207def CheckDoNotSubmitOnCommit(input_api, output_api):
208 return input_api.canned_checks.CheckDoNotSubmit(input_api, output_api)
Garrett Beatyf41670f2022-08-29 22:26:42 +0000209
210
211def CheckPatchFormatted(input_api, output_api):
212 # TODO(https://crbug.com/979330) If clang-format is fixed for non-chromium
213 # repos, remove check_clang_format=False so that proto files can be formatted
214 return input_api.canned_checks.CheckPatchFormatted(input_api,
215 output_api,
216 check_clang_format=False)