summaryrefslogtreecommitdiffstats
path: root/PRESUBMIT_test.py
diff options
context:
space:
mode:
authorthakis <thakis@chromium.org>2015-02-23 09:00:37 -0800
committerCommit bot <commit-bot@chromium.org>2015-02-23 17:01:24 +0000
commit4f02f4cd6c8b9087ecd286c905b15e253d88d82f (patch)
treeddddecc466cd4ac08007913660bddebae1c3f77f /PRESUBMIT_test.py
parent6377bfd717ccf6488462d6d22248f28918c247b5 (diff)
downloadchromium_src-4f02f4cd6c8b9087ecd286c905b15e253d88d82f.zip
chromium_src-4f02f4cd6c8b9087ecd286c905b15e253d88d82f.tar.gz
chromium_src-4f02f4cd6c8b9087ecd286c905b15e253d88d82f.tar.bz2
Make `git cl try` to the same as hitting cq (except for presubmit).
PRESUBMIT.py used to contain a handwritten list of bots to trigger on `git cl try`. It was always out of sync with what the cq did. https://codereview.chromium.org/823823002 awesomely made `git cl try` do the same thing as the commit queue, but it kept a few special-case rules around. It's better to not have these special cases: Once the developer hits cq, all the "intelligently" skipped bots need to be run anyways. More importantly, there should be only one system to decide what to run and it should be identical for cq and `git cl try`. Currently, we think this system is the `analyze` step, and it does a decent job of that already. BUG=none (but made possible by 443613) Review URL: https://codereview.chromium.org/921323003 Cr-Commit-Position: refs/heads/master@{#317597}
Diffstat (limited to 'PRESUBMIT_test.py')
-rwxr-xr-xPRESUBMIT_test.py8
1 files changed, 0 insertions, 8 deletions
diff --git a/PRESUBMIT_test.py b/PRESUBMIT_test.py
index 5c93d98..ad391d3 100755
--- a/PRESUBMIT_test.py
+++ b/PRESUBMIT_test.py
@@ -375,14 +375,6 @@ class BadExtensionsTest(unittest.TestCase):
results = PRESUBMIT._CheckPatchFiles(mock_input_api, MockOutputApi())
self.assertEqual(0, len(results))
- def testOnlyOwnersFiles(self):
- mock_change = MockChange([
- 'some/path/OWNERS',
- 'A\Windows\Path\OWNERS',
- ])
- results = PRESUBMIT.GetPreferredTryMasters(None, mock_change)
- self.assertEqual({}, results)
-
class CheckSingletonInHeadersTest(unittest.TestCase):
def testSingletonInArbitraryHeader(self):