Rename workflow-summary-tab to workflow-summary (#994)

* Move summary tab and update imports

* update component names

Author: Assem-Uber

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

@@ -4,7 +4,7 @@ import { render } from '@/test-utils/rtl';
 
 import losslessJsonStringify from '@/utils/lossless-json-stringify';
 
-import WorkflowSummaryTabJsonView from '../workflow-history-event-details-json';
+import WorkflowHistoryEventDetailsJson from '../workflow-history-event-details-json';
 
 jest.mock('@/components/copy-text-button/copy-text-button', () =>
   jest.fn(({ textToCopy }) => <div>Copy Button: {textToCopy}</div>)
@@ -22,15 +22,15 @@ describe('WorkflowHistoryEventDetailsJson', () => {
 
   it('renders correctly with initial props', () => {
     const { getByText } = render(
-      <WorkflowSummaryTabJsonView entryValue={losslessInputJson} />
+      <WorkflowHistoryEventDetailsJson entryValue={losslessInputJson} />
     );
 
     expect(getByText('PrettyJson Mock')).toBeInTheDocument();
   });
 
   it('renders copy text button and pass the correct text', () => {
     const { getByText } = render(
-      <WorkflowSummaryTabJsonView entryValue={losslessInputJson} />
+      <WorkflowHistoryEventDetailsJson entryValue={losslessInputJson} />
     );
 
     const copyButton = getByText(/Copy Button/);

src/views/workflow-page/config/workflow-page-tabs.config.ts

@@ -11,7 +11,7 @@ import WorkflowDiagnostics from '@/views/workflow-diagnostics/workflow-diagnosti
 import WorkflowHistoryWrapper from '@/views/workflow-history/workflow-history-wrapper/workflow-history-wrapper';
 import WorkflowQueries from '@/views/workflow-queries/workflow-queries';
 import WorkflowStackTrace from '@/views/workflow-stack-trace/workflow-stack-trace';
-import WorkflowSummaryTab from '@/views/workflow-summary-tab/workflow-summary-tab';
+import WorkflowSummary from '@/views/workflow-summary/workflow-summary';
 
 import getWorkflowPageErrorConfig from '../helpers/get-workflow-page-error-config';
 import WorkflowPageDiagnosticsBadge from '../workflow-page-diagnostics-badge/workflow-page-diagnostics-badge';
@@ -24,7 +24,7 @@ const workflowPageTabsConfig: WorkflowPageTabsConfig<
   summary: {
     title: 'Summary',
     artwork: MdListAlt,
-    content: WorkflowSummaryTab,
+    content: WorkflowSummary,
     getErrorConfig: (err) =>
       getWorkflowPageErrorConfig(
         err,

src/views/workflow-summary-tab/__tests__/workflow-summary-tab.test.tsx renamed to src/views/workflow-summary/__tests__/workflow-summary.test.tsx

@@ -7,23 +7,22 @@ import { type GetWorkflowExecutionHistoryResponse } from '@/__generated__/proto-
 import { startWorkflowExecutionEvent } from '@/views/workflow-history/__fixtures__/workflow-history-single-events';
 import { type WorkflowPageTabContentProps } from '@/views/workflow-page/workflow-page-tab-content/workflow-page-tab-content.types';
 
-import WorkflowSummaryTab from '../workflow-summary-tab';
+import WorkflowSummary from '../workflow-summary';
 
-jest.mock('../workflow-summary-tab-details/workflow-summary-tab-details', () =>
-  jest.fn(() => <div>MockWorkflowSummaryTabDetails</div>)
+jest.mock('../workflow-summary-details/workflow-summary-details', () =>
+  jest.fn(() => <div>MockWorkflowSummaryDetails</div>)
 );
 
-jest.mock(
-  '../workflow-summary-tab-json-view/workflow-summary-tab-json-view',
-  () => jest.fn(() => <div>MockWorkflowSummaryTabJsonView</div>)
+jest.mock('../workflow-summary-json-view/workflow-summary-json-view', () =>
+  jest.fn(() => <div>MockWorkflowSummaryJsonView</div>)
 );
 
 jest.mock(
-  '../workflow-summary-tab-diagnostics-banner/workflow-summary-tab-diagnostics-banner',
-  () => jest.fn(() => <div>MockWorkflowSummaryTabDiagnosticsBanner</div>)
+  '../workflow-summary-diagnostics-banner/workflow-summary-diagnostics-banner',
+  () => jest.fn(() => <div>MockWorkflowSummaryDiagnosticsBanner</div>)
 );
 
-describe('WorkflowSummaryTab', () => {
+describe('WorkflowSummary', () => {
   beforeEach(() => {
     jest.clearAllMocks();
   });
@@ -39,7 +38,7 @@ describe('WorkflowSummaryTab', () => {
   it('should render tab deatils, JSON view, and diagnostics banner', async () => {
     render(
       <Suspense>
-        <WorkflowSummaryTab params={params} />
+        <WorkflowSummary params={params} />
       </Suspense>,
       {
         endpointsMocks: [
@@ -70,13 +69,13 @@ describe('WorkflowSummaryTab', () => {
       }
     );
     expect(
-      await screen.findByText('MockWorkflowSummaryTabDetails')
+      await screen.findByText('MockWorkflowSummaryDetails')
     ).toBeInTheDocument();
     expect(
-      await screen.findByText('MockWorkflowSummaryTabJsonView')
+      await screen.findByText('MockWorkflowSummaryJsonView')
     ).toBeInTheDocument();
     expect(
-      await screen.findByText('MockWorkflowSummaryTabDiagnosticsBanner')
+      await screen.findByText('MockWorkflowSummaryDiagnosticsBanner')
     ).toBeInTheDocument();
   });
 });

src/views/workflow-summary-tab/config/workflow-summary-tab-details.config.ts renamed to src/views/workflow-summary/config/workflow-summary-details.config.ts

@@ -6,9 +6,9 @@ import WorkflowStatusTag from '@/views/shared/workflow-status-tag/workflow-statu
 import getWorkflowStatusTagProps from '@/views/workflow-page/helpers/get-workflow-status-tag-props';
 
 import WorkflowEventDetailsExecutionLink from '../../shared/workflow-event-details-execution-link/workflow-event-details-execution-link';
-import { type WorkflowSummaryTabDetailsConfig } from '../workflow-summary-tab-details/workflow-summary-tab-details.types';
+import { type WorkflowSummaryDetailsConfig } from '../workflow-summary-details/workflow-summary-details.types';
 
-const workflowSummaryTabDetailsConfig: WorkflowSummaryTabDetailsConfig[] = [
+const workflowSummaryDetailsConfig: WorkflowSummaryDetailsConfig[] = [
   {
     key: 'status',
     getLabel: () => 'Status',
@@ -146,4 +146,4 @@ const workflowSummaryTabDetailsConfig: WorkflowSummaryTabDetailsConfig[] = [
   },
 ];
 
-export default workflowSummaryTabDetailsConfig;
+export default workflowSummaryDetailsConfig;

src/views/workflow-summary-tab/helpers/__tests__/get-workflow-result-json.test.ts renamed to src/views/workflow-summary/helpers/__tests__/get-workflow-result-json.test.ts

@@ -10,14 +10,14 @@ import {
   startWorkflowExecutionEvent,
 } from '@/views/workflow-history/__fixtures__/workflow-history-single-events';
 
-import WorkflowSummaryTabDetails from '../workflow-summary-tab-details';
+import WorkflowSummaryDetails from '../workflow-summary-details';
 import {
-  type WorkflowSummaryTabDetailsConfig,
+  type WorkflowSummaryDetailsConfig,
   type Props,
-} from '../workflow-summary-tab-details.types';
+} from '../workflow-summary-details.types';
 
 jest.mock(
-  '../../config/workflow-summary-tab-details.config',
+  '../../config/workflow-summary-details.config',
   () =>
     [
       {
@@ -37,7 +37,7 @@ jest.mock(
         getLabel: () => 'Hidden Label 3',
         valueComponent: () => <span>Hidden Value 3</span>,
       },
-    ] satisfies WorkflowSummaryTabDetailsConfig[]
+    ] satisfies WorkflowSummaryDetailsConfig[]
 );
 
 const params: Props['decodedPageUrlParams'] = {
@@ -56,7 +56,7 @@ const mockWorkflowDetails: DescribeWorkflowResponse = {
   workflowExecutionInfo: null,
 };
 
-describe('WorkflowSummaryTabDetails', () => {
+describe('WorkflowSummaryDetails', () => {
   // TODO @assem.hafez enhance typing for formattedFirstHistoryEvent
   //@ts-expect-error - TS is complaining about the type of formattedFirstHistoryEvent
   const formattedFirstHistoryEvent: FormattedHistoryEventForType<'WorkflowExecutionStarted'> =
@@ -66,7 +66,7 @@ describe('WorkflowSummaryTabDetails', () => {
   );
   it('should render workflow type name from firstHistoryEvent', () => {
     render(
-      <WorkflowSummaryTabDetails
+      <WorkflowSummaryDetails
         firstHistoryEvent={startWorkflowExecutionEvent}
         closeHistoryEvent={completeWorkflowExecutionEvent}
         formattedFirstHistoryEvent={formattedFirstHistoryEvent}
@@ -82,7 +82,7 @@ describe('WorkflowSummaryTabDetails', () => {
 
   it('should render all detail rows that are not hidden', () => {
     render(
-      <WorkflowSummaryTabDetails
+      <WorkflowSummaryDetails
         firstHistoryEvent={startWorkflowExecutionEvent}
         closeHistoryEvent={completeWorkflowExecutionEvent}
         formattedFirstHistoryEvent={formattedFirstHistoryEvent}
@@ -100,7 +100,7 @@ describe('WorkflowSummaryTabDetails', () => {
 
   it('should not render detail rows that are hidden', () => {
     render(
-      <WorkflowSummaryTabDetails
+      <WorkflowSummaryDetails
         firstHistoryEvent={startWorkflowExecutionEvent}
         closeHistoryEvent={completeWorkflowExecutionEvent}
         formattedFirstHistoryEvent={formattedFirstHistoryEvent}

src/views/workflow-summary-tab/helpers/get-workflow-result-json.ts renamed to src/views/workflow-summary/helpers/get-workflow-result-json.ts

@@ -5,12 +5,12 @@ import { LabelMedium } from 'baseui/typography';
 
 import useStyletronClasses from '@/hooks/use-styletron-classes';
 
-import workflowSummaryTabDetailsConfig from '../config/workflow-summary-tab-details.config';
+import workflowSummaryDetailsConfig from '../config/workflow-summary-details.config';
 
-import { cssStyles } from './workflow-summary-tab-details.styles';
-import { type Props } from './workflow-summary-tab-details.types';
+import { cssStyles } from './workflow-summary-details.styles';
+import { type Props } from './workflow-summary-details.types';
 
-export default function WorkflowSummaryTabDetails({
+export default function WorkflowSummaryDetails({
   firstHistoryEvent,
   closeHistoryEvent,
   formattedFirstHistoryEvent,
@@ -32,7 +32,7 @@ export default function WorkflowSummaryTabDetails({
         </LabelMedium>
       </div>
       <div>
-        {workflowSummaryTabDetailsConfig
+        {workflowSummaryDetailsConfig
           .filter(
             (c) =>
               !c.hide ||

src/views/workflow-summary-tab/workflow-summary-tab-details/__tests__/workflow-summary-tab-details.test.tsx renamed to src/views/workflow-summary/workflow-summary-details/__tests__/workflow-summary-details.test.tsx

@@ -17,11 +17,11 @@ export type Props = {
   decodedPageUrlParams: WorkflowPageTabContentProps['params'];
 };
 
-export type WorkflowSummaryTabDetailsComponent =
+export type WorkflowSummaryDetailsComponent =
   | keyof JSX.IntrinsicElements
   | React.JSXElementConstructor<any>;
 
-export type WorkflowSummaryTabDetailsConfig = {
+export type WorkflowSummaryDetailsConfig = {
   key: string;
   getLabel: () => string;
   valueComponent: React.ComponentType<{