Add timer display on resource monitor view. (#1091)

Co-authored-by: yuxuan-ms <[email protected]>
morganstanley · Jun 5, 2024 · 79a3c29 · 79a3c29
1 parent ab67547
commit 79a3c29
Show file tree

Hide file tree

Showing 7 changed files with 466 additions and 227 deletions.
diff --git a/doc/newsfragments/1384_changed.report_ui.rst b/doc/newsfragments/1384_changed.report_ui.rst
@@ -0,0 +1 @@
+Add event timing view on report UI
diff --git a/testplan/web_ui/testing/src/AssertionPane/AssertionTypes/LogfileMatchAssertion.js b/testplan/web_ui/testing/src/AssertionPane/AssertionTypes/LogfileMatchAssertion.js
@@ -37,10 +37,11 @@ const LogfileMatchAssertion = ({ assertion }) => {
 
   const timeoutMsg =
     (assertion.passed ? "Passed" : "Failed") + ` in ${timeout} seconds.`;
-  const entries = [...results, ...failure].map((entry) => {
+  const entries = [...results, ...failure].map((entry, index) => {
     const { matched, pattern, start_pos: startPos, end_pos: endPos } = entry;
     return (
       <LogfileMatchEntry
+        key={`logmatch${index}`}
         matched={matched}
         pattern={pattern}
         startPos={startPos}

diff --git a/...c/AssertionPane/AssertionTypes/__tests__/__snapshots__/LogfileMatchAssertion.test.js.snap b/...c/AssertionPane/AssertionTypes/__tests__/__snapshots__/LogfileMatchAssertion.test.js.snap
@@ -11,18 +11,21 @@ exports[`LogfileMatchAssertion shallow renders the correct HTML structure 1`] =
   </div>
   <LogfileMatchEntry
     endPos="<inode 110000, position 5>"
+    key="logmatch0"
     matched="okok"
     pattern=".*ok.*"
     startPos="<BOF>"
   />
   <LogfileMatchEntry
     endPos="<inode 110000, position 10>"
+    key="logmatch1"
     matched="okok"
     pattern=".*ok.*"
     startPos="<inode 110000, position 5>"
   />
   <LogfileMatchEntry
     endPos="<inode 110000, position 15>"
+    key="logmatch2"
     matched={null}
     pattern=".*ok.*"
     startPos="<inode 110000, position 10>"