blob: 0c9b71edab82493cc65dea07c1e1ebc0a230adc0 [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
szager@chromium.org34071032014-03-18 17:23:48 +000011import fnmatch
12import os
13
dpranke@chromium.org627ea672011-03-11 23:29:03 +000014
Vadim Shtayura09098852018-06-21 22:50:30 +000015# CIPD ensure manifest for checking CIPD client itself.
16CIPD_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 Shtayura333617b2018-08-07 20:46:38 +000021$VerifiedPlatform linux-arm64 linux-armv6l
22$VerifiedPlatform linux-mips64 linux-mips64le linux-mipsle
Robert Iannucci7d47eb52017-12-06 12:18:18 -080023
24%s %s
25'''
26
27
agablef39c3332016-09-26 09:35:42 -070028def 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.org55cb27e2016-05-16 17:54:40 +000035 ]
agablef39c3332016-09-26 09:35:42 -070036 black_list = list(input_api.DEFAULT_BLACK_LIST)
szager@chromium.org34071032014-03-18 17:23:48 +000037 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.orgff9a2172012-04-24 16:55:32 +000047 disabled_warnings = [
48 'R0401', # Cyclic import
49 'W0613', # Unused argument
50 ]
agablef39c3332016-09-26 09:35:42 -070051 return input_api.canned_checks.GetPylint(
scottmg@chromium.org69ae86a2014-01-17 03:55:45 +000052 input_api,
53 output_api,
agablef39c3332016-09-26 09:35:42 -070054 white_list=white_list,
scottmg@chromium.org69ae86a2014-01-17 03:55:45 +000055 black_list=black_list,
56 disabled_warnings=disabled_warnings)
agablef39c3332016-09-26 09:35:42 -070057
58
Edward Lemura877ee62019-09-03 20:23:17 +000059def CommonChecks(input_api, output_api, tests_to_black_list, run_on_python3):
agablef39c3332016-09-26 09:35:42 -070060 results = []
61 results.extend(input_api.canned_checks.CheckOwners(input_api, output_api))
Edward Lesmescb62e482018-04-19 18:29:35 -040062 results.extend(input_api.canned_checks.CheckOwnersFormat(
63 input_api, output_api))
Aaron Gable1318a302019-05-21 21:46:07 +000064 results.extend(input_api.canned_checks.CheckJsonParses(
65 input_api, output_api))
Vadim Shtayura7e50ee32018-07-26 17:43:51 +000066
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 Lemurbbd70fd2019-09-26 18:19:41 +000071 tests_to_black_list = [
72 r'.*auth_test\.py$',
Edward Lemurbbd70fd2019-09-26 18:19:41 +000073 r'.*git_cl_test\.py$',
74 r'.*git_common_test\.py$',
Edward Lemurbbd70fd2019-09-26 18:19:41 +000075 r'.*git_hyper_blame_test\.py$',
Edward Lemurb800fde2020-01-10 23:04:44 +000076 r'.*git_map_test\.py$',
Edward Lemurbbd70fd2019-09-26 18:19:41 +000077 r'.*git_number_test\.py$',
78 r'.*git_rebase_update_test\.py$',
79 r'.*ninjalog_uploader_test\.py$',
Edward Lemurbbd70fd2019-09-26 18:19:41 +000080 r'.*recipes_test\.py$',
81 r'.*roll_dep_test\.py$',
Edward Lemurbbd70fd2019-09-26 18:19:41 +000082 r'.*subprocess2_test\.py$',
Edward Lemurbbd70fd2019-09-26 18:19:41 +000083 ]
Vadim Shtayura7e50ee32018-07-26 17:43:51 +000084
scottmg@chromium.org69ae86a2014-01-17 03:55:45 +000085 # TODO(maruel): Make sure at least one file is modified first.
86 # TODO(maruel): If only tests are modified, only run them.
agablef39c3332016-09-26 09:35:42 -070087 tests = DepotToolsPylint(input_api, output_api)
Vadim Shtayura7e50ee32018-07-26 17:43:51 +000088 tests.extend(input_api.canned_checks.GetUnitTestsInDirectory(
scottmg@chromium.org69ae86a2014-01-17 03:55:45 +000089 input_api,
90 output_api,
91 'tests',
Vadim Shtayura7e50ee32018-07-26 17:43:51 +000092 whitelist=tests_to_white_list,
Edward Lemura877ee62019-09-03 20:23:17 +000093 blacklist=tests_to_black_list,
94 run_on_python3=run_on_python3))
Robert Iannucci7d47eb52017-12-06 12:18:18 -080095
96 # Validate CIPD manifests.
97 root = input_api.os_path.normpath(
98 input_api.os_path.abspath(input_api.PresubmitLocalPath()))
99 rel_file = lambda rel: input_api.os_path.join(root, rel)
100 cipd_manifests = set(rel_file(input_api.os_path.join(*x)) for x in (
101 ('cipd_manifest.txt',),
Robert Iannucciee67b972019-12-10 22:46:56 +0000102 ('bootstrap', 'manifest.txt'),
103 ('bootstrap', 'manifest_bleeding_edge.txt'),
Robert Iannucci7d47eb52017-12-06 12:18:18 -0800104
105 # Also generate a file for the cipd client itself.
106 ('cipd_client_version',),
107 ))
108 affected_manifests = input_api.AffectedFiles(
109 include_deletes=False,
110 file_filter=lambda x:
111 input_api.os_path.normpath(x.AbsoluteLocalPath()) in cipd_manifests)
112 for path in affected_manifests:
113 path = path.AbsoluteLocalPath()
114 if path.endswith('.txt'):
115 tests.append(input_api.canned_checks.CheckCIPDManifest(
116 input_api, output_api, path=path))
117 else:
118 pkg = 'infra/tools/cipd/${platform}'
119 ver = input_api.ReadFile(path)
120 tests.append(input_api.canned_checks.CheckCIPDManifest(
Vadim Shtayura09098852018-06-21 22:50:30 +0000121 input_api, output_api,
122 content=CIPD_CLIENT_ENSURE_FILE_TEMPLATE % (pkg, ver)))
Vadim Shtayuraf9b48452018-09-18 00:22:26 +0000123 tests.append(input_api.canned_checks.CheckCIPDClientDigests(
124 input_api, output_api, client_version_file=path))
Robert Iannucci7d47eb52017-12-06 12:18:18 -0800125
scottmg@chromium.org69ae86a2014-01-17 03:55:45 +0000126 results.extend(input_api.RunTests(tests))
maruel@chromium.org3c9f7ca2011-04-01 14:52:38 +0000127 return results
maruel@chromium.org2a74d372011-03-29 19:05:50 +0000128
129
maruel@chromium.orge94aedc2010-12-13 21:11:30 +0000130def CheckChangeOnUpload(input_api, output_api):
maruel@chromium.orgd52417c2011-09-02 20:13:17 +0000131 # Do not run integration tests on upload since they are way too slow.
132 tests_to_black_list = [
133 r'^checkout_test\.py$',
Vadim Shtayura7e50ee32018-07-26 17:43:51 +0000134 r'^cipd_bootstrap_test\.py$',
maruel@chromium.orgd52417c2011-09-02 20:13:17 +0000135 r'^gclient_smoketest\.py$',
Vadim Shtayura7e50ee32018-07-26 17:43:51 +0000136 ]
Edward Lemura877ee62019-09-03 20:23:17 +0000137 # TODO(ehmaldonado): Run Python 3 tests on upload once Python 3 is
138 # bootstrapped on Linux and Mac.
139 return CommonChecks(input_api, output_api, tests_to_black_list, False)
maruel@chromium.orge94aedc2010-12-13 21:11:30 +0000140
141
maruel@chromium.orgce30ef72009-05-25 15:20:42 +0000142def CheckChangeOnCommit(input_api, output_api):
143 output = []
Edward Lemura877ee62019-09-03 20:23:17 +0000144 output.extend(CommonChecks(input_api, output_api, [], True))
maruel@chromium.orge94aedc2010-12-13 21:11:30 +0000145 output.extend(input_api.canned_checks.CheckDoNotSubmit(
146 input_api,
147 output_api))
maruel@chromium.org7b305e82009-05-19 18:24:20 +0000148 return output