[email protected] | c920ad2 | 2012-02-02 18:26:04 | [diff] [blame] | 1 | # Copyright (c) 2012 The Chromium Authors. All rights reserved. |
[email protected] | 3d23524 | 2009-05-15 12:40:48 | [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 |
[email protected] | 2a74d37 | 2011-03-29 19:05:50 | [diff] [blame] | 8 | details on the presubmit API built into depot_tools. |
[email protected] | 3d23524 | 2009-05-15 12:40:48 | [diff] [blame] | 9 | """ |
| 10 | |
[email protected] | 3407103 | 2014-03-18 17:23:48 | [diff] [blame] | 11 | import fnmatch |
| 12 | import os |
| 13 | |
[email protected] | 627ea67 | 2011-03-11 23:29:03 | [diff] [blame] | 14 | |
[email protected] | d52417c | 2011-09-02 20:13:17 | [diff] [blame] | 15 | def CommonChecks(input_api, output_api, tests_to_black_list): |
[email protected] | 3c9f7ca | 2011-04-01 14:52:38 | [diff] [blame] | 16 | results = [] |
[email protected] | 3c9f7ca | 2011-04-01 14:52:38 | [diff] [blame] | 17 | results.extend(input_api.canned_checks.CheckOwners(input_api, output_api)) |
[email protected] | bf38a7e | 2010-12-14 18:15:54 | [diff] [blame] | 18 | black_list = list(input_api.DEFAULT_BLACK_LIST) + [ |
[email protected] | f7bb4cf | 2015-12-04 23:30:38 | [diff] [blame] | 19 | r'^\.recipe_deps[\/\\].*', |
[email protected] | 4c39694 | 2016-02-02 16:13:01 | [diff] [blame] | 20 | r'^infra[\/\\]\.recipe_deps[\/\\].*', |
[email protected] | 5d0dc43 | 2011-01-03 02:40:37 | [diff] [blame] | 21 | r'^cpplint\.py$', |
[email protected] | 63c2c1d | 2011-09-07 21:41:55 | [diff] [blame] | 22 | r'^cpplint_chromium\.py$', |
[email protected] | 7cc38e8 | 2015-01-08 22:28:34 | [diff] [blame] | 23 | r'^external_bin[\/\\].+', |
[email protected] | 154c36c | 2013-12-12 14:13:44 | [diff] [blame] | 24 | r'^python[0-9]*_bin[\/\\].+', |
[email protected] | f7bb4cf | 2015-12-04 23:30:38 | [diff] [blame] | 25 | r'^recipes\.py$', |
[email protected] | c920ad2 | 2012-02-02 18:26:04 | [diff] [blame] | 26 | r'^site-packages-py[0-9]\.[0-9][\/\\].+', |
[email protected] | ccbb781 | 2011-04-06 13:36:23 | [diff] [blame] | 27 | r'^svn_bin[\/\\].+', |
[email protected] | 150aa7b | 2015-11-09 19:20:55 | [diff] [blame] | 28 | r'^testing_support[\/\\]_rietveld[\/\\].+'] |
[email protected] | 3407103 | 2014-03-18 17:23:48 | [diff] [blame] | 29 | if os.path.exists('.gitignore'): |
| 30 | with open('.gitignore') as fh: |
| 31 | lines = [l.strip() for l in fh.readlines()] |
| 32 | black_list.extend([fnmatch.translate(l) for l in lines if |
| 33 | l and not l.startswith('#')]) |
| 34 | if os.path.exists('.git/info/exclude'): |
| 35 | with open('.git/info/exclude') as fh: |
| 36 | lines = [l.strip() for l in fh.readlines()] |
| 37 | black_list.extend([fnmatch.translate(l) for l in lines if |
| 38 | l and not l.startswith('#')]) |
[email protected] | ff9a217 | 2012-04-24 16:55:32 | [diff] [blame] | 39 | disabled_warnings = [ |
| 40 | 'R0401', # Cyclic import |
| 41 | 'W0613', # Unused argument |
| 42 | ] |
[email protected] | 69ae86a | 2014-01-17 03:55:45 | [diff] [blame] | 43 | pylint = input_api.canned_checks.GetPylint( |
| 44 | input_api, |
| 45 | output_api, |
| 46 | white_list=[r'.*\.py$'], |
| 47 | black_list=black_list, |
| 48 | disabled_warnings=disabled_warnings) |
| 49 | # TODO(maruel): Make sure at least one file is modified first. |
| 50 | # TODO(maruel): If only tests are modified, only run them. |
| 51 | unit_tests = input_api.canned_checks.GetUnitTestsInDirectory( |
| 52 | input_api, |
| 53 | output_api, |
| 54 | 'tests', |
| 55 | whitelist=[r'.*test\.py$'], |
| 56 | blacklist=tests_to_black_list) |
| 57 | tests = pylint |
| 58 | if not input_api.platform.startswith(('cygwin', 'win32')): |
| 59 | tests.extend(unit_tests) |
| 60 | else: |
| 61 | print('Warning: not running unit tests on Windows') |
| 62 | results.extend(input_api.RunTests(tests)) |
[email protected] | 3c9f7ca | 2011-04-01 14:52:38 | [diff] [blame] | 63 | return results |
[email protected] | 2a74d37 | 2011-03-29 19:05:50 | [diff] [blame] | 64 | |
| 65 | |
[email protected] | 899e1c1 | 2011-04-07 17:03:18 | [diff] [blame] | 66 | def RunGitClTests(input_api, output_api): |
[email protected] | 2a74d37 | 2011-03-29 19:05:50 | [diff] [blame] | 67 | """Run all the shells scripts in the directory test. |
| 68 | """ |
[email protected] | c98c0c5 | 2011-04-06 13:39:43 | [diff] [blame] | 69 | if input_api.platform == 'win32': |
| 70 | # Skip for now as long as the test scripts are bash scripts. |
| 71 | return [] |
| 72 | |
[email protected] | 2a74d37 | 2011-03-29 19:05:50 | [diff] [blame] | 73 | # First loads a local Rietveld instance. |
| 74 | import sys |
| 75 | old_sys_path = sys.path |
| 76 | try: |
| 77 | sys.path = [input_api.PresubmitLocalPath()] + sys.path |
[email protected] | 0927b7e | 2011-11-11 16:06:22 | [diff] [blame] | 78 | from testing_support import local_rietveld |
[email protected] | 2a74d37 | 2011-03-29 19:05:50 | [diff] [blame] | 79 | server = local_rietveld.LocalRietveld() |
| 80 | finally: |
| 81 | sys.path = old_sys_path |
| 82 | |
[email protected] | ccbb781 | 2011-04-06 13:36:23 | [diff] [blame] | 83 | results = [] |
[email protected] | 2a74d37 | 2011-03-29 19:05:50 | [diff] [blame] | 84 | try: |
| 85 | # Start a local rietveld instance to test against. |
| 86 | server.start_server() |
| 87 | test_path = input_api.os_path.abspath( |
| 88 | input_api.os_path.join(input_api.PresubmitLocalPath(), 'tests')) |
[email protected] | ccbb781 | 2011-04-06 13:36:23 | [diff] [blame] | 89 | for test in input_api.os_listdir(test_path): |
[email protected] | 2a74d37 | 2011-03-29 19:05:50 | [diff] [blame] | 90 | # test-lib.sh is not an actual test so it should not be run. The other |
| 91 | # tests are tests known to fail. |
| 92 | DISABLED_TESTS = ( |
| 93 | 'owners.sh', 'push-from-logs.sh', 'rename.sh', 'test-lib.sh') |
| 94 | if test in DISABLED_TESTS or not test.endswith('.sh'): |
| 95 | continue |
| 96 | |
| 97 | print('Running %s' % test) |
[email protected] | ccbb781 | 2011-04-06 13:36:23 | [diff] [blame] | 98 | try: |
[email protected] | 899e1c1 | 2011-04-07 17:03:18 | [diff] [blame] | 99 | if input_api.verbose: |
[email protected] | ccbb781 | 2011-04-06 13:36:23 | [diff] [blame] | 100 | input_api.subprocess.check_call( |
| 101 | [input_api.os_path.join(test_path, test)], cwd=test_path) |
| 102 | else: |
| 103 | input_api.subprocess.check_output( |
[email protected] | 87e6d33 | 2011-09-09 19:01:28 | [diff] [blame] | 104 | [input_api.os_path.join(test_path, test)], cwd=test_path, |
| 105 | stderr=input_api.subprocess.STDOUT) |
[email protected] | ccbb781 | 2011-04-06 13:36:23 | [diff] [blame] | 106 | except (OSError, input_api.subprocess.CalledProcessError), e: |
| 107 | results.append(output_api.PresubmitError('%s failed\n%s' % (test, e))) |
[email protected] | 2a74d37 | 2011-03-29 19:05:50 | [diff] [blame] | 108 | except local_rietveld.Failure, e: |
[email protected] | ccbb781 | 2011-04-06 13:36:23 | [diff] [blame] | 109 | results.append(output_api.PresubmitError('\n'.join(str(i) for i in e.args))) |
[email protected] | 2a74d37 | 2011-03-29 19:05:50 | [diff] [blame] | 110 | finally: |
| 111 | server.stop_server() |
[email protected] | ccbb781 | 2011-04-06 13:36:23 | [diff] [blame] | 112 | return results |
[email protected] | ce30ef7 | 2009-05-25 15:20:42 | [diff] [blame] | 113 | |
| 114 | |
[email protected] | e94aedc | 2010-12-13 21:11:30 | [diff] [blame] | 115 | def CheckChangeOnUpload(input_api, output_api): |
[email protected] | d52417c | 2011-09-02 20:13:17 | [diff] [blame] | 116 | # Do not run integration tests on upload since they are way too slow. |
| 117 | tests_to_black_list = [ |
| 118 | r'^checkout_test\.py$', |
| 119 | r'^gclient_smoketest\.py$', |
| 120 | r'^scm_unittest\.py$', |
[email protected] | 5856562 | 2013-03-17 23:18:37 | [diff] [blame] | 121 | r'^subprocess2_test\.py$', |
[email protected] | d52417c | 2011-09-02 20:13:17 | [diff] [blame] | 122 | ] |
| 123 | return CommonChecks(input_api, output_api, tests_to_black_list) |
[email protected] | e94aedc | 2010-12-13 21:11:30 | [diff] [blame] | 124 | |
| 125 | |
[email protected] | ce30ef7 | 2009-05-25 15:20:42 | [diff] [blame] | 126 | def CheckChangeOnCommit(input_api, output_api): |
| 127 | output = [] |
[email protected] | d52417c | 2011-09-02 20:13:17 | [diff] [blame] | 128 | output.extend(CommonChecks(input_api, output_api, [])) |
[email protected] | e94aedc | 2010-12-13 21:11:30 | [diff] [blame] | 129 | output.extend(input_api.canned_checks.CheckDoNotSubmit( |
| 130 | input_api, |
| 131 | output_api)) |
[email protected] | d52417c | 2011-09-02 20:13:17 | [diff] [blame] | 132 | output.extend(RunGitClTests(input_api, output_api)) |
[email protected] | 7b305e8 | 2009-05-19 18:24:20 | [diff] [blame] | 133 | return output |