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

Unified Diff: tests/gclient_test.py

Issue 8298002: Do not sort solutions anymore, only dependencies specified in DEPS files. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: Created 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « gclient.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/gclient_test.py
diff --git a/tests/gclient_test.py b/tests/gclient_test.py
index e547c8a6eb1f9bf0bb104614d5620e538fb5a591..767808dc30ab6222936fa8aafdf6dd72cfa80a87 100755
--- a/tests/gclient_test.py
+++ b/tests/gclient_test.py
@@ -157,20 +157,12 @@ class GclientTest(trial_dir.TestCase):
actual[3:])
self.assertEquals(3, len(obj.dependencies))
- self.assertEquals('bar', obj.dependencies[0].name)
- self.assertEquals('bar/empty', obj.dependencies[1].name)
- self.assertEquals('foo', obj.dependencies[2].name)
+ self.assertEquals('foo', obj.dependencies[0].name)
+ self.assertEquals('bar', obj.dependencies[1].name)
+ self.assertEquals('bar/empty', obj.dependencies[2].name)
self._check_requirements(
obj.dependencies[0],
{
- 'foo/dir1/dir2': ['bar', 'bar/empty', 'foo', 'foo/dir1'],
- })
- self._check_requirements(
- obj.dependencies[1],
- {})
- self._check_requirements(
- obj.dependencies[2],
- {
'foo/dir1': ['bar', 'bar/empty', 'foo'],
'foo/dir1/dir2/dir3':
['bar', 'bar/empty', 'foo', 'foo/dir1', 'foo/dir1/dir2'],
@@ -182,6 +174,14 @@ class GclientTest(trial_dir.TestCase):
'foo/dir1/dir2/dir3/dir4'],
})
self._check_requirements(
+ obj.dependencies[1],
+ {
+ 'foo/dir1/dir2': ['bar', 'bar/empty', 'foo', 'foo/dir1'],
+ })
+ self._check_requirements(
+ obj.dependencies[2],
+ {})
+ self._check_requirements(
obj,
{
'foo': [],
« no previous file with comments | « gclient.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698