commit eadad92e4aee31691ebac12d33a90375a701b29f
author Andrii Shyshkalov <tandrii@chromium.org> Thu Jan 26 09:38:30 2017 +0100
committer Andrii Shyshkalov <tandrii@chromium.org> Thu Jan 26 08:46:11 2017 +0000
tree 1ee019d5ca652b7b3377305b786785ddf273da9e
parent 8039be7cc841e547dd25f9b6917b42df80e57734

git cl try: reduce number of RPCs to Gerrit from 3 to 1.

R=agable@chromium.org
BUG=681704

Change-Id: I3c0ab781e0357e0855d50157c5653df57f68ac9a
Reviewed-on: https://chromium-review.googlesource.com/432336
Reviewed-by: Aaron Gable <agable@chromium.org>

tests/git_cl_test.py

diff --git a/tests/git_cl_test.py b/tests/git_cl_test.py
index 0cafc70..c622201 100755
--- a/tests/git_cl_test.py
+++ b/tests/git_cl_test.py
@@ -2278,8 +2278,24 @@
         ((['git', 'config', 'branch.feature.gerritissue'],), '123456'),
         ((['git', 'config', 'branch.feature.gerritserver'],),
          'https://chromium-review.googlesource.com'),
-        (('GetChangeDetail', 'chromium-review.googlesource.com', '123456', []),
-         { 'status': 'OPEN' }),
+        (('GetChangeDetail', 'chromium-review.googlesource.com', '123456',
+         ['DETAILED_ACCOUNTS', 'ALL_REVISIONS', 'CURRENT_COMMIT']), {
+          'project': 'depot_tools',
+          'status': 'OPEN',
+          'owner': {'email': 'owner@e.mail'},
+          'revisions': {
+            'deadbeaf':  {
+              '_number': 6,
+            },
+            'beeeeeef': {
+              '_number': 7,
+              'fetch': {'http': {
+                'url': 'https://chromium.googlesource.com/depot_tools',
+                'ref': 'refs/changes/56/123456/7'
+              }},
+            },
+          },
+        }),
         ((['git', 'config', 'branch.feature.merge'],), 'feature'),
         ((['git', 'config', 'branch.feature.remote'],), 'origin'),
         ((['get_or_create_merge_base', 'feature', 'feature'],),
@@ -2370,14 +2386,11 @@
         ((['git', 'config', 'branch.feature.gerritissue'],), '123456'),
         ((['git', 'config', 'branch.feature.gerritserver'],),
          'https://chromium-review.googlesource.com'),
-        (('GetChangeDetail', 'chromium-review.googlesource.com', '123456', []),
-         { 'status': 'OPEN' }),
         (('GetChangeDetail', 'chromium-review.googlesource.com', '123456',
-         ['DETAILED_ACCOUNTS']),
-         {'owner': {'email': 'owner@e.mail'}}),
-        (('GetChangeDetail', 'chromium-review.googlesource.com', '123456',
-         ['ALL_REVISIONS', 'CURRENT_COMMIT']), {
+         ['DETAILED_ACCOUNTS', 'ALL_REVISIONS', 'CURRENT_COMMIT']), {
           'project': 'depot_tools',
+          'status': 'OPEN',
+          'owner': {'email': 'owner@e.mail'},
           'revisions': {
             'deadbeaf':  {
               '_number': 6,