Skip to content

feat(testing): implement comprehensive test pyramid and fix architecture - #17

Merged
Opselon merged 1 commit into
masterfrom
feat-auth-flow-e2e
Nov 3, 2025
Merged

feat(testing): implement comprehensive test pyramid and fix architecture#17
Opselon merged 1 commit into
masterfrom
feat-auth-flow-e2e

Conversation

@Opselon

@Opselon Opselon commented Nov 3, 2025

Copy link
Copy Markdown
Owner

This commit introduces a comprehensive, multi-layered testing strategy for the MasirYar platform, including unit, integration, and E2E tests.

It also refactors the IdentityService to adhere to Clean Architecture principles by moving the IJwtTokenGenerator and IPasswordHasher interfaces to the Application layer.

Key changes include:

  • A full suite of unit tests for the IdentityService, NotificationService, and CoachingService.
  • Integration tests for the IdentityService and NotificationService using Testcontainers.
  • Integration tests for the CoachingService using WireMock.Net.
  • A full suite of E2E tests for the frontend-web application using Playwright.
  • An updated ci.yml to run the full test suite and a Docker Compose smoke test.

This commit introduces a comprehensive, multi-layered testing strategy for the MasirYar platform, including unit, integration, and E2E tests.

It also refactors the `IdentityService` to adhere to Clean Architecture principles by moving the `IJwtTokenGenerator` and `IPasswordHasher` interfaces to the `Application` layer.

Key changes include:
- A full suite of unit tests for the `IdentityService`, `NotificationService`, and `CoachingService`.
- Integration tests for the `IdentityService` and `NotificationService` using Testcontainers.
- Integration tests for the `CoachingService` using `WireMock.Net`.
- A full suite of E2E tests for the `frontend-web` application using Playwright.
- An updated `ci.yml` to run the full test suite and a Docker Compose smoke test.
@Opselon
Opselon merged commit e58ea44 into master Nov 3, 2025
1 check failed
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.

1 participant