gclient: print a warning if a dep would get deleted or moved in the future

tests/gclient_smoketest.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.
 
 """Smoke tests for gclient.py.
 
 Shell out 'gclient' and run basic conformance tests.
 
 This test assumes GClientSmokeBase.URL_BASE is valid.
 """
 
 import logging
 import os
 import re
 import subprocess
 import sys
 import unittest
 
 ROOT_DIR = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
 sys.path.insert(0, ROOT_DIR)
 
 from testing_support.fake_repos import join, write
 from testing_support.fake_repos import FakeReposTestBase, FakeRepoTransitive
+# TODO(borenet): Uncomment these includes once 
+# gclient_scm.SCMWrapper._DeleteOrMove is enabled.
+#from testing_support.fake_repos import FakeReposTestBase, FakeRepoTransitive, \
+#                                       FakeRepoSkiaDEPS
 
-import gclient_utils
+
+#import gclient_utils
+#import scm as gclient_scm
 
 import subprocess2
 
 GCLIENT_PATH = os.path.join(ROOT_DIR, 'gclient')
 COVERAGE = False
 
 
 class GClientSmokeBase(FakeReposTestBase):
   def setUp(self):
     super(GClientSmokeBase, self).setUp()
@@ skipping 719 matching lines @@
       return
     self.gclient(['config', self.svn_base + 'trunk/src/'])
     self.gclient(['sync'])
     # Cripple the checkout.
     os.remove(join(self.root_dir, '.gclient_entries'))
     src = join(self.root_dir, 'src')
     res = self.gclient(['sync', '--jobs', '1'], src)
     self.checkBlock(res[0],
                     ['running', 'running', 'running'])
 
-  def testUnversionedRepository(self):
-    # Check that gclient automatically deletes crippled SVN repositories.
-    if not self.enabled:
-      return
-    self.gclient(['config', self.svn_base + 'trunk/src/'])
-    cmd = ['sync', '--jobs', '1', '--delete_unversioned_trees', '--reset']
-    self.assertEquals(0, self.gclient(cmd)[-1])
-    third_party = join(self.root_dir, 'src', 'third_party')
-    subprocess2.check_call(['svn', 'propset', '-q', 'svn:ignore', 'foo', '.'],
-                           cwd=third_party)
-
-    # Cripple src/third_party/foo and make sure gclient still succeeds.
-    gclient_utils.rmtree(join(third_party, 'foo', '.svn'))
-    self.assertEquals(0, self.gclient(cmd)[-1])
+  # TODO(borenet): Uncomment this test once 
+  # gclient_scm.SCMWrapper._DeleteOrMove is enabled.
+#   def testUnversionedRepository(self):
+#     # Check that gclient automatically deletes crippled SVN repositories.
+#     if not self.enabled:
+#       return
+#     self.gclient(['config', self.svn_base + 'trunk/src/'])
+#     cmd = ['sync', '--jobs', '1', '--delete_unversioned_trees', '--reset']
+#     self.assertEquals(0, self.gclient(cmd)[-1])
+#     third_party = join(self.root_dir, 'src', 'third_party')
+#     subprocess2.check_call(['svn', 'propset', '-q', 'svn:ignore', 'foo', '.'],
+#                            cwd=third_party)
+# 
+#     # Cripple src/third_party/foo and make sure gclient still succeeds.
+#     gclient_utils.rmtree(join(third_party, 'foo', '.svn'))
+#     self.assertEquals(0, self.gclient(cmd)[-1])
 
 
 class GClientSmokeSVNTransitive(GClientSmokeBase):
   FAKE_REPOS_CLASS = FakeRepoTransitive
 
   def setUp(self):
     super(GClientSmokeSVNTransitive, self).setUp()
     self.enabled = self.FAKE_REPOS.set_up_svn()
 
   def testSyncTransitive(self):
@@ skipping 558 matching lines @@
         ('git', 'repo_2@' + self.githash('repo_2', 1)[:7], 'src/repo2'),
         ('git', 'repo_3', 'src/repo2/repo_renamed'),
         ('svn', 'trunk/third_party/foo@1', 'src/third_party/foo'),
       ]
     expected = [(scm, bases[scm] + url, os.path.join(self.root_dir, path))
                 for (scm, url, path) in expected_source]
 
     self.assertEquals(sorted(entries), sorted(expected))
 
 
+# TODO(borenet): Uncomment the following tests once
+# gclient_scm.SCMWrapper._DeleteOrMove is enabled.
+# class SkiaDEPSTransitionSmokeTest(GClientSmokeBase):
+#   """Simulate the behavior of bisect bots as they transition across the Skia
+#   DEPS change."""
+# 
+#   FAKE_REPOS_CLASS = FakeRepoSkiaDEPS
+# 
+#   def setUp(self):
+#     super(SkiaDEPSTransitionSmokeTest, self).setUp()
+#     self.enabled = (self.FAKE_REPOS.set_up_git() and
+#                     self.FAKE_REPOS.set_up_svn())
+# 
+#   def testSkiaDEPSChangeSVN(self):
+#     if not self.enabled:
+#       return
+# 
+#     # Create an initial checkout:
+#     # - Single checkout at the root.
+#     # - Multiple checkouts in a shared subdirectory.
+#     self.gclient(['config', '--spec',
+#         'solutions=['
+#         '{"name": "src",'
+#         ' "url": "' + self.svn_base + 'trunk/src/",'
+#         '}]'])
+# 
+#     checkout_path = os.path.join(self.root_dir, 'src')
+#     skia = os.path.join(checkout_path, 'third_party', 'skia')
+#     skia_gyp = os.path.join(skia, 'gyp')
+#     skia_include = os.path.join(skia, 'include')
+#     skia_src = os.path.join(skia, 'src')
+# 
+#     gyp_svn_url = self.svn_base + 'skia/gyp'
+#     include_svn_url = self.svn_base + 'skia/include'
+#     src_svn_url = self.svn_base + 'skia/src'
+#     skia_git_url = self.git_base + 'repo_1'
+# 
+#     # Initial sync. Verify that we get the expected checkout.
+#     res = self.gclient(['sync', '--deps', 'mac', '--revision', 'src@2'])
+#     self.assertEqual(res[2], 0, 'Initial sync failed.')
+#     self.assertEqual(gclient_scm.SVN.CaptureLocalInfo([], skia_gyp)['URL'],
+#                      gyp_svn_url)
+#     self.assertEqual(gclient_scm.SVN.CaptureLocalInfo([],
+#                                                       skia_include)['URL'],
+#                      include_svn_url)
+#     self.assertEqual(gclient_scm.SVN.CaptureLocalInfo([], skia_src)['URL'],
+#                      src_svn_url)
+# 
+#     # Verify that the sync succeeds. Verify that we have the  expected merged
+#     # checkout.
+#     res = self.gclient(['sync', '--deps', 'mac', '--revision', 'src@3'])
+#     self.assertEqual(res[2], 0, 'DEPS change sync failed.')
+#     self.assertEqual(gclient_scm.GIT.Capture(['config', 'remote.origin.url'],
+#                                              skia), skia_git_url)
+# 
+#     # Sync again. Verify that we still have the expected merged checkout.
+#     res = self.gclient(['sync', '--deps', 'mac', '--revision', 'src@3'])
+#     self.assertEqual(res[2], 0, 'Subsequent sync failed.')
+#     self.assertEqual(gclient_scm.GIT.Capture(['config', 'remote.origin.url'],
+#                                              skia), skia_git_url)
+# 
+#     # Sync back to the original DEPS. Verify that we get the original
+#     # structure.
+#     res = self.gclient(['sync', '--deps', 'mac', '--revision', 'src@2'])
+#     self.assertEqual(res[2], 0, 'Reverse sync failed.')
+#     self.assertEqual(gclient_scm.SVN.CaptureLocalInfo([], skia_gyp)['URL'],
+#                      gyp_svn_url)
+#     self.assertEqual(gclient_scm.SVN.CaptureLocalInfo([],
+#                                                       skia_include)['URL'],
+#                      include_svn_url)
+#     self.assertEqual(gclient_scm.SVN.CaptureLocalInfo([], skia_src)['URL'],
+#                      src_svn_url)
+# 
+#     # Sync again. Verify that we still have the original structure.
+#     res = self.gclient(['sync', '--deps', 'mac', '--revision', 'src@2'])
+#     self.assertEqual(res[2], 0, 'Subsequent sync #2 failed.')
+#     self.assertEqual(gclient_scm.SVN.CaptureLocalInfo([], skia_gyp)['URL'],
+#                      gyp_svn_url)
+#     self.assertEqual(gclient_scm.SVN.CaptureLocalInfo([],
+#                                                       skia_include)['URL'],
+#                      include_svn_url)
+#     self.assertEqual(gclient_scm.SVN.CaptureLocalInfo([], skia_src)['URL'],
+#                      src_svn_url)
+# 
+#   def testSkiaDEPSChangeGit(self):
+#     if not self.enabled:
+#       return
+# 
+#     # Create an initial checkout:
+#     # - Single checkout at the root.
+#     # - Multiple checkouts in a shared subdirectory.
+#     self.gclient(['config', '--spec',
+#         'solutions=['
+#         '{"name": "src",'
+#         ' "url": "' + self.git_base + 'repo_2",'
+#         '}]'])
+# 
+#     checkout_path = os.path.join(self.root_dir, 'src')
+#     skia = os.path.join(checkout_path, 'third_party', 'skia')
+#     skia_gyp = os.path.join(skia, 'gyp')
+#     skia_include = os.path.join(skia, 'include')
+#     skia_src = os.path.join(skia, 'src')
+# 
+#     gyp_git_url = self.git_base + 'repo_3'
+#     include_git_url = self.git_base + 'repo_4'
+#     src_git_url = self.git_base + 'repo_5'
+#     skia_git_url = self.FAKE_REPOS.git_base + 'repo_1'
+# 
+#     pre_hash = self.githash('repo_2', 1)
+#     post_hash = self.githash('repo_2', 2)
+# 
+#     # Initial sync. Verify that we get the expected checkout.
+#     res = self.gclient(['sync', '--deps', 'mac', '--revision',
+#                         'src@%s' % pre_hash])
+#     self.assertEqual(res[2], 0, 'Initial sync failed.')
+#     self.assertEqual(gclient_scm.GIT.Capture(['config', 'remote.origin.url'],
+#                                              skia_gyp), gyp_git_url)
+#     self.assertEqual(gclient_scm.GIT.Capture(['config', 'remote.origin.url'],
+#                                              skia_include), include_git_url)
+#     self.assertEqual(gclient_scm.GIT.Capture(['config', 'remote.origin.url'],
+#                                              skia_src), src_git_url)
+# 
+#     # Verify that the sync succeeds. Verify that we have the  expected merged
+#     # checkout.
+#     res = self.gclient(['sync', '--deps', 'mac', '--revision',
+#                         'src@%s' % post_hash])
+#     self.assertEqual(res[2], 0, 'DEPS change sync failed.')
+#     self.assertEqual(gclient_scm.GIT.Capture(['config', 'remote.origin.url'],
+#                                              skia), skia_git_url)
+# 
+#     # Sync again. Verify that we still have the expected merged checkout.
+#     res = self.gclient(['sync', '--deps', 'mac', '--revision',
+#                         'src@%s' % post_hash])
+#     self.assertEqual(res[2], 0, 'Subsequent sync failed.')
+#     self.assertEqual(gclient_scm.GIT.Capture(['config', 'remote.origin.url'],
+#                                              skia), skia_git_url)
+# 
+#     # Sync back to the original DEPS. Verify that we get the original
+#     # structure.
+#     res = self.gclient(['sync', '--deps', 'mac', '--revision',
+#                         'src@%s' % pre_hash])
+#     self.assertEqual(res[2], 0, 'Reverse sync failed.')
+#     self.assertEqual(gclient_scm.GIT.Capture(['config', 'remote.origin.url'],
+#                                              skia_gyp), gyp_git_url)
+#     self.assertEqual(gclient_scm.GIT.Capture(['config', 'remote.origin.url'],
+#                                              skia_include), include_git_url)
+#     self.assertEqual(gclient_scm.GIT.Capture(['config', 'remote.origin.url'],
+#                                              skia_src), src_git_url)
+# 
+#     # Sync again. Verify that we still have the original structure.
+#     res = self.gclient(['sync', '--deps', 'mac', '--revision',
+#                         'src@%s' % pre_hash])
+#     self.assertEqual(res[2], 0, 'Subsequent sync #2 failed.')
+#     self.assertEqual(gclient_scm.GIT.Capture(['config', 'remote.origin.url'],
+#                                              skia_gyp), gyp_git_url)
+#     self.assertEqual(gclient_scm.GIT.Capture(['config', 'remote.origin.url'],
+#                                              skia_include), include_git_url)
+#     self.assertEqual(gclient_scm.GIT.Capture(['config', 'remote.origin.url'],
+#                                              skia_src), src_git_url)
+
+
 class GClientSmokeFromCheckout(GClientSmokeBase):
   # WebKit abuses this. It has a .gclient and a DEPS from a checkout.
   def setUp(self):
     super(GClientSmokeFromCheckout, self).setUp()
     self.enabled = self.FAKE_REPOS.set_up_svn()
     os.rmdir(self.root_dir)
     if self.enabled:
       usr, pwd = self.FAKE_REPOS.USERS[0]
       subprocess2.check_call(
           ['svn', 'checkout', self.svn_base + '/trunk/webkit',
@@ skipping 87 matching lines @@
 
   if '-c' in sys.argv:
     COVERAGE = True
     sys.argv.remove('-c')
     if os.path.exists('.coverage'):
       os.remove('.coverage')
     os.environ['COVERAGE_FILE'] = os.path.join(
         os.path.dirname(os.path.dirname(os.path.abspath(__file__))),
         '.coverage')
   unittest.main()