test: unify i18next mocks into centralized helpers

[tomerqodo]

Benchmark PR from qodo-benchmark#437

[greptile-apps]

Greptile Summary

• Refactors i18next mocking across frontend tests by creating a centralized helper function createReactI18nextMock in web/test/i18n-mock.ts to replace duplicate inline mocks
• Updates 20+ test files to use the centralized helper instead of maintaining individual react-i18next mock implementations
• Enhances testing documentation and templates to guide developers towards using the standardized mocking approach rather than creating custom inline mocks

Important Files Changed

Filename	Overview
web/test/i18n-mock.ts	New centralized helper file that provides createReactI18nextMock function with TypeScript support for consistent i18n mocking
web/vitest.setup.ts	Updated global test setup to use centralized i18n mock helper instead of inline implementation
web/testing/testing.md	Enhanced testing documentation with guidance on using centralized i18n helpers over manual mock definitions

Confidence score: 5/5

• This PR is extremely safe to merge with minimal risk as it's purely a refactoring effort that consolidates duplicate code
• Score reflects thorough implementation across the entire frontend test suite with consistent patterns and well-documented changes
• No files require special attention as the changes maintain identical test functionality while improving maintainability

Sequence Diagram

sequenceDiagram
    participant Developer
    participant TestFile
    participant I18nMockHelper
    participant ViTest
    participant GlobalSetup

    Developer->>TestFile: "Write/update test file"
    TestFile->>I18nMockHelper: "Import createReactI18nextMock"
    TestFile->>I18nMockHelper: "Call createReactI18nextMock({ 'key': 'value' })"
    I18nMockHelper->>TestFile: "Return mock config"
    TestFile->>ViTest: "vi.mock('react-i18next', mockConfig)"
    ViTest->>GlobalSetup: "Load vitest.setup.ts"
    GlobalSetup->>ViTest: "Apply global i18n mock"
    ViTest->>TestFile: "Override with custom translations"
    TestFile->>TestFile: "Render component"
    TestFile->>TestFile: "Assert using custom translation"

Loading

[greptile-apps]

Additional Comments (1)

1. web/app/components/plugins/plugin-mutation-model/index.spec.tsx, line 18-27 (link)

   style: These i18n-config mocks may be inconsistent with the centralized i18next mocking approach. Consider moving these to the centralized helpers as well.

   _{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

   Context Used: Context from dashboard - .cursorrules (source). Should these remaining i18n-config mocks also be moved to the centralized helpers for consistency?

_{21 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}