Fix files and add tests

Signed-off-by: Adhitya Mamallan <[email protected]>

Author: adhityamamallan

src/views/workflow-history-v2/workflow-history-details-row-json/__tests__/workflow-history-details-row-json.test.tsx

@@ -0,0 +1,23 @@
+import { render, screen } from '@/test-utils/rtl';
+
+import WorkflowHistoryDetailsRowJson from '../workflow-history-details-row-json';
+
+describe(WorkflowHistoryDetailsRowJson.name, () => {
+  it('renders the stringified JSON value', () => {
+    render(
+      <WorkflowHistoryDetailsRowJson
+        value={{ key: 'value', nested: { number: 123 } }}
+        isNegative={false}
+        label="test-label"
+        domain="test-domain"
+        cluster="test-cluster"
+        workflowId="test-workflow-id"
+        runId="test-run-id"
+      />
+    );
+
+    expect(
+      screen.getByText('{"key":"value","nested":{"number":123}}')
+    ).toBeInTheDocument();
+  });
+});

src/views/workflow-history-v2/workflow-history-details-row-tooltip-json/__tests__/workflow-history-details-row-tooltip-json.test.tsx

@@ -0,0 +1,38 @@
+import { render, screen } from '@/test-utils/rtl';
+
+import WorkflowHistoryDetailsRowTooltipJson from '../workflow-history-details-row-tooltip-json';
+
+jest.mock(
+  '@/views/workflow-history/workflow-history-event-details-json/workflow-history-event-details-json',
+  () =>
+    jest.fn(({ entryValue, isNegative }) => (
+      <div data-testid="event-details-json">
+        Event Details Json: {JSON.stringify(entryValue)}
+        {isNegative && ' (negative)'}
+      </div>
+    ))
+);
+
+describe(WorkflowHistoryDetailsRowTooltipJson.name, () => {
+  it('renders the label and passes value to WorkflowHistoryEventDetailsJson', () => {
+    render(
+      <WorkflowHistoryDetailsRowTooltipJson
+        value={{ key: 'value', nested: { number: 123 } }}
+        label="test-label"
+        isNegative={false}
+        domain="test-domain"
+        cluster="test-cluster"
+        workflowId="test-workflow-id"
+        runId="test-run-id"
+      />
+    );
+
+    expect(screen.getByText('test-label')).toBeInTheDocument();
+    expect(screen.getByTestId('event-details-json')).toBeInTheDocument();
+    expect(
+      screen.getByText(
+        /Event Details Json: \{"key":"value","nested":\{"number":123\}\}/
+      )
+    ).toBeInTheDocument();
+  });
+});

src/views/workflow-history-v2/workflow-history-event-group/workflow-history-event-group.tsx

@@ -85,20 +85,13 @@ export default function WorkflowHistoryEventGroup({
         ? [
             getSummaryTabContentEntry({
               groupId: eventGroup.firstEventId ?? 'unknown',
-              summaryDetails: summaryDetailsEntries.flatMap(
-                ([_eventId, { eventDetails }]) => eventDetails
-              ),
+              summaryDetails: groupSummaryDetails,
             }),
           ]
         : []),
       ...groupDetailsEntries,
     ],
-    [
-      eventGroup.firstEventId,
-      groupDetailsEntries,
-      summaryDetailsEntries,
-      groupSummaryDetails.length,
-    ]
+    [eventGroup.firstEventId, groupDetailsEntries, groupSummaryDetails]
   );
 
   return (