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

Unified Diff: tests/gclient_smoketest.py

Issue 2087313002: Remove all safesync_url functionality from gclient (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Patch Set: Fix smoketest Created 4 years, 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tests/gclient_scm_test.py ('k') | tests/gclient_test.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/gclient_smoketest.py
diff --git a/tests/gclient_smoketest.py b/tests/gclient_smoketest.py
index 50e7fd72c0ac2d20e2bad5a41bc5a52016f57c6e..51c5b4eef6a10ce9c5396a52567f5a0999013071 100755
--- a/tests/gclient_smoketest.py
+++ b/tests/gclient_smoketest.py
@@ -205,52 +205,36 @@ class GClientSmoke(GClientSmokeBase):
test(['config', self.svn_base + 'trunk/src/'],
('solutions = [\n'
- ' { "name" : "src",\n'
+ ' {\n'
+ ' "name" : "src",\n'
' "url" : "%strunk/src",\n'
' "deps_file" : "DEPS",\n'
' "managed" : True,\n'
- ' "custom_deps" : {\n'
- ' },\n'
- ' "safesync_url": "",\n'
+ ' "custom_deps" : {},\n'
' },\n'
']\n'
'cache_dir = None\n') % self.svn_base)
test(['config', self.git_base + 'repo_1', '--name', 'src'],
('solutions = [\n'
- ' { "name" : "src",\n'
+ ' {\n'
+ ' "name" : "src",\n'
' "url" : "%srepo_1",\n'
' "deps_file" : "DEPS",\n'
' "managed" : True,\n'
- ' "custom_deps" : {\n'
- ' },\n'
- ' "safesync_url": "",\n'
+ ' "custom_deps" : {},\n'
' },\n'
']\n'
'cache_dir = None\n') % self.git_base)
- test(['config', 'foo', 'faa'],
- 'solutions = [\n'
- ' { "name" : "foo",\n'
- ' "url" : "foo",\n'
- ' "deps_file" : "DEPS",\n'
- ' "managed" : True,\n'
- ' "custom_deps" : {\n'
- ' },\n'
- ' "safesync_url": "faa",\n'
- ' },\n'
- ']\n'
- 'cache_dir = None\n')
-
test(['config', 'foo', '--deps', 'blah'],
'solutions = [\n'
- ' { "name" : "foo",\n'
+ ' {\n'
+ ' "name" : "foo",\n'
' "url" : "foo",\n'
' "deps_file" : "blah",\n'
' "managed" : True,\n'
- ' "custom_deps" : {\n'
- ' },\n'
- ' "safesync_url": "",\n'
+ ' "custom_deps" : {},\n'
' },\n'
']\n'
'cache_dir = None\n')
@@ -259,7 +243,7 @@ class GClientSmoke(GClientSmokeBase):
os.remove(p)
results = self.gclient(['config', 'foo', 'faa', 'fuu'])
- err = ('Usage: gclient.py config [options] [url] [safesync url]\n\n'
+ err = ('Usage: gclient.py config [options] [url]\n\n'
'gclient.py: error: Inconsistent arguments. Use either --spec or one'
' or 2 args\n')
self.check(('', err, 2), results)
@@ -306,7 +290,6 @@ class GClientSmokeGIT(GClientSmokeBase):
def testSync(self):
if not self.enabled:
return
- # TODO(maruel): safesync.
self.gclient(['config', self.git_base + 'repo_1', '--name', 'src'])
# Test unversioned checkout.
self.parseGclient(
@@ -386,7 +369,6 @@ class GClientSmokeGIT(GClientSmokeBase):
def testSyncJobs(self):
if not self.enabled:
return
- # TODO(maruel): safesync.
self.gclient(['config', self.git_base + 'repo_1', '--name', 'src'])
# Test unversioned checkout.
self.parseGclient(
« no previous file with comments | « tests/gclient_scm_test.py ('k') | tests/gclient_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698