Skip to content

Feat/setup vitest testing, update ESLint and synchronise with main#39

Merged
Pinkish-Warrior merged 8 commits intomainfrom
feat/setup-vitest-testing
Dec 16, 2025
Merged

Feat/setup vitest testing, update ESLint and synchronise with main#39
Pinkish-Warrior merged 8 commits intomainfrom
feat/setup-vitest-testing

Conversation

@Pinkish-Warrior
Copy link
Contributor

This Pull Request introduces comprehensive unit testing for the backend, updates the ESLint configuration for typed linting, and synchronises the feature branch with the latest changes from main.
Key Changes:

  • Vitest Integration:
    • Adds Vitest as the primary unit testing framework for the backend.
    • Includes initial test files for llm.service.ts and ai.controller.ts, along with basic setup and mocks.
    • Configures vitest.config.ts for proper test execution and coverage.
  • ESLint Configuration Update:
    • Updates eslint.config.js and tsconfig.eslint.json to enable typed linting across the project, including test files.
    • Resolves ESLint parsing errors in test files.
    • Fixes a no-undef error for Express types in upload.ts.
  • Documentation:
    • Updates README.md and adds docs/VITEST.md and docs/LINTS.md to reflect the new testing and linting setup and
      available scripts.
  • Main Branch Synchronisation:
    • Successfully merged the latest main branch changes (including recent linting infrastructure updates) into this
      feature branch, resolving all conflicts locally.

Why these changes?
This work significantly enhances code quality and reliability by:

  • Enabling robust unit testing of core backend services and controllers.
  • Ensuring consistent code style and preventing common errors through updated ESLint rules.
  • Keeping the codebase up-to-date with the latest main changes, minimising future integration issues.

Areas for Review:

  • Please review the new test files in backend/src/tests for coverage and correctness.
  • Review the updated ESLint configuration to ensure it aligns with project standards.
  • Verify the README.md, docs/VITEST.md, and docs/LINTS.md updates.

Copy link
Contributor

@MariShopen MariShopen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all good for me

@Pinkish-Warrior Pinkish-Warrior merged commit 867a4b4 into main Dec 16, 2025
1 check passed
@Pinkish-Warrior Pinkish-Warrior deleted the feat/setup-vitest-testing branch December 16, 2025 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments