test: enhance test suite for coverage

Author: pganesh-apphelix

src/optimizer-page/scan-results/PreviousRunLinkTable.test.tsx

@@ -0,0 +1,87 @@
+import { render, screen, fireEvent } from '@testing-library/react';
+import { IntlProvider } from '@edx/frontend-platform/i18n';
+import PreviousRunLinkTable from './PreviousRunLinkTable';
+
+const intlWrapper = (ui: React.ReactElement) => render(
+  <IntlProvider locale="en" messages={{}}>
+    {ui}
+  </IntlProvider>,
+);
+
+describe('PreviousRunLinkTable', () => {
+  const mockUnitWithLinks = {
+    id: 'unit-2',
+    displayName: 'Unit with previous run links',
+    blocks: [
+      {
+        id: 'block-1',
+        displayName: 'Block with links',
+        url: 'http://example.com',
+        brokenLinks: [],
+        lockedLinks: [],
+        externalForbiddenLinks: [],
+        previousRunLinks: ['https://prev-link.com/1'],
+      },
+    ],
+  };
+
+  const mockUnitNoLinks = {
+    id: 'unit-1',
+    displayName: 'Unit without links',
+    blocks: [
+      {
+        id: 'block-1',
+        displayName: 'Block without links',
+        url: 'http://example.com',
+        brokenLinks: [],
+        lockedLinks: [],
+        externalForbiddenLinks: [],
+        previousRunLinks: [],
+      },
+    ],
+  };
+
+  it('renders table with previous run links', () => {
+    intlWrapper(<PreviousRunLinkTable unit={mockUnitWithLinks} />);
+    expect(screen.getByText('Unit with previous run links')).toBeInTheDocument();
+
+    mockUnitWithLinks.blocks[0].previousRunLinks.forEach(link => {
+      expect(screen.getByText(link)).toBeInTheDocument();
+    });
+  });
+
+  it('renders nothing if no previous run links', () => {
+    const { container } = intlWrapper(<PreviousRunLinkTable unit={mockUnitNoLinks} />);
+    expect(container.firstChild).toBeNull();
+  });
+
+  it('opens previous run link in new tab on click', () => {
+    const originalOpen = window.open;
+    window.open = jest.fn();
+    intlWrapper(<PreviousRunLinkTable unit={mockUnitWithLinks} />);
+    fireEvent.click(screen.getByText('https://prev-link.com/1'));
+    expect(window.open).toHaveBeenCalledWith('https://prev-link.com/1', '_blank');
+    window.open = originalOpen;
+  });
+
+  it('handles missing displayName gracefully', () => {
+    const unit = {
+      id: 'unit-3',
+      displayName: 'Unit with test links',
+      blocks: [
+        {
+          id: 'block-3',
+          displayName: '',
+          url: 'http://example.com/block3',
+          brokenLinks: [],
+          lockedLinks: [],
+          externalForbiddenLinks: [],
+          previousRunLinks: ['https://prev-link.com/1'],
+        },
+      ],
+    };
+    intlWrapper(<PreviousRunLinkTable unit={unit} />);
+    expect(screen.getByText('Go to block')).toBeInTheDocument();
+    expect(screen.getByText('https://prev-link.com/1')).toBeInTheDocument();
+  });
+});

src/optimizer-page/scan-results/PreviousRunSectionCollapsible.test.tsx

@@ -0,0 +1,52 @@
+import { render, screen, fireEvent } from '@testing-library/react';
+import React from 'react';
+import PreviousRunSectionCollapsible from './PreviousRunSectionCollapsible';
+
+describe('PreviousRunSectionCollapsible', () => {
+  const defaultProps = {
+    index: 1,
+    handleToggle: jest.fn(),
+    isOpen: false,
+    hasPrevAndIsOpen: false,
+    hasNextAndIsOpen: false,
+    title: 'Section Title',
+    children: <div>Section Content</div>,
+    previousRunLinksCount: 3,
+    className: 'test-class',
+  };
+
+  it('renders with closed state and correct title/count', () => {
+    render(<PreviousRunSectionCollapsible {...defaultProps} />);
+    expect(screen.getByText('Section Title')).toBeInTheDocument();
+    expect(screen.getByText('3')).toBeInTheDocument();
+  });
+
+  it('renders with open state and shows children', () => {
+    render(<PreviousRunSectionCollapsible {...defaultProps} isOpen />);
+    expect(screen.getByText('Section Content')).toBeInTheDocument();
+  });
+
+  it('shows dash when previousRunLinksCount is 0', () => {
+    render(<PreviousRunSectionCollapsible {...defaultProps} previousRunLinksCount={0} />);
+    expect(screen.getByText('-')).toBeInTheDocument();
+  });
+
+  it('calls handleToggle with index when header is clicked', () => {
+    const handleToggle = jest.fn();
+    render(<PreviousRunSectionCollapsible {...defaultProps} handleToggle={handleToggle} />);
+    // Click the header icon (which is inside the header div)
+    const header = screen.getByText('Section Title').closest('.section-collapsible-header-item');
+    if (header) {
+      fireEvent.click(header);
+    } else {
+      // fallback: click the whole header area
+      fireEvent.click(screen.getByText('Section Title'));
+    }
+    expect(handleToggle).toHaveBeenCalledWith(1);
+  });
+
+  it('applies custom className', () => {
+    render(<PreviousRunSectionCollapsible {...defaultProps} className="my-custom-class" />);
+    expect(screen.getByText('Section Title').parentElement?.parentElement).toHaveClass('my-custom-class');
+  });
+});