3305 - Make the length of the short revision hash constant across Tre…

…eherder (mozilla#7914) * 3305 - Make the length of the short revision hash constant with bugzilla * 3305 - Make the length of the short revision hash constant with bugzilla
florinbilt · Mar 1, 2024 · 2054b20 · 2054b20
1 parent c644257
commit 2054b20
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/tests/ui/perfherder/graphs-view/graphs_view_test.jsx b/tests/ui/perfherder/graphs-view/graphs_view_test.jsx
@@ -206,7 +206,7 @@ test('Using select query param displays tooltip for correct datapoint', async ()
   const graphTooltip = await waitFor(() => getByTestId('graphTooltip'));
   const expectedRevision = '3afb892abb74c6d281f3e66431408cbb2e16b8c4';
   const revision = await waitFor(() =>
-    getByText(expectedRevision.slice(0, 13)),
+    getByText(expectedRevision.slice(0, 12)),
   );
   const repoName = await waitFor(() => getByTestId('repoName'));
   const platform = await waitFor(() => getByTestId('platform'));
@@ -226,7 +226,7 @@ test('Using select query param displays tooltip for correct datapoint with repli
   const graphTooltip = await waitFor(() => getByTestId('graphTooltip'));
   const expectedRevision = '3afb892abb74c6d281f3e66431408cbb2e16b8c4';
   const revision = await waitFor(() =>
-    getByText(expectedRevision.slice(0, 13)),
+    getByText(expectedRevision.slice(0, 12)),
   );
   const repoName = await waitFor(() => getByTestId('repoName'));
   const platform = await waitFor(() => getByTestId('platform'));

diff --git a/ui/perfherder/graphs/GraphTooltip.jsx b/ui/perfherder/graphs/GraphTooltip.jsx
@@ -212,7 +212,7 @@ const GraphTooltip = ({
           <div>
             <span>
               <a href={pushUrl} target="_blank" rel="noopener noreferrer">
-                {dataPointDetails.revision.slice(0, 13)}
+                {dataPointDetails.revision.slice(0, 12)}
               </a>{' '}
               {(dataPointDetails.jobId || prevRevision) && '('}
               {dataPointDetails.jobId && (