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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: tests/gclient_scm_test.py

Issue 8771042: Enforces using cwd in all svn calls. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: Created 9 years 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 side-by-side diff with in-line comments
Download patch
« gclient_scm.py ('K') | « testing_support/fake_repos.py ('k') | tests/presubmit_unittest.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/gclient_scm_test.py
diff --git a/tests/gclient_scm_test.py b/tests/gclient_scm_test.py
index a0e24aa11d2c04dd1524164251e3a9c0a8e18b34..bc5d9a1259f2d21b3a630ecaaddb3301111861f8 100755
--- a/tests/gclient_scm_test.py
+++ b/tests/gclient_scm_test.py
@@ -156,7 +156,7 @@ class SVNWrapperTestCase(BaseTestCase):
options = self.Options(verbose=True)
gclient_scm.os.path.isdir(self.base_path).AndReturn(False)
gclient_scm.os.path.exists(self.base_path).AndReturn(False)
- gclient_scm.scm.SVN.Capture(['--version']
+ gclient_scm.scm.SVN.Capture(['--version'], None
).AndReturn('svn, version 1.5.1 (r32289)')
# It'll to a checkout instead.
gclient_scm.os.path.exists(join(self.base_path, '.git')).AndReturn(False)
@@ -197,7 +197,8 @@ class SVNWrapperTestCase(BaseTestCase):
gclient_scm.os.makedirs(parent)
gclient_scm.os.path.exists(parent).AndReturn(True)
files_list = self.mox.CreateMockAnything()
- gclient_scm.scm.SVN.Capture(['--version']).AndReturn('svn, version 1.6')
+ gclient_scm.scm.SVN.Capture(['--version'], None
+ ).AndReturn('svn, version 1.6')
gclient_scm.scm.SVN.RunAndGetFileList(
options.verbose,
['checkout', self.url, self.base_path, '--force', '--ignore-externals'],
@@ -216,7 +217,7 @@ class SVNWrapperTestCase(BaseTestCase):
options = self.Options(verbose=True)
gclient_scm.os.path.isdir(self.base_path).AndReturn(True)
gclient_scm.os.path.isdir(join(self.base_path, '.svn')).AndReturn(True)
- gclient_scm.scm.SVN.CaptureStatus(self.base_path).AndReturn([])
+ gclient_scm.scm.SVN.CaptureStatus(None, self.base_path).AndReturn([])
gclient_scm.os.path.isdir(self.base_path).AndReturn(True)
gclient_scm.scm.SVN.RunAndGetFileList(
options.verbose,
@@ -237,7 +238,7 @@ class SVNWrapperTestCase(BaseTestCase):
items = [
('~ ', 'a'),
]
- gclient_scm.scm.SVN.CaptureStatus(self.base_path).AndReturn(items)
+ gclient_scm.scm.SVN.CaptureStatus(None, self.base_path).AndReturn(items)
file_path = join(self.base_path, 'a')
gclient_scm.os.path.exists(file_path).AndReturn(True)
gclient_scm.os.path.isfile(file_path).AndReturn(False)
@@ -266,7 +267,7 @@ class SVNWrapperTestCase(BaseTestCase):
items = [
('~ ', '.'),
]
- gclient_scm.scm.SVN.CaptureStatus(self.base_path).AndReturn(items)
+ gclient_scm.scm.SVN.CaptureStatus(None, self.base_path).AndReturn(items)
file_path = join(self.base_path, '.')
gclient_scm.os.path.exists(file_path).AndReturn(True)
gclient_scm.os.path.isfile(file_path).AndReturn(False)
@@ -317,7 +318,7 @@ class SVNWrapperTestCase(BaseTestCase):
gclient_scm.os.makedirs(parent)
gclient_scm.os.path.exists(parent).AndReturn(True)
files_list = self.mox.CreateMockAnything()
- gclient_scm.scm.SVN.Capture(['--version']
+ gclient_scm.scm.SVN.Capture(['--version'], None
).AndReturn('svn, version 1.5.1 (r32289)')
gclient_scm.scm.SVN.RunAndGetFileList(
options.verbose,
@@ -344,17 +345,17 @@ class SVNWrapperTestCase(BaseTestCase):
# Verify no locked files.
dotted_path = join(self.base_path, '.')
- gclient_scm.scm.SVN.CaptureStatus(dotted_path).AndReturn([])
+ gclient_scm.scm.SVN.CaptureStatus(None, dotted_path).AndReturn([])
# Checkout or update.
gclient_scm.os.path.exists(self.base_path).AndReturn(True)
- gclient_scm.scm.SVN.CaptureInfo(dotted_path).AndReturn(file_info)
+ gclient_scm.scm.SVN.CaptureInfo(None, dotted_path).AndReturn(file_info)
# Cheat a bit here.
- gclient_scm.scm.SVN.CaptureInfo(file_info['URL']).AndReturn(file_info)
+ gclient_scm.scm.SVN.CaptureInfo(file_info['URL'], None).AndReturn(file_info)
additional_args = []
if options.manually_grab_svn_rev:
additional_args = ['--revision', str(file_info['Revision'])]
- gclient_scm.scm.SVN.Capture(['--version']
+ gclient_scm.scm.SVN.Capture(['--version'], None
).AndReturn('svn, version 1.5.1 (r32289)')
additional_args.extend(['--force', '--ignore-externals'])
files_list = []
@@ -377,14 +378,14 @@ class SVNWrapperTestCase(BaseTestCase):
# Checks to make sure that we support svn co --depth.
gclient_scm.scm.SVN.current_version = None
- gclient_scm.scm.SVN.Capture(['--version']
+ gclient_scm.scm.SVN.Capture(['--version'], None
).AndReturn('svn, version 1.5.1 (r32289)')
gclient_scm.os.path.exists(join(self.base_path, '.svn')).AndReturn(False)
gclient_scm.os.path.exists(join(self.base_path, 'DEPS')).AndReturn(False)
# Verify no locked files.
dotted_path = join(self.base_path, '.')
- gclient_scm.scm.SVN.CaptureStatus(dotted_path).AndReturn([])
+ gclient_scm.scm.SVN.CaptureStatus(None, dotted_path).AndReturn([])
# When checking out a single file, we issue an svn checkout and svn update.
files_list = self.mox.CreateMockAnything()
@@ -402,8 +403,8 @@ class SVNWrapperTestCase(BaseTestCase):
gclient_scm.os.path.exists(join(self.base_path, '.git')).AndReturn(False)
gclient_scm.os.path.exists(join(self.base_path, '.hg')).AndReturn(False)
gclient_scm.os.path.exists(self.base_path).AndReturn(True)
- gclient_scm.scm.SVN.CaptureInfo(dotted_path).AndReturn(file_info)
- gclient_scm.scm.SVN.CaptureInfo(file_info['URL']).AndReturn(file_info)
+ gclient_scm.scm.SVN.CaptureInfo(None, dotted_path).AndReturn(file_info)
+ gclient_scm.scm.SVN.CaptureInfo(file_info['URL'], None).AndReturn(file_info)
self.mox.ReplayAll()
scm = self._scm_wrapper(url=self.url, root_dir=self.root_dir,
@@ -416,7 +417,7 @@ class SVNWrapperTestCase(BaseTestCase):
# Checks to make sure that we support svn co --depth.
gclient_scm.scm.SVN.current_version = None
- gclient_scm.scm.SVN.Capture(['--version']
+ gclient_scm.scm.SVN.Capture(['--version'], None
).AndReturn('svn, version 1.4.4 (r25188)')
gclient_scm.os.path.exists(self.base_path).AndReturn(True)
@@ -440,7 +441,7 @@ class SVNWrapperTestCase(BaseTestCase):
# Checks to make sure that we support svn co --depth.
gclient_scm.scm.SVN.current_version = None
- gclient_scm.scm.SVN.Capture(['--version']
+ gclient_scm.scm.SVN.Capture(['--version'], None
).AndReturn('svn, version 1.5.1 (r32289)')
gclient_scm.os.path.exists(join(self.base_path, '.svn')).AndReturn(False)
# If DEPS already exists, assume we're upgrading from svn1.4, so delete
@@ -449,7 +450,8 @@ class SVNWrapperTestCase(BaseTestCase):
gclient_scm.os.remove(join(self.base_path, 'DEPS'))
# Verify no locked files.
- gclient_scm.scm.SVN.CaptureStatus(join(self.base_path, '.')).AndReturn([])
+ gclient_scm.scm.SVN.CaptureStatus(
+ None, join(self.base_path, '.')).AndReturn([])
# When checking out a single file, we issue an svn checkout and svn update.
files_list = self.mox.CreateMockAnything()
@@ -468,8 +470,8 @@ class SVNWrapperTestCase(BaseTestCase):
gclient_scm.os.path.exists(join(self.base_path, '.hg')).AndReturn(False)
gclient_scm.os.path.exists(self.base_path).AndReturn(True)
gclient_scm.scm.SVN.CaptureInfo(
- join(self.base_path, ".")).AndReturn(file_info)
- gclient_scm.scm.SVN.CaptureInfo(file_info['URL']).AndReturn(file_info)
+ None, join(self.base_path, ".")).AndReturn(file_info)
+ gclient_scm.scm.SVN.CaptureInfo(file_info['URL'], None).AndReturn(file_info)
self.mox.ReplayAll()
scm = self._scm_wrapper(url=self.url, root_dir=self.root_dir,
@@ -486,12 +488,13 @@ class SVNWrapperTestCase(BaseTestCase):
}
# Checks to make sure that we support svn co --depth.
gclient_scm.scm.SVN.current_version = None
- gclient_scm.scm.SVN.Capture(['--version']
+ gclient_scm.scm.SVN.Capture(['--version'], None
).AndReturn('svn, version 1.5.1 (r32289)')
gclient_scm.os.path.exists(join(self.base_path, '.svn')).AndReturn(True)
# Verify no locked files.
- gclient_scm.scm.SVN.CaptureStatus(join(self.base_path, '.')).AndReturn([])
+ gclient_scm.scm.SVN.CaptureStatus(None, join(self.base_path, '.')
+ ).AndReturn([])
# Now we fall back on scm.update().
files_list = self.mox.CreateMockAnything()
@@ -499,8 +502,8 @@ class SVNWrapperTestCase(BaseTestCase):
gclient_scm.os.path.exists(join(self.base_path, '.hg')).AndReturn(False)
gclient_scm.os.path.exists(self.base_path).AndReturn(True)
gclient_scm.scm.SVN.CaptureInfo(
- join(self.base_path, '.')).AndReturn(file_info)
- gclient_scm.scm.SVN.CaptureInfo(file_info['URL']).AndReturn(file_info)
+ None, join(self.base_path, '.')).AndReturn(file_info)
+ gclient_scm.scm.SVN.CaptureInfo(file_info['URL'], None).AndReturn(file_info)
self.mox.ReplayAll()
scm = self._scm_wrapper(url=self.url, root_dir=self.root_dir,
« gclient_scm.py ('K') | « testing_support/fake_repos.py ('k') | tests/presubmit_unittest.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698