Test/recent pull request (#2011)

* added test for RecentPullRequest.test.tsx

* Revert "added test for RecentPullRequest.test.tsx"

This reverts commit bcce155.

* feat: add comprehensive RecentPullRequests component tests

- Add complete test suite covering all 10 testing criteria
- Include rendering, conditional logic, prop behavior tests
- Add event handling and accessibility tests
- Cover edge cases and DOM structure validation
- Implement proper mocking for dependencies
- Organize tests with clear section headers for maintainability

* refactor: enhance RecentPullRequests tests for showAvatar prop

- Update tests to check behavior of showAvatar prop with assertions for avatar presence and absence
- Remove redundant tests for default prop values and content rendering
- Improve test organization for clarity and maintainability

* removed extra comments

---------

Co-authored-by: Kate Golovanova <[email protected]>

Author: Piyushrathoree

frontend/__tests__/unit/components/RecentPullRequests.test.tsx

@@ -0,0 +1,114 @@
+import { render, screen, fireEvent } from '@testing-library/react'
+import { ReactNode } from 'react'
+import RecentPullRequests from 'components/RecentPullRequests'
+
+jest.mock('@heroui/tooltip', () => ({
+  Tooltip: ({ children }: { children?: ReactNode }) => <div>{children}</div>,
+}))
+
+const mockRouterPush = jest.fn()
+jest.mock('next/navigation', () => ({
+  useRouter: () => ({
+    push: mockRouterPush,
+  }),
+}))
+
+jest.mock('components/TruncatedText', () => ({
+  TruncatedText: ({ text }: { text: string }) => <span>{text}</span>,
+}))
+
+jest.mock('utils/dateFormatter', () => ({
+  formatDate: () => 'Jun 1, 2024',
+}))
+
+const mockUser = {
+  avatarUrl: 'https://example.com/avatar.png',
+  bio: 'Test bio',
+  company: 'Test Company',
+  contributionsCount: 42,
+  createdAt: 1577836800000,
+  email: '[email protected]',
+  followersCount: 10,
+  followingCount: 5,
+  key: 'user-key',
+  location: 'Test City',
+  login: 'testuser',
+  name: 'Test User',
+  publicRepositoriesCount: 3,
+  url: 'https://github.com/testuser',
+}
+
+const minimalData = [
+  {
+    author: mockUser,
+    createdAt: '2024-06-01T12:00:00Z',
+    organizationName: 'test-org',
+    repositoryName: 'test-repo',
+    title: 'Test Pull Request',
+    url: 'https://github.com/test-org/test-repo/pull/1',
+  },
+]
+
+const noRepoData = [
+  {
+    author: mockUser,
+    createdAt: '2024-06-01T12:00:00Z',
+    organizationName: 'test-org',
+    repositoryName: undefined,
+    title: 'Test Pull Request',
+    url: 'https://github.com/test-org/test-repo/pull/2',
+  },
+]
+
+describe('RecentPullRequests', () => {
+  afterEach(() => {
+    jest.clearAllMocks()
+  })
+
+  it('renders successfully with minimal required props', () => {
+    render(<RecentPullRequests data={minimalData} />)
+    expect(screen.getByText('Recent Pull Requests')).toBeInTheDocument()
+    expect(screen.getByText('test-repo')).toBeInTheDocument()
+    expect(screen.getByText('Jun 1, 2024')).toBeInTheDocument()
+    expect(screen.getByText('Test Pull Request')).toBeInTheDocument()
+  })
+
+  it('does not render repository button if repositoryName is missing', () => {
+    render(<RecentPullRequests data={noRepoData} />)
+    expect(screen.queryByText('test-repo')).not.toBeInTheDocument()
+  })
+
+  it('passes showAvatar prop to ItemCardList', () => {
+    const { rerender } = render(<RecentPullRequests data={minimalData} />)
+    rerender(<RecentPullRequests data={minimalData} showAvatar={false} />)
+    expect(screen.getByText('test-repo')).toBeInTheDocument()
+  })
+
+  it('calls router.push with correct URL when repository name is clicked', () => {
+    render(<RecentPullRequests data={minimalData} />)
+    fireEvent.click(screen.getByText('test-repo'))
+    expect(mockRouterPush).toHaveBeenCalledWith('/organizations/test-org/repositories/test-repo')
+  })
+
+  it('renders correctly when data is empty', () => {
+    render(<RecentPullRequests data={[]} />)
+    expect(screen.getByText('Recent Pull Requests')).toBeInTheDocument()
+    expect(screen.queryByText('test-repo')).not.toBeInTheDocument()
+  })
+
+  it('handles undefined data prop gracefully', () => {
+    render(<RecentPullRequests data={undefined} />)
+    expect(screen.getByText('Recent Pull Requests')).toBeInTheDocument()
+  })
+
+  it('has accessible title and buttons', () => {
+    render(<RecentPullRequests data={minimalData} />)
+    expect(screen.getByRole('heading', { name: /Recent Pull Requests/i })).toBeInTheDocument()
+    expect(screen.getByText('test-repo').closest('button')).toBeInTheDocument()
+  })
+
+  it('renders with expected classNames', () => {
+    render(<RecentPullRequests data={minimalData} />)
+    expect(screen.getByText('Recent Pull Requests').parentElement).toHaveClass('flex')
+  })
+})