Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(10)

Side by Side Diff: tests/gclient_smoketest.py

Issue 25322002: Add support in gclient for pre-DEPS hooks (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Patch Set: Add asserts, tests Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« gclient.py ('K') | « testing_support/fake_repos.py ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """Smoke tests for gclient.py. 6 """Smoke tests for gclient.py.
7 7
8 Shell out 'gclient' and run basic conformance tests. 8 Shell out 'gclient' and run basic conformance tests.
9 9
10 This test assumes GClientSmokeBase.URL_BASE is valid. 10 This test assumes GClientSmokeBase.URL_BASE is valid.
(...skipping 1049 matching lines...) Expand 10 before | Expand all | Expand 10 after
1060 ['running', 'running']) 1060 ['running', 'running'])
1061 self.assertEquals(1, len(out[0])) 1061 self.assertEquals(1, len(out[0]))
1062 self.assertEquals(1, len(out[1])) 1062 self.assertEquals(1, len(out[1]))
1063 tree = self.mangle_git_tree(('repo_1@2', 'src'), 1063 tree = self.mangle_git_tree(('repo_1@2', 'src'),
1064 ('repo_2@1', 'src/repo2'), 1064 ('repo_2@1', 'src/repo2'),
1065 ('repo_3@2', 'src/repo2/repo_renamed')) 1065 ('repo_3@2', 'src/repo2/repo_renamed'))
1066 tree['src/git_hooked1'] = 'git_hooked1' 1066 tree['src/git_hooked1'] = 'git_hooked1'
1067 tree['src/git_hooked2'] = 'git_hooked2' 1067 tree['src/git_hooked2'] = 'git_hooked2'
1068 self.assertTree(tree) 1068 self.assertTree(tree)
1069 1069
1070 def testPreDepsHooks(self):
1071 if not self.enabled:
1072 return
1073 self.gclient(['config', self.git_base + 'repo_5', '--name', 'src'])
1074 expectation = [
1075 ('running', self.root_dir), # git clone repo_5
1076 ('running', self.root_dir + '/src'), # git checkout src
1077 ('running', self.root_dir), # pre-deps hook
1078 ('running', self.root_dir), # git clone repo_1
1079 ('running', self.root_dir + '/src/repo1'), # git checkout repo1
1080 ('running', self.root_dir), # git clone repo_1
1081 ('running', self.root_dir + '/src/repo2'), # git checkout repo2
1082 ]
1083 out = self.parseGclient(['sync', '--deps', 'mac', '--jobs=1',
1084 '--revision', 'src@' + self.githash('repo_5', 2)],
1085 expectation)
1086 self.assertEquals(2, len(out[2]))
1087 self.assertEquals('pre-deps hook', out[2][1])
1088 tree = self.mangle_git_tree(('repo_5@2', 'src'),
1089 ('repo_1@2', 'src/repo1'),
1090 ('repo_2@1', 'src/repo2')
1091 )
1092 tree['src/git_pre_deps_hooked'] = 'git_pre_deps_hooked'
1093 self.assertTree(tree)
1094
1095 os.remove(join(self.root_dir, 'src', 'git_pre_deps_hooked'))
1096
1097 # Pre-DEPS hooks run with runhooks.
1098 self.gclient(['runhooks', '--deps', 'mac'])
1099 tree = self.mangle_git_tree(('repo_5@2', 'src'),
1100 ('repo_1@2', 'src/repo1'),
1101 ('repo_2@1', 'src/repo2')
1102 )
1103 tree['src/git_pre_deps_hooked'] = 'git_pre_deps_hooked'
1104 self.assertTree(tree)
1105
1106 os.remove(join(self.root_dir, 'src', 'git_pre_deps_hooked'))
1107
1108 # Pre-DEPS hooks do not run with --nohooks.
1109 self.gclient(['sync', '--deps', 'mac', '--nohooks',
1110 '--revision', 'src@' + self.githash('repo_5', 2)])
1111 tree = self.mangle_git_tree(('repo_5@2', 'src'),
1112 ('repo_1@2', 'src/repo1'),
1113 ('repo_2@1', 'src/repo2')
1114 )
1115 self.assertTree(tree)
1116
1117 def testPreDepsHooksError(self):
1118 if not self.enabled:
1119 return
1120 self.gclient(['config', self.git_base + 'repo_5', '--name', 'src'])
1121 expectated_stdout = [
1122 ('running', self.root_dir), # git clone repo_5
1123 ('running', self.root_dir + '/src'), # git checkout src
1124 ('running', self.root_dir), # pre-deps hook
1125 ('running', self.root_dir), # pre-deps hook (fails)
1126 ]
1127 expected_stderr = ('Error: Command /usr/bin/python -c import sys; '
1128 'sys.exit(1) returned non-zero exit status 1 in %s\n'
1129 % self.root_dir)
1130 stdout, stderr, retcode = self.gclient(['sync', '--deps', 'mac', '--jobs=1',
1131 '--revision',
1132 'src@' + self.githash('repo_5', 3)])
1133 self.assertEquals(stderr, expected_stderr)
1134 self.assertEquals(2, retcode)
1135 self.checkBlock(stdout, expectated_stdout)
1136
1070 def testRevInfo(self): 1137 def testRevInfo(self):
1071 if not self.enabled: 1138 if not self.enabled:
1072 return 1139 return
1073 self.gclient(['config', self.git_base + 'repo_1', '--name', 'src']) 1140 self.gclient(['config', self.git_base + 'repo_1', '--name', 'src'])
1074 self.gclient(['sync', '--deps', 'mac']) 1141 self.gclient(['sync', '--deps', 'mac'])
1075 results = self.gclient(['revinfo', '--deps', 'mac']) 1142 results = self.gclient(['revinfo', '--deps', 'mac'])
1076 out = ('src: %(base)srepo_1\n' 1143 out = ('src: %(base)srepo_1\n'
1077 'src/repo2: %(base)srepo_2@%(hash2)s\n' 1144 'src/repo2: %(base)srepo_2@%(hash2)s\n'
1078 'src/repo2/repo_renamed: %(base)srepo_3\n' % 1145 'src/repo2/repo_renamed: %(base)srepo_3\n' %
1079 { 1146 {
(...skipping 288 matching lines...) Expand 10 before | Expand all | Expand 10 after
1368 1435
1369 if '-c' in sys.argv: 1436 if '-c' in sys.argv:
1370 COVERAGE = True 1437 COVERAGE = True
1371 sys.argv.remove('-c') 1438 sys.argv.remove('-c')
1372 if os.path.exists('.coverage'): 1439 if os.path.exists('.coverage'):
1373 os.remove('.coverage') 1440 os.remove('.coverage')
1374 os.environ['COVERAGE_FILE'] = os.path.join( 1441 os.environ['COVERAGE_FILE'] = os.path.join(
1375 os.path.dirname(os.path.dirname(os.path.abspath(__file__))), 1442 os.path.dirname(os.path.dirname(os.path.abspath(__file__))),
1376 '.coverage') 1443 '.coverage')
1377 unittest.main() 1444 unittest.main()
OLDNEW
« gclient.py ('K') | « testing_support/fake_repos.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698