Reapply 48271 with fix (again)

tests/fake_repos.py

 #!/usr/bin/python
 # Copyright (c) 2010 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Generate fake repositories for testing."""
 
 import logging
 import os
 import re
@@ skipping 187 matching lines @@
         fs[k] = fs[k] % { 'rev': rev }
       fs['trunk/src/DEPS'] = DEPS
       return fs
 
     # Testing:
     # - dependency disapear
     # - dependency renamed
     # - versioned and unversioned reference
     # - relative and full reference
     # - deps_os
-    # TODO(maruel):
     # - var
     # - hooks
+    # TODO(maruel):
     # - File
+    # - $matching_files
+    # - use_relative_paths
     self._commit_svn(file_system(1, """
+vars = {
+  'DummyVariable': 'third_party',
+}
 deps = {
   'src/other': 'svn://%(host)s/svn/trunk/other',
-  'src/third_party/fpp': '/trunk/third_party/foo',
+  'src/third_party/fpp': '/trunk/' + Var('DummyVariable') + '/foo',
 }
 deps_os = {
   'mac': {
     'src/third_party/prout': '/trunk/third_party/prout',
   },
 }""" % { 'host': '127.0.0.1' }))
 
     self._commit_svn(file_system(2, """
 deps = {
   'src/other': 'svn://%(host)s/svn/trunk/other',
   'src/third_party/foo': '/trunk/third_party/foo@1',
 }
+# I think this is wrong to have the hooks run from the base of the gclient
+# checkout. It's maybe a bit too late to change that behavior.
+hooks = [
+  {
+    'pattern': '.',
+    'action': ['python', '-c',
+               'open(\\'src/hooked1\\', \\'w\\').write(\\'hooked1\\')'],
+  },
+  {
+    # Should not be run.
+    'pattern': 'nonexistent',
+    'action': ['python', '-c',
+               'open(\\'src/hooked2\\', \\'w\\').write(\\'hooked2\\')'],
+  },
+]
 """ % { 'host': '127.0.0.1' }))
 
   def setUpGIT(self):
     """Creates git repositories and start the servers."""
     assert not self.gitdaemon
     rmtree(self.git_root)
     for repo in ['repo_%d' % r for r in range(1, 5)]:
       check_call(['git', 'init', '-q', join(self.git_root, repo)])
       self.git_hashes[repo] = []
 
     # Testing:
     # - dependency disapear
     # - dependency renamed
     # - versioned and unversioned reference
     # - relative and full reference
     # - deps_os
-    # TODO(maruel):
     # - var
     # - hooks
+    # TODO(maruel):
     # - File
+    # - $matching_files
+    # - use_relative_paths
     self._commit_git('repo_1', {
       'DEPS': """
+vars = {
+  'DummyVariable': 'repo',
+}
 deps = {
   'src/repo2': 'git://%(host)s/git/repo_2',
-  'src/repo2/repo3': '/repo_3',
+  'src/repo2/repo3': '/' + Var('DummyVariable') + '_3',
 }
 deps_os = {
   'mac': {
     'src/repo4': '/repo_4',
   },
 }""" % { 'host': '127.0.0.1' },
       'origin': 'git/repo_1@1\n',
     })
 
     self._commit_git('repo_2', {
@@ skipping 19 matching lines @@
     self._commit_git('repo_4', {
       'origin': "git/repo_4@2\n"
     })
 
     self._commit_git('repo_1', {
       'DEPS': """
 deps = {
   'src/repo2': 'git://%(host)s/git/repo_2@%(hash)s',
   'src/repo2/repo_renamed': '/repo_3',
 }
-""" % { 'host': '127.0.0.1', 'hash': self.git_hashes['repo_2'][0][0] },
+# I think this is wrong to have the hooks run from the base of the gclient
+# checkout. It's maybe a bit too late to change that behavior.
+hooks = [
+  {
+    'pattern': '.',
+    'action': ['python', '-c',
+               'open(\\'src/hooked1\\', \\'w\\').write(\\'hooked1\\')'],
+  },
+  {
+    # Should not be run.
+    'pattern': 'nonexistent',
+    'action': ['python', '-c',
+               'open(\\'src/hooked2\\', \\'w\\').write(\\'hooked2\\')'],
+  },
+]
+""" % {
+      # TODO(maruel): http://crosbug.com/3591 We need to strip the hash.. duh.
+      'host': '127.0.0.1', 'hash': self.git_hashes['repo_2'][0][0][:7] },
       'origin': "git/repo_1@2\n"
     })
 
     # Start the daemon.
     cmd = ['git', 'daemon', '--export-all', '--base-path=' + self.repos_dir]
     if self.local_only:
       cmd.append('--listen=127.0.0.1')
     logging.debug(cmd)
     self.gitdaemon.append(Popen(cmd, cwd=self.repos_dir))
 
@@ skipping 38 matching lines @@
     sys.stdin.readline()
   except KeyboardInterrupt:
     fake.leak = True
   finally:
     fake.tearDown()
   return 0
 
 
 if __name__ == '__main__':
   sys.exit(main(sys.argv))