chrisha@chromium.org | c920ad2 | 2012-02-02 18:26:04 +0000 | [diff] [blame] | 1 | # Copyright (c) 2012 The Chromium Authors. All rights reserved. |
maruel@chromium.org | 3d23524 | 2009-05-15 12:40:48 +0000 | [diff] [blame] | 2 | # 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 | |
| 7 | See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts for |
maruel@chromium.org | 2a74d37 | 2011-03-29 19:05:50 +0000 | [diff] [blame] | 8 | details on the presubmit API built into depot_tools. |
maruel@chromium.org | 3d23524 | 2009-05-15 12:40:48 +0000 | [diff] [blame] | 9 | """ |
| 10 | |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 11 | PRESUBMIT_VERSION = '2.0.0' |
| 12 | |
szager@chromium.org | 3407103 | 2014-03-18 17:23:48 +0000 | [diff] [blame] | 13 | import fnmatch |
| 14 | import os |
Edward Lemur | 595eb19 | 2020-03-12 21:53:52 +0000 | [diff] [blame] | 15 | import sys |
szager@chromium.org | 3407103 | 2014-03-18 17:23:48 +0000 | [diff] [blame] | 16 | |
Dirk Pranke | 61bf6e8 | 2021-04-23 00:50:21 +0000 | [diff] [blame] | 17 | # 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.org | 627ea67 | 2011-03-11 23:29:03 +0000 | [diff] [blame] | 23 | |
Vadim Shtayura | 0909885 | 2018-06-21 22:50:30 +0000 | [diff] [blame] | 24 | # CIPD ensure manifest for checking CIPD client itself. |
| 25 | CIPD_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 Shtayura | 333617b | 2018-08-07 20:46:38 +0000 | [diff] [blame] | 30 | $VerifiedPlatform linux-arm64 linux-armv6l |
| 31 | $VerifiedPlatform linux-mips64 linux-mips64le linux-mipsle |
Robert Iannucci | 7d47eb5 | 2017-12-06 12:18:18 -0800 | [diff] [blame] | 32 | |
| 33 | %s %s |
| 34 | ''' |
| 35 | |
Edward Lemur | 595eb19 | 2020-03-12 21:53:52 +0000 | [diff] [blame] | 36 | # Timeout for a test to be executed. |
Edward Lesmes | ae3586b | 2020-03-23 21:21:14 +0000 | [diff] [blame] | 37 | TEST_TIMEOUT_S = 330 # 5m 30s |
Edward Lemur | 595eb19 | 2020-03-12 21:53:52 +0000 | [diff] [blame] | 38 | |
| 39 | |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 40 | def CheckPylint(input_api, output_api): |
agable | f39c333 | 2016-09-26 09:35:42 -0700 | [diff] [blame] | 41 | """Gather all the pylint logic into one place to make it self-contained.""" |
Ayu Ishii | 81a9074 | 2020-07-07 19:53:00 +0000 | [diff] [blame] | 42 | files_to_check = [ |
agable | f39c333 | 2016-09-26 09:35:42 -0700 | [diff] [blame] | 43 | r'^[^/]*\.py$', |
| 44 | r'^testing_support/[^/]*\.py$', |
| 45 | r'^tests/[^/]*\.py$', |
| 46 | r'^recipe_modules/.*\.py$', # Allow recursive search in recipe modules. |
tandrii@chromium.org | 55cb27e | 2016-05-16 17:54:40 +0000 | [diff] [blame] | 47 | ] |
local_bot | 9af33fa | 2020-07-09 03:26:20 +0000 | [diff] [blame] | 48 | files_to_skip = list(input_api.DEFAULT_FILES_TO_SKIP) |
szager@chromium.org | 3407103 | 2014-03-18 17:23:48 +0000 | [diff] [blame] | 49 | if os.path.exists('.gitignore'): |
| 50 | with open('.gitignore') as fh: |
| 51 | lines = [l.strip() for l in fh.readlines()] |
Ayu Ishii | 81a9074 | 2020-07-07 19:53:00 +0000 | [diff] [blame] | 52 | files_to_skip.extend([fnmatch.translate(l) for l in lines if |
szager@chromium.org | 3407103 | 2014-03-18 17:23:48 +0000 | [diff] [blame] | 53 | 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 Ishii | 81a9074 | 2020-07-07 19:53:00 +0000 | [diff] [blame] | 57 | files_to_skip.extend([fnmatch.translate(l) for l in lines if |
szager@chromium.org | 3407103 | 2014-03-18 17:23:48 +0000 | [diff] [blame] | 58 | l and not l.startswith('#')]) |
maruel@chromium.org | ff9a217 | 2012-04-24 16:55:32 +0000 | [diff] [blame] | 59 | disabled_warnings = [ |
Aravind Vasudevan | c5f0cbb | 2022-01-24 23:56:57 +0000 | [diff] [blame] | 60 | '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.org | ff9a217 | 2012-04-24 16:55:32 +0000 | [diff] [blame] | 73 | ] |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 74 | return input_api.RunTests(input_api.canned_checks.GetPylint( |
scottmg@chromium.org | 69ae86a | 2014-01-17 03:55:45 +0000 | [diff] [blame] | 75 | input_api, |
| 76 | output_api, |
local_bot | 9af33fa | 2020-07-09 03:26:20 +0000 | [diff] [blame] | 77 | files_to_check=files_to_check, |
| 78 | files_to_skip=files_to_skip, |
Aravind Vasudevan | c5f0cbb | 2022-01-24 23:56:57 +0000 | [diff] [blame] | 79 | disabled_warnings=disabled_warnings, |
| 80 | version='2.7'), parallel=False) |
agable | f39c333 | 2016-09-26 09:35:42 -0700 | [diff] [blame] | 81 | |
| 82 | |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 83 | def CheckRecipes(input_api, output_api): |
Josip Sokcevic | 9eee47a | 2021-07-13 17:38:53 +0000 | [diff] [blame] | 84 | file_filter = lambda x: x.LocalPath() == 'infra/config/recipes.cfg' |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 85 | return input_api.canned_checks.CheckJsonParses(input_api, output_api, |
| 86 | file_filter=file_filter) |
Dirk Pranke | 61bf6e8 | 2021-04-23 00:50:21 +0000 | [diff] [blame] | 87 | |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 88 | def CheckPythonVersion(input_api, output_api): |
Dirk Pranke | 61bf6e8 | 2021-04-23 00:50:21 +0000 | [diff] [blame] | 89 | # The tests here are assuming this is not defined, so raise an error |
| 90 | # if it is. |
| 91 | if 'USE_PYTHON3' in globals(): |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 92 | return [output_api.PresubmitError( |
Dirk Pranke | 61bf6e8 | 2021-04-23 00:50:21 +0000 | [diff] [blame] | 93 | 'USE_PYTHON3 is defined; update the tests in //PRESUBMIT.py and ' |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 94 | '//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 Pranke | 61bf6e8 | 2021-04-23 00:50:21 +0000 | [diff] [blame] | 99 | |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 100 | |
| 101 | def CheckJsonFiles(input_api, output_api): |
| 102 | return input_api.canned_checks.CheckJsonParses( |
| 103 | input_api, output_api) |
| 104 | |
| 105 | |
| 106 | def 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 Shtayura | 7e50ee3 | 2018-07-26 17:43:51 +0000 | [diff] [blame] | 109 | |
| 110 | # Run only selected tests on Windows. |
Ayu Ishii | 81a9074 | 2020-07-07 19:53:00 +0000 | [diff] [blame] | 111 | test_to_run_list = [r'.*test\.py$'] |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 112 | tests_to_skip_list = [] |
Vadim Shtayura | 7e50ee3 | 2018-07-26 17:43:51 +0000 | [diff] [blame] | 113 | if input_api.platform.startswith(('cygwin', 'win32')): |
| 114 | print('Warning: skipping most unit tests on Windows') |
Josip Sokcevic | 4940cc4 | 2021-10-05 23:55:34 +0000 | [diff] [blame] | 115 | tests_to_skip_list.extend([ |
Edward Lemur | bbd70fd | 2019-09-26 18:19:41 +0000 | [diff] [blame] | 116 | r'.*auth_test\.py$', |
Edward Lemur | bbd70fd | 2019-09-26 18:19:41 +0000 | [diff] [blame] | 117 | r'.*git_common_test\.py$', |
Edward Lemur | bbd70fd | 2019-09-26 18:19:41 +0000 | [diff] [blame] | 118 | r'.*git_hyper_blame_test\.py$', |
Edward Lemur | b800fde | 2020-01-10 23:04:44 +0000 | [diff] [blame] | 119 | r'.*git_map_test\.py$', |
Edward Lemur | bbd70fd | 2019-09-26 18:19:41 +0000 | [diff] [blame] | 120 | r'.*ninjalog_uploader_test\.py$', |
Edward Lemur | bbd70fd | 2019-09-26 18:19:41 +0000 | [diff] [blame] | 121 | r'.*recipes_test\.py$', |
Josip Sokcevic | 4940cc4 | 2021-10-05 23:55:34 +0000 | [diff] [blame] | 122 | ]) |
Josip Sokcevic | 11c32b5 | 2021-11-19 01:10:50 +0000 | [diff] [blame] | 123 | py2_only_tests = [ |
Josip Sokcevic | 11c32b5 | 2021-11-19 01:10:50 +0000 | [diff] [blame] | 124 | 'recipes_test.py', |
| 125 | ] |
Vadim Shtayura | 7e50ee3 | 2018-07-26 17:43:51 +0000 | [diff] [blame] | 126 | |
Takuto Ikuta | a657331 | 2022-01-20 00:24:57 +0000 | [diff] [blame] | 127 | py3_only_tests = ['ninjalog_uploader_test.py'] |
| 128 | |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 129 | tests = input_api.canned_checks.GetUnitTestsInDirectory( |
scottmg@chromium.org | 69ae86a | 2014-01-17 03:55:45 +0000 | [diff] [blame] | 130 | input_api, |
| 131 | output_api, |
| 132 | 'tests', |
Josip Sokcevic | c6aa151 | 2020-08-25 23:47:13 +0000 | [diff] [blame] | 133 | files_to_check=test_to_run_list, |
Takuto Ikuta | a657331 | 2022-01-20 00:24:57 +0000 | [diff] [blame] | 134 | files_to_skip=tests_to_skip_list + py2_only_tests + py3_only_tests, |
Josip Sokcevic | 11c32b5 | 2021-11-19 01:10:50 +0000 | [diff] [blame] | 135 | run_on_python3=True) |
Robert Iannucci | 7d47eb5 | 2017-12-06 12:18:18 -0800 | [diff] [blame] | 136 | |
Josip Sokcevic | 11c32b5 | 2021-11-19 01:10:50 +0000 | [diff] [blame] | 137 | # TODO: once py3 compatbile, remove those tests |
Takuto Ikuta | a657331 | 2022-01-20 00:24:57 +0000 | [diff] [blame] | 138 | tests.extend( |
| 139 | input_api.canned_checks.GetUnitTestsInDirectory( |
| 140 | input_api, |
| 141 | output_api, |
| 142 | 'tests', |
| 143 | files_to_check=py2_only_tests, |
| 144 | files_to_skip=tests_to_skip_list, |
| 145 | run_on_python3=False)) |
| 146 | |
| 147 | # TODO: use this for all tests when py2 support is dropped. |
| 148 | tests.extend( |
| 149 | input_api.canned_checks.GetUnitTestsInDirectory( |
| 150 | input_api, |
| 151 | output_api, |
| 152 | 'tests', |
| 153 | files_to_check=py3_only_tests, |
| 154 | files_to_skip=tests_to_skip_list, |
| 155 | run_on_python3=True, |
| 156 | run_on_python2=False)) |
Josip Sokcevic | 4940cc4 | 2021-10-05 23:55:34 +0000 | [diff] [blame] | 157 | |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 158 | return input_api.RunTests(tests) |
| 159 | |
| 160 | |
| 161 | def CheckCIPDManifest(input_api, output_api): |
Robert Iannucci | 7d47eb5 | 2017-12-06 12:18:18 -0800 | [diff] [blame] | 162 | # Validate CIPD manifests. |
| 163 | root = input_api.os_path.normpath( |
| 164 | input_api.os_path.abspath(input_api.PresubmitLocalPath())) |
| 165 | rel_file = lambda rel: input_api.os_path.join(root, rel) |
| 166 | cipd_manifests = set(rel_file(input_api.os_path.join(*x)) for x in ( |
| 167 | ('cipd_manifest.txt',), |
Robert Iannucci | ee67b97 | 2019-12-10 22:46:56 +0000 | [diff] [blame] | 168 | ('bootstrap', 'manifest.txt'), |
| 169 | ('bootstrap', 'manifest_bleeding_edge.txt'), |
Robert Iannucci | 7d47eb5 | 2017-12-06 12:18:18 -0800 | [diff] [blame] | 170 | |
| 171 | # Also generate a file for the cipd client itself. |
| 172 | ('cipd_client_version',), |
| 173 | )) |
| 174 | affected_manifests = input_api.AffectedFiles( |
| 175 | include_deletes=False, |
| 176 | file_filter=lambda x: |
| 177 | input_api.os_path.normpath(x.AbsoluteLocalPath()) in cipd_manifests) |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 178 | tests = [] |
Robert Iannucci | 7d47eb5 | 2017-12-06 12:18:18 -0800 | [diff] [blame] | 179 | for path in affected_manifests: |
| 180 | path = path.AbsoluteLocalPath() |
| 181 | if path.endswith('.txt'): |
| 182 | tests.append(input_api.canned_checks.CheckCIPDManifest( |
| 183 | input_api, output_api, path=path)) |
| 184 | else: |
| 185 | pkg = 'infra/tools/cipd/${platform}' |
| 186 | ver = input_api.ReadFile(path) |
| 187 | tests.append(input_api.canned_checks.CheckCIPDManifest( |
Vadim Shtayura | 0909885 | 2018-06-21 22:50:30 +0000 | [diff] [blame] | 188 | input_api, output_api, |
| 189 | content=CIPD_CLIENT_ENSURE_FILE_TEMPLATE % (pkg, ver))) |
Vadim Shtayura | f9b4845 | 2018-09-18 00:22:26 +0000 | [diff] [blame] | 190 | tests.append(input_api.canned_checks.CheckCIPDClientDigests( |
| 191 | input_api, output_api, client_version_file=path)) |
Robert Iannucci | 7d47eb5 | 2017-12-06 12:18:18 -0800 | [diff] [blame] | 192 | |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 193 | return input_api.RunTests(tests) |
maruel@chromium.org | 2a74d37 | 2011-03-29 19:05:50 +0000 | [diff] [blame] | 194 | |
| 195 | |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 196 | def CheckOwnersFormat(input_api, output_api): |
| 197 | return input_api.canned_checks.CheckOwnersFormat(input_api, output_api) |
maruel@chromium.org | e94aedc | 2010-12-13 21:11:30 +0000 | [diff] [blame] | 198 | |
| 199 | |
Josip Sokcevic | 57c928c | 2021-11-02 22:56:28 +0000 | [diff] [blame] | 200 | def CheckOwnersOnUpload(input_api, output_api): |
| 201 | return input_api.canned_checks.CheckOwners(input_api, output_api, |
| 202 | allow_tbr=False) |
| 203 | |
| 204 | def CheckDoNotSubmitOnCommit(input_api, output_api): |
| 205 | return input_api.canned_checks.CheckDoNotSubmit(input_api, output_api) |