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 | |
szager@chromium.org | 3407103 | 2014-03-18 17:23:48 +0000 | [diff] [blame] | 11 | import fnmatch |
| 12 | import os |
| 13 | |
dpranke@chromium.org | 627ea67 | 2011-03-11 23:29:03 +0000 | [diff] [blame] | 14 | |
Vadim Shtayura | 0909885 | 2018-06-21 22:50:30 +0000 | [diff] [blame] | 15 | # CIPD ensure manifest for checking CIPD client itself. |
| 16 | CIPD_CLIENT_ENSURE_FILE_TEMPLATE = r''' |
| 17 | # Full supported. |
| 18 | $VerifiedPlatform linux-amd64 mac-amd64 windows-amd64 windows-386 |
| 19 | # Best effort support. |
| 20 | $VerifiedPlatform linux-386 linux-ppc64 linux-ppc64le linux-s390x |
Vadim Shtayura | 333617b | 2018-08-07 20:46:38 +0000 | [diff] [blame] | 21 | $VerifiedPlatform linux-arm64 linux-armv6l |
| 22 | $VerifiedPlatform linux-mips64 linux-mips64le linux-mipsle |
Robert Iannucci | 7d47eb5 | 2017-12-06 12:18:18 -0800 | [diff] [blame] | 23 | |
| 24 | %s %s |
| 25 | ''' |
| 26 | |
| 27 | |
agable | f39c333 | 2016-09-26 09:35:42 -0700 | [diff] [blame] | 28 | def DepotToolsPylint(input_api, output_api): |
| 29 | """Gather all the pylint logic into one place to make it self-contained.""" |
| 30 | white_list = [ |
| 31 | r'^[^/]*\.py$', |
| 32 | r'^testing_support/[^/]*\.py$', |
| 33 | r'^tests/[^/]*\.py$', |
| 34 | r'^recipe_modules/.*\.py$', # Allow recursive search in recipe modules. |
tandrii@chromium.org | 55cb27e | 2016-05-16 17:54:40 +0000 | [diff] [blame] | 35 | ] |
agable | f39c333 | 2016-09-26 09:35:42 -0700 | [diff] [blame] | 36 | black_list = list(input_api.DEFAULT_BLACK_LIST) |
szager@chromium.org | 3407103 | 2014-03-18 17:23:48 +0000 | [diff] [blame] | 37 | if os.path.exists('.gitignore'): |
| 38 | with open('.gitignore') as fh: |
| 39 | lines = [l.strip() for l in fh.readlines()] |
| 40 | black_list.extend([fnmatch.translate(l) for l in lines if |
| 41 | l and not l.startswith('#')]) |
| 42 | if os.path.exists('.git/info/exclude'): |
| 43 | with open('.git/info/exclude') as fh: |
| 44 | lines = [l.strip() for l in fh.readlines()] |
| 45 | black_list.extend([fnmatch.translate(l) for l in lines if |
| 46 | l and not l.startswith('#')]) |
maruel@chromium.org | ff9a217 | 2012-04-24 16:55:32 +0000 | [diff] [blame] | 47 | disabled_warnings = [ |
| 48 | 'R0401', # Cyclic import |
| 49 | 'W0613', # Unused argument |
| 50 | ] |
agable | f39c333 | 2016-09-26 09:35:42 -0700 | [diff] [blame] | 51 | return input_api.canned_checks.GetPylint( |
scottmg@chromium.org | 69ae86a | 2014-01-17 03:55:45 +0000 | [diff] [blame] | 52 | input_api, |
| 53 | output_api, |
agable | f39c333 | 2016-09-26 09:35:42 -0700 | [diff] [blame] | 54 | white_list=white_list, |
scottmg@chromium.org | 69ae86a | 2014-01-17 03:55:45 +0000 | [diff] [blame] | 55 | black_list=black_list, |
| 56 | disabled_warnings=disabled_warnings) |
agable | f39c333 | 2016-09-26 09:35:42 -0700 | [diff] [blame] | 57 | |
| 58 | |
Edward Lemur | a877ee6 | 2019-09-03 20:23:17 +0000 | [diff] [blame] | 59 | def CommonChecks(input_api, output_api, tests_to_black_list, run_on_python3): |
agable | f39c333 | 2016-09-26 09:35:42 -0700 | [diff] [blame] | 60 | results = [] |
| 61 | results.extend(input_api.canned_checks.CheckOwners(input_api, output_api)) |
Edward Lesmes | cb62e48 | 2018-04-19 18:29:35 -0400 | [diff] [blame] | 62 | results.extend(input_api.canned_checks.CheckOwnersFormat( |
| 63 | input_api, output_api)) |
Aaron Gable | 1318a30 | 2019-05-21 21:46:07 +0000 | [diff] [blame] | 64 | results.extend(input_api.canned_checks.CheckJsonParses( |
| 65 | input_api, output_api)) |
Vadim Shtayura | 7e50ee3 | 2018-07-26 17:43:51 +0000 | [diff] [blame] | 66 | |
| 67 | # Run only selected tests on Windows. |
| 68 | tests_to_white_list = [r'.*test\.py$'] |
| 69 | if input_api.platform.startswith(('cygwin', 'win32')): |
| 70 | print('Warning: skipping most unit tests on Windows') |
Edward Lemur | bbd70fd | 2019-09-26 18:19:41 +0000 | [diff] [blame] | 71 | tests_to_black_list = [ |
| 72 | r'.*auth_test\.py$', |
Edward Lemur | bbd70fd | 2019-09-26 18:19:41 +0000 | [diff] [blame] | 73 | r'.*git_cl_test\.py$', |
| 74 | r'.*git_common_test\.py$', |
Edward Lemur | bbd70fd | 2019-09-26 18:19:41 +0000 | [diff] [blame] | 75 | r'.*git_hyper_blame_test\.py$', |
Edward Lemur | b800fde | 2020-01-10 23:04:44 +0000 | [diff] [blame^] | 76 | r'.*git_map_test\.py$', |
Edward Lemur | bbd70fd | 2019-09-26 18:19:41 +0000 | [diff] [blame] | 77 | r'.*git_number_test\.py$', |
| 78 | r'.*git_rebase_update_test\.py$', |
| 79 | r'.*ninjalog_uploader_test\.py$', |
Edward Lemur | bbd70fd | 2019-09-26 18:19:41 +0000 | [diff] [blame] | 80 | r'.*recipes_test\.py$', |
| 81 | r'.*roll_dep_test\.py$', |
| 82 | r'.*scm_unittest\.py$', |
| 83 | r'.*subprocess2_test\.py$', |
Edward Lemur | bbd70fd | 2019-09-26 18:19:41 +0000 | [diff] [blame] | 84 | ] |
Vadim Shtayura | 7e50ee3 | 2018-07-26 17:43:51 +0000 | [diff] [blame] | 85 | |
scottmg@chromium.org | 69ae86a | 2014-01-17 03:55:45 +0000 | [diff] [blame] | 86 | # TODO(maruel): Make sure at least one file is modified first. |
| 87 | # TODO(maruel): If only tests are modified, only run them. |
agable | f39c333 | 2016-09-26 09:35:42 -0700 | [diff] [blame] | 88 | tests = DepotToolsPylint(input_api, output_api) |
Vadim Shtayura | 7e50ee3 | 2018-07-26 17:43:51 +0000 | [diff] [blame] | 89 | tests.extend(input_api.canned_checks.GetUnitTestsInDirectory( |
scottmg@chromium.org | 69ae86a | 2014-01-17 03:55:45 +0000 | [diff] [blame] | 90 | input_api, |
| 91 | output_api, |
| 92 | 'tests', |
Vadim Shtayura | 7e50ee3 | 2018-07-26 17:43:51 +0000 | [diff] [blame] | 93 | whitelist=tests_to_white_list, |
Edward Lemur | a877ee6 | 2019-09-03 20:23:17 +0000 | [diff] [blame] | 94 | blacklist=tests_to_black_list, |
| 95 | run_on_python3=run_on_python3)) |
Robert Iannucci | 7d47eb5 | 2017-12-06 12:18:18 -0800 | [diff] [blame] | 96 | |
| 97 | # Validate CIPD manifests. |
| 98 | root = input_api.os_path.normpath( |
| 99 | input_api.os_path.abspath(input_api.PresubmitLocalPath())) |
| 100 | rel_file = lambda rel: input_api.os_path.join(root, rel) |
| 101 | cipd_manifests = set(rel_file(input_api.os_path.join(*x)) for x in ( |
| 102 | ('cipd_manifest.txt',), |
Robert Iannucci | ee67b97 | 2019-12-10 22:46:56 +0000 | [diff] [blame] | 103 | ('bootstrap', 'manifest.txt'), |
| 104 | ('bootstrap', 'manifest_bleeding_edge.txt'), |
Robert Iannucci | 7d47eb5 | 2017-12-06 12:18:18 -0800 | [diff] [blame] | 105 | |
| 106 | # Also generate a file for the cipd client itself. |
| 107 | ('cipd_client_version',), |
| 108 | )) |
| 109 | affected_manifests = input_api.AffectedFiles( |
| 110 | include_deletes=False, |
| 111 | file_filter=lambda x: |
| 112 | input_api.os_path.normpath(x.AbsoluteLocalPath()) in cipd_manifests) |
| 113 | for path in affected_manifests: |
| 114 | path = path.AbsoluteLocalPath() |
| 115 | if path.endswith('.txt'): |
| 116 | tests.append(input_api.canned_checks.CheckCIPDManifest( |
| 117 | input_api, output_api, path=path)) |
| 118 | else: |
| 119 | pkg = 'infra/tools/cipd/${platform}' |
| 120 | ver = input_api.ReadFile(path) |
| 121 | tests.append(input_api.canned_checks.CheckCIPDManifest( |
Vadim Shtayura | 0909885 | 2018-06-21 22:50:30 +0000 | [diff] [blame] | 122 | input_api, output_api, |
| 123 | content=CIPD_CLIENT_ENSURE_FILE_TEMPLATE % (pkg, ver))) |
Vadim Shtayura | f9b4845 | 2018-09-18 00:22:26 +0000 | [diff] [blame] | 124 | tests.append(input_api.canned_checks.CheckCIPDClientDigests( |
| 125 | input_api, output_api, client_version_file=path)) |
Robert Iannucci | 7d47eb5 | 2017-12-06 12:18:18 -0800 | [diff] [blame] | 126 | |
scottmg@chromium.org | 69ae86a | 2014-01-17 03:55:45 +0000 | [diff] [blame] | 127 | results.extend(input_api.RunTests(tests)) |
maruel@chromium.org | 3c9f7ca | 2011-04-01 14:52:38 +0000 | [diff] [blame] | 128 | return results |
maruel@chromium.org | 2a74d37 | 2011-03-29 19:05:50 +0000 | [diff] [blame] | 129 | |
| 130 | |
maruel@chromium.org | e94aedc | 2010-12-13 21:11:30 +0000 | [diff] [blame] | 131 | def CheckChangeOnUpload(input_api, output_api): |
maruel@chromium.org | d52417c | 2011-09-02 20:13:17 +0000 | [diff] [blame] | 132 | # Do not run integration tests on upload since they are way too slow. |
| 133 | tests_to_black_list = [ |
| 134 | r'^checkout_test\.py$', |
Vadim Shtayura | 7e50ee3 | 2018-07-26 17:43:51 +0000 | [diff] [blame] | 135 | r'^cipd_bootstrap_test\.py$', |
maruel@chromium.org | d52417c | 2011-09-02 20:13:17 +0000 | [diff] [blame] | 136 | r'^gclient_smoketest\.py$', |
Vadim Shtayura | 7e50ee3 | 2018-07-26 17:43:51 +0000 | [diff] [blame] | 137 | ] |
Edward Lemur | a877ee6 | 2019-09-03 20:23:17 +0000 | [diff] [blame] | 138 | # TODO(ehmaldonado): Run Python 3 tests on upload once Python 3 is |
| 139 | # bootstrapped on Linux and Mac. |
| 140 | return CommonChecks(input_api, output_api, tests_to_black_list, False) |
maruel@chromium.org | e94aedc | 2010-12-13 21:11:30 +0000 | [diff] [blame] | 141 | |
| 142 | |
maruel@chromium.org | ce30ef7 | 2009-05-25 15:20:42 +0000 | [diff] [blame] | 143 | def CheckChangeOnCommit(input_api, output_api): |
| 144 | output = [] |
Edward Lemur | a877ee6 | 2019-09-03 20:23:17 +0000 | [diff] [blame] | 145 | output.extend(CommonChecks(input_api, output_api, [], True)) |
maruel@chromium.org | e94aedc | 2010-12-13 21:11:30 +0000 | [diff] [blame] | 146 | output.extend(input_api.canned_checks.CheckDoNotSubmit( |
| 147 | input_api, |
| 148 | output_api)) |
maruel@chromium.org | 7b305e8 | 2009-05-19 18:24:20 +0000 | [diff] [blame] | 149 | return output |