Export mercurial repo & revision in debug report

bot/code_review_bot/revisions.py

@@ -74,7 +74,8 @@ class Revision(object):
     def __init__(self, api, try_task, update_build=True):
         assert isinstance(api, PhabricatorAPI)
         assert isinstance(try_task, dict)
-        self.repository = None
+        self.repository = None  # a try repo where the revision is stored
+        self.target_repository = None  # the target repo where the patch may land
         self.files = []
         self.lines = {}
         self.patch = None
@@ -300,12 +301,14 @@ def is_decision_task(task):
         if decision_env.get("GECKO_HEAD_REPOSITORY") == REPO_GECKO_TRY:
             # Mozilla-Central Try
             self.mercurial_revision = decision_env.get("GECKO_HEAD_REV")
-            self.repository = "mozilla-central"
+            self.repository = "try"
+            self.target_repository = "mozilla-central"
 
         elif decision_env.get("NSS_HEAD_REPOSITORY") == REPO_NSS_TRY:
             # NSS Try
             self.mercurial_revision = decision_env.get("NSS_HEAD_REVISION")
-            self.repository = "nss"
+            self.repository = "nss-try"
+            self.target_repository = "nss"
 
         else:
             raise Exception("Unsupported decision task")
@@ -328,4 +331,8 @@ def as_dict(self):
             # Extra infos for frontend
             "title": self.revision["fields"].get("title"),
             "bugzilla_id": self.revision["fields"].get("bugzilla.bug-id"),
+            # Extra infos for backend
+            "repository": self.repository,
+            "target_repository": self.target_repository,
+            "mercurial_revision": self.mercurial_revision,
         }

bot/code_review_bot/workflow.py

@@ -154,7 +154,8 @@ def index(self, revision, **kwargs):
         payload["try_group_id"] = settings.try_group_id
 
         # Always add the repository we are working on
-        payload["repository"] = revision.repository
+        # This is mainly used by the frontend to list & filter diffs
+        payload["repository"] = revision.target_repository
 
         # Add restartable flag for monitoring
         payload["monitoring_restart"] = payload["state"] == "error" and payload.get(

bot/tests/test_reporter_debug.py

@@ -12,6 +12,25 @@ def test_publication(tmpdir, mock_issues, mock_revision):
     """
     from code_review_bot.report.debug import DebugReporter
 
+    # Load description from Taskcluster tasks
+    mock_revision.setup_try(
+        {
+            # Base information are retrieved from the decision task
+            "decision": {
+                "task": {
+                    "payload": {
+                        "image": "taskcluster/decision",
+                        "env": {
+                            "GECKO_HEAD_REV": "deadc0ffee",
+                            "GECKO_HEAD_REPOSITORY": "https://hg.mozilla.org/try",
+                            "GECKO_BASE_REPOSITORY": "https://hg.mozilla.org/mozilla-central",
+                        },
+                    }
+                }
+            }
+        }
+    )
+
     report_dir = str(tmpdir.mkdir("public").realpath())
     report_path = os.path.join(report_dir, "report.json")
     assert not os.path.exists(report_path)
@@ -36,6 +55,9 @@ def test_publication(tmpdir, mock_issues, mock_revision):
         "phid": "PHID-DREV-zzzzz",
         "title": "Static Analysis tests",
         "has_clang_files": False,
+        "repository": "try",
+        "target_repository": "mozilla-central",
+        "mercurial_revision": "deadc0ffee",
     }
 
     assert "time" in report

bot/tests/test_workflow.py

@@ -25,7 +25,7 @@ class MockRevision(Revision):
     def __init__(self, namespaces, details, repository):
         self._namespaces = namespaces
         self._details = details
-        self.repository = repository
+        self.target_repository = repository
 
     @property
     def namespaces(self):