Address change request

Author: beatrice-acasandrei

Diff for: tests/ui/mock/alert_summaries.json

@@ -136,6 +136,7 @@
       }
     ],
     "related_alerts": [],
+    "duplicated_summaries_ids": [],
     "status": 0,
     "bug_number": null,
     "bug_updated": null,
@@ -223,6 +224,7 @@
       }
     ],
     "related_alerts": [],
+    "duplicated_summaries_ids": [],
     "status": 0,
     "bug_number": null,
     "bug_updated": null,
@@ -358,6 +360,7 @@
       }
     ],
     "related_alerts": [],
+    "duplicated_summaries_ids": [],
     "status": 0,
     "bug_number": null,
     "bug_updated": null,

Diff for: tests/webapp/api/test_performance_alertsummary_api.py

@@ -94,6 +94,7 @@ def test_alert_summaries_get(
         "push_timestamp",
         "prev_push_revision",
         "performance_tags",
+        "duplicated_summaries_ids",
     }
     assert len(resp.json()["results"][0]["alerts"]) == 1
     assert set(resp.json()["results"][0]["alerts"][0].keys()) == {
@@ -174,6 +175,7 @@ def test_alert_summaries_get_onhold(
         "push_timestamp",
         "prev_push_revision",
         "performance_tags",
+        "duplicated_summaries_ids",
     }
     assert len(resp.json()["results"][0]["alerts"]) == 1
     assert set(resp.json()["results"][0]["alerts"][0].keys()) == {

Diff for: ui/perfherder/alerts/AlertHeader.jsx

@@ -111,25 +111,6 @@ const AlertHeader = ({
               PerfCompare comparison
             </a>
           </Row>
-          {alertSummary.duplicated_summaries_ids.length > 0 && (
-            <Row className="m-0 px-0 py-0">
-              Duplicated summaries:
-              {alertSummary.duplicated_summaries_ids.map((id, index) => (
-                <Link
-                  className="text-dark mr-1"
-                  target="_blank"
-                  to={`./alerts?id=${id}&hideDwnToInv=0`}
-                  id={`alert summary ${id.toString()} title`}
-                  data-testid={`alert summary ${alertSummary.id.toString()} title`}
-                  style={{ marginLeft: '5px' }}
-                >
-                  Alert #{id}
-                  {alertSummary.duplicated_summaries_ids.length - 1 !== index &&
-                    ', '}
-                </Link>
-              ))}
-            </Row>
-          )}
         </Col>
         <Col className="p-0" xs="auto">
           {inEditMode ? (
@@ -260,6 +241,24 @@ const AlertHeader = ({
           />
         </Col>
       </Row>
+      {alertSummary.duplicated_summaries_ids.length > 0 && (
+        <Row>
+          Duplicated summaries:
+          {alertSummary.duplicated_summaries_ids.map((id, index) => (
+            <Link
+              className="text-dark mr-1"
+              target="_blank"
+              to={`./alerts?id=${id}&hideDwnToInv=0`}
+              id={`duplicated alert summary ${id.toString()} `}
+              style={{ marginLeft: '5px' }}
+            >
+              Alert #{id}
+              {alertSummary.duplicated_summaries_ids.length - 1 !== index &&
+                ', '}
+            </Link>
+          ))}
+        </Row>
+      )}
       <Row>
         {performanceTags.length > 0 && (
           <Col className="p-0" xs="auto">