commit 0f790421b29596227054e934c0add868e385d0d9
author Gavin Mak <gavinmak@google.com> Mon Feb 01 19:06:37 2021 +0000
committer LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Feb 01 19:06:37 2021 +0000
tree 651a715587dad90495ece5ad5a4e359197b4cee4
parent 037b0821fc21e7101649fbc9031b0654e528bb03

Update ListOwners for GerritClient in owners_client

The REST endpoint used by GerritClient.ListOwners needs to be updated as
of https://gerrit-review.googlesource.com/c/plugins/code-owners/+/293806

Change-Id: I5c142f16443b157c9d8edfa5bcf418cbedd2b193
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2658609
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Gavin Mak <gavinmak@google.com>

owners_client.py

diff --git a/owners_client.py b/owners_client.py
index fed27af..0d2f505 100644
--- a/owners_client.py
+++ b/owners_client.py
@@ -203,4 +203,4 @@
     # random.
     data = gerrit_util.GetOwnersForFile(
         self._host, self._project, self._branch, path)
-    return [d['account']['email'] for d in data]
+    return [d['account']['email'] for d in data['code_owners']]

tests/owners_client_test.py

diff --git a/tests/owners_client_test.py b/tests/owners_client_test.py
index 9aaa849..4f727b3 100644
--- a/tests/owners_client_test.py
+++ b/tests/owners_client_test.py
@@ -27,23 +27,25 @@
 
 
 def _get_owners():
-  return [
-    {
-      "account": {
-        "email": 'approver@example.com'
+  return {
+    "code_owners": [
+      {
+        "account": {
+          "email": 'approver@example.com'
+        }
+      },
+      {
+        "account": {
+          "email": 'reviewer@example.com'
+        },
+      },
+      {
+        "account": {
+          "email": 'missing@example.com'
+        },
       }
-    },
-    {
-      "account": {
-        "email": 'reviewer@example.com'
-      },
-    },
-    {
-      "account": {
-        "email": 'missing@example.com'
-      },
-    }
-  ]
+    ]
+  }