feat: Add user authentication system #42

Open

matiasdev wants to merge 3 commits into main from feature/auth

matiasdev commented 2 hours ago

Esta PR implementa un sistema completo de autenticación de usuarios siguiendo clean architecture y principios SOLID.

✨ Cambios incluidos:

Factory method para creación de usuarios
Validación de datos con Value Objects
Repository pattern para persistencia
Email service con eventos de dominio

🧪 Testing:

✅ Unit tests para UserService
✅ Integration tests para el flujo completo
✅ Coverage: 95%

tech-lead reviewed 1 hour ago Approved

Excelente trabajo! 🎉 Me gusta especialmente cómo separaste las responsabilidades.

El uso del Factory Pattern para la creación de usuarios es muy limpio. LGTM! 🚀

src/services/UserService.js

+25 -3

              
              -  async register(email, password) {
            
              -    const user = await this.createUser(email, password);
            
              -    return user;
            
              ...
            
              +  async register(userData) {
            
              +    // 1. Validación
            
              +    this.validate(userData);
            
              +    
            
              +    // 2. Creación con Factory
            
              +    const user = User.create(userData);
            
              +    
            
              +    // 3. Persistencia
            
              +    await this.repository.save(user);
            
              +    
            
              +    // 4. Eventos
            
              +    await this.sendWelcomeEmail(user);
            
              +    
            
              +    return user;
            
              +  }

All checks have passed

Tests (18/18 passed)

ESLint (no errors)

Coverage > 90%

Merge via squash and merge

Reviewers

tech-lead

Labels

enhancement clean-code