diff --git a/.dockerignore b/.dockerignore new file mode 100644 index 0000000000..f7ccb323c6 --- /dev/null +++ b/.dockerignore @@ -0,0 +1,50 @@ +# Maven +target/ +pom.xml.tag +pom.xml.releaseBackup +pom.xml.versionsBackup +pom.xml.next +release.properties +dependency-reduced-pom.xml +buildNumber.properties +.mvn/timing.properties +.mvn/wrapper/maven-wrapper.jar + +# IDE +.idea/ +*.iml +*.iws +*.ipr +.vscode/ +.project +.classpath +.settings/ +.metadata/ + +# Git +.git/ +.gitignore +.gitattributes + +# Docker +Dockerfile +.dockerignore +docker-compose.yml + +# Documentation +*.md +!README.md + +# Logs +*.log + +# OS +.DS_Store +Thumbs.db + +# Build artifacts +*.jar +*.war +*.ear +*.class + diff --git a/.gitattributes b/.gitattributes new file mode 100644 index 0000000000..3b41682ac5 --- /dev/null +++ b/.gitattributes @@ -0,0 +1,2 @@ +/mvnw text eol=lf +*.cmd text eol=crlf diff --git a/.gitignore b/.gitignore index 67045665db..38b8d9ab31 100644 --- a/.gitignore +++ b/.gitignore @@ -1,104 +1,37 @@ -# Logs -logs +HELP.md +target/ +!.mvn/wrapper/maven-wrapper.jar +!**/src/main/**/target/ +!**/src/test/**/target/ + +### STS ### +.apt_generated +.classpath +.factorypath +.project +.settings +.springBeans +.sts4-cache + +### IntelliJ IDEA ### +.idea +*.iws +*.iml +*.ipr + +### NetBeans ### +/nbproject/private/ +/nbbuild/ +/dist/ +/nbdist/ +/.nb-gradle/ +build/ +!**/src/main/**/build/ +!**/src/test/**/build/ + +### VS Code ### +.vscode/ + +### Docker ### *.log -npm-debug.log* -yarn-debug.log* -yarn-error.log* -lerna-debug.log* -# Diagnostic reports (https://nodejs.org/api/report.html) -report.[0-9]*.[0-9]*.[0-9]*.[0-9]*.json - -# Runtime data -pids -*.pid -*.seed -*.pid.lock - -# Directory for instrumented libs generated by jscoverage/JSCover -lib-cov - -# Coverage directory used by tools like istanbul -coverage -*.lcov - -# nyc test coverage -.nyc_output - -# Grunt intermediate storage (https://gruntjs.com/creating-plugins#storing-task-files) -.grunt - -# Bower dependency directory (https://bower.io/) -bower_components - -# node-waf configuration -.lock-wscript - -# Compiled binary addons (https://nodejs.org/api/addons.html) -build/Release - -# Dependency directories -node_modules/ -jspm_packages/ - -# TypeScript v1 declaration files -typings/ - -# TypeScript cache -*.tsbuildinfo - -# Optional npm cache directory -.npm - -# Optional eslint cache -.eslintcache - -# Microbundle cache -.rpt2_cache/ -.rts2_cache_cjs/ -.rts2_cache_es/ -.rts2_cache_umd/ - -# Optional REPL history -.node_repl_history - -# Output of 'npm pack' -*.tgz - -# Yarn Integrity file -.yarn-integrity - -# dotenv environment variables file -.env -.env.test - -# parcel-bundler cache (https://parceljs.org/) -.cache - -# Next.js build output -.next - -# Nuxt.js build / generate output -.nuxt -dist - -# Gatsby files -.cache/ -# Comment in the public line in if your project uses Gatsby and *not* Next.js -# https://nextjs.org/blog/next-9-1#public-directory-support -# public - -# vuepress build output -.vuepress/dist - -# Serverless directories -.serverless/ - -# FuseBox cache -.fusebox/ - -# DynamoDB Local files -.dynamodb/ - -# TernJS port file -.tern-port diff --git a/.mvn/wrapper/maven-wrapper.properties b/.mvn/wrapper/maven-wrapper.properties new file mode 100644 index 0000000000..8dea6c227c --- /dev/null +++ b/.mvn/wrapper/maven-wrapper.properties @@ -0,0 +1,3 @@ +wrapperVersion=3.3.4 +distributionType=only-script +distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.9.12/apache-maven-3.9.12-bin.zip diff --git a/Dockerfile b/Dockerfile new file mode 100644 index 0000000000..5fcef25b09 --- /dev/null +++ b/Dockerfile @@ -0,0 +1,25 @@ +# Build stage +FROM maven:3.9.5-eclipse-temurin-21 AS build +WORKDIR /app + +# Copy parent POM and common module +COPY pom.xml . +COPY common common + +# Copy transaction-service +COPY transaction-service transaction-service + +# Build only transaction-service and its dependencies +RUN mvn clean package -DskipTests -pl transaction-service -am + +# Run stage +FROM eclipse-temurin:21-jre-alpine +WORKDIR /app + +COPY --from=build /app/transaction-service/target/*.jar app.jar + +EXPOSE 8080 + +ENTRYPOINT ["java", "-jar", "app.jar"] + + diff --git a/README.md b/README.md index b067a71026..fef131ef0c 100644 --- a/README.md +++ b/README.md @@ -1,82 +1,484 @@ -# Yape Code Challenge :rocket: +# Yape Challenge - Microservicios de Transacciones -Our code challenge will let you marvel us with your Jedi coding skills :smile:. +Sistema de microservicios para gestión de transacciones con validación anti-fraude, **optimizado para alto volumen**. -Don't forget that the proper way to submit your work is to fork the repo and create a PR :wink: ... have fun !! +## ✨ Características Principales -- [Problem](#problem) -- [Tech Stack](#tech_stack) -- [Send us your challenge](#send_us_your_challenge) +- ✅ **Arquitectura de Microservicios** con comunicación asíncrona vía Kafka +- ✅ **Event Sourcing** implementado para auditoría completa de transacciones +- ✅ **CQRS** (Command Query Responsibility Segregation) con Command Bus y Query Bus +- ✅ **Redis Cache Distribuido** para lecturas de alta velocidad (5-20ms) +- ✅ **Optimizado para Alto Volumen** (5K-10K lecturas/seg, 100-200 escrituras/seg) +- ✅ **API REST** con Spring Boot 3.2.0 y Java 21 +- ✅ **Validación Anti-Fraude** en tiempo real (rechaza transacciones > 1000) +- ✅ **PostgreSQL 16** con JSONB para Event Store +- ✅ **Apache Kafka** para mensajería asíncrona entre servicios +- ✅ **Docker y Docker Compose** para deployment simplificado +- ✅ **HikariCP** con connection pooling optimizado (50 conexiones) +- ✅ **Event Store API** para auditoría y debugging de eventos -# Problem +## 🏗️ Arquitectura -Every time a financial transaction is created it must be validated by our anti-fraud microservice and then the same service sends a message back to update the transaction status. -For now, we have only three transaction statuses: +Este proyecto implementa una arquitectura de microservicios con los siguientes componentes: -
    -
  1. pending
    2. -
  2. approved
    3. -
  3. rejected
    4. -
+- **Transaction Service**: API REST para gestión de transacciones con **Event Sourcing** y **Redis Cache** +- **Anti-Fraud Service**: Servicio de validación anti-fraude +- **Common**: Librería compartida con DTOs y utilidades +- **PostgreSQL**: Base de datos para transacciones y Event Store +- **Redis**: Caché distribuido para optimización de lecturas +- **Kafka**: Message broker para comunicación asíncrona entre servicios -Every transaction with a value greater than 1000 should be rejected. +### 🎯 Patrones Implementados -```mermaid - flowchart LR - Transaction -- Save Transaction with pending Status --> transactionDatabase[(Database)] - Transaction --Send transaction Created event--> Anti-Fraud - Anti-Fraud -- Send transaction Status Approved event--> Transaction - Anti-Fraud -- Send transaction Status Rejected event--> Transaction - Transaction -- Update transaction Status event--> transactionDatabase[(Database)] +- **Event Sourcing**: Todos los cambios se almacenan como eventos inmutables +- **CQRS**: Separación entre comandos (escritura) y queries (lectura) +- **Domain Events**: Eventos del dominio del negocio +- **Event Store**: Persistencia de eventos en PostgreSQL con JSONB +- **Distributed Caching**: Redis para lecturas de alta velocidad (5-20ms) +- **Cache-Aside Pattern**: Estrategia de caché con invalidación automática +- **Read Model**: Proyección optimizada para consultas +- **Message Broker**: Kafka para integración entre servicios + +## 📁 Estructura del Proyecto + +``` +yape-challenge/ +├── transaction-service/ # Microservicio de transacciones (EVENT SOURCING + CQRS) +│ ├── Dockerfile +│ ├── README.md +│ └── src/ +│ ├── main/ +│ │ ├── java/com/yape/challenge/transaction/ +│ │ │ ├── TransactionServiceApplication.java +│ │ │ ├── application/ # CQRS - Commands & Queries +│ │ │ │ ├── bus/ # Command Bus & Query Bus +│ │ │ │ ├── command/ # CreateTransactionCommand +│ │ │ │ ├── query/ # GetTransactionQuery +│ │ │ │ ├── handler/ # Command & Query Handlers +│ │ │ │ └── dto/ # Request/Response DTOs +│ │ │ ├── domain/ +│ │ │ │ ├── entity/ # Transaction (JPA Entity) +│ │ │ │ ├── event/ # Domain Events +│ │ │ │ │ ├── TransactionCreatedEvent +│ │ │ │ │ ├── TransactionStatusUpdatedEvent +│ │ │ │ │ └── TransactionDomainEvent (base) +│ │ │ │ └── service/ # TransactionAggregateService +│ │ │ ├── infrastructure/ +│ │ │ │ ├── eventstore/ # Event Store Implementation +│ │ │ │ │ ├── EventStore +│ │ │ │ │ ├── DomainEventEntity (JPA) +│ │ │ │ │ └── DomainEventRepository +│ │ │ │ ├── repository/ # JPA Repositories +│ │ │ │ │ └── TransactionRepository +│ │ │ │ ├── kafka/ # Kafka Producers/Consumers +│ │ │ │ │ ├── TransactionProducer +│ │ │ │ │ └── TransactionStatusConsumer +│ │ │ │ └── config/ # Redis, Kafka, JPA Config +│ │ │ └── presentation/ # REST Controllers +│ │ │ ├── controller/ +│ │ │ │ ├── TransactionController +│ │ │ │ └── EventStoreController +│ │ │ └── exception/ # Global Exception Handler +│ │ └── resources/ +│ │ ├── application.yml # Local profile +│ │ ├── application-docker.yml # Docker profile +│ │ └── db/ +│ │ └── data.sql # Initial Data (Transfer Types) +│ └── test/ +│ └── java/ # Unit & Integration Tests +├── antifraud-service/ # Microservicio anti-fraude +│ ├── Dockerfile +│ ├── README.md +│ └── src/ +│ ├── main/ +│ │ ├── java/com/yape/challenge/antifraud/ +│ │ │ ├── AntiFraudApplication.java +│ │ │ ├── service/ # AntiFraudService +│ │ │ ├── kafka/ # Kafka Consumer/Producer +│ │ │ └── config/ # Kafka Configuration +│ │ └── resources/ +│ │ ├── application.yml +│ │ └── application-docker.yml +│ └── test/ +│ └── java/ # Unit Tests +├── common/ # Módulo compartido (DTOs y Kafka) +│ ├── pom.xml +│ └── src/ +│ └── main/ +│ └── java/com/yape/challenge/common/ +│ ├── dto/ # DTOs compartidos +│ │ ├── TransactionCreatedEvent +│ │ ├── TransactionStatusEvent +│ │ └── TransactionStatus (enum) +│ └── kafka/ +│ └── KafkaTopics # Nombres de topics +├── docker-compose.yml # Orquestación de servicios +│ # Servicios: postgres, redis, zookeeper, kafka, kafka-ui, +│ # transaction-service, antifraud-service +├── pom.xml # POM principal del monorepo +├── README.md # Esta documentación +└── Yape-Challenge.postman_collection.json # Colección de Postman ``` -# Tech Stack +### Descripción de Módulos -
    -
  1. Node. You can use any framework you want (i.e. Nestjs with an ORM like TypeOrm or Prisma)
    2. -
  2. Any database
    3. -
  3. Kafka
    4. -
+#### Transaction Service +- **Puerto**: 8080 +- **Base de datos**: PostgreSQL (transacciones + event store) +- **Caché**: Redis (lecturas optimizadas) +- **Patrones**: Event Sourcing, CQRS, Domain Events, Cache-Aside -We do provide a `Dockerfile` to help you get started with a dev environment. +#### Anti-Fraud Service +- **Puerto**: 8081 +- **Función**: Validación de transacciones en tiempo real +- **Regla**: Rechaza transacciones con valor > 1000 -You must have two resources: +#### Common +- Módulo compartido entre servicios +- DTOs para eventos de Kafka +- Constantes de topics de Kafka +- No tiene puerto, es una librería -1. Resource to create a transaction that must containt: +## 🚀 Inicio Rápido + +### Prerrequisitos + +- Docker y Docker Compose +- Java 21 (si quieres ejecutar sin Docker) +- Maven 3.9+ (si quieres compilar localmente) + +### Ejecutar todo el stack + +```bash +# Construir y ejecutar todos los servicios +docker compose up --build + +# O en modo detached (background) +docker compose up -d --build +``` + +Los servicios estarán disponibles en: +- **Transaction Service**: http://localhost:8080 +- **Anti-Fraud Service**: http://localhost:8081 +- **Kafka UI**: http://localhost:8090 +- **PostgreSQL**: localhost:5432 (usuario: yapeuser, db: yape_transactions) +- **Redis**: localhost:6379 + +### Endpoints de Monitoreo (Actuator) + +#### Transaction Service +- Health: http://localhost:8080/actuator/health +- Metrics: http://localhost:8080/actuator/metrics + +#### Anti-Fraud Service +- Health: http://localhost:8081/actuator/health +- Metrics: http://localhost:8081/actuator/metrics + +### Ejecutar servicios individuales + +Cada microservicio puede ejecutarse de forma independiente. Ver el README en cada directorio: +- [Transaction Service README](./transaction-service/README.md) +- [Anti-Fraud Service README](./antifraud-service/README.md) + +## 📡 API Endpoints + +### Transaction Service + +#### Crear Transacción +```bash +POST http://localhost:8080/api/v1/transactions +Content-Type: application/json -```json { - "accountExternalIdDebit": "Guid", - "accountExternalIdCredit": "Guid", + "accountExternalIdDebit": "Guid1", + "accountExternalIdCredit": "Guid2", "tranferTypeId": 1, - "value": 120 + "value": 120.00 } ``` -2. Resource to retrieve a transaction +**Respuesta exitosa (201 Created):** +```json +{ + "transactionExternalId": "550e8400-e29b-41d4-a716-446655440000", + "transactionStatus": "PENDING", + "transactionType": 1, + "value": 120.00, + "createdAt": "2026-01-04T10:30:00Z" +} +``` +#### Obtener Transacción por ID +```bash +GET http://localhost:8080/api/v1/transactions/{externalId} +``` + +**Respuesta exitosa (200 OK):** ```json { - "transactionExternalId": "Guid", - "transactionType": { - "name": "" - }, - "transactionStatus": { - "name": "" - }, - "value": 120, - "createdAt": "Date" + "transactionExternalId": "550e8400-e29b-41d4-a716-446655440000", + "transactionStatus": "APPROVED", + "transactionType": 1, + "value": 120.00, + "createdAt": "2026-01-04T10:30:00Z" } ``` -## Optional +### Event Store API (Auditoría y Debug) + +#### Obtener eventos de una transacción +```bash +GET http://localhost:8080/api/v1/events/transaction/{transactionId} +``` + +#### Obtener todos los eventos +```bash +GET http://localhost:8080/api/v1/events/all +``` + +#### Obtener eventos por tipo +```bash +GET http://localhost:8080/api/v1/events/type/{eventType} +``` + +#### Verificar si existe una transacción +```bash +GET http://localhost:8080/api/v1/events/transaction/{transactionId}/exists +``` + +#### Contar eventos de una transacción +```bash +GET http://localhost:8080/api/v1/events/transaction/{transactionId}/count +``` + +## 🔄 Flujo de Transacciones + +### Flujo de Creación (Write Path - CQRS Command) + +1. **Cliente** envía solicitud POST a `/api/v1/transactions` +2. **Transaction Service** recibe el request y valida los datos +3. **Command Bus** despacha el `CreateTransactionCommand` +4. **Transaction Aggregate Service** crea los eventos de dominio: + - `TransactionCreatedEvent` (estado inicial: PENDING) +5. **Event Store** persiste los eventos en PostgreSQL (JSONB) +6. **Transaction Repository** actualiza la proyección (read model) +7. **Redis Cache** almacena la transacción para lecturas rápidas +8. **Kafka Producer** publica evento `transaction-created` a Kafka +9. **Anti-Fraud Service** consume el evento de Kafka +10. **Anti-Fraud Service** valida la transacción: + - ✅ APPROVED si valor ≤ 1000 + - ❌ REJECTED si valor > 1000 +11. **Anti-Fraud Service** publica resultado a Kafka topic `transaction-status` +12. **Transaction Service** consume el resultado de validación +13. **Transaction Aggregate Service** crea evento `TransactionStatusUpdatedEvent` +14. **Event Store** persiste el nuevo evento +15. **Transaction Repository** actualiza la proyección con el nuevo estado +16. **Redis Cache** invalida la entrada en caché (evict) +17. Siguiente lectura reconstruirá el estado desde el Event Store + +### Flujo de Consulta (Read Path - CQRS Query) + +#### Con Cache Hit (80-90% de los casos): +1. **Cliente** envía GET a `/api/v1/transactions/{id}` +2. **Query Bus** despacha el `GetTransactionQuery` +3. **Redis Cache** devuelve la transacción (5-20ms) +4. Respuesta al cliente + +#### Con Cache Miss: +1. **Cliente** envía GET a `/api/v1/transactions/{id}` +2. **Query Bus** despacha el `GetTransactionQuery` +3. **Redis Cache** no encuentra la transacción +4. **Event Store** reconstruye el estado desde eventos +5. **Redis Cache** almacena el resultado (cache-aside pattern) +6. Respuesta al cliente + +### Ventajas del Event Sourcing + +- ✅ **Auditoría completa**: Cada cambio queda registrado +- ✅ **Reconstrucción temporal**: Se puede ver el estado en cualquier momento +- ✅ **Debug facilitado**: API `/api/v1/events` para inspección +- ✅ **Escrituras optimizadas**: Solo INSERT (append-only) +- ✅ **Sin locks**: No hay UPDATE que bloquee lecturas + +## 🛠️ Tecnologías + +### Backend +- **Java 21**: Lenguaje de programación (LTS) +- **Spring Boot 3.2.0**: Framework principal +- **Spring Data JPA**: Persistencia de datos con Hibernate +- **Spring Data Redis**: Integración con Redis para caché distribuido +- **Spring Cache**: Abstracción de caché con anotaciones +- **Spring Kafka**: Integración con Apache Kafka +- **MapStruct 1.5.5**: Mapeo de objetos DTO/Entity +- **Lombok 1.18.30**: Reducción de boilerplate code +- **Resilience4j 2.3.0**: Circuit breaker y patrones de resiliencia -You can use any approach to store transaction data but you should consider that we may deal with high volume scenarios where we have a huge amount of writes and reads for the same data at the same time. How would you tackle this requirement? +### Base de Datos +- **PostgreSQL 16**: Base de datos relacional + - Almacenamiento de transacciones (read model) + - Event Store con tipo JSONB para eventos +- **Redis 7**: Caché distribuido en memoria + - Estrategia: Cache-aside pattern + - TTL: 5 minutos configurables + - Política de evicción: allkeys-lru + - Max memory: 512MB -You can use Graphql; +### Mensajería +- **Apache Kafka 7.5.0**: Message broker + - Topics: + - `transaction-created`: Transacciones nuevas + - `transaction-status`: Resultado de validación anti-fraude +- **Confluent Zookeeper 7.5.0**: Coordinación de Kafka +- **Kafka UI**: Interfaz web para monitoreo (puerto 8090) -# Send us your challenge +### Infraestructura +- **Docker**: Contenedorización +- **Docker Compose**: Orquestación de servicios +- **Maven 3.9+**: Gestión de dependencias +- **HikariCP**: Connection pooling optimizado + - Pool size: 50 conexiones máximas + - Prepared statement cache habilitado + - Leak detection configurado + +### Arquitectura +- **Monorepo Multi-módulo**: Gestión unificada con Maven + - `common`: Librería compartida (DTOs, eventos Kafka) + - `transaction-service`: API REST y gestión de transacciones + - `antifraud-service`: Validación anti-fraude + +## 🚀 Optimización para Alto Volumen + +Este proyecto está optimizado para manejar **alto volumen de lecturas y escrituras concurrentes**: + +### Estrategias Implementadas + +1. **Redis Distributed Cache** + - Cache-aside pattern + - TTL configurable por tipo de dato + - Invalidación automática en actualizaciones + - Cache hit rate esperado: 80-90% + +2. **Event Sourcing** + - Append-only pattern (solo INSERT) + - Sin locks de actualización + - Escrituras optimizadas + +3. **CQRS** + - Separación de modelos lectura/escritura + - Escalado independiente + +4. **Connection Pooling Optimizado** + - HikariCP con 50 conexiones max + - Prepared statement cache + - Leak detection + +### Métricas de Performance + +| Métrica | Sin Cache | Con Cache | +|---------|-----------|-----------| +| Throughput lecturas | 100-200/seg | 5,000-10,000/seg | +| Latencia P95 lectura | 150ms | 5-20ms | +| Cache hit rate | 0% | 80-90% | + +### Configuraciones Clave + +- **HikariCP**: Pool de 50 conexiones con prepared statements cache +- **Redis TTL**: 5 minutos (configurable en application.yml) +- **Kafka**: Async processing con retry configurado +- **PostgreSQL**: JSONB para Event Store, índices optimizados + +## 📋 Comandos Útiles + +### Docker Compose + +```bash +# Ver logs de todos los servicios +docker compose logs -f + +# Ver logs de un servicio específico +docker compose logs -f transaction-service + +# Detener todos los servicios +docker compose down + +# Detener y eliminar volúmenes +docker compose down -v + +# Reconstruir un servicio específico +docker compose build transaction-service + +# Reiniciar un servicio +docker compose restart transaction-service +``` + +### Maven + +```bash +# Compilar todo el proyecto +mvn clean install + +# Compilar sin tests +mvn clean install -DskipTests + +# Compilar solo un módulo +mvn clean install -pl transaction-service -am + +# Ejecutar tests +mvn test +``` + +## 🧪 Testing + +```bash +# Ejecutar todos los tests +mvn test + +# Ejecutar tests de un módulo específico +mvn test -pl transaction-service +``` + +## 📚 Documentación Adicional + +- [Transaction Service README](./transaction-service/README.md) - Documentación del servicio de transacciones +- [Anti-Fraud Service README](./antifraud-service/README.md) - Documentación del servicio anti-fraude +- [Common Module README](./common/README.md) - Documentación del módulo compartido +- [Postman Collection](./Yape-Challenge.postman_collection.json) - Colección de Postman con ejemplos de API + +## 🔍 Monitoreo + +### Kafka UI +Accede a http://localhost:8090 para: +- Ver topics de Kafka +- Monitorear mensajes +- Ver estado de consumers + +### PostgreSQL +```bash +# Conectarse a la base de datos +docker exec -it yape-postgres psql -U yapeuser -d yape_transactions +``` + +## 📝 Notas + +- Los Dockerfiles están ubicados en cada directorio de microservicio +- Cada servicio puede construirse y ejecutarse de forma independiente +- El módulo `common` contiene código compartido entre servicios +- La configuración usa perfiles de Spring para diferentes entornos + +## 🐛 Troubleshooting + +### Los servicios no se conectan a Kafka +Verifica que Kafka esté saludable: +```bash +docker compose ps kafka +``` + +### Error de conexión a PostgreSQL +Asegúrate de que PostgreSQL esté listo: +```bash +docker compose ps postgres +``` -When you finish your challenge, after forking a repository, you **must** open a pull request to our repository. There are no limitations to the implementation, you can follow the programming paradigm, modularization, and style that you feel is the most appropriate solution. +### Puerto ya en uso +Si algún puerto está ocupado, puedes cambiarlos en `docker-compose.yml`. -If you have any questions, please let us know. diff --git a/Yape-Challenge.postman_collection.json b/Yape-Challenge.postman_collection.json new file mode 100644 index 0000000000..11df28e83f --- /dev/null +++ b/Yape-Challenge.postman_collection.json @@ -0,0 +1,288 @@ +{ + "info": { + "_postman_id": "bd15a21f-512c-41e9-9864-044f6c4568a6", + "name": "Yape", + "schema": "https://schema.getpostman.com/json/collection/v2.1.0/collection.json", + "_exporter_id": "13458143" + }, + "item": [ + { + "name": "transaction-service", + "item": [ + { + "name": "actuator", + "item": [ + { + "name": "Health", + "request": { + "method": "GET", + "header": [], + "url": { + "raw": "http://localhost:8080/actuator/health", + "protocol": "http", + "host": [ + "localhost" + ], + "port": "8080", + "path": [ + "actuator", + "health" + ] + } + }, + "response": [] + }, + { + "name": "Metrics", + "request": { + "method": "GET", + "header": [], + "url": { + "raw": "http://localhost:8080/actuator/metrics", + "protocol": "http", + "host": [ + "localhost" + ], + "port": "8080", + "path": [ + "actuator", + "metrics" + ] + } + }, + "response": [] + } + ] + }, + { + "name": "transaction", + "item": [ + { + "name": "get transaction", + "request": { + "method": "GET", + "header": [], + "url": { + "raw": "http://localhost:8080/api/v1/transactions/250d26ea-9342-459f-b45a-1762943275af", + "protocol": "http", + "host": [ + "localhost" + ], + "port": "8080", + "path": [ + "api", + "v1", + "transactions", + "250d26ea-9342-459f-b45a-1762943275af" + ] + } + }, + "response": [] + }, + { + "name": "create transaction", + "request": { + "method": "POST", + "header": [], + "body": { + "mode": "raw", + "raw": "{\n \"accountExternalIdDebit\": \"4720478c-3d11-411f-9f79-f8923eb33e9e\",\n \"accountExternalIdCredit\": \"c85500ab-7177-455c-b311-c2f2e26659a6\",\n \"tranferTypeId\": 1,\n \"value\": 150.60\n}", + "options": { + "raw": { + "language": "json" + } + } + }, + "url": { + "raw": "http://localhost:8080/api/v1/transactions", + "protocol": "http", + "host": [ + "localhost" + ], + "port": "8080", + "path": [ + "api", + "v1", + "transactions" + ] + } + }, + "response": [] + } + ] + }, + { + "name": "event-store", + "item": [ + { + "name": "Get all events", + "request": { + "method": "GET", + "header": [], + "url": { + "raw": "http://localhost:8080/api/v1/events/transaction/ce767f62-9e3a-4155-8765-e9b6a443de33", + "protocol": "http", + "host": [ + "localhost" + ], + "port": "8080", + "path": [ + "api", + "v1", + "events", + "transaction", + "ce767f62-9e3a-4155-8765-e9b6a443de33" + ] + } + }, + "response": [] + }, + { + "name": "Get event count for a specific transaction", + "request": { + "method": "GET", + "header": [], + "url": { + "raw": "http://localhost:8080/api/v1/events/transaction/ce767f62-9e3a-4155-8765-e9b6a443de33/count", + "protocol": "http", + "host": [ + "localhost" + ], + "port": "8080", + "path": [ + "api", + "v1", + "events", + "transaction", + "ce767f62-9e3a-4155-8765-e9b6a443de33", + "count" + ] + } + }, + "response": [] + }, + { + "name": "Get all events by event type", + "request": { + "method": "GET", + "header": [], + "url": { + "raw": "http://localhost:8080/api/v1/events/type/TransactionStatusChangedDomainEvent", + "protocol": "http", + "host": [ + "localhost" + ], + "port": "8080", + "path": [ + "api", + "v1", + "events", + "type", + "TransactionStatusChangedDomainEvent" + ] + } + }, + "response": [] + }, + { + "name": "Get all transaction events", + "request": { + "method": "GET", + "header": [], + "url": { + "raw": "http://localhost:8080/api/v1/events/all", + "protocol": "http", + "host": [ + "localhost" + ], + "port": "8080", + "path": [ + "api", + "v1", + "events", + "all" + ] + } + }, + "response": [] + }, + { + "name": "Check if transaction exists in event store", + "request": { + "method": "GET", + "header": [], + "url": { + "raw": "http://localhost:8080/api/v1/events/transaction/ce767f62-9e3a-4155-8765-e9b6a443de33/exists", + "protocol": "http", + "host": [ + "localhost" + ], + "port": "8080", + "path": [ + "api", + "v1", + "events", + "transaction", + "ce767f62-9e3a-4155-8765-e9b6a443de33", + "exists" + ] + } + }, + "response": [] + } + ] + } + ] + }, + { + "name": "antifraud-service", + "item": [ + { + "name": "actuator", + "item": [ + { + "name": "Health", + "request": { + "method": "GET", + "header": [], + "url": { + "raw": "http://localhost:8081/actuator/health", + "protocol": "http", + "host": [ + "localhost" + ], + "port": "8081", + "path": [ + "actuator", + "health" + ] + } + }, + "response": [] + }, + { + "name": "Metrics", + "request": { + "method": "GET", + "header": [], + "url": { + "raw": "http://localhost:8081/actuator/metrics", + "protocol": "http", + "host": [ + "localhost" + ], + "port": "8081", + "path": [ + "actuator", + "metrics" + ] + } + }, + "response": [] + } + ] + } + ] + } + ] +} \ No newline at end of file diff --git a/antifraud-service/.dockerignore b/antifraud-service/.dockerignore new file mode 100644 index 0000000000..ddfe79561c --- /dev/null +++ b/antifraud-service/.dockerignore @@ -0,0 +1,9 @@ +target/ +*.md +.git +.gitignore +*.iml +.idea/ +.DS_Store +*.log + diff --git a/antifraud-service/Dockerfile b/antifraud-service/Dockerfile new file mode 100644 index 0000000000..a6bc958347 --- /dev/null +++ b/antifraud-service/Dockerfile @@ -0,0 +1,20 @@ +# Build stage +FROM maven:3.9.5-eclipse-temurin-21 AS build +WORKDIR /app + +# Copy the entire project (parent POM needs to see all modules) +COPY . . + +# Build only antifraud-service and its dependencies +RUN mvn clean package -DskipTests -pl antifraud-service -am + +# Run stage +FROM eclipse-temurin:21-jre-alpine +WORKDIR /app + +COPY --from=build /app/antifraud-service/target/*.jar app.jar + +EXPOSE 8081 + +ENTRYPOINT ["java", "-jar", "app.jar"] + diff --git a/antifraud-service/README.md b/antifraud-service/README.md new file mode 100644 index 0000000000..4ba42ed9b3 --- /dev/null +++ b/antifraud-service/README.md @@ -0,0 +1,237 @@ +# Anti-Fraud Service + +Microservicio de validación anti-fraude para Yape Challenge. Valida transacciones en tiempo real mediante eventos de Kafka. + +## 🎯 Características + +- ✅ **Validación en Tiempo Real**: Procesamiento de transacciones vía Kafka +- ✅ **Reglas de Negocio**: Validación de montos y detección de fraude +- ✅ **Comunicación Asíncrona**: Integración completa con Kafka +- ✅ **Arquitectura Reactiva**: Procesamiento event-driven +- ✅ **Spring Boot 3.2**: Framework moderno y optimizado + +## 🚀 Ejecución + +### Con Docker Compose (recomendado) + +Desde la raíz del proyecto: +```bash +docker-compose up antifraud-service --build +``` + +### Con Docker + +Construir la imagen: +```bash +docker build -t antifraud-service:latest -f antifraud-service/Dockerfile . +``` + +Ejecutar el contenedor: +```bash +docker run -p 8081:8081 \ + -e SPRING_PROFILES_ACTIVE=docker,antifraud \ + -e SPRING_KAFKA_BOOTSTRAP_SERVERS=kafka:29092 \ + antifraud-service:latest +``` + +### Con Maven + +Compilar y ejecutar localmente: +```bash +cd antifraud-service +mvn clean package -DskipTests +java -jar target/antifraud-service-*.jar +``` + +**Nota**: Asegúrate de tener Kafka ejecutándose localmente. + +## 📡 Funcionalidad + +### Puerto +- **8081** + +### Validación Anti-Fraude + +Este servicio escucha eventos de transacciones desde Kafka y aplica las siguientes reglas: + +#### Reglas de Validación + +1. **Monto Máximo**: Transacciones con valor > 1000 son rechazadas +2. **Estados Posibles**: + - ✅ `APPROVED`: Transacción válida (valor ≤ 1000) + - ❌ `REJECTED`: Transacción rechazada (valor > 1000) + +### Flujo de Validación + +1. Consume evento `transaction-created` de Kafka +2. Extrae el monto de la transacción +3. Aplica reglas de validación +4. Publica resultado al topic `transaction-status` +5. Transaction Service actualiza el estado + +### Actuator (Monitoreo) + +- Health: `http://localhost:8081/actuator/health` +- Metrics: `http://localhost:8081/actuator/metrics` + +## 🔧 Configuración + +### Variables de Entorno + +| Variable | Default | Descripción | +|----------|---------|-------------| +| `SPRING_PROFILES_ACTIVE` | - | Perfil activo (docker, antifraud) | +| `SPRING_KAFKA_BOOTSTRAP_SERVERS` | localhost:9092 | Servidores de Kafka | +| `SERVER_PORT` | 8081 | Puerto del servicio | + +### Configuración de Kafka + +```yaml +spring: + kafka: + consumer: + group-id: antifraud-service-group + auto-offset-reset: earliest + producer: + key-serializer: StringSerializer + value-serializer: JsonSerializer +``` + +## 🏗️ Arquitectura + +### Estructura de Paquetes + +``` +com.yape.challenge.antifraud/ +├── AntiFraudApplication.java +├── service/ # Lógica de validación +│ └── AntiFraudService +├── kafka/ # Consumers y Producers +│ ├── TransactionEventConsumer +│ └── TransactionStatusProducer +└── config/ # Configuraciones + └── KafkaConfig +``` + +### Componentes Principales + +- **AntiFraudService**: Lógica de validación de transacciones +- **TransactionEventConsumer**: Consumidor de eventos de Kafka +- **TransactionStatusProducer**: Productor de resultados a Kafka + +## 📊 Topics de Kafka + +| Tipo | Topic | Descripción | +|------|-------|-------------| +| **Consume** | `transaction-created` | Transacciones nuevas del Transaction Service | +| **Produce** | `transaction-status` | Resultado de validación (APPROVED/REJECTED) | + +### Ejemplo de Evento Consumido + +```json +{ + "transactionExternalId": "550e8400-e29b-41d4-a716-446655440000", + "accountExternalIdDebit": "Guid1", + "accountExternalIdCredit": "Guid2", + "tranferTypeId": 1, + "value": 1500.00, + "createdAt": "2026-01-04T10:30:00Z" +} +``` + +### Ejemplo de Evento Producido + +```json +{ + "transactionExternalId": "550e8400-e29b-41d4-a716-446655440000", + "status": "REJECTED" +} +``` + +## 🧪 Testing + +```bash +# Ejecutar tests +mvn test + +# Ejecutar tests con cobertura +mvn test jacoco:report +``` + +### Escenarios de Test + +- ✅ Validación de transacción válida (valor ≤ 1000) +- ✅ Rechazo de transacción fraudulenta (valor > 1000) +- ✅ Consumo correcto de eventos de Kafka +- ✅ Publicación correcta de resultados + +## 🔍 Debugging + +### Ver logs en Docker + +```bash +docker-compose logs -f antifraud-service +``` + +### Monitorear Kafka UI + +Accede a http://localhost:8090 para: +- Ver mensajes en topics +- Monitorear consumers +- Ver estado de procesamiento + +## ⚙️ Reglas de Negocio + +### Límites de Transacción + +| Monto | Estado | Descripción | +|-------|--------|-------------| +| ≤ 1000 | ✅ APPROVED | Transacción válida | +| > 1000 | ❌ REJECTED | Posible fraude detectado | + +### Extensibilidad + +El servicio está diseñado para agregar fácilmente nuevas reglas: + +```java +@Service +public class AntiFraudService { + + private static final BigDecimal FRAUD_THRESHOLD = new BigDecimal("1000"); + + private TransactionStatus determineStatus(BigDecimal value) { + // Regla 1: Validar monto máximo + if (value.compareTo(FRAUD_THRESHOLD) > 0) { + return TransactionStatus.REJECTED; + } + + // Agregar más reglas aquí: + // - Validar frecuencia de transacciones + // - Validar patrones sospechosos + // - Validar horarios inusuales + // - etc. + + return TransactionStatus.APPROVED; + } +} +``` + +## 📈 Performance + +- **Throughput**: 100-500 eventos/seg +- **Latencia promedio**: 50-100ms por evento +- **Consumer group**: antifraud-service-group + +## 🔗 Referencias + +- [README Principal](../README.md) +- [Transaction Service](../transaction-service/README.md) + +## 📝 Notas + +- El servicio no tiene base de datos propia (stateless) +- Toda la comunicación es asíncrona vía Kafka +- Se puede escalar horizontalmente agregando más instancias +- Cada instancia procesará diferentes particiones del topic + + diff --git a/antifraud-service/pom.xml b/antifraud-service/pom.xml new file mode 100644 index 0000000000..09aefaa4b7 --- /dev/null +++ b/antifraud-service/pom.xml @@ -0,0 +1,80 @@ + + + 4.0.0 + + + com.yape + yape-challenge + 0.0.1-SNAPSHOT + + + antifraud-service + antifraud-service + Anti-fraud microservice + + + + + com.yape + common + ${project.version} + + + + + org.springframework.boot + spring-boot-starter-web + + + + + org.springframework.kafka + spring-kafka + + + + + org.projectlombok + lombok + true + + + + + org.springframework.boot + spring-boot-starter-actuator + + + + + org.springframework.boot + spring-boot-starter-test + test + + + + + org.springframework.kafka + spring-kafka-test + test + + + + + + + org.springframework.boot + spring-boot-maven-plugin + + + org.apache.maven.plugins + maven-compiler-plugin + + + + + + diff --git a/antifraud-service/src/main/java/com/yape/challenge/antifraud/AntiFraudApplication.java b/antifraud-service/src/main/java/com/yape/challenge/antifraud/AntiFraudApplication.java new file mode 100644 index 0000000000..bf91de7e90 --- /dev/null +++ b/antifraud-service/src/main/java/com/yape/challenge/antifraud/AntiFraudApplication.java @@ -0,0 +1,12 @@ +package com.yape.challenge.antifraud; + +import org.springframework.boot.SpringApplication; +import org.springframework.boot.autoconfigure.SpringBootApplication; + +@SpringBootApplication +public class AntiFraudApplication { + + public static void main(String[] args) { + SpringApplication.run(AntiFraudApplication.class, args); + } +} diff --git a/antifraud-service/src/main/java/com/yape/challenge/antifraud/config/KafkaConsumerConfig.java b/antifraud-service/src/main/java/com/yape/challenge/antifraud/config/KafkaConsumerConfig.java new file mode 100644 index 0000000000..99ebdd1f79 --- /dev/null +++ b/antifraud-service/src/main/java/com/yape/challenge/antifraud/config/KafkaConsumerConfig.java @@ -0,0 +1,50 @@ +package com.yape.challenge.antifraud.config; + +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import org.apache.kafka.clients.consumer.ConsumerConfig; +import org.apache.kafka.common.serialization.StringDeserializer; +import org.springframework.beans.factory.annotation.Value; +import org.springframework.context.annotation.Bean; +import org.springframework.context.annotation.Configuration; +import org.springframework.kafka.annotation.EnableKafka; +import org.springframework.kafka.config.ConcurrentKafkaListenerContainerFactory; +import org.springframework.kafka.core.ConsumerFactory; +import org.springframework.kafka.core.DefaultKafkaConsumerFactory; +import org.springframework.kafka.support.serializer.JsonDeserializer; + +import java.util.HashMap; +import java.util.Map; + +@EnableKafka +@Configuration +public class KafkaConsumerConfig { + + @Value("${spring.kafka.bootstrap-servers}") + private String bootstrapServers; + + @Value("${spring.kafka.consumer.group-id}") + private String groupId; + + @Bean + public ConsumerFactory consumerFactory() { + Map props = new HashMap<>(); + props.put(ConsumerConfig.BOOTSTRAP_SERVERS_CONFIG, bootstrapServers); + props.put(ConsumerConfig.GROUP_ID_CONFIG, groupId); + props.put(ConsumerConfig.KEY_DESERIALIZER_CLASS_CONFIG, StringDeserializer.class); + props.put(ConsumerConfig.VALUE_DESERIALIZER_CLASS_CONFIG, JsonDeserializer.class); + props.put(ConsumerConfig.AUTO_OFFSET_RESET_CONFIG, "earliest"); + props.put(JsonDeserializer.TRUSTED_PACKAGES, "*"); + props.put(JsonDeserializer.VALUE_DEFAULT_TYPE, TransactionCreatedEvent.class.getName()); + return new DefaultKafkaConsumerFactory<>(props); + } + + @Bean + public ConcurrentKafkaListenerContainerFactory kafkaListenerContainerFactory() { + ConcurrentKafkaListenerContainerFactory factory = + new ConcurrentKafkaListenerContainerFactory<>(); + factory.setConsumerFactory(consumerFactory()); + factory.setConcurrency(3); + return factory; + } +} + diff --git a/antifraud-service/src/main/java/com/yape/challenge/antifraud/config/KafkaProducerConfig.java b/antifraud-service/src/main/java/com/yape/challenge/antifraud/config/KafkaProducerConfig.java new file mode 100644 index 0000000000..6138eb1d4a --- /dev/null +++ b/antifraud-service/src/main/java/com/yape/challenge/antifraud/config/KafkaProducerConfig.java @@ -0,0 +1,41 @@ +package com.yape.challenge.antifraud.config; + +import com.yape.challenge.common.dto.TransactionStatusEvent; +import org.apache.kafka.clients.producer.ProducerConfig; +import org.apache.kafka.common.serialization.StringSerializer; +import org.springframework.beans.factory.annotation.Value; +import org.springframework.context.annotation.Bean; +import org.springframework.context.annotation.Configuration; +import org.springframework.kafka.core.DefaultKafkaProducerFactory; +import org.springframework.kafka.core.KafkaTemplate; +import org.springframework.kafka.core.ProducerFactory; +import org.springframework.kafka.support.serializer.JsonSerializer; + +import java.util.HashMap; +import java.util.Map; + +@Configuration +public class KafkaProducerConfig { + + @Value("${spring.kafka.bootstrap-servers}") + private String bootstrapServers; + + @Bean + public ProducerFactory producerFactory() { + Map configProps = new HashMap<>(); + configProps.put(ProducerConfig.BOOTSTRAP_SERVERS_CONFIG, bootstrapServers); + configProps.put(ProducerConfig.KEY_SERIALIZER_CLASS_CONFIG, StringSerializer.class); + configProps.put(ProducerConfig.VALUE_SERIALIZER_CLASS_CONFIG, JsonSerializer.class); + configProps.put(ProducerConfig.ACKS_CONFIG, "all"); + configProps.put(ProducerConfig.RETRIES_CONFIG, 3); + configProps.put(ProducerConfig.ENABLE_IDEMPOTENCE_CONFIG, true); + configProps.put(JsonSerializer.ADD_TYPE_INFO_HEADERS, false); + return new DefaultKafkaProducerFactory<>(configProps); + } + + @Bean + public KafkaTemplate kafkaTemplate() { + return new KafkaTemplate<>(producerFactory()); + } +} + diff --git a/antifraud-service/src/main/java/com/yape/challenge/antifraud/kafka/TransactionCreatedConsumer.java b/antifraud-service/src/main/java/com/yape/challenge/antifraud/kafka/TransactionCreatedConsumer.java new file mode 100644 index 0000000000..351854ad02 --- /dev/null +++ b/antifraud-service/src/main/java/com/yape/challenge/antifraud/kafka/TransactionCreatedConsumer.java @@ -0,0 +1,30 @@ +package com.yape.challenge.antifraud.kafka; + +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import com.yape.challenge.common.kafka.KafkaTopics; +import com.yape.challenge.antifraud.service.AntiFraudService; +import lombok.RequiredArgsConstructor; +import lombok.extern.slf4j.Slf4j; +import org.springframework.kafka.annotation.KafkaListener; +import org.springframework.stereotype.Component; + +@Component +@RequiredArgsConstructor +@Slf4j +public class TransactionCreatedConsumer { + + private final AntiFraudService antiFraudService; + + @KafkaListener(topics = KafkaTopics.TRANSACTION_CREATED, groupId = "${spring.kafka.consumer.group-id}") + public void consumeTransactionCreated(TransactionCreatedEvent event) { + log.info("Received transaction created event: {}", event); + + try { + antiFraudService.validateTransaction(event); + } catch (Exception e) { + log.error("Error processing transaction: {}", event.getTransactionExternalId(), e); + throw e; + } + } +} + diff --git a/antifraud-service/src/main/java/com/yape/challenge/antifraud/service/AntiFraudService.java b/antifraud-service/src/main/java/com/yape/challenge/antifraud/service/AntiFraudService.java new file mode 100644 index 0000000000..02193dc948 --- /dev/null +++ b/antifraud-service/src/main/java/com/yape/challenge/antifraud/service/AntiFraudService.java @@ -0,0 +1,45 @@ +package com.yape.challenge.antifraud.service; + +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import com.yape.challenge.common.dto.TransactionStatus; +import com.yape.challenge.common.dto.TransactionStatusEvent; +import com.yape.challenge.common.kafka.KafkaTopics; +import lombok.RequiredArgsConstructor; +import lombok.extern.slf4j.Slf4j; +import org.springframework.kafka.core.KafkaTemplate; +import org.springframework.stereotype.Service; + +import java.math.BigDecimal; + +@Service +@RequiredArgsConstructor +@Slf4j +public class AntiFraudService { + + private static final BigDecimal FRAUD_THRESHOLD = new BigDecimal("1000"); + private final KafkaTemplate kafkaTemplate; + + public void validateTransaction(TransactionCreatedEvent event) { + log.info("Validating transaction: {}", event.getTransactionExternalId()); + + TransactionStatusEvent statusEvent = TransactionStatusEvent.builder() + .transactionExternalId(event.getTransactionExternalId()) + .status(determineStatus(event.getValue())) + .build(); + + kafkaTemplate.send(KafkaTopics.TRANSACTION_STATUS_UPDATED, + event.getTransactionExternalId().toString(), + statusEvent); + + log.info("Transaction validation completed. ExternalId: {}, Status: {}", + event.getTransactionExternalId(), statusEvent.getStatus()); + } + + private TransactionStatus determineStatus(BigDecimal value) { + if (value.compareTo(FRAUD_THRESHOLD) > 0) { + return TransactionStatus.REJECTED; + } + return TransactionStatus.APPROVED; + } +} + diff --git a/antifraud-service/src/main/resources/application-docker.yml b/antifraud-service/src/main/resources/application-docker.yml new file mode 100644 index 0000000000..298ed9e797 --- /dev/null +++ b/antifraud-service/src/main/resources/application-docker.yml @@ -0,0 +1,37 @@ +spring: + application: + name: antifraud-service + + kafka: + bootstrap-servers: ${SPRING_KAFKA_BOOTSTRAP_SERVERS:localhost:9092} + consumer: + group-id: antifraud-service-group + auto-offset-reset: earliest + key-deserializer: org.apache.kafka.common.serialization.StringDeserializer + value-deserializer: org.springframework.kafka.support.serializer.JsonDeserializer + properties: + spring.json.trusted.packages: "*" + producer: + key-serializer: org.apache.kafka.common.serialization.StringSerializer + value-serializer: org.springframework.kafka.support.serializer.JsonSerializer + properties: + spring.json.add.type.headers: false + +server: + port: 8081 + +management: + endpoints: + web: + exposure: + include: health,info,metrics,prometheus + endpoint: + health: + show-details: always + +logging: + level: + com.yape.challenge.antifraud: INFO + org.springframework.kafka: WARN + org.apache.kafka: WARN + diff --git a/antifraud-service/src/main/resources/application.yml b/antifraud-service/src/main/resources/application.yml new file mode 100644 index 0000000000..a08d62e549 --- /dev/null +++ b/antifraud-service/src/main/resources/application.yml @@ -0,0 +1,39 @@ +spring: + application: + name: antifraud-service + + kafka: + bootstrap-servers: localhost:9092 + consumer: + group-id: antifraud-service-group + auto-offset-reset: earliest + key-deserializer: org.apache.kafka.common.serialization.StringDeserializer + value-deserializer: org.springframework.kafka.support.serializer.JsonDeserializer + properties: + spring.json.trusted.packages: "*" + producer: + key-serializer: org.apache.kafka.common.serialization.StringSerializer + value-serializer: org.springframework.kafka.support.serializer.JsonSerializer + properties: + spring.json.add.type.headers: false + +server: + port: 8081 + +management: + endpoints: + web: + exposure: + include: health,info,metrics + endpoint: + health: + show-details: always + +logging: + level: + com.yape.challenge.antifraud: INFO + org.springframework.kafka: WARN + org.apache.kafka: WARN + + + diff --git a/antifraud-service/src/test/java/com/yape/challenge/antifraud/AntiFraudApplicationTest.java b/antifraud-service/src/test/java/com/yape/challenge/antifraud/AntiFraudApplicationTest.java new file mode 100644 index 0000000000..c56665aad8 --- /dev/null +++ b/antifraud-service/src/test/java/com/yape/challenge/antifraud/AntiFraudApplicationTest.java @@ -0,0 +1,48 @@ +package com.yape.challenge.antifraud; + +import org.junit.jupiter.api.DisplayName; +import org.junit.jupiter.api.Test; +import org.springframework.beans.factory.annotation.Autowired; +import org.springframework.boot.test.context.SpringBootTest; +import org.springframework.context.ApplicationContext; +import org.springframework.kafka.core.KafkaTemplate; +import org.springframework.test.context.TestPropertySource; + +import static org.junit.jupiter.api.Assertions.*; + +@SpringBootTest +@TestPropertySource(properties = { + "spring.kafka.bootstrap-servers=localhost:9092", + "spring.kafka.consumer.group-id=test-group" +}) +@DisplayName("AntiFraud Application Tests") +class AntiFraudApplicationTest { + + @Autowired + private ApplicationContext applicationContext; + + @Test + @DisplayName("Should load application context") + void shouldLoadApplicationContext() { + assertNotNull(applicationContext); + } + + @Test + @DisplayName("Should have antifraud service bean") + void shouldHaveAntiFraudServiceBean() { + assertTrue(applicationContext.containsBean("antiFraudService")); + } + + @Test + @DisplayName("Should have transaction created consumer bean") + void shouldHaveTransactionCreatedConsumerBean() { + assertTrue(applicationContext.containsBean("transactionCreatedConsumer")); + } + + @Test + @DisplayName("Should have kafka template bean") + void shouldHaveKafkaTemplateBean() { + assertNotNull(applicationContext.getBean(KafkaTemplate.class)); + } +} + diff --git a/antifraud-service/src/test/java/com/yape/challenge/antifraud/config/KafkaConsumerConfigTest.java b/antifraud-service/src/test/java/com/yape/challenge/antifraud/config/KafkaConsumerConfigTest.java new file mode 100644 index 0000000000..063d4aa2ef --- /dev/null +++ b/antifraud-service/src/test/java/com/yape/challenge/antifraud/config/KafkaConsumerConfigTest.java @@ -0,0 +1,61 @@ +package com.yape.challenge.antifraud.config; + +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import org.apache.kafka.clients.consumer.ConsumerConfig; +import org.apache.kafka.common.serialization.StringDeserializer; +import org.junit.jupiter.api.DisplayName; +import org.junit.jupiter.api.Test; +import org.springframework.beans.factory.annotation.Autowired; +import org.springframework.boot.test.context.SpringBootTest; +import org.springframework.kafka.config.ConcurrentKafkaListenerContainerFactory; +import org.springframework.kafka.core.ConsumerFactory; +import org.springframework.kafka.support.serializer.JsonDeserializer; +import org.springframework.test.context.TestPropertySource; + +import static org.junit.jupiter.api.Assertions.*; + +@SpringBootTest +@TestPropertySource(properties = { + "spring.kafka.bootstrap-servers=localhost:9092", + "spring.kafka.consumer.group-id=test-group" +}) +@DisplayName("Kafka Consumer Config Tests") +class KafkaConsumerConfigTest { + + @Autowired + private ConsumerFactory consumerFactory; + + @Autowired + private ConcurrentKafkaListenerContainerFactory kafkaListenerContainerFactory; + + @Test + @DisplayName("Should create consumer factory bean") + void shouldCreateConsumerFactoryBean() { + assertNotNull(consumerFactory); + } + + @Test + @DisplayName("Should create kafka listener container factory bean") + void shouldCreateKafkaListenerContainerFactoryBean() { + assertNotNull(kafkaListenerContainerFactory); + } + + @Test + @DisplayName("Should configure consumer factory with correct properties") + void shouldConfigureConsumerFactoryWithCorrectProperties() { + var configMap = consumerFactory.getConfigurationProperties(); + + assertEquals("localhost:9092", configMap.get(ConsumerConfig.BOOTSTRAP_SERVERS_CONFIG)); + assertEquals("test-group", configMap.get(ConsumerConfig.GROUP_ID_CONFIG)); + assertEquals(StringDeserializer.class, configMap.get(ConsumerConfig.KEY_DESERIALIZER_CLASS_CONFIG)); + assertEquals(JsonDeserializer.class, configMap.get(ConsumerConfig.VALUE_DESERIALIZER_CLASS_CONFIG)); + assertEquals("earliest", configMap.get(ConsumerConfig.AUTO_OFFSET_RESET_CONFIG)); + } + + @Test + @DisplayName("Should configure kafka listener container factory with consumer factory") + void shouldConfigureKafkaListenerContainerFactoryWithConsumerFactory() { + assertNotNull(kafkaListenerContainerFactory.getConsumerFactory()); + } +} + diff --git a/antifraud-service/src/test/java/com/yape/challenge/antifraud/config/KafkaProducerConfigTest.java b/antifraud-service/src/test/java/com/yape/challenge/antifraud/config/KafkaProducerConfigTest.java new file mode 100644 index 0000000000..90d57057d9 --- /dev/null +++ b/antifraud-service/src/test/java/com/yape/challenge/antifraud/config/KafkaProducerConfigTest.java @@ -0,0 +1,61 @@ +package com.yape.challenge.antifraud.config; + +import com.yape.challenge.common.dto.TransactionStatusEvent; +import org.apache.kafka.clients.producer.ProducerConfig; +import org.apache.kafka.common.serialization.StringSerializer; +import org.junit.jupiter.api.DisplayName; +import org.junit.jupiter.api.Test; +import org.springframework.beans.factory.annotation.Autowired; +import org.springframework.boot.test.context.SpringBootTest; +import org.springframework.kafka.core.KafkaTemplate; +import org.springframework.kafka.core.ProducerFactory; +import org.springframework.kafka.support.serializer.JsonSerializer; +import org.springframework.test.context.TestPropertySource; + +import static org.junit.jupiter.api.Assertions.*; + +@SpringBootTest +@TestPropertySource(properties = { + "spring.kafka.bootstrap-servers=localhost:9092" +}) +@DisplayName("Kafka Producer Config Tests") +class KafkaProducerConfigTest { + + @Autowired + private ProducerFactory producerFactory; + + @Autowired + private KafkaTemplate kafkaTemplate; + + @Test + @DisplayName("Should create producer factory bean") + void shouldCreateProducerFactoryBean() { + assertNotNull(producerFactory); + } + + @Test + @DisplayName("Should create kafka template bean") + void shouldCreateKafkaTemplateBean() { + assertNotNull(kafkaTemplate); + } + + @Test + @DisplayName("Should configure producer factory with correct properties") + void shouldConfigureProducerFactoryWithCorrectProperties() { + var configMap = producerFactory.getConfigurationProperties(); + + assertEquals("localhost:9092", configMap.get(ProducerConfig.BOOTSTRAP_SERVERS_CONFIG)); + assertEquals(StringSerializer.class, configMap.get(ProducerConfig.KEY_SERIALIZER_CLASS_CONFIG)); + assertEquals(JsonSerializer.class, configMap.get(ProducerConfig.VALUE_SERIALIZER_CLASS_CONFIG)); + assertEquals("all", configMap.get(ProducerConfig.ACKS_CONFIG)); + assertEquals(3, configMap.get(ProducerConfig.RETRIES_CONFIG)); + assertEquals(true, configMap.get(ProducerConfig.ENABLE_IDEMPOTENCE_CONFIG)); + } + + @Test + @DisplayName("Should configure kafka template with producer factory") + void shouldConfigureKafkaTemplateWithProducerFactory() { + assertNotNull(kafkaTemplate.getProducerFactory()); + } +} + diff --git a/antifraud-service/src/test/java/com/yape/challenge/antifraud/config/TestKafkaProducerConfig.java b/antifraud-service/src/test/java/com/yape/challenge/antifraud/config/TestKafkaProducerConfig.java new file mode 100644 index 0000000000..ddb37215e3 --- /dev/null +++ b/antifraud-service/src/test/java/com/yape/challenge/antifraud/config/TestKafkaProducerConfig.java @@ -0,0 +1,38 @@ +package com.yape.challenge.antifraud.config; + +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import org.apache.kafka.clients.producer.ProducerConfig; +import org.apache.kafka.common.serialization.StringSerializer; +import org.springframework.beans.factory.annotation.Value; +import org.springframework.boot.test.context.TestConfiguration; +import org.springframework.context.annotation.Bean; +import org.springframework.kafka.core.DefaultKafkaProducerFactory; +import org.springframework.kafka.core.KafkaTemplate; +import org.springframework.kafka.core.ProducerFactory; +import org.springframework.kafka.support.serializer.JsonSerializer; + +import java.util.HashMap; +import java.util.Map; + +@TestConfiguration +public class TestKafkaProducerConfig { + + @Value("${spring.embedded.kafka.brokers}") + private String bootstrapServers; + + @Bean + public ProducerFactory testProducerFactory() { + Map configProps = new HashMap<>(); + configProps.put(ProducerConfig.BOOTSTRAP_SERVERS_CONFIG, bootstrapServers); + configProps.put(ProducerConfig.KEY_SERIALIZER_CLASS_CONFIG, StringSerializer.class); + configProps.put(ProducerConfig.VALUE_SERIALIZER_CLASS_CONFIG, JsonSerializer.class); + configProps.put(JsonSerializer.ADD_TYPE_INFO_HEADERS, false); + return new DefaultKafkaProducerFactory<>(configProps); + } + + @Bean + public KafkaTemplate testKafkaTemplate() { + return new KafkaTemplate<>(testProducerFactory()); + } +} + diff --git a/antifraud-service/src/test/java/com/yape/challenge/antifraud/integration/AntiFraudServiceIntegrationTest.java b/antifraud-service/src/test/java/com/yape/challenge/antifraud/integration/AntiFraudServiceIntegrationTest.java new file mode 100644 index 0000000000..1f7f5776c6 --- /dev/null +++ b/antifraud-service/src/test/java/com/yape/challenge/antifraud/integration/AntiFraudServiceIntegrationTest.java @@ -0,0 +1,199 @@ +package com.yape.challenge.antifraud.integration; + +import com.yape.challenge.antifraud.config.TestKafkaProducerConfig; +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import com.yape.challenge.common.dto.TransactionStatus; +import com.yape.challenge.common.dto.TransactionStatusEvent; +import com.yape.challenge.common.kafka.KafkaTopics; +import org.apache.kafka.clients.consumer.ConsumerConfig; +import org.apache.kafka.clients.consumer.ConsumerRecord; +import org.apache.kafka.common.serialization.StringDeserializer; +import org.junit.jupiter.api.AfterEach; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.DisplayName; +import org.junit.jupiter.api.Test; +import org.springframework.beans.factory.annotation.Autowired; +import org.springframework.boot.test.context.SpringBootTest; +import org.springframework.context.annotation.Import; +import org.springframework.kafka.core.DefaultKafkaConsumerFactory; +import org.springframework.kafka.core.KafkaTemplate; +import org.springframework.kafka.listener.ContainerProperties; +import org.springframework.kafka.listener.KafkaMessageListenerContainer; +import org.springframework.kafka.listener.MessageListener; +import org.springframework.kafka.support.serializer.JsonDeserializer; +import org.springframework.kafka.test.context.EmbeddedKafka; +import org.springframework.kafka.test.utils.ContainerTestUtils; +import org.springframework.test.annotation.DirtiesContext; + +import java.math.BigDecimal; +import java.util.HashMap; +import java.util.Map; +import java.util.UUID; +import java.util.concurrent.BlockingQueue; +import java.util.concurrent.LinkedBlockingQueue; +import java.util.concurrent.TimeUnit; + +import static org.junit.jupiter.api.Assertions.*; + +@SpringBootTest(properties = "spring.kafka.bootstrap-servers=${spring.embedded.kafka.brokers}") +@Import(TestKafkaProducerConfig.class) +@DirtiesContext(classMode = DirtiesContext.ClassMode.AFTER_EACH_TEST_METHOD) +@EmbeddedKafka( + partitions = 3, + topics = {KafkaTopics.TRANSACTION_CREATED, KafkaTopics.TRANSACTION_STATUS_UPDATED} +) +@DisplayName("AntiFraud Service Integration Tests") +class AntiFraudServiceIntegrationTest { + + @Autowired + private KafkaTemplate producerTemplate; + + private KafkaMessageListenerContainer container; + private BlockingQueue> records; + + @BeforeEach + void setUp() { + records = new LinkedBlockingQueue<>(); + + Map consumerProps = new HashMap<>(); + consumerProps.put(ConsumerConfig.BOOTSTRAP_SERVERS_CONFIG, System.getProperty("spring.embedded.kafka.brokers")); + consumerProps.put(ConsumerConfig.GROUP_ID_CONFIG, "test-integration-group"); + consumerProps.put(ConsumerConfig.KEY_DESERIALIZER_CLASS_CONFIG, StringDeserializer.class); + consumerProps.put(ConsumerConfig.VALUE_DESERIALIZER_CLASS_CONFIG, JsonDeserializer.class); + consumerProps.put(ConsumerConfig.AUTO_OFFSET_RESET_CONFIG, "earliest"); + consumerProps.put(JsonDeserializer.TRUSTED_PACKAGES, "*"); + consumerProps.put(JsonDeserializer.VALUE_DEFAULT_TYPE, TransactionStatusEvent.class.getName()); + + DefaultKafkaConsumerFactory consumerFactory = + new DefaultKafkaConsumerFactory<>(consumerProps); + + ContainerProperties containerProperties = new ContainerProperties(KafkaTopics.TRANSACTION_STATUS_UPDATED); + container = new KafkaMessageListenerContainer<>(consumerFactory, containerProperties); + container.setupMessageListener((MessageListener) records::add); + container.start(); + ContainerTestUtils.waitForAssignment(container, 3); + } + + @AfterEach + void tearDown() { + if (container != null) { + container.stop(); + } + } + + @Test + @DisplayName("Should approve transaction when value is below threshold") + void shouldApproveTransactionWhenValueIsBelowThreshold() throws InterruptedException { + // Given + UUID transactionId = UUID.randomUUID(); + TransactionCreatedEvent event = TransactionCreatedEvent.builder() + .transactionExternalId(transactionId) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("500.00")) + .build(); + + // When + producerTemplate.send(KafkaTopics.TRANSACTION_CREATED, transactionId.toString(), event); + + // Then + ConsumerRecord received = records.poll(10, TimeUnit.SECONDS); + assertNotNull(received, "Should receive a status event"); + + TransactionStatusEvent statusEvent = received.value(); + assertEquals(transactionId, statusEvent.getTransactionExternalId()); + assertEquals(TransactionStatus.APPROVED, statusEvent.getStatus()); + } + + @Test + @DisplayName("Should reject transaction when value exceeds threshold") + void shouldRejectTransactionWhenValueExceedsThreshold() throws InterruptedException { + // Given + UUID transactionId = UUID.randomUUID(); + TransactionCreatedEvent event = TransactionCreatedEvent.builder() + .transactionExternalId(transactionId) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("2000.00")) + .build(); + + // When + producerTemplate.send(KafkaTopics.TRANSACTION_CREATED, transactionId.toString(), event); + + // Then + ConsumerRecord received = records.poll(10, TimeUnit.SECONDS); + assertNotNull(received, "Should receive a status event"); + + TransactionStatusEvent statusEvent = received.value(); + assertEquals(transactionId, statusEvent.getTransactionExternalId()); + assertEquals(TransactionStatus.REJECTED, statusEvent.getStatus()); + } + + @Test + @DisplayName("Should process multiple transactions in sequence") + void shouldProcessMultipleTransactionsInSequence() throws InterruptedException { + // Given + UUID transactionId1 = UUID.randomUUID(); + UUID transactionId2 = UUID.randomUUID(); + + TransactionCreatedEvent event1 = TransactionCreatedEvent.builder() + .transactionExternalId(transactionId1) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("500.00")) + .build(); + + TransactionCreatedEvent event2 = TransactionCreatedEvent.builder() + .transactionExternalId(transactionId2) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("1500.00")) + .build(); + + // When + producerTemplate.send(KafkaTopics.TRANSACTION_CREATED, transactionId1.toString(), event1); + producerTemplate.send(KafkaTopics.TRANSACTION_CREATED, transactionId2.toString(), event2); + + // Then + ConsumerRecord received1 = records.poll(10, TimeUnit.SECONDS); + ConsumerRecord received2 = records.poll(10, TimeUnit.SECONDS); + + assertNotNull(received1, "Should receive first status event"); + assertNotNull(received2, "Should receive second status event"); + + TransactionStatusEvent statusEvent1 = received1.value(); + assertEquals(transactionId1, statusEvent1.getTransactionExternalId()); + assertEquals(TransactionStatus.APPROVED, statusEvent1.getStatus()); + + TransactionStatusEvent statusEvent2 = received2.value(); + assertEquals(transactionId2, statusEvent2.getTransactionExternalId()); + assertEquals(TransactionStatus.REJECTED, statusEvent2.getStatus()); + } + + @Test + @DisplayName("Should use transaction id as message key") + void shouldUseTransactionIdAsMessageKey() throws InterruptedException { + // Given + UUID transactionId = UUID.randomUUID(); + TransactionCreatedEvent event = TransactionCreatedEvent.builder() + .transactionExternalId(transactionId) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("100.00")) + .build(); + + // When + producerTemplate.send(KafkaTopics.TRANSACTION_CREATED, transactionId.toString(), event); + + // Then + ConsumerRecord received = records.poll(10, TimeUnit.SECONDS); + assertNotNull(received, "Should receive a status event"); + assertEquals(transactionId.toString(), received.key()); + } +} + diff --git a/antifraud-service/src/test/java/com/yape/challenge/antifraud/kafka/TransactionCreatedConsumerTest.java b/antifraud-service/src/test/java/com/yape/challenge/antifraud/kafka/TransactionCreatedConsumerTest.java new file mode 100644 index 0000000000..2294f2d693 --- /dev/null +++ b/antifraud-service/src/test/java/com/yape/challenge/antifraud/kafka/TransactionCreatedConsumerTest.java @@ -0,0 +1,121 @@ +package com.yape.challenge.antifraud.kafka; + +import com.yape.challenge.antifraud.service.AntiFraudService; +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.DisplayName; +import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.extension.ExtendWith; +import org.mockito.InjectMocks; +import org.mockito.Mock; +import org.mockito.junit.jupiter.MockitoExtension; + +import java.math.BigDecimal; +import java.util.UUID; + +import static org.junit.jupiter.api.Assertions.*; +import static org.mockito.Mockito.*; + +@ExtendWith(MockitoExtension.class) +@DisplayName("Transaction Created Consumer Tests") +class TransactionCreatedConsumerTest { + + @Mock + private AntiFraudService antiFraudService; + + @InjectMocks + private TransactionCreatedConsumer transactionCreatedConsumer; + + private TransactionCreatedEvent event; + + @BeforeEach + void setUp() { + event = TransactionCreatedEvent.builder() + .transactionExternalId(UUID.randomUUID()) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("500.00")) + .build(); + } + + @Test + @DisplayName("Should successfully consume and process transaction created event") + void shouldSuccessfullyConsumeAndProcessTransactionCreatedEvent() { + // Given + doNothing().when(antiFraudService).validateTransaction(any(TransactionCreatedEvent.class)); + + // When + transactionCreatedConsumer.consumeTransactionCreated(event); + + // Then + verify(antiFraudService, times(1)).validateTransaction(event); + } + + @Test + @DisplayName("Should propagate exception when validation fails") + void shouldPropagateExceptionWhenValidationFails() { + // Given + RuntimeException exception = new RuntimeException("Validation failed"); + doThrow(exception).when(antiFraudService).validateTransaction(any(TransactionCreatedEvent.class)); + + // When & Then + RuntimeException thrown = assertThrows(RuntimeException.class, () -> + transactionCreatedConsumer.consumeTransactionCreated(event) + ); + + assertEquals("Validation failed", thrown.getMessage()); + verify(antiFraudService, times(1)).validateTransaction(event); + } + + @Test + @DisplayName("Should call validate transaction with correct event data") + void shouldCallValidateTransactionWithCorrectEventData() { + // Given + doNothing().when(antiFraudService).validateTransaction(any(TransactionCreatedEvent.class)); + + // When + transactionCreatedConsumer.consumeTransactionCreated(event); + + // Then + verify(antiFraudService).validateTransaction(argThat(e -> + e.getTransactionExternalId().equals(event.getTransactionExternalId()) && + e.getAccountExternalIdDebit().equals(event.getAccountExternalIdDebit()) && + e.getAccountExternalIdCredit().equals(event.getAccountExternalIdCredit()) && + e.getValue().equals(event.getValue()) + )); + } + + @Test + @DisplayName("Should handle multiple consecutive events") + void shouldHandleMultipleConsecutiveEvents() { + // Given + TransactionCreatedEvent event1 = TransactionCreatedEvent.builder() + .transactionExternalId(UUID.randomUUID()) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("100.00")) + .build(); + + TransactionCreatedEvent event2 = TransactionCreatedEvent.builder() + .transactionExternalId(UUID.randomUUID()) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("200.00")) + .build(); + + doNothing().when(antiFraudService).validateTransaction(any(TransactionCreatedEvent.class)); + + // When + transactionCreatedConsumer.consumeTransactionCreated(event1); + transactionCreatedConsumer.consumeTransactionCreated(event2); + + // Then + verify(antiFraudService, times(2)).validateTransaction(any(TransactionCreatedEvent.class)); + verify(antiFraudService).validateTransaction(event1); + verify(antiFraudService).validateTransaction(event2); + } +} + diff --git a/antifraud-service/src/test/java/com/yape/challenge/antifraud/service/AntiFraudServiceTest.java b/antifraud-service/src/test/java/com/yape/challenge/antifraud/service/AntiFraudServiceTest.java new file mode 100644 index 0000000000..64b772a96c --- /dev/null +++ b/antifraud-service/src/test/java/com/yape/challenge/antifraud/service/AntiFraudServiceTest.java @@ -0,0 +1,206 @@ +package com.yape.challenge.antifraud.service; + +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import com.yape.challenge.common.dto.TransactionStatus; +import com.yape.challenge.common.dto.TransactionStatusEvent; +import com.yape.challenge.common.kafka.KafkaTopics; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.DisplayName; +import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.extension.ExtendWith; +import org.mockito.ArgumentCaptor; +import org.mockito.Captor; +import org.mockito.InjectMocks; +import org.mockito.Mock; +import org.mockito.junit.jupiter.MockitoExtension; +import org.springframework.kafka.core.KafkaTemplate; + +import java.math.BigDecimal; +import java.util.UUID; + +import static org.junit.jupiter.api.Assertions.*; +import static org.mockito.ArgumentMatchers.*; +import static org.mockito.Mockito.*; + +@ExtendWith(MockitoExtension.class) +@DisplayName("AntiFraud Service Tests") +class AntiFraudServiceTest { + + @Mock + private KafkaTemplate kafkaTemplate; + + @InjectMocks + private AntiFraudService antiFraudService; + + @Captor + private ArgumentCaptor statusEventCaptor; + + @Captor + private ArgumentCaptor topicCaptor; + + @Captor + private ArgumentCaptor keyCaptor; + + private UUID transactionId; + + @BeforeEach + void setUp() { + transactionId = UUID.randomUUID(); + } + + @Test + @DisplayName("Should approve transaction when value is less than threshold") + void shouldApproveTransactionWhenValueIsLessThanThreshold() { + // Given + TransactionCreatedEvent event = TransactionCreatedEvent.builder() + .transactionExternalId(transactionId) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("500.00")) + .build(); + + // When + antiFraudService.validateTransaction(event); + + // Then + verify(kafkaTemplate).send(topicCaptor.capture(), keyCaptor.capture(), statusEventCaptor.capture()); + + assertEquals(KafkaTopics.TRANSACTION_STATUS_UPDATED, topicCaptor.getValue()); + assertEquals(transactionId.toString(), keyCaptor.getValue()); + + TransactionStatusEvent statusEvent = statusEventCaptor.getValue(); + assertEquals(transactionId, statusEvent.getTransactionExternalId()); + assertEquals(TransactionStatus.APPROVED, statusEvent.getStatus()); + } + + @Test + @DisplayName("Should approve transaction when value is exactly at threshold") + void shouldApproveTransactionWhenValueIsAtThreshold() { + // Given + TransactionCreatedEvent event = TransactionCreatedEvent.builder() + .transactionExternalId(transactionId) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("1000.00")) + .build(); + + // When + antiFraudService.validateTransaction(event); + + // Then + verify(kafkaTemplate).send(anyString(), anyString(), statusEventCaptor.capture()); + + TransactionStatusEvent statusEvent = statusEventCaptor.getValue(); + assertEquals(transactionId, statusEvent.getTransactionExternalId()); + assertEquals(TransactionStatus.APPROVED, statusEvent.getStatus()); + } + + @Test + @DisplayName("Should reject transaction when value exceeds threshold") + void shouldRejectTransactionWhenValueExceedsThreshold() { + // Given + TransactionCreatedEvent event = TransactionCreatedEvent.builder() + .transactionExternalId(transactionId) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("1000.01")) + .build(); + + // When + antiFraudService.validateTransaction(event); + + // Then + verify(kafkaTemplate).send(anyString(), anyString(), statusEventCaptor.capture()); + + TransactionStatusEvent statusEvent = statusEventCaptor.getValue(); + assertEquals(transactionId, statusEvent.getTransactionExternalId()); + assertEquals(TransactionStatus.REJECTED, statusEvent.getStatus()); + } + + @Test + @DisplayName("Should reject transaction when value is much higher than threshold") + void shouldRejectTransactionWhenValueIsMuchHigherThanThreshold() { + // Given + TransactionCreatedEvent event = TransactionCreatedEvent.builder() + .transactionExternalId(transactionId) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("5000.00")) + .build(); + + // When + antiFraudService.validateTransaction(event); + + // Then + verify(kafkaTemplate).send(anyString(), anyString(), statusEventCaptor.capture()); + + TransactionStatusEvent statusEvent = statusEventCaptor.getValue(); + assertEquals(transactionId, statusEvent.getTransactionExternalId()); + assertEquals(TransactionStatus.REJECTED, statusEvent.getStatus()); + } + + @Test + @DisplayName("Should approve transaction with zero value") + void shouldApproveTransactionWithZeroValue() { + // Given + TransactionCreatedEvent event = TransactionCreatedEvent.builder() + .transactionExternalId(transactionId) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(BigDecimal.ZERO) + .build(); + + // When + antiFraudService.validateTransaction(event); + + // Then + verify(kafkaTemplate).send(anyString(), anyString(), statusEventCaptor.capture()); + + TransactionStatusEvent statusEvent = statusEventCaptor.getValue(); + assertEquals(TransactionStatus.APPROVED, statusEvent.getStatus()); + } + + @Test + @DisplayName("Should use correct topic for publishing status event") + void shouldUseCorrectTopicForPublishingStatusEvent() { + // Given + TransactionCreatedEvent event = TransactionCreatedEvent.builder() + .transactionExternalId(transactionId) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("100.00")) + .build(); + + // When + antiFraudService.validateTransaction(event); + + // Then + verify(kafkaTemplate).send(eq(KafkaTopics.TRANSACTION_STATUS_UPDATED), anyString(), any()); + } + + @Test + @DisplayName("Should use transaction external id as kafka message key") + void shouldUseTransactionExternalIdAsKafkaMessageKey() { + // Given + TransactionCreatedEvent event = TransactionCreatedEvent.builder() + .transactionExternalId(transactionId) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("100.00")) + .build(); + + // When + antiFraudService.validateTransaction(event); + + // Then + verify(kafkaTemplate).send(anyString(), eq(transactionId.toString()), any()); + } +} + diff --git a/antifraud-service/src/test/resources/application.yml b/antifraud-service/src/test/resources/application.yml new file mode 100644 index 0000000000..ad74c9ce58 --- /dev/null +++ b/antifraud-service/src/test/resources/application.yml @@ -0,0 +1,19 @@ +spring: + kafka: + bootstrap-servers: localhost:9092 + consumer: + group-id: antifraud-test-group + auto-offset-reset: earliest + key-deserializer: org.apache.kafka.common.serialization.StringDeserializer + value-deserializer: org.springframework.kafka.support.serializer.JsonDeserializer + properties: + spring.json.trusted.packages: "*" + producer: + key-serializer: org.apache.kafka.common.serialization.StringSerializer + value-serializer: org.springframework.kafka.support.serializer.JsonSerializer + +logging: + level: + com.yape.challenge: DEBUG + org.springframework.kafka: DEBUG + diff --git a/common/README.md b/common/README.md new file mode 100644 index 0000000000..2a98a15aae --- /dev/null +++ b/common/README.md @@ -0,0 +1,207 @@ +# Common Module + +Módulo compartido que contiene clases y utilidades comunes utilizadas por todos los microservicios del proyecto Yape Challenge. + +## 📦 Contenido + +Este módulo incluye: + +- **DTOs**: Data Transfer Objects compartidos entre servicios +- **Kafka Topics**: Constantes con nombres de topics de Kafka +- **Enums**: Enumeraciones comunes (estados de transacción) + +## 🏗️ Estructura + +``` +common/ +├── pom.xml +└── src/ + └── main/ + └── java/ + └── com/yape/challenge/common/ + ├── dto/ + │ ├── TransactionCreatedEvent.java + │ ├── TransactionStatusEvent.java + │ └── TransactionStatus.java + └── kafka/ + └── KafkaTopics.java +``` + +## 📄 Clases Principales + +### DTOs + +#### TransactionCreatedEvent +Evento publicado cuando se crea una nueva transacción. + +**Campos:** +- `transactionExternalId` (UUID): ID externo de la transacción +- `accountExternalIdDebit` (String): ID de cuenta de débito +- `accountExternalIdCredit` (String): ID de cuenta de crédito +- `tranferTypeId` (Integer): ID del tipo de transferencia +- `value` (BigDecimal): Monto de la transacción +- `createdAt` (LocalDateTime): Fecha y hora de creación + +**Topic Kafka:** `transaction-created` + +#### TransactionStatusEvent +Evento publicado cuando se actualiza el estado de una transacción. + +**Campos:** +- `transactionExternalId` (UUID): ID externo de la transacción +- `status` (TransactionStatus): Nuevo estado de la transacción + +**Topic Kafka:** `transaction-status` + +#### TransactionStatus (Enum) +Estados posibles de una transacción: + +- `PENDING`: Transacción creada, pendiente de validación +- `APPROVED`: Transacción aprobada por anti-fraude +- `REJECTED`: Transacción rechazada por anti-fraude + +### Kafka Topics + +#### KafkaTopics +Constantes con los nombres de los topics de Kafka: + +```java +public class KafkaTopics { + public static final String TRANSACTION_CREATED = "transaction-created"; + public static final String TRANSACTION_STATUS_UPDATED = "transaction-status"; +} +``` + +## 🔧 Uso + +### Como Dependencia Maven + +En los módulos `transaction-service` y `antifraud-service`, este módulo se incluye como dependencia: + +```xml + + com.yape + common + 0.0.1-SNAPSHOT + +``` + +### Ejemplo de Uso + +#### En Transaction Service (Productor) + +```java +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import com.yape.challenge.common.kafka.KafkaTopics; + +@Service +public class TransactionProducer { + + private final KafkaTemplate kafkaTemplate; + + public void publishTransactionCreated(TransactionCreatedEvent event) { + kafkaTemplate.send( + KafkaTopics.TRANSACTION_CREATED, + event.getTransactionExternalId().toString(), + event + ); + } +} +``` + +#### En Anti-Fraud Service (Consumidor) + +```java +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import com.yape.challenge.common.dto.TransactionStatusEvent; +import com.yape.challenge.common.dto.TransactionStatus; +import com.yape.challenge.common.kafka.KafkaTopics; + +@Service +public class AntiFraudService { + + @KafkaListener(topics = KafkaTopics.TRANSACTION_CREATED) + public void handleTransactionCreated(TransactionCreatedEvent event) { + // Validar transacción + TransactionStatus status = validate(event); + + // Publicar resultado + TransactionStatusEvent statusEvent = TransactionStatusEvent.builder() + .transactionExternalId(event.getTransactionExternalId()) + .status(status) + .build(); + + kafkaTemplate.send(KafkaTopics.TRANSACTION_STATUS_UPDATED, statusEvent); + } +} +``` + +## 📋 Compilación + +Este módulo se compila automáticamente como parte del proyecto principal: + +```bash +# Desde la raíz del proyecto +mvn clean install + +# Solo el módulo common +mvn clean install -pl common +``` + +## 🧪 Testing + +```bash +# Ejecutar tests del módulo +mvn test -pl common +``` + +## 📝 Notas + +- Este módulo **NO** tiene dependencias de Spring Boot +- Es una librería Java pura con POJOs +- Se usa tanto en `transaction-service` como en `antifraud-service` +- No genera un JAR ejecutable, solo una librería +- Todos los DTOs usan Lombok para reducir boilerplate + +## 🔗 Referencias + +- [README Principal](../README.md) +- [Transaction Service](../transaction-service/README.md) +- [Anti-Fraud Service](../antifraud-service/README.md) + +## 📊 Diagrama de Dependencias + +``` +┌─────────────────────┐ +│ transaction- │ +│ service │ +└──────────┬──────────┘ + │ + │ depends on + │ + ▼ +┌─────────────────────┐ +│ common │◄──────────┐ +│ (shared lib) │ │ +└─────────────────────┘ │ + │ + depends on + │ + ┌──────────────────────┘ + │ +┌──────────▼──────────┐ +│ antifraud- │ +│ service │ +└─────────────────────┘ +``` + +## 🎯 Propósito + +Este módulo existe para: + +1. **Evitar duplicación de código**: DTOs usados por múltiples servicios +2. **Mantener contratos consistentes**: Mismos DTOs para Kafka +3. **Facilitar el mantenimiento**: Cambios en un solo lugar +4. **Type safety**: Usar enums en lugar de strings para estados +5. **Documentación centralizada**: Constantes de topics en un lugar + diff --git a/common/pom.xml b/common/pom.xml new file mode 100644 index 0000000000..22fd07c619 --- /dev/null +++ b/common/pom.xml @@ -0,0 +1,49 @@ + + + 4.0.0 + + + com.yape + yape-challenge + 0.0.1-SNAPSHOT + + + common + common + Common module with shared DTOs and constants + + + + + org.projectlombok + lombok + true + + + + + org.springframework.boot + spring-boot-starter-validation + + + + + com.fasterxml.jackson.core + jackson-databind + + + + + + + org.apache.maven.plugins + maven-compiler-plugin + + + + + + diff --git a/common/src/main/java/com/yape/challenge/common/dto/TransactionCreatedEvent.java b/common/src/main/java/com/yape/challenge/common/dto/TransactionCreatedEvent.java new file mode 100644 index 0000000000..084c9e5ffa --- /dev/null +++ b/common/src/main/java/com/yape/challenge/common/dto/TransactionCreatedEvent.java @@ -0,0 +1,24 @@ +package com.yape.challenge.common.dto; + +import lombok.AllArgsConstructor; +import lombok.Builder; +import lombok.Data; +import lombok.NoArgsConstructor; + +import java.math.BigDecimal; +import java.util.UUID; + +@Data +@Builder +@NoArgsConstructor +@AllArgsConstructor +public class TransactionCreatedEvent { + + private UUID transactionExternalId; + private UUID accountExternalIdDebit; + private UUID accountExternalIdCredit; + private Integer transferTypeId; + private BigDecimal value; + private TransactionStatus status; +} + diff --git a/common/src/main/java/com/yape/challenge/common/dto/TransactionStatus.java b/common/src/main/java/com/yape/challenge/common/dto/TransactionStatus.java new file mode 100644 index 0000000000..bc1d969203 --- /dev/null +++ b/common/src/main/java/com/yape/challenge/common/dto/TransactionStatus.java @@ -0,0 +1,7 @@ +package com.yape.challenge.common.dto; + +public enum TransactionStatus { + REJECTED, + APPROVED, + PENDING +} diff --git a/common/src/main/java/com/yape/challenge/common/dto/TransactionStatusEvent.java b/common/src/main/java/com/yape/challenge/common/dto/TransactionStatusEvent.java new file mode 100644 index 0000000000..fe04a0ec6d --- /dev/null +++ b/common/src/main/java/com/yape/challenge/common/dto/TransactionStatusEvent.java @@ -0,0 +1,19 @@ +package com.yape.challenge.common.dto; + +import lombok.AllArgsConstructor; +import lombok.Builder; +import lombok.Data; +import lombok.NoArgsConstructor; + +import java.util.UUID; + +@Data +@Builder +@NoArgsConstructor +@AllArgsConstructor +public class TransactionStatusEvent { + + private UUID transactionExternalId; + private TransactionStatus status; +} + diff --git a/common/src/main/java/com/yape/challenge/common/kafka/KafkaTopics.java b/common/src/main/java/com/yape/challenge/common/kafka/KafkaTopics.java new file mode 100644 index 0000000000..4bbe9aeaf3 --- /dev/null +++ b/common/src/main/java/com/yape/challenge/common/kafka/KafkaTopics.java @@ -0,0 +1,12 @@ +package com.yape.challenge.common.kafka; + +public class KafkaTopics { + + public static final String TRANSACTION_CREATED = "transaction-created"; + public static final String TRANSACTION_STATUS_UPDATED = "transaction-status-updated"; + + private KafkaTopics() { + throw new IllegalStateException("Constants class"); + } +} + diff --git a/docker-compose.yml b/docker-compose.yml index 0e8807f21c..1b1efb867b 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -1,25 +1,147 @@ -version: "3.7" +version: '3.8' + services: + # PostgreSQL Database postgres: - image: postgres:14 + image: postgres:16-alpine + container_name: yape-postgres + environment: + - POSTGRES_DB=yape_transactions + - POSTGRES_USER=yapeuser + - POSTGRES_PASSWORD=YapePass2026 ports: - "5432:5432" - environment: - - POSTGRES_USER=postgres - - POSTGRES_PASSWORD=postgres + volumes: + - postgres-data:/var/lib/postgresql/data + healthcheck: + test: ["CMD-SHELL", "pg_isready -U yapeuser -d yape_transactions"] + interval: 10s + timeout: 5s + retries: 5 + networks: + - yape-network + + # Redis Cache + redis: + image: redis:7-alpine + container_name: yape-redis + ports: + - "6379:6379" + command: > + redis-server + --maxmemory 512mb + --maxmemory-policy allkeys-lru + --save "" + --appendonly no + healthcheck: + test: ["CMD", "redis-cli", "ping"] + interval: 10s + timeout: 5s + retries: 5 + networks: + - yape-network + + # Zookeeper for Kafka zookeeper: - image: confluentinc/cp-zookeeper:5.5.3 + image: confluentinc/cp-zookeeper:7.5.0 + container_name: yape-zookeeper environment: ZOOKEEPER_CLIENT_PORT: 2181 + ZOOKEEPER_TICK_TIME: 2000 + ports: + - "2181:2181" + networks: + - yape-network + + # Kafka kafka: - image: confluentinc/cp-enterprise-kafka:5.5.3 - depends_on: [zookeeper] + image: confluentinc/cp-kafka:7.5.0 + container_name: yape-kafka + depends_on: + - zookeeper + ports: + - "9092:9092" + - "29092:29092" environment: - KAFKA_ZOOKEEPER_CONNECT: "zookeeper:2181" + KAFKA_BROKER_ID: 1 + KAFKA_ZOOKEEPER_CONNECT: zookeeper:2181 KAFKA_ADVERTISED_LISTENERS: PLAINTEXT://kafka:29092,PLAINTEXT_HOST://localhost:9092 KAFKA_LISTENER_SECURITY_PROTOCOL_MAP: PLAINTEXT:PLAINTEXT,PLAINTEXT_HOST:PLAINTEXT - KAFKA_BROKER_ID: 1 + KAFKA_INTER_BROKER_LISTENER_NAME: PLAINTEXT KAFKA_OFFSETS_TOPIC_REPLICATION_FACTOR: 1 - KAFKA_JMX_PORT: 9991 + KAFKA_AUTO_CREATE_TOPICS_ENABLE: "true" + healthcheck: + test: kafka-broker-api-versions --bootstrap-server localhost:9092 || exit 1 + interval: 10s + timeout: 5s + retries: 5 + networks: + - yape-network + + # Kafka UI + kafka-ui: + image: provectuslabs/kafka-ui:latest + container_name: yape-kafka-ui + depends_on: + - kafka + ports: + - "8090:8080" + environment: + KAFKA_CLUSTERS_0_NAME: local + KAFKA_CLUSTERS_0_BOOTSTRAPSERVERS: kafka:29092 + KAFKA_CLUSTERS_0_ZOOKEEPER: zookeeper:2181 + networks: + - yape-network + + # Transaction Service + transaction-service: + build: + context: . + dockerfile: transaction-service/Dockerfile + container_name: yape-transaction-service + depends_on: + postgres: + condition: service_healthy + kafka: + condition: service_healthy + redis: + condition: service_healthy ports: - - 9092:9092 + - "8080:8080" + environment: + - SPRING_PROFILES_ACTIVE=docker + - SPRING_DATASOURCE_URL=jdbc:postgresql://postgres:5432/yape_transactions + - SPRING_DATASOURCE_USERNAME=yapeuser + - SPRING_DATASOURCE_PASSWORD=YapePass2026 + - SPRING_KAFKA_BOOTSTRAP_SERVERS=kafka:29092 + - SPRING_REDIS_HOST=redis + - SPRING_REDIS_PORT=6379 + networks: + - yape-network + restart: on-failure + + # Anti-Fraud Service + antifraud-service: + build: + context: . + dockerfile: antifraud-service/Dockerfile + container_name: yape-antifraud-service + depends_on: + kafka: + condition: service_healthy + ports: + - "8081:8081" + environment: + - SPRING_PROFILES_ACTIVE=docker,antifraud + - SPRING_KAFKA_BOOTSTRAP_SERVERS=kafka:29092 + networks: + - yape-network + restart: on-failure + +networks: + yape-network: + driver: bridge + +volumes: + postgres-data: + diff --git a/mvnw b/mvnw new file mode 100755 index 0000000000..bd8896bf22 --- /dev/null +++ b/mvnw @@ -0,0 +1,295 @@ +#!/bin/sh +# ---------------------------------------------------------------------------- +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file +# distributed with this work for additional information +# regarding copyright ownership. The ASF licenses this file +# to you under the Apache License, Version 2.0 (the +# "License"); you may not use this file except in compliance +# with the License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, +# software distributed under the License is distributed on an +# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY +# KIND, either express or implied. See the License for the +# specific language governing permissions and limitations +# under the License. +# ---------------------------------------------------------------------------- + +# ---------------------------------------------------------------------------- +# Apache Maven Wrapper startup batch script, version 3.3.4 +# +# Optional ENV vars +# ----------------- +# JAVA_HOME - location of a JDK home dir, required when download maven via java source +# MVNW_REPOURL - repo url base for downloading maven distribution +# MVNW_USERNAME/MVNW_PASSWORD - user and password for downloading maven +# MVNW_VERBOSE - true: enable verbose log; debug: trace the mvnw script; others: silence the output +# ---------------------------------------------------------------------------- + +set -euf +[ "${MVNW_VERBOSE-}" != debug ] || set -x + +# OS specific support. +native_path() { printf %s\\n "$1"; } +case "$(uname)" in +CYGWIN* | MINGW*) + [ -z "${JAVA_HOME-}" ] || JAVA_HOME="$(cygpath --unix "$JAVA_HOME")" + native_path() { cygpath --path --windows "$1"; } + ;; +esac + +# set JAVACMD and JAVACCMD +set_java_home() { + # For Cygwin and MinGW, ensure paths are in Unix format before anything is touched + if [ -n "${JAVA_HOME-}" ]; then + if [ -x "$JAVA_HOME/jre/sh/java" ]; then + # IBM's JDK on AIX uses strange locations for the executables + JAVACMD="$JAVA_HOME/jre/sh/java" + JAVACCMD="$JAVA_HOME/jre/sh/javac" + else + JAVACMD="$JAVA_HOME/bin/java" + JAVACCMD="$JAVA_HOME/bin/javac" + + if [ ! -x "$JAVACMD" ] || [ ! -x "$JAVACCMD" ]; then + echo "The JAVA_HOME environment variable is not defined correctly, so mvnw cannot run." >&2 + echo "JAVA_HOME is set to \"$JAVA_HOME\", but \"\$JAVA_HOME/bin/java\" or \"\$JAVA_HOME/bin/javac\" does not exist." >&2 + return 1 + fi + fi + else + JAVACMD="$( + 'set' +e + 'unset' -f command 2>/dev/null + 'command' -v java + )" || : + JAVACCMD="$( + 'set' +e + 'unset' -f command 2>/dev/null + 'command' -v javac + )" || : + + if [ ! -x "${JAVACMD-}" ] || [ ! -x "${JAVACCMD-}" ]; then + echo "The java/javac command does not exist in PATH nor is JAVA_HOME set, so mvnw cannot run." >&2 + return 1 + fi + fi +} + +# hash string like Java String::hashCode +hash_string() { + str="${1:-}" h=0 + while [ -n "$str" ]; do + char="${str%"${str#?}"}" + h=$(((h * 31 + $(LC_CTYPE=C printf %d "'$char")) % 4294967296)) + str="${str#?}" + done + printf %x\\n $h +} + +verbose() { :; } +[ "${MVNW_VERBOSE-}" != true ] || verbose() { printf %s\\n "${1-}"; } + +die() { + printf %s\\n "$1" >&2 + exit 1 +} + +trim() { + # MWRAPPER-139: + # Trims trailing and leading whitespace, carriage returns, tabs, and linefeeds. + # Needed for removing poorly interpreted newline sequences when running in more + # exotic environments such as mingw bash on Windows. + printf "%s" "${1}" | tr -d '[:space:]' +} + +scriptDir="$(dirname "$0")" +scriptName="$(basename "$0")" + +# parse distributionUrl and optional distributionSha256Sum, requires .mvn/wrapper/maven-wrapper.properties +while IFS="=" read -r key value; do + case "${key-}" in + distributionUrl) distributionUrl=$(trim "${value-}") ;; + distributionSha256Sum) distributionSha256Sum=$(trim "${value-}") ;; + esac +done <"$scriptDir/.mvn/wrapper/maven-wrapper.properties" +[ -n "${distributionUrl-}" ] || die "cannot read distributionUrl property in $scriptDir/.mvn/wrapper/maven-wrapper.properties" + +case "${distributionUrl##*/}" in +maven-mvnd-*bin.*) + MVN_CMD=mvnd.sh _MVNW_REPO_PATTERN=/maven/mvnd/ + case "${PROCESSOR_ARCHITECTURE-}${PROCESSOR_ARCHITEW6432-}:$(uname -a)" in + *AMD64:CYGWIN* | *AMD64:MINGW*) distributionPlatform=windows-amd64 ;; + :Darwin*x86_64) distributionPlatform=darwin-amd64 ;; + :Darwin*arm64) distributionPlatform=darwin-aarch64 ;; + :Linux*x86_64*) distributionPlatform=linux-amd64 ;; + *) + echo "Cannot detect native platform for mvnd on $(uname)-$(uname -m), use pure java version" >&2 + distributionPlatform=linux-amd64 + ;; + esac + distributionUrl="${distributionUrl%-bin.*}-$distributionPlatform.zip" + ;; +maven-mvnd-*) MVN_CMD=mvnd.sh _MVNW_REPO_PATTERN=/maven/mvnd/ ;; +*) MVN_CMD="mvn${scriptName#mvnw}" _MVNW_REPO_PATTERN=/org/apache/maven/ ;; +esac + +# apply MVNW_REPOURL and calculate MAVEN_HOME +# maven home pattern: ~/.m2/wrapper/dists/{apache-maven-,maven-mvnd--}/ +[ -z "${MVNW_REPOURL-}" ] || distributionUrl="$MVNW_REPOURL$_MVNW_REPO_PATTERN${distributionUrl#*"$_MVNW_REPO_PATTERN"}" +distributionUrlName="${distributionUrl##*/}" +distributionUrlNameMain="${distributionUrlName%.*}" +distributionUrlNameMain="${distributionUrlNameMain%-bin}" +MAVEN_USER_HOME="${MAVEN_USER_HOME:-${HOME}/.m2}" +MAVEN_HOME="${MAVEN_USER_HOME}/wrapper/dists/${distributionUrlNameMain-}/$(hash_string "$distributionUrl")" + +exec_maven() { + unset MVNW_VERBOSE MVNW_USERNAME MVNW_PASSWORD MVNW_REPOURL || : + exec "$MAVEN_HOME/bin/$MVN_CMD" "$@" || die "cannot exec $MAVEN_HOME/bin/$MVN_CMD" +} + +if [ -d "$MAVEN_HOME" ]; then + verbose "found existing MAVEN_HOME at $MAVEN_HOME" + exec_maven "$@" +fi + +case "${distributionUrl-}" in +*?-bin.zip | *?maven-mvnd-?*-?*.zip) ;; +*) die "distributionUrl is not valid, must match *-bin.zip or maven-mvnd-*.zip, but found '${distributionUrl-}'" ;; +esac + +# prepare tmp dir +if TMP_DOWNLOAD_DIR="$(mktemp -d)" && [ -d "$TMP_DOWNLOAD_DIR" ]; then + clean() { rm -rf -- "$TMP_DOWNLOAD_DIR"; } + trap clean HUP INT TERM EXIT +else + die "cannot create temp dir" +fi + +mkdir -p -- "${MAVEN_HOME%/*}" + +# Download and Install Apache Maven +verbose "Couldn't find MAVEN_HOME, downloading and installing it ..." +verbose "Downloading from: $distributionUrl" +verbose "Downloading to: $TMP_DOWNLOAD_DIR/$distributionUrlName" + +# select .zip or .tar.gz +if ! command -v unzip >/dev/null; then + distributionUrl="${distributionUrl%.zip}.tar.gz" + distributionUrlName="${distributionUrl##*/}" +fi + +# verbose opt +__MVNW_QUIET_WGET=--quiet __MVNW_QUIET_CURL=--silent __MVNW_QUIET_UNZIP=-q __MVNW_QUIET_TAR='' +[ "${MVNW_VERBOSE-}" != true ] || __MVNW_QUIET_WGET='' __MVNW_QUIET_CURL='' __MVNW_QUIET_UNZIP='' __MVNW_QUIET_TAR=v + +# normalize http auth +case "${MVNW_PASSWORD:+has-password}" in +'') MVNW_USERNAME='' MVNW_PASSWORD='' ;; +has-password) [ -n "${MVNW_USERNAME-}" ] || MVNW_USERNAME='' MVNW_PASSWORD='' ;; +esac + +if [ -z "${MVNW_USERNAME-}" ] && command -v wget >/dev/null; then + verbose "Found wget ... using wget" + wget ${__MVNW_QUIET_WGET:+"$__MVNW_QUIET_WGET"} "$distributionUrl" -O "$TMP_DOWNLOAD_DIR/$distributionUrlName" || die "wget: Failed to fetch $distributionUrl" +elif [ -z "${MVNW_USERNAME-}" ] && command -v curl >/dev/null; then + verbose "Found curl ... using curl" + curl ${__MVNW_QUIET_CURL:+"$__MVNW_QUIET_CURL"} -f -L -o "$TMP_DOWNLOAD_DIR/$distributionUrlName" "$distributionUrl" || die "curl: Failed to fetch $distributionUrl" +elif set_java_home; then + verbose "Falling back to use Java to download" + javaSource="$TMP_DOWNLOAD_DIR/Downloader.java" + targetZip="$TMP_DOWNLOAD_DIR/$distributionUrlName" + cat >"$javaSource" <<-END + public class Downloader extends java.net.Authenticator + { + protected java.net.PasswordAuthentication getPasswordAuthentication() + { + return new java.net.PasswordAuthentication( System.getenv( "MVNW_USERNAME" ), System.getenv( "MVNW_PASSWORD" ).toCharArray() ); + } + public static void main( String[] args ) throws Exception + { + setDefault( new Downloader() ); + java.nio.file.Files.copy( java.net.URI.create( args[0] ).toURL().openStream(), java.nio.file.Paths.get( args[1] ).toAbsolutePath().normalize() ); + } + } + END + # For Cygwin/MinGW, switch paths to Windows format before running javac and java + verbose " - Compiling Downloader.java ..." + "$(native_path "$JAVACCMD")" "$(native_path "$javaSource")" || die "Failed to compile Downloader.java" + verbose " - Running Downloader.java ..." + "$(native_path "$JAVACMD")" -cp "$(native_path "$TMP_DOWNLOAD_DIR")" Downloader "$distributionUrl" "$(native_path "$targetZip")" +fi + +# If specified, validate the SHA-256 sum of the Maven distribution zip file +if [ -n "${distributionSha256Sum-}" ]; then + distributionSha256Result=false + if [ "$MVN_CMD" = mvnd.sh ]; then + echo "Checksum validation is not supported for maven-mvnd." >&2 + echo "Please disable validation by removing 'distributionSha256Sum' from your maven-wrapper.properties." >&2 + exit 1 + elif command -v sha256sum >/dev/null; then + if echo "$distributionSha256Sum $TMP_DOWNLOAD_DIR/$distributionUrlName" | sha256sum -c - >/dev/null 2>&1; then + distributionSha256Result=true + fi + elif command -v shasum >/dev/null; then + if echo "$distributionSha256Sum $TMP_DOWNLOAD_DIR/$distributionUrlName" | shasum -a 256 -c >/dev/null 2>&1; then + distributionSha256Result=true + fi + else + echo "Checksum validation was requested but neither 'sha256sum' or 'shasum' are available." >&2 + echo "Please install either command, or disable validation by removing 'distributionSha256Sum' from your maven-wrapper.properties." >&2 + exit 1 + fi + if [ $distributionSha256Result = false ]; then + echo "Error: Failed to validate Maven distribution SHA-256, your Maven distribution might be compromised." >&2 + echo "If you updated your Maven version, you need to update the specified distributionSha256Sum property." >&2 + exit 1 + fi +fi + +# unzip and move +if command -v unzip >/dev/null; then + unzip ${__MVNW_QUIET_UNZIP:+"$__MVNW_QUIET_UNZIP"} "$TMP_DOWNLOAD_DIR/$distributionUrlName" -d "$TMP_DOWNLOAD_DIR" || die "failed to unzip" +else + tar xzf${__MVNW_QUIET_TAR:+"$__MVNW_QUIET_TAR"} "$TMP_DOWNLOAD_DIR/$distributionUrlName" -C "$TMP_DOWNLOAD_DIR" || die "failed to untar" +fi + +# Find the actual extracted directory name (handles snapshots where filename != directory name) +actualDistributionDir="" + +# First try the expected directory name (for regular distributions) +if [ -d "$TMP_DOWNLOAD_DIR/$distributionUrlNameMain" ]; then + if [ -f "$TMP_DOWNLOAD_DIR/$distributionUrlNameMain/bin/$MVN_CMD" ]; then + actualDistributionDir="$distributionUrlNameMain" + fi +fi + +# If not found, search for any directory with the Maven executable (for snapshots) +if [ -z "$actualDistributionDir" ]; then + # enable globbing to iterate over items + set +f + for dir in "$TMP_DOWNLOAD_DIR"/*; do + if [ -d "$dir" ]; then + if [ -f "$dir/bin/$MVN_CMD" ]; then + actualDistributionDir="$(basename "$dir")" + break + fi + fi + done + set -f +fi + +if [ -z "$actualDistributionDir" ]; then + verbose "Contents of $TMP_DOWNLOAD_DIR:" + verbose "$(ls -la "$TMP_DOWNLOAD_DIR")" + die "Could not find Maven distribution directory in extracted archive" +fi + +verbose "Found extracted Maven distribution directory: $actualDistributionDir" +printf %s\\n "$distributionUrl" >"$TMP_DOWNLOAD_DIR/$actualDistributionDir/mvnw.url" +mv -- "$TMP_DOWNLOAD_DIR/$actualDistributionDir" "$MAVEN_HOME" || [ -d "$MAVEN_HOME" ] || die "fail to move MAVEN_HOME" + +clean || : +exec_maven "$@" diff --git a/mvnw.cmd b/mvnw.cmd new file mode 100644 index 0000000000..92450f9327 --- /dev/null +++ b/mvnw.cmd @@ -0,0 +1,189 @@ +<# : batch portion +@REM ---------------------------------------------------------------------------- +@REM Licensed to the Apache Software Foundation (ASF) under one +@REM or more contributor license agreements. See the NOTICE file +@REM distributed with this work for additional information +@REM regarding copyright ownership. The ASF licenses this file +@REM to you under the Apache License, Version 2.0 (the +@REM "License"); you may not use this file except in compliance +@REM with the License. You may obtain a copy of the License at +@REM +@REM http://www.apache.org/licenses/LICENSE-2.0 +@REM +@REM Unless required by applicable law or agreed to in writing, +@REM software distributed under the License is distributed on an +@REM "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY +@REM KIND, either express or implied. See the License for the +@REM specific language governing permissions and limitations +@REM under the License. +@REM ---------------------------------------------------------------------------- + +@REM ---------------------------------------------------------------------------- +@REM Apache Maven Wrapper startup batch script, version 3.3.4 +@REM +@REM Optional ENV vars +@REM MVNW_REPOURL - repo url base for downloading maven distribution +@REM MVNW_USERNAME/MVNW_PASSWORD - user and password for downloading maven +@REM MVNW_VERBOSE - true: enable verbose log; others: silence the output +@REM ---------------------------------------------------------------------------- + +@IF "%__MVNW_ARG0_NAME__%"=="" (SET __MVNW_ARG0_NAME__=%~nx0) +@SET __MVNW_CMD__= +@SET __MVNW_ERROR__= +@SET __MVNW_PSMODULEP_SAVE=%PSModulePath% +@SET PSModulePath= +@FOR /F "usebackq tokens=1* delims==" %%A IN (`powershell -noprofile "& {$scriptDir='%~dp0'; $script='%__MVNW_ARG0_NAME__%'; icm -ScriptBlock ([Scriptblock]::Create((Get-Content -Raw '%~f0'))) -NoNewScope}"`) DO @( + IF "%%A"=="MVN_CMD" (set __MVNW_CMD__=%%B) ELSE IF "%%B"=="" (echo %%A) ELSE (echo %%A=%%B) +) +@SET PSModulePath=%__MVNW_PSMODULEP_SAVE% +@SET __MVNW_PSMODULEP_SAVE= +@SET __MVNW_ARG0_NAME__= +@SET MVNW_USERNAME= +@SET MVNW_PASSWORD= +@IF NOT "%__MVNW_CMD__%"=="" ("%__MVNW_CMD__%" %*) +@echo Cannot start maven from wrapper >&2 && exit /b 1 +@GOTO :EOF +: end batch / begin powershell #> + +$ErrorActionPreference = "Stop" +if ($env:MVNW_VERBOSE -eq "true") { + $VerbosePreference = "Continue" +} + +# calculate distributionUrl, requires .mvn/wrapper/maven-wrapper.properties +$distributionUrl = (Get-Content -Raw "$scriptDir/.mvn/wrapper/maven-wrapper.properties" | ConvertFrom-StringData).distributionUrl +if (!$distributionUrl) { + Write-Error "cannot read distributionUrl property in $scriptDir/.mvn/wrapper/maven-wrapper.properties" +} + +switch -wildcard -casesensitive ( $($distributionUrl -replace '^.*/','') ) { + "maven-mvnd-*" { + $USE_MVND = $true + $distributionUrl = $distributionUrl -replace '-bin\.[^.]*$',"-windows-amd64.zip" + $MVN_CMD = "mvnd.cmd" + break + } + default { + $USE_MVND = $false + $MVN_CMD = $script -replace '^mvnw','mvn' + break + } +} + +# apply MVNW_REPOURL and calculate MAVEN_HOME +# maven home pattern: ~/.m2/wrapper/dists/{apache-maven-,maven-mvnd--}/ +if ($env:MVNW_REPOURL) { + $MVNW_REPO_PATTERN = if ($USE_MVND -eq $False) { "/org/apache/maven/" } else { "/maven/mvnd/" } + $distributionUrl = "$env:MVNW_REPOURL$MVNW_REPO_PATTERN$($distributionUrl -replace "^.*$MVNW_REPO_PATTERN",'')" +} +$distributionUrlName = $distributionUrl -replace '^.*/','' +$distributionUrlNameMain = $distributionUrlName -replace '\.[^.]*$','' -replace '-bin$','' + +$MAVEN_M2_PATH = "$HOME/.m2" +if ($env:MAVEN_USER_HOME) { + $MAVEN_M2_PATH = "$env:MAVEN_USER_HOME" +} + +if (-not (Test-Path -Path $MAVEN_M2_PATH)) { + New-Item -Path $MAVEN_M2_PATH -ItemType Directory | Out-Null +} + +$MAVEN_WRAPPER_DISTS = $null +if ((Get-Item $MAVEN_M2_PATH).Target[0] -eq $null) { + $MAVEN_WRAPPER_DISTS = "$MAVEN_M2_PATH/wrapper/dists" +} else { + $MAVEN_WRAPPER_DISTS = (Get-Item $MAVEN_M2_PATH).Target[0] + "/wrapper/dists" +} + +$MAVEN_HOME_PARENT = "$MAVEN_WRAPPER_DISTS/$distributionUrlNameMain" +$MAVEN_HOME_NAME = ([System.Security.Cryptography.SHA256]::Create().ComputeHash([byte[]][char[]]$distributionUrl) | ForEach-Object {$_.ToString("x2")}) -join '' +$MAVEN_HOME = "$MAVEN_HOME_PARENT/$MAVEN_HOME_NAME" + +if (Test-Path -Path "$MAVEN_HOME" -PathType Container) { + Write-Verbose "found existing MAVEN_HOME at $MAVEN_HOME" + Write-Output "MVN_CMD=$MAVEN_HOME/bin/$MVN_CMD" + exit $? +} + +if (! $distributionUrlNameMain -or ($distributionUrlName -eq $distributionUrlNameMain)) { + Write-Error "distributionUrl is not valid, must end with *-bin.zip, but found $distributionUrl" +} + +# prepare tmp dir +$TMP_DOWNLOAD_DIR_HOLDER = New-TemporaryFile +$TMP_DOWNLOAD_DIR = New-Item -Itemtype Directory -Path "$TMP_DOWNLOAD_DIR_HOLDER.dir" +$TMP_DOWNLOAD_DIR_HOLDER.Delete() | Out-Null +trap { + if ($TMP_DOWNLOAD_DIR.Exists) { + try { Remove-Item $TMP_DOWNLOAD_DIR -Recurse -Force | Out-Null } + catch { Write-Warning "Cannot remove $TMP_DOWNLOAD_DIR" } + } +} + +New-Item -Itemtype Directory -Path "$MAVEN_HOME_PARENT" -Force | Out-Null + +# Download and Install Apache Maven +Write-Verbose "Couldn't find MAVEN_HOME, downloading and installing it ..." +Write-Verbose "Downloading from: $distributionUrl" +Write-Verbose "Downloading to: $TMP_DOWNLOAD_DIR/$distributionUrlName" + +$webclient = New-Object System.Net.WebClient +if ($env:MVNW_USERNAME -and $env:MVNW_PASSWORD) { + $webclient.Credentials = New-Object System.Net.NetworkCredential($env:MVNW_USERNAME, $env:MVNW_PASSWORD) +} +[Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12 +$webclient.DownloadFile($distributionUrl, "$TMP_DOWNLOAD_DIR/$distributionUrlName") | Out-Null + +# If specified, validate the SHA-256 sum of the Maven distribution zip file +$distributionSha256Sum = (Get-Content -Raw "$scriptDir/.mvn/wrapper/maven-wrapper.properties" | ConvertFrom-StringData).distributionSha256Sum +if ($distributionSha256Sum) { + if ($USE_MVND) { + Write-Error "Checksum validation is not supported for maven-mvnd. `nPlease disable validation by removing 'distributionSha256Sum' from your maven-wrapper.properties." + } + Import-Module $PSHOME\Modules\Microsoft.PowerShell.Utility -Function Get-FileHash + if ((Get-FileHash "$TMP_DOWNLOAD_DIR/$distributionUrlName" -Algorithm SHA256).Hash.ToLower() -ne $distributionSha256Sum) { + Write-Error "Error: Failed to validate Maven distribution SHA-256, your Maven distribution might be compromised. If you updated your Maven version, you need to update the specified distributionSha256Sum property." + } +} + +# unzip and move +Expand-Archive "$TMP_DOWNLOAD_DIR/$distributionUrlName" -DestinationPath "$TMP_DOWNLOAD_DIR" | Out-Null + +# Find the actual extracted directory name (handles snapshots where filename != directory name) +$actualDistributionDir = "" + +# First try the expected directory name (for regular distributions) +$expectedPath = Join-Path "$TMP_DOWNLOAD_DIR" "$distributionUrlNameMain" +$expectedMvnPath = Join-Path "$expectedPath" "bin/$MVN_CMD" +if ((Test-Path -Path $expectedPath -PathType Container) -and (Test-Path -Path $expectedMvnPath -PathType Leaf)) { + $actualDistributionDir = $distributionUrlNameMain +} + +# If not found, search for any directory with the Maven executable (for snapshots) +if (!$actualDistributionDir) { + Get-ChildItem -Path "$TMP_DOWNLOAD_DIR" -Directory | ForEach-Object { + $testPath = Join-Path $_.FullName "bin/$MVN_CMD" + if (Test-Path -Path $testPath -PathType Leaf) { + $actualDistributionDir = $_.Name + } + } +} + +if (!$actualDistributionDir) { + Write-Error "Could not find Maven distribution directory in extracted archive" +} + +Write-Verbose "Found extracted Maven distribution directory: $actualDistributionDir" +Rename-Item -Path "$TMP_DOWNLOAD_DIR/$actualDistributionDir" -NewName $MAVEN_HOME_NAME | Out-Null +try { + Move-Item -Path "$TMP_DOWNLOAD_DIR/$MAVEN_HOME_NAME" -Destination $MAVEN_HOME_PARENT | Out-Null +} catch { + if (! (Test-Path -Path "$MAVEN_HOME" -PathType Container)) { + Write-Error "fail to move MAVEN_HOME" + } +} finally { + try { Remove-Item $TMP_DOWNLOAD_DIR -Recurse -Force | Out-Null } + catch { Write-Warning "Cannot remove $TMP_DOWNLOAD_DIR" } +} + +Write-Output "MVN_CMD=$MAVEN_HOME/bin/$MVN_CMD" diff --git a/pom.xml b/pom.xml new file mode 100644 index 0000000000..9fa07c41f0 --- /dev/null +++ b/pom.xml @@ -0,0 +1,106 @@ + + + 4.0.0 + + org.springframework.boot + spring-boot-starter-parent + 3.2.0 + + + com.yape + yape-challenge + 0.0.1-SNAPSHOT + pom + yape-challenge + Yape Challenge - Multi-module monorepo + + + common + transaction-service + antifraud-service + + + + + + + + + + + + + + + + + 21 + 1.5.5.Final + 1.18.30 + 2.3.0 + + + + + + org.mapstruct + mapstruct + ${mapstruct.version} + + + io.github.resilience4j + resilience4j-bom + ${resilience4j.version} + pom + import + + + + + + + + + org.springframework.boot + spring-boot-maven-plugin + + + + org.projectlombok + lombok + + + + + + org.apache.maven.plugins + maven-compiler-plugin + 3.11.0 + + 21 + 21 + + + org.mapstruct + mapstruct-processor + ${mapstruct.version} + + + org.projectlombok + lombok + ${lombok.version} + + + org.projectlombok + lombok-mapstruct-binding + 0.2.0 + + + + + + + + + diff --git a/transaction-service/.dockerignore b/transaction-service/.dockerignore new file mode 100644 index 0000000000..30b8716f4b --- /dev/null +++ b/transaction-service/.dockerignore @@ -0,0 +1,9 @@ +target/ +*.log +.DS_Store +.idea/ +*.iml +.gitignore +.git +*.md + diff --git a/transaction-service/Dockerfile b/transaction-service/Dockerfile new file mode 100644 index 0000000000..f5fc115ffa --- /dev/null +++ b/transaction-service/Dockerfile @@ -0,0 +1,20 @@ + +# Build stage +FROM maven:3.9.5-eclipse-temurin-21 AS build +WORKDIR /app + +# Copy the entire project (parent POM needs to see all modules) +COPY . . + +# Build only transaction-service and its dependencies +RUN mvn clean package -DskipTests -pl transaction-service -am + +# Run stage +FROM eclipse-temurin:21-jre-alpine +WORKDIR /app + +COPY --from=build /app/transaction-service/target/*.jar app.jar + +EXPOSE 8080 + +ENTRYPOINT ["java", "-jar", "app.jar"] diff --git a/transaction-service/README.md b/transaction-service/README.md new file mode 100644 index 0000000000..fd34adfd6d --- /dev/null +++ b/transaction-service/README.md @@ -0,0 +1,248 @@ +# Transaction Service + +Microservicio de gestión de transacciones para Yape Challenge. Implementa **Event Sourcing** y **CQRS** para garantizar auditoría completa y alto rendimiento. + +## 🎯 Características + +- ✅ **Event Sourcing**: Todos los cambios se guardan como eventos +- ✅ **CQRS**: Separación de comandos y consultas +- ✅ **Redis Cache**: Lecturas optimizadas (5-20ms) +- ✅ **PostgreSQL**: Almacenamiento de transacciones y Event Store +- ✅ **Apache Kafka**: Comunicación asíncrona con Anti-Fraud Service +- ✅ **API REST**: Endpoints para crear y consultar transacciones +- ✅ **Event Store API**: Endpoints para auditoría y debugging + +## 🚀 Ejecución + +### Con Docker Compose (recomendado) + +Desde la raíz del proyecto: +```bash +docker-compose up transaction-service --build +``` + +### Con Docker + +Construir la imagen: +```bash +docker build -t transaction-service:latest -f transaction-service/Dockerfile . +``` + +Ejecutar el contenedor: +```bash +docker run -p 8080:8080 \ + -e SPRING_PROFILES_ACTIVE=docker \ + -e SPRING_DATASOURCE_URL=jdbc:postgresql://postgres:5432/yape_transactions \ + -e SPRING_DATASOURCE_USERNAME=yapeuser \ + -e SPRING_DATASOURCE_PASSWORD=YapePass2026 \ + -e SPRING_KAFKA_BOOTSTRAP_SERVERS=kafka:29092 \ + -e SPRING_REDIS_HOST=redis \ + -e SPRING_REDIS_PORT=6379 \ + transaction-service:latest +``` + +### Con Maven + +Compilar y ejecutar localmente: +```bash +cd transaction-service +mvn clean package -DskipTests +java -jar target/transaction-service-*.jar +``` + +**Nota**: Asegúrate de tener PostgreSQL, Redis y Kafka ejecutándose localmente. + +## 📡 Endpoints + +### Puerto +- **8080** + +### API de Transacciones + +#### Crear Transacción +```bash +POST /api/v1/transactions +Content-Type: application/json + +{ + "accountExternalIdDebit": "Guid1", + "accountExternalIdCredit": "Guid2", + "tranferTypeId": 1, + "value": 120.00 +} +``` + +**Respuesta (201 Created):** +```json +{ + "transactionExternalId": "550e8400-e29b-41d4-a716-446655440000", + "transactionStatus": "PENDING", + "transactionType": 1, + "value": 120.00, + "createdAt": "2026-01-04T10:30:00Z" +} +``` + +#### Obtener Transacción +```bash +GET /api/v1/transactions/{externalId} +``` + +**Respuesta (200 OK):** +```json +{ + "transactionExternalId": "550e8400-e29b-41d4-a716-446655440000", + "transactionStatus": "APPROVED", + "transactionType": 1, + "value": 120.00, + "createdAt": "2026-01-04T10:30:00Z" +} +``` + +### API del Event Store (Auditoría) + +#### Obtener eventos de una transacción +```bash +GET /api/v1/events/transaction/{transactionId} +``` + +#### Obtener todos los eventos +```bash +GET /api/v1/events/all +``` + +#### Obtener eventos por tipo +```bash +GET /api/v1/events/type/{eventType} +``` + +Tipos disponibles: +- `TransactionCreatedEvent` +- `TransactionStatusUpdatedEvent` + +#### Verificar si existe una transacción +```bash +GET /api/v1/events/transaction/{transactionId}/exists +``` + +#### Contar eventos de una transacción +```bash +GET /api/v1/events/transaction/{transactionId}/count +``` + +### Actuator (Monitoreo) + +- Health: `http://localhost:8080/actuator/health` +- Metrics: `http://localhost:8080/actuator/metrics` +- Circuit Breakers: `http://localhost:8080/actuator/circuitbreakers` + +## 🔧 Configuración + +### Variables de Entorno + +| Variable | Default | Descripción | +|----------|---------|-------------| +| `SPRING_PROFILES_ACTIVE` | - | Perfil activo (docker, local) | +| `SPRING_DATASOURCE_URL` | jdbc:postgresql://localhost:5432/yape_transactions | URL de PostgreSQL | +| `SPRING_DATASOURCE_USERNAME` | yapeuser | Usuario de PostgreSQL | +| `SPRING_DATASOURCE_PASSWORD` | YapePass2026 | Contraseña de PostgreSQL | +| `SPRING_KAFKA_BOOTSTRAP_SERVERS` | localhost:9092 | Servidores de Kafka | +| `SPRING_REDIS_HOST` | localhost | Host de Redis | +| `SPRING_REDIS_PORT` | 6379 | Puerto de Redis | +| `SERVER_PORT` | 8080 | Puerto del servicio | + +## 🏗️ Arquitectura + +### Patrones Implementados + +- **Event Sourcing**: Almacenamiento de eventos de dominio +- **CQRS**: Command Bus y Query Bus +- **Domain Events**: Eventos de negocio +- **Cache-Aside**: Patrón de caché con Redis +- **Repository Pattern**: Acceso a datos +- **Aggregate Pattern**: TransactionAggregateService + +### Estructura de Paquetes + +``` +com.yape.challenge.transaction/ +├── application/ # CQRS - Commands & Queries +│ ├── bus/ # Command Bus & Query Bus +│ ├── command/ # Commands +│ ├── query/ # Queries +│ ├── handler/ # Handlers +│ └── dto/ # DTOs +├── domain/ +│ ├── entity/ # Entities (JPA) +│ ├── event/ # Domain Events +│ └── service/ # Domain Services +├── infrastructure/ +│ ├── eventstore/ # Event Store +│ ├── repository/ # Repositories +│ ├── kafka/ # Kafka Producers/Consumers +│ └── config/ # Configuraciones +└── presentation/ # REST Controllers + ├── controller/ + └── exception/ +``` + +## 📊 Topics de Kafka + +- **Produce**: `transaction-created` - Notifica nueva transacción al Anti-Fraud Service +- **Consume**: `transaction-status` - Recibe resultado de validación anti-fraude + +## 🗄️ Base de Datos + +### Tablas + +- `transactions`: Read model (proyección) +- `domain_events`: Event Store (eventos de dominio en JSONB) + +### Inicialización + +El archivo `db/data.sql` contiene datos iniciales: +- Tipos de transferencia (Transfer Types) + +## 🧪 Testing + +```bash +# Ejecutar tests +mvn test + +# Ejecutar tests con cobertura +mvn test jacoco:report +``` + +## 🔍 Debugging + +### Ver eventos de una transacción + +```bash +curl http://localhost:8080/api/v1/events/transaction/{transactionId} +``` + +### Ver todos los eventos + +```bash +curl http://localhost:8080/api/v1/events/all +``` + +### Ver logs en Docker + +```bash +docker-compose logs -f transaction-service +``` + +## 📈 Performance + +- **Throughput lecturas**: 5,000-10,000 req/seg (con cache) +- **Latencia P95**: 5-20ms (con cache hit) +- **Cache hit rate esperado**: 80-90% + +## 🔗 Referencias + +- [README Principal](../README.md) +- [Anti-Fraud Service](../antifraud-service/README.md) + + + diff --git a/transaction-service/pom.xml b/transaction-service/pom.xml new file mode 100644 index 0000000000..be047e4f5e --- /dev/null +++ b/transaction-service/pom.xml @@ -0,0 +1,152 @@ + + + 4.0.0 + + + com.yape + yape-challenge + 0.0.1-SNAPSHOT + + + transaction-service + transaction-service + Transaction microservice + + + + + com.yape + common + ${project.version} + + + + + org.springframework.boot + spring-boot-starter-web + + + + + org.springframework.boot + spring-boot-starter-data-jpa + + + + + org.springframework.boot + spring-boot-starter-validation + + + + + org.springframework.kafka + spring-kafka + + + + + org.postgresql + postgresql + runtime + + + + + org.springframework.boot + spring-boot-starter-data-redis + + + io.lettuce + lettuce-core + + + + + org.springframework.boot + spring-boot-starter-cache + + + + + org.projectlombok + lombok + true + + + + + org.mapstruct + mapstruct + + + + + com.fasterxml.jackson.datatype + jackson-datatype-jsr310 + + + + + org.springframework.boot + spring-boot-starter-actuator + + + + + io.github.resilience4j + resilience4j-spring-boot3 + + + + + io.github.resilience4j + resilience4j-micrometer + + + + + org.springframework.boot + spring-boot-starter-aop + + + + + org.springframework.boot + spring-boot-starter-test + test + + + + + org.springframework.kafka + spring-kafka-test + test + + + + + com.h2database + h2 + test + + + + + + + org.springframework.boot + spring-boot-maven-plugin + + + org.apache.maven.plugins + maven-compiler-plugin + + + + + + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/TransactionServiceApplication.java b/transaction-service/src/main/java/com/yape/challenge/transaction/TransactionServiceApplication.java new file mode 100644 index 0000000000..e59bdf07ec --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/TransactionServiceApplication.java @@ -0,0 +1,14 @@ +package com.yape.challenge.transaction; + +import org.springframework.boot.SpringApplication; +import org.springframework.boot.autoconfigure.SpringBootApplication; + +@SpringBootApplication +public class TransactionServiceApplication { + + public static void main(String[] args) { + SpringApplication.run(TransactionServiceApplication.class, args); + } + +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/application/bus/CommandBus.java b/transaction-service/src/main/java/com/yape/challenge/transaction/application/bus/CommandBus.java new file mode 100644 index 0000000000..0785e4a390 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/application/bus/CommandBus.java @@ -0,0 +1,66 @@ +package com.yape.challenge.transaction.application.bus; + +import com.yape.challenge.transaction.application.handler.CommandHandler; +import lombok.extern.slf4j.Slf4j; +import org.springframework.context.ApplicationContext; +import org.springframework.core.ResolvableType; +import org.springframework.stereotype.Component; + +/** + * Command Bus to dispatch commands to their respective handlers + */ +@Component +@Slf4j +public class CommandBus { + + private final ApplicationContext applicationContext; + + public CommandBus(ApplicationContext applicationContext) { + this.applicationContext = applicationContext; + } + + /** + * Dispatches a command to its corresponding handler + * @param command Command to dispatch + * @param Command type + * @param Response type + * @return Response from handler + */ + public R dispatch(C command) { + log.debug("Dispatching command: {}", command.getClass().getSimpleName()); + + CommandHandler handler = findHandler(command); + + if (handler == null) { + throw new IllegalStateException("No handler found for command: " + command.getClass().getName()); + } + + return handler.handle(command); + } + + @SuppressWarnings("unchecked") + private CommandHandler findHandler(C command) { + // Get all CommandHandler beans + var handlers = applicationContext.getBeansOfType(CommandHandler.class); + + CommandHandler foundHandler = null; + + for (CommandHandler handler : handlers.values()) { + // Check if this handler can handle the command by examining its generic type + ResolvableType handlerType = ResolvableType.forClass(handler.getClass()).as(CommandHandler.class); + ResolvableType[] generics = handlerType.getGenerics(); + + if (generics.length == 2) { + Class commandType = generics[0].resolve(); + if (commandType != null && commandType.isAssignableFrom(command.getClass())) { + if (foundHandler != null) { + throw new IllegalStateException("Multiple handlers found for command: " + command.getClass().getName()); + } + foundHandler = (CommandHandler) handler; + } + } + } + + return foundHandler; + } +} diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/application/bus/QueryBus.java b/transaction-service/src/main/java/com/yape/challenge/transaction/application/bus/QueryBus.java new file mode 100644 index 0000000000..2b28dc1990 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/application/bus/QueryBus.java @@ -0,0 +1,64 @@ +package com.yape.challenge.transaction.application.bus; + +import com.yape.challenge.transaction.application.handler.QueryHandler; +import lombok.extern.slf4j.Slf4j; +import org.springframework.context.ApplicationContext; +import org.springframework.core.ResolvableType; +import org.springframework.stereotype.Component; + +/** + * Query Bus to dispatch queries to their respective handlers + */ +@Component +@Slf4j +public class QueryBus { + + private final ApplicationContext applicationContext; + + public QueryBus(ApplicationContext applicationContext) { + this.applicationContext = applicationContext; + } + + /** + * Dispatches a query to its corresponding handler + * + * @param query Query to dispatch + * @param Query type + * @param Response type + * @return Response from handler + */ + public R dispatch(Q query) { + log.debug("Dispatching query: {}", query.getClass().getSimpleName()); + + QueryHandler handler = findHandler(query); + + if (handler == null) { + throw new IllegalStateException("No handler found for query: " + query.getClass().getName()); + } + + return handler.handle(query); + } + + @SuppressWarnings("unchecked") + private QueryHandler findHandler(Q query) { + // Get all QueryHandler beans + var handlers = applicationContext.getBeansOfType(QueryHandler.class); + + // Find the handler that can handle this query type + for (var handler : handlers.values()) { + // Check if this handler can handle the query by examining its generic types + ResolvableType handlerType = ResolvableType.forClass(handler.getClass()).as(QueryHandler.class); + ResolvableType[] generics = handlerType.getGenerics(); + + if (generics.length == 2) { + Class queryType = generics[0].resolve(); + if (queryType != null && queryType.isAssignableFrom(query.getClass())) { + return handler; + } + } + } + + return null; + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/application/command/CreateTransactionCommand.java b/transaction-service/src/main/java/com/yape/challenge/transaction/application/command/CreateTransactionCommand.java new file mode 100644 index 0000000000..43c0092796 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/application/command/CreateTransactionCommand.java @@ -0,0 +1,19 @@ +package com.yape.challenge.transaction.application.command; + +import lombok.Builder; +import lombok.Data; + +import java.math.BigDecimal; +import java.util.UUID; + +/** + * Command to create a new transaction + */ +@Data +@Builder +public class CreateTransactionCommand { + private UUID accountExternalIdDebit; + private UUID accountExternalIdCredit; + private Integer tranferTypeId; + private BigDecimal value; +} diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/application/command/UpdateTransactionStatusCommand.java b/transaction-service/src/main/java/com/yape/challenge/transaction/application/command/UpdateTransactionStatusCommand.java new file mode 100644 index 0000000000..0682d6c4dd --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/application/command/UpdateTransactionStatusCommand.java @@ -0,0 +1,18 @@ +package com.yape.challenge.transaction.application.command; + +import com.yape.challenge.common.dto.TransactionStatus; +import lombok.Builder; +import lombok.Data; + +import java.util.UUID; + +/** + * Command to update transaction status + */ +@Data +@Builder +public class UpdateTransactionStatusCommand { + private UUID externalId; + private TransactionStatus status; +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/application/dto/request/CreateTransactionRequest.java b/transaction-service/src/main/java/com/yape/challenge/transaction/application/dto/request/CreateTransactionRequest.java new file mode 100644 index 0000000000..b1a18a9733 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/application/dto/request/CreateTransactionRequest.java @@ -0,0 +1,34 @@ +package com.yape.challenge.transaction.application.dto.request; + +import jakarta.validation.constraints.NotNull; +import jakarta.validation.constraints.Positive; +import lombok.AllArgsConstructor; +import lombok.Builder; +import lombok.Data; +import lombok.NoArgsConstructor; + +import java.math.BigDecimal; +import java.util.UUID; + +@Data +@Builder +@NoArgsConstructor +@AllArgsConstructor +public class CreateTransactionRequest { + + @NotNull(message = "accountExternalIdDebit is required") + private UUID accountExternalIdDebit; + + @NotNull(message = "accountExternalIdCredit is required") + private UUID accountExternalIdCredit; + + @NotNull(message = "tranferTypeId is required") + @Positive(message = "tranferTypeId must be positive") + private Integer tranferTypeId; + + @NotNull(message = "value is required") + @Positive(message = "value must be positive") + private BigDecimal value; +} + + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/application/dto/response/TransactionResponse.java b/transaction-service/src/main/java/com/yape/challenge/transaction/application/dto/response/TransactionResponse.java new file mode 100644 index 0000000000..c023051eee --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/application/dto/response/TransactionResponse.java @@ -0,0 +1,44 @@ +package com.yape.challenge.transaction.application.dto.response; + +import lombok.AllArgsConstructor; +import lombok.Builder; +import lombok.Data; +import lombok.NoArgsConstructor; + +import java.math.BigDecimal; +import java.time.LocalDateTime; +import java.util.UUID; + +@Data +@Builder +@NoArgsConstructor +@AllArgsConstructor +public class TransactionResponse { + + private UUID transactionExternalId; + + private TransactionTypeDto transactionType; + + private TransactionStatusDto transactionStatus; + + private BigDecimal value; + + private LocalDateTime createdAt; + + @Data + @Builder + @NoArgsConstructor + @AllArgsConstructor + public static class TransactionTypeDto { + private String name; + } + + @Data + @Builder + @NoArgsConstructor + @AllArgsConstructor + public static class TransactionStatusDto { + private String name; + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/CommandHandler.java b/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/CommandHandler.java new file mode 100644 index 0000000000..10c81d8093 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/CommandHandler.java @@ -0,0 +1,10 @@ +package com.yape.challenge.transaction.application.handler; + +/** + * Generic Command Handler interface + * @param Command type + * @param Response type + */ +public interface CommandHandler { + R handle(C command); +} diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/QueryHandler.java b/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/QueryHandler.java new file mode 100644 index 0000000000..845d9d2dfc --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/QueryHandler.java @@ -0,0 +1,11 @@ +package com.yape.challenge.transaction.application.handler; + +/** + * Generic Query Handler interface + * @param Query type + * @param Response type + */ +public interface QueryHandler { + R handle(Q query); +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/command/CreateTransactionCommandHandler.java b/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/command/CreateTransactionCommandHandler.java new file mode 100644 index 0000000000..50f61f1aad --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/command/CreateTransactionCommandHandler.java @@ -0,0 +1,97 @@ +package com.yape.challenge.transaction.application.handler.command; + +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import com.yape.challenge.common.dto.TransactionStatus; +import com.yape.challenge.common.kafka.KafkaTopics; +import com.yape.challenge.transaction.application.command.CreateTransactionCommand; +import com.yape.challenge.transaction.application.dto.response.TransactionResponse; +import com.yape.challenge.transaction.application.handler.CommandHandler; +import com.yape.challenge.transaction.application.mapper.TransactionMapper; +import com.yape.challenge.transaction.domain.entity.Transaction; +import com.yape.challenge.transaction.domain.entity.TransactionType; +import com.yape.challenge.transaction.domain.event.TransactionCreatedDomainEvent; +import com.yape.challenge.transaction.infrastructure.eventstore.EventStore; +import com.yape.challenge.transaction.infrastructure.kafka.producer.KafkaProducerService; +import com.yape.challenge.transaction.infrastructure.repository.TransactionRepository; +import com.yape.challenge.transaction.infrastructure.repository.TransactionTypeRepository; +import lombok.RequiredArgsConstructor; +import lombok.extern.slf4j.Slf4j; +import org.springframework.stereotype.Component; +import org.springframework.transaction.annotation.Transactional; + +import java.time.LocalDateTime; +import java.util.UUID; + +/** + * Handler for CreateTransactionCommand with Event Sourcing + */ +@Component +@RequiredArgsConstructor +@Slf4j +public class CreateTransactionCommandHandler implements CommandHandler { + + private final EventStore eventStore; + private final TransactionRepository transactionRepository; + private final TransactionTypeRepository transactionTypeRepository; + private final TransactionMapper transactionMapper; + private final KafkaProducerService kafkaProducerService; + + @Override + @Transactional + public TransactionResponse handle(CreateTransactionCommand command) { + log.info("Handling CreateTransactionCommand with Event Sourcing: {}", command); + + // 1. Generate unique transaction ID + UUID transactionId = UUID.randomUUID(); + + // 2. Create domain event + TransactionCreatedDomainEvent domainEvent = TransactionCreatedDomainEvent.builder() + .aggregateId(transactionId) + .accountExternalIdDebit(command.getAccountExternalIdDebit()) + .accountExternalIdCredit(command.getAccountExternalIdCredit()) + .transferTypeId(command.getTranferTypeId()) + .value(command.getValue()) + .occurredAt(LocalDateTime.now()) + .build(); + + // 3. Save event to Event Store (persistence) + eventStore.saveEvent(domainEvent); + log.info("Domain event persisted in Event Store for transaction: {}", transactionId); + + // 4. Apply event to create aggregate and save read model + Transaction transaction = applyEvent(domainEvent); + Transaction savedTransaction = transactionRepository.save(transaction); + log.info("Transaction read model saved with externalId: {}", savedTransaction.getExternalId()); + + // 5. Get transaction type + TransactionType transactionType = transactionTypeRepository.findById(command.getTranferTypeId()) + .orElseThrow(() -> new IllegalArgumentException("Transaction type not found")); + + // 6. Publish integration event to Kafka + TransactionCreatedEvent kafkaEvent = transactionMapper.toCreatedEvent(savedTransaction); + kafkaProducerService.sendTransactionCreatedEvent( + KafkaTopics.TRANSACTION_CREATED, + savedTransaction.getExternalId().toString(), + kafkaEvent + ); + log.info("Integration event published to Kafka for externalId: {}", savedTransaction.getExternalId()); + + return transactionMapper.toResponse(savedTransaction, transactionType); + } + + /** + * Apply domain event to create transaction aggregate + */ + private Transaction applyEvent(TransactionCreatedDomainEvent event) { + return Transaction.builder() + .externalId(event.getAggregateId()) + .accountExternalIdDebit(event.getAccountExternalIdDebit()) + .accountExternalIdCredit(event.getAccountExternalIdCredit()) + .transferTypeId(event.getTransferTypeId()) + .value(event.getValue()) + .status(TransactionStatus.PENDING) + .createdAt(event.getOccurredAt()) + .updatedAt(event.getOccurredAt()) + .build(); + } +} diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/command/UpdateTransactionStatusCommandHandler.java b/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/command/UpdateTransactionStatusCommandHandler.java new file mode 100644 index 0000000000..0832a99800 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/command/UpdateTransactionStatusCommandHandler.java @@ -0,0 +1,70 @@ +package com.yape.challenge.transaction.application.handler.command; + +import com.yape.challenge.transaction.application.command.UpdateTransactionStatusCommand; +import com.yape.challenge.transaction.application.handler.CommandHandler; +import com.yape.challenge.transaction.domain.entity.Transaction; +import com.yape.challenge.transaction.domain.event.TransactionStatusChangedDomainEvent; +import com.yape.challenge.transaction.infrastructure.eventstore.EventStore; +import com.yape.challenge.transaction.infrastructure.repository.TransactionRepository; +import lombok.RequiredArgsConstructor; +import lombok.extern.slf4j.Slf4j; +import org.springframework.cache.annotation.CacheEvict; +import org.springframework.stereotype.Component; +import org.springframework.transaction.annotation.Transactional; + +import java.time.LocalDateTime; + +/** + * Handler for UpdateTransactionStatusCommand with Event Sourcing + * Implements cache invalidation for consistency + */ +@Component +@RequiredArgsConstructor +@Slf4j +public class UpdateTransactionStatusCommandHandler implements CommandHandler { + + private final EventStore eventStore; + private final TransactionRepository transactionRepository; + + @Override + @Transactional + @CacheEvict(value = "transactions", key = "#command.externalId.toString()") + public Void handle(UpdateTransactionStatusCommand command) { + log.info("Handling UpdateTransactionStatusCommand with Event Sourcing: {}", command); + + // 1. Get current transaction state + Transaction transaction = transactionRepository.findByExternalId(command.getExternalId()) + .orElseThrow(() -> new IllegalArgumentException("Transaction not found: " + command.getExternalId())); + + // 2. Check if status actually changed + if (transaction.getStatus() == command.getStatus()) { + log.info("Transaction status unchanged: {}", command.getStatus()); + return null; + } + + // 3. Create domain event for status change + TransactionStatusChangedDomainEvent domainEvent = TransactionStatusChangedDomainEvent.builder() + .aggregateId(command.getExternalId()) + .oldStatus(transaction.getStatus()) + .newStatus(command.getStatus()) + .reason("Status updated via antifraud validation") + .occurredAt(LocalDateTime.now()) + .build(); + + // 4. Save event to Event Store (persistence) + eventStore.saveEvent(domainEvent); + log.info("Domain event persisted in Event Store for transaction: {} - Status change: {} -> {}", + command.getExternalId(), domainEvent.getOldStatus(), domainEvent.getNewStatus()); + + // 5. Apply event to update aggregate and save read model + transaction.setStatus(command.getStatus()); + transaction.setUpdatedAt(domainEvent.getOccurredAt()); + transactionRepository.save(transaction); + + log.info("Transaction status updated successfully and cache invalidated for externalId: {} - New status: {}", + command.getExternalId(), command.getStatus()); + + return null; + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/query/GetTransactionQueryHandler.java b/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/query/GetTransactionQueryHandler.java new file mode 100644 index 0000000000..956655a59e --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/application/handler/query/GetTransactionQueryHandler.java @@ -0,0 +1,46 @@ +package com.yape.challenge.transaction.application.handler.query; + +import com.yape.challenge.transaction.application.dto.response.TransactionResponse; +import com.yape.challenge.transaction.application.handler.QueryHandler; +import com.yape.challenge.transaction.application.mapper.TransactionMapper; +import com.yape.challenge.transaction.application.query.GetTransactionQuery; +import com.yape.challenge.transaction.domain.entity.Transaction; +import com.yape.challenge.transaction.domain.entity.TransactionType; +import com.yape.challenge.transaction.infrastructure.repository.TransactionRepository; +import com.yape.challenge.transaction.infrastructure.repository.TransactionTypeRepository; +import com.yape.challenge.transaction.presentation.exception.ResourceNotFoundException; +import lombok.RequiredArgsConstructor; +import lombok.extern.slf4j.Slf4j; +import org.springframework.cache.annotation.Cacheable; +import org.springframework.stereotype.Component; +import org.springframework.transaction.annotation.Transactional; + +/** + * Handler for GetTransactionQuery + * Implements caching for high volume read optimization + */ +@Component +@RequiredArgsConstructor +@Slf4j +public class GetTransactionQueryHandler implements QueryHandler { + + private final TransactionRepository transactionRepository; + private final TransactionTypeRepository transactionTypeRepository; + private final TransactionMapper transactionMapper; + + @Override + @Transactional(readOnly = true) + @Cacheable(value = "transactions", key = "#query.externalId.toString()", unless = "#result == null") + public TransactionResponse handle(GetTransactionQuery query) { + log.info("Handling GetTransactionQuery: {} (cache miss)", query); + + Transaction transaction = transactionRepository.findByExternalId(query.getExternalId()) + .orElseThrow(() -> new ResourceNotFoundException("Transaction not found")); + + TransactionType transactionType = transactionTypeRepository.findById(transaction.getTransferTypeId()) + .orElseThrow(() -> new IllegalArgumentException("Transaction type not found")); + + return transactionMapper.toResponse(transaction, transactionType); + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/application/mapper/TransactionMapper.java b/transaction-service/src/main/java/com/yape/challenge/transaction/application/mapper/TransactionMapper.java new file mode 100644 index 0000000000..2f0d1a8357 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/application/mapper/TransactionMapper.java @@ -0,0 +1,54 @@ +package com.yape.challenge.transaction.application.mapper; + +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import com.yape.challenge.transaction.application.dto.request.CreateTransactionRequest; +import com.yape.challenge.transaction.application.dto.response.TransactionResponse; +import com.yape.challenge.transaction.domain.entity.Transaction; +import com.yape.challenge.transaction.domain.entity.TransactionType; +import org.mapstruct.Mapper; +import org.mapstruct.Mapping; +import org.mapstruct.Named; + +@Mapper(componentModel = "spring") +public interface TransactionMapper { + + @Mapping(target = "id", ignore = true) + @Mapping(target = "externalId", ignore = true) + @Mapping(target = "status", ignore = true) + @Mapping(target = "createdAt", ignore = true) + @Mapping(target = "updatedAt", ignore = true) + @Mapping(target = "transferTypeId", source = "tranferTypeId") + Transaction toEntity(CreateTransactionRequest request); + + @Mapping(target = "transactionExternalId", source = "externalId") + @Mapping(target = "status", source = "transaction.status") + TransactionCreatedEvent toCreatedEvent(Transaction transaction); + + @Mapping(target = "transactionExternalId", source = "transaction.externalId") + @Mapping(target = "transactionType", source = "transactionType", qualifiedByName = "mapTransactionType") + @Mapping(target = "transactionStatus", source = "transaction.status", qualifiedByName = "mapTransactionStatus") + @Mapping(target = "value", source = "transaction.value") + @Mapping(target = "createdAt", source = "transaction.createdAt") + TransactionResponse toResponse(Transaction transaction, TransactionType transactionType); + + @Named("mapTransactionType") + default TransactionResponse.TransactionTypeDto mapTransactionType(TransactionType transactionType) { + if (transactionType == null) { + return null; + } + return TransactionResponse.TransactionTypeDto.builder() + .name(transactionType.getName()) + .build(); + } + + @Named("mapTransactionStatus") + default TransactionResponse.TransactionStatusDto mapTransactionStatus(com.yape.challenge.common.dto.TransactionStatus status) { + if (status == null) { + return null; + } + return TransactionResponse.TransactionStatusDto.builder() + .name(status.name()) + .build(); + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/application/query/GetTransactionQuery.java b/transaction-service/src/main/java/com/yape/challenge/transaction/application/query/GetTransactionQuery.java new file mode 100644 index 0000000000..2be6277a6e --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/application/query/GetTransactionQuery.java @@ -0,0 +1,16 @@ +package com.yape.challenge.transaction.application.query; + +import lombok.Builder; +import lombok.Data; + +import java.util.UUID; + +/** + * Query to get a transaction by external ID + */ +@Data +@Builder +public class GetTransactionQuery { + private UUID externalId; +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/domain/entity/DomainEvent.java b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/entity/DomainEvent.java new file mode 100644 index 0000000000..6a155aa13a --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/entity/DomainEvent.java @@ -0,0 +1,59 @@ +package com.yape.challenge.transaction.domain.entity; + +import jakarta.persistence.*; +import lombok.AllArgsConstructor; +import lombok.Builder; +import lombok.Data; +import lombok.NoArgsConstructor; +import org.hibernate.annotations.JdbcTypeCode; +import org.hibernate.type.SqlTypes; + +import java.time.LocalDateTime; +import java.util.UUID; + +/** + * Entity representing a domain event in the Event Store + */ +@Entity +@Table(name = "domain_events", indexes = { + @Index(name = "idx_domain_events_aggregate_id", columnList = "aggregate_id"), + @Index(name = "idx_domain_events_event_type", columnList = "event_type"), + @Index(name = "idx_domain_events_occurred_at", columnList = "occurred_at"), + @Index(name = "idx_domain_events_aggregate_type", columnList = "aggregate_type") +}, uniqueConstraints = { + @UniqueConstraint(name = "uk_aggregate_version", columnNames = {"aggregate_id", "version"}) +}) +@Data +@Builder +@NoArgsConstructor +@AllArgsConstructor +public class DomainEvent { + + @Id + @GeneratedValue(strategy = GenerationType.IDENTITY) + private Long id; + + @Column(name = "aggregate_id", nullable = false) + private UUID aggregateId; + + @Column(name = "aggregate_type", nullable = false, length = 100) + private String aggregateType; + + @Column(name = "event_type", nullable = false, length = 100) + private String eventType; + + @JdbcTypeCode(SqlTypes.JSON) + @Column(name = "event_data", nullable = false, columnDefinition = "jsonb") + private String eventData; + + @JdbcTypeCode(SqlTypes.JSON) + @Column(name = "metadata", columnDefinition = "jsonb") + private String metadata; + + @Column(name = "version", nullable = false) + private Integer version; + + @Column(name = "occurred_at", nullable = false) + private LocalDateTime occurredAt; +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/domain/entity/Transaction.java b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/entity/Transaction.java new file mode 100644 index 0000000000..d3c275d866 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/entity/Transaction.java @@ -0,0 +1,68 @@ +package com.yape.challenge.transaction.domain.entity; + +import com.yape.challenge.common.dto.TransactionStatus; +import jakarta.persistence.*; +import lombok.AllArgsConstructor; +import lombok.Builder; +import lombok.Data; +import lombok.NoArgsConstructor; +import org.hibernate.annotations.CreationTimestamp; +import org.hibernate.annotations.UpdateTimestamp; + +import java.math.BigDecimal; +import java.time.LocalDateTime; +import java.util.UUID; + +@Entity +@Table(name = "transactions", indexes = { + @Index(name = "idx_external_id", columnList = "external_id", unique = true), + @Index(name = "idx_status_created", columnList = "status, created_at") +}) +@Data +@Builder +@NoArgsConstructor +@AllArgsConstructor +public class Transaction { + + @Id + @GeneratedValue(strategy = GenerationType.IDENTITY) + private Long id; + + @Column(name = "external_id", nullable = false, unique = true, updatable = false) + private UUID externalId; + + @Column(name = "account_external_id_debit", nullable = false) + private UUID accountExternalIdDebit; + + @Column(name = "account_external_id_credit", nullable = false) + private UUID accountExternalIdCredit; + + @Column(name = "transfer_type_id", nullable = false) + private Integer transferTypeId; + + @Column(name = "\"value\"", nullable = false, precision = 19, scale = 2) + private BigDecimal value; + + @Enumerated(EnumType.STRING) + @Column(nullable = false, length = 20) + private TransactionStatus status; + + @CreationTimestamp + @Column(name = "created_at", nullable = false, updatable = false) + private LocalDateTime createdAt; + + @UpdateTimestamp + @Column(name = "updated_at", nullable = false) + private LocalDateTime updatedAt; + + @PrePersist + protected void onCreate() { + if (externalId == null) { + externalId = UUID.randomUUID(); + } + if (status == null) { + status = TransactionStatus.PENDING; + } + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/domain/entity/TransactionType.java b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/entity/TransactionType.java new file mode 100644 index 0000000000..39d60d421d --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/entity/TransactionType.java @@ -0,0 +1,27 @@ +package com.yape.challenge.transaction.domain.entity; + +import jakarta.persistence.*; +import lombok.AllArgsConstructor; +import lombok.Builder; +import lombok.Data; +import lombok.NoArgsConstructor; + +@Entity +@Table(name = "transaction_types") +@Data +@Builder +@NoArgsConstructor +@AllArgsConstructor +public class TransactionType { + + @Id + @GeneratedValue(strategy = GenerationType.IDENTITY) + private Integer id; + + @Column(nullable = false, unique = true, length = 50) + private String name; + + @Column(length = 255) + private String description; +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/domain/event/TransactionCreatedDomainEvent.java b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/event/TransactionCreatedDomainEvent.java new file mode 100644 index 0000000000..06e9ac5bd5 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/event/TransactionCreatedDomainEvent.java @@ -0,0 +1,48 @@ +package com.yape.challenge.transaction.domain.event; + +import com.fasterxml.jackson.annotation.JsonCreator; +import com.fasterxml.jackson.annotation.JsonIgnoreProperties; +import com.fasterxml.jackson.annotation.JsonProperty; +import lombok.Builder; +import lombok.Data; + +import java.math.BigDecimal; +import java.time.LocalDateTime; +import java.util.UUID; + +/** + * Domain event fired when a transaction is created + */ +@Data +@Builder +@JsonIgnoreProperties(ignoreUnknown = true) +public class TransactionCreatedDomainEvent implements TransactionDomainEvent { + private final UUID aggregateId; + private final UUID accountExternalIdDebit; + private final UUID accountExternalIdCredit; + private final Integer transferTypeId; + private final BigDecimal value; + private final LocalDateTime occurredAt; + + @JsonCreator + public TransactionCreatedDomainEvent( + @JsonProperty("aggregateId") UUID aggregateId, + @JsonProperty("accountExternalIdDebit") UUID accountExternalIdDebit, + @JsonProperty("accountExternalIdCredit") UUID accountExternalIdCredit, + @JsonProperty("transferTypeId") Integer transferTypeId, + @JsonProperty("value") BigDecimal value, + @JsonProperty("occurredAt") LocalDateTime occurredAt) { + this.aggregateId = aggregateId; + this.accountExternalIdDebit = accountExternalIdDebit; + this.accountExternalIdCredit = accountExternalIdCredit; + this.transferTypeId = transferTypeId; + this.value = value; + this.occurredAt = occurredAt; + } + + @Override + public String getEventType() { + return "TransactionCreatedDomainEvent"; + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/domain/event/TransactionDomainEvent.java b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/event/TransactionDomainEvent.java new file mode 100644 index 0000000000..e9e99d4cb1 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/event/TransactionDomainEvent.java @@ -0,0 +1,15 @@ +package com.yape.challenge.transaction.domain.event; + +import java.time.LocalDateTime; +import java.util.UUID; + +/** + * Base interface for all transaction domain events + */ +public interface TransactionDomainEvent { + UUID getAggregateId(); + LocalDateTime getOccurredAt(); + String getEventType(); +} + + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/domain/event/TransactionStatusChangedDomainEvent.java b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/event/TransactionStatusChangedDomainEvent.java new file mode 100644 index 0000000000..7002a08055 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/event/TransactionStatusChangedDomainEvent.java @@ -0,0 +1,45 @@ +package com.yape.challenge.transaction.domain.event; + +import com.fasterxml.jackson.annotation.JsonCreator; +import com.fasterxml.jackson.annotation.JsonIgnoreProperties; +import com.fasterxml.jackson.annotation.JsonProperty; +import com.yape.challenge.common.dto.TransactionStatus; +import lombok.Builder; +import lombok.Data; + +import java.time.LocalDateTime; +import java.util.UUID; + +/** + * Domain event fired when a transaction status changes + */ +@Data +@Builder +@JsonIgnoreProperties(ignoreUnknown = true) +public class TransactionStatusChangedDomainEvent implements TransactionDomainEvent { + private final UUID aggregateId; + private final TransactionStatus oldStatus; + private final TransactionStatus newStatus; + private final String reason; + private final LocalDateTime occurredAt; + + @JsonCreator + public TransactionStatusChangedDomainEvent( + @JsonProperty("aggregateId") UUID aggregateId, + @JsonProperty("oldStatus") TransactionStatus oldStatus, + @JsonProperty("newStatus") TransactionStatus newStatus, + @JsonProperty("reason") String reason, + @JsonProperty("occurredAt") LocalDateTime occurredAt) { + this.aggregateId = aggregateId; + this.oldStatus = oldStatus; + this.newStatus = newStatus; + this.reason = reason; + this.occurredAt = occurredAt; + } + + @Override + public String getEventType() { + return "TransactionStatusChangedDomainEvent"; + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/domain/service/TransactionAggregateService.java b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/service/TransactionAggregateService.java new file mode 100644 index 0000000000..b476993cf4 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/domain/service/TransactionAggregateService.java @@ -0,0 +1,90 @@ +package com.yape.challenge.transaction.domain.service; + +import com.yape.challenge.common.dto.TransactionStatus; +import com.yape.challenge.transaction.domain.entity.Transaction; +import com.yape.challenge.transaction.domain.event.TransactionCreatedDomainEvent; +import com.yape.challenge.transaction.domain.event.TransactionDomainEvent; +import com.yape.challenge.transaction.domain.event.TransactionStatusChangedDomainEvent; +import com.yape.challenge.transaction.infrastructure.eventstore.EventStore; +import lombok.RequiredArgsConstructor; +import lombok.extern.slf4j.Slf4j; +import org.springframework.stereotype.Service; + +import java.util.List; +import java.util.UUID; + +/** + * Service to rebuild Transaction aggregates from domain events + */ +@Service +@RequiredArgsConstructor +@Slf4j +public class TransactionAggregateService { + + private final EventStore eventStore; + + /** + * Rebuild a transaction aggregate from its event history + */ + public Transaction rebuildFromEvents(UUID transactionId) { + log.info("Rebuilding transaction aggregate from events: {}", transactionId); + + List events = eventStore.getEvents(transactionId); + + if (events.isEmpty()) { + throw new IllegalArgumentException("No events found for transaction: " + transactionId); + } + + Transaction transaction = new Transaction(); + transaction.setExternalId(transactionId); + + // Apply each event in order + for (TransactionDomainEvent event : events) { + applyEvent(transaction, event); + } + + log.info("Transaction aggregate rebuilt successfully: {}", transactionId); + return transaction; + } + + /** + * Apply a single event to the transaction aggregate + */ + public void applyEvent(Transaction transaction, TransactionDomainEvent event) { + switch (event) { + case TransactionCreatedDomainEvent created -> { + transaction.setAccountExternalIdDebit(created.getAccountExternalIdDebit()); + transaction.setAccountExternalIdCredit(created.getAccountExternalIdCredit()); + transaction.setTransferTypeId(created.getTransferTypeId()); + transaction.setValue(created.getValue()); + transaction.setStatus(TransactionStatus.PENDING); + transaction.setCreatedAt(created.getOccurredAt()); + transaction.setUpdatedAt(created.getOccurredAt()); + log.debug("Applied TransactionCreatedDomainEvent to aggregate: {}", transaction.getExternalId()); + } + case TransactionStatusChangedDomainEvent statusChanged -> { + transaction.setStatus(statusChanged.getNewStatus()); + transaction.setUpdatedAt(statusChanged.getOccurredAt()); + log.debug("Applied TransactionStatusChangedDomainEvent to aggregate: {} - New status: {}", + transaction.getExternalId(), statusChanged.getNewStatus()); + } + default -> + log.warn("Unknown event type: {}", event.getClass().getName()); + } + } + + /** + * Check if a transaction exists in the event store + */ + public boolean transactionExists(UUID transactionId) { + return eventStore.aggregateExists(transactionId); + } + + /** + * Get the number of events for a transaction + */ + public long getEventCount(UUID transactionId) { + return eventStore.getEventCount(transactionId); + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/config/CacheConfig.java b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/config/CacheConfig.java new file mode 100644 index 0000000000..cee5bca385 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/config/CacheConfig.java @@ -0,0 +1,83 @@ +package com.yape.challenge.transaction.infrastructure.config; + +import com.fasterxml.jackson.databind.ObjectMapper; +import com.fasterxml.jackson.databind.SerializationFeature; +import com.fasterxml.jackson.datatype.jsr310.JavaTimeModule; +import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty; +import org.springframework.cache.CacheManager; +import org.springframework.cache.annotation.EnableCaching; +import org.springframework.context.annotation.Bean; +import org.springframework.context.annotation.Configuration; +import org.springframework.data.redis.cache.RedisCacheConfiguration; +import org.springframework.data.redis.cache.RedisCacheManager; +import org.springframework.data.redis.connection.RedisConnectionFactory; +import org.springframework.data.redis.serializer.GenericJackson2JsonRedisSerializer; +import org.springframework.data.redis.serializer.RedisSerializationContext; +import org.springframework.data.redis.serializer.StringRedisSerializer; + +import java.time.Duration; + +/** + * Redis cache configuration for high volume read optimization + * Implements distributed caching to reduce database load + */ +@Configuration +@EnableCaching +@ConditionalOnProperty(name = "spring.cache.type", havingValue = "redis", matchIfMissing = true) +public class CacheConfig { + + /** + * Configure ObjectMapper for Redis serialization with Java Time support + * Enables default typing to preserve type information during serialization/deserialization + */ + @Bean + public ObjectMapper redisCacheObjectMapper() { + ObjectMapper objectMapper = new ObjectMapper(); + objectMapper.registerModule(new JavaTimeModule()); + objectMapper.disable(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS); + objectMapper.findAndRegisterModules(); + // Enable default typing to avoid ClassCastException when deserializing from Redis + objectMapper.activateDefaultTyping( + objectMapper.getPolymorphicTypeValidator(), + ObjectMapper.DefaultTyping.NON_FINAL + ); + return objectMapper; + } + + /** + * Configure Redis Cache Manager with custom TTLs for different cache regions + */ + @Bean + public CacheManager cacheManager(RedisConnectionFactory connectionFactory) { + // Default cache configuration + RedisCacheConfiguration defaultConfig = RedisCacheConfiguration.defaultCacheConfig() + .entryTtl(Duration.ofMinutes(5)) + .serializeKeysWith( + RedisSerializationContext.SerializationPair.fromSerializer( + new StringRedisSerializer() + ) + ) + .serializeValuesWith( + RedisSerializationContext.SerializationPair.fromSerializer( + new GenericJackson2JsonRedisSerializer(redisCacheObjectMapper()) + ) + ) + .disableCachingNullValues(); + + // Build cache manager with specific cache configurations + return RedisCacheManager.builder(connectionFactory) + .cacheDefaults(defaultConfig) + // Transaction cache: 10 minutes (frequent reads) + .withCacheConfiguration("transactions", + defaultConfig.entryTtl(Duration.ofMinutes(10))) + // Transaction types: 1 hour (rarely changes) + .withCacheConfiguration("transactionTypes", + defaultConfig.entryTtl(Duration.ofHours(1))) + // Transaction list: 2 minutes (changes frequently) + .withCacheConfiguration("transactionList", + defaultConfig.entryTtl(Duration.ofMinutes(2))) + .transactionAware() + .build(); + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/config/ObjectMapperConfig.java b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/config/ObjectMapperConfig.java new file mode 100644 index 0000000000..5fe95b6dac --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/config/ObjectMapperConfig.java @@ -0,0 +1,30 @@ +package com.yape.challenge.transaction.infrastructure.config; + +import com.fasterxml.jackson.databind.ObjectMapper; +import com.fasterxml.jackson.databind.SerializationFeature; +import com.fasterxml.jackson.datatype.jsr310.JavaTimeModule; +import org.springframework.context.annotation.Bean; +import org.springframework.context.annotation.Configuration; +import org.springframework.context.annotation.Primary; + +/** + * Configuration for ObjectMapper with Java Time support + */ +@Configuration +public class ObjectMapperConfig { + + @Bean + @Primary + public ObjectMapper objectMapper() { + ObjectMapper objectMapper = new ObjectMapper(); + + // Register JavaTimeModule for LocalDateTime, LocalDate, etc. + objectMapper.registerModule(new JavaTimeModule()); + + // Disable writing dates as timestamps + objectMapper.disable(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS); + + return objectMapper; + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/config/Resilience4jConfig.java b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/config/Resilience4jConfig.java new file mode 100644 index 0000000000..7f143a3d5b --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/config/Resilience4jConfig.java @@ -0,0 +1,56 @@ +package com.yape.challenge.transaction.infrastructure.config; + +import io.github.resilience4j.circuitbreaker.CircuitBreakerRegistry; +import io.github.resilience4j.core.registry.EntryAddedEvent; +import io.github.resilience4j.core.registry.EntryRemovedEvent; +import io.github.resilience4j.core.registry.EntryReplacedEvent; +import lombok.extern.slf4j.Slf4j; +import org.springframework.context.annotation.Bean; +import org.springframework.context.annotation.Configuration; + +/** + * Configuration for Resilience4j Circuit Breaker + * Provides custom logging for Circuit Breaker events + */ +@Configuration +@Slf4j +public class Resilience4jConfig { + + @Bean + public CircuitBreakerRegistry circuitBreakerRegistry() { + CircuitBreakerRegistry registry = CircuitBreakerRegistry.ofDefaults(); + + // Register event listeners for Circuit Breaker lifecycle + registry.getEventPublisher() + .onEntryAdded(this::onCircuitBreakerAdded) + .onEntryRemoved(this::onCircuitBreakerRemoved) + .onEntryReplaced(this::onCircuitBreakerReplaced); + + return registry; + } + + private void onCircuitBreakerAdded(EntryAddedEvent event) { + log.info("Circuit Breaker '{}' added", event.getAddedEntry().getName()); + + // Register state transition listener + event.getAddedEntry().getEventPublisher() + .onStateTransition(e -> log.warn("Circuit Breaker '{}' state changed: {} -> {}", + event.getAddedEntry().getName(), + e.getStateTransition().getFromState(), + e.getStateTransition().getToState())) + .onError(e -> log.error("Circuit Breaker '{}' recorded error: {}", + event.getAddedEntry().getName(), + e.getThrowable().getMessage())) + .onSuccess(e -> log.debug("Circuit Breaker '{}' recorded success", + event.getAddedEntry().getName())); + } + + private void onCircuitBreakerRemoved(EntryRemovedEvent event) { + log.info("Circuit Breaker '{}' removed", event.getRemovedEntry().getName()); + } + + private void onCircuitBreakerReplaced(EntryReplacedEvent event) { + log.info("Circuit Breaker '{}' replaced", event.getNewEntry().getName()); + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/eventstore/EventStore.java b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/eventstore/EventStore.java new file mode 100644 index 0000000000..1b86906aeb --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/eventstore/EventStore.java @@ -0,0 +1,174 @@ +package com.yape.challenge.transaction.infrastructure.eventstore; + +import com.fasterxml.jackson.core.JsonProcessingException; +import com.fasterxml.jackson.databind.ObjectMapper; +import com.yape.challenge.transaction.domain.entity.DomainEvent; +import com.yape.challenge.transaction.domain.event.TransactionCreatedDomainEvent; +import com.yape.challenge.transaction.domain.event.TransactionDomainEvent; +import com.yape.challenge.transaction.domain.event.TransactionStatusChangedDomainEvent; +import com.yape.challenge.transaction.infrastructure.repository.DomainEventRepository; +import io.github.resilience4j.circuitbreaker.annotation.CircuitBreaker; +import lombok.RequiredArgsConstructor; +import lombok.extern.slf4j.Slf4j; +import org.springframework.stereotype.Component; +import org.springframework.transaction.annotation.Transactional; + +import java.time.LocalDateTime; +import java.util.HashMap; +import java.util.List; +import java.util.Map; +import java.util.UUID; +import java.util.stream.Collectors; + +/** + * Event Store implementation for persisting and retrieving domain events + */ +@Component +@RequiredArgsConstructor +@Slf4j +public class EventStore { + + private final DomainEventRepository domainEventRepository; + private final ObjectMapper objectMapper; + + private static final String AGGREGATE_TYPE = "Transaction"; + + /** + * Save a domain event to the event store + */ + @Transactional + @CircuitBreaker(name = "database", fallbackMethod = "saveEventFallback") + public void saveEvent(TransactionDomainEvent event) { + try { + UUID aggregateId = event.getAggregateId(); + + // Get the last version for this aggregate + Integer lastVersion = domainEventRepository + .findLastVersionByAggregateId(aggregateId) + .orElse(0); + + // Serialize event data to JSON + String eventData = objectMapper.writeValueAsString(event); + + // Create metadata + Map metadata = new HashMap<>(); + metadata.put("timestamp", LocalDateTime.now()); + metadata.put("eventClass", event.getClass().getName()); + String metadataJson = objectMapper.writeValueAsString(metadata); + + // Create and save domain event + DomainEvent domainEvent = DomainEvent.builder() + .aggregateId(aggregateId) + .aggregateType(AGGREGATE_TYPE) + .eventType(event.getEventType()) + .eventData(eventData) + .metadata(metadataJson) + .version(lastVersion + 1) + .occurredAt(event.getOccurredAt()) + .build(); + + domainEventRepository.save(domainEvent); + log.info("Event saved: {} for aggregate: {}, version: {}", + event.getEventType(), aggregateId, lastVersion + 1); + + } catch (JsonProcessingException e) { + log.error("Error serializing event: {}", event, e); + throw new RuntimeException("Failed to serialize event", e); + } + } + + /** + * Fallback method for saveEvent when database is not available + */ + private void saveEventFallback(TransactionDomainEvent event, Exception e) { + log.error("Database circuit breaker is OPEN or error occurred. Event: {}, Error: {}", + event.getAggregateId(), e.getMessage()); + throw new RuntimeException("Database service is currently unavailable. Please try again later.", e); + } + + /** + * Get all events for a specific aggregate + */ + @Transactional(readOnly = true) + @CircuitBreaker(name = "database", fallbackMethod = "getEventsFallback") + public List getEvents(UUID aggregateId) { + List domainEvents = domainEventRepository + .findByAggregateIdOrderByVersionAsc(aggregateId); + + return domainEvents.stream() + .map(this::deserializeEvent) + .toList(); + } + + /** + * Fallback method for getEvents when database is not available + */ + private List getEventsFallback(UUID aggregateId, Exception e) { + log.error("Database circuit breaker is OPEN or error occurred. Aggregate: {}, Error: {}", + aggregateId, e.getMessage()); + throw new RuntimeException("Database service is currently unavailable. Please try again later.", e); + } + + /** + * Check if an aggregate exists in the event store + */ + @Transactional(readOnly = true) + public boolean aggregateExists(UUID aggregateId) { + return domainEventRepository.existsByAggregateId(aggregateId); + } + + /** + * Get event count for an aggregate + */ + @Transactional(readOnly = true) + public long getEventCount(UUID aggregateId) { + return domainEventRepository.countByAggregateId(aggregateId); + } + + /** + * Deserialize a domain event from JSON + */ + private TransactionDomainEvent deserializeEvent(DomainEvent domainEvent) { + try { + String eventType = domainEvent.getEventType(); + String eventData = domainEvent.getEventData(); + + return switch (eventType) { + case "TransactionCreatedDomainEvent" -> + objectMapper.readValue(eventData, TransactionCreatedDomainEvent.class); + case "TransactionStatusChangedDomainEvent" -> + objectMapper.readValue(eventData, TransactionStatusChangedDomainEvent.class); + default -> { + log.error("Unknown event type: {}", eventType); + throw new IllegalArgumentException("Unknown event type: " + eventType); + } + }; + } catch (JsonProcessingException e) { + log.error("Error deserializing event: {}", domainEvent, e); + throw new RuntimeException("Failed to deserialize event", e); + } + } + + /** + * Get all events by type + */ + @Transactional(readOnly = true) + public List getEventsByType(String eventType) { + return domainEventRepository.findByEventTypeOrderByOccurredAtDesc(eventType) + .stream() + .map(this::deserializeEvent) + .toList(); + } + + /** + * Get all events for the aggregate type + */ + @Transactional(readOnly = true) + public List getAllTransactionEvents() { + return domainEventRepository.findByAggregateTypeOrderByOccurredAtDesc(AGGREGATE_TYPE) + .stream() + .map(this::deserializeEvent) + .toList(); + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/config/KafkaConsumerConfig.java b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/config/KafkaConsumerConfig.java new file mode 100644 index 0000000000..818861f717 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/config/KafkaConsumerConfig.java @@ -0,0 +1,50 @@ +package com.yape.challenge.transaction.infrastructure.kafka.config; + +import com.yape.challenge.common.dto.TransactionStatusEvent; +import org.apache.kafka.clients.consumer.ConsumerConfig; +import org.apache.kafka.common.serialization.StringDeserializer; +import org.springframework.beans.factory.annotation.Value; +import org.springframework.context.annotation.Bean; +import org.springframework.context.annotation.Configuration; +import org.springframework.kafka.annotation.EnableKafka; +import org.springframework.kafka.config.ConcurrentKafkaListenerContainerFactory; +import org.springframework.kafka.core.ConsumerFactory; +import org.springframework.kafka.core.DefaultKafkaConsumerFactory; +import org.springframework.kafka.support.serializer.JsonDeserializer; + +import java.util.HashMap; +import java.util.Map; + +@EnableKafka +@Configuration +public class KafkaConsumerConfig { + + @Value("${spring.kafka.bootstrap-servers}") + private String bootstrapServers; + + @Value("${spring.kafka.consumer.group-id}") + private String groupId; + + @Bean + public ConsumerFactory consumerFactory() { + Map props = new HashMap<>(); + props.put(ConsumerConfig.BOOTSTRAP_SERVERS_CONFIG, bootstrapServers); + props.put(ConsumerConfig.GROUP_ID_CONFIG, groupId); + props.put(ConsumerConfig.KEY_DESERIALIZER_CLASS_CONFIG, StringDeserializer.class); + props.put(ConsumerConfig.VALUE_DESERIALIZER_CLASS_CONFIG, JsonDeserializer.class); + props.put(ConsumerConfig.AUTO_OFFSET_RESET_CONFIG, "earliest"); + props.put(JsonDeserializer.TRUSTED_PACKAGES, "*"); + props.put(JsonDeserializer.VALUE_DEFAULT_TYPE, TransactionStatusEvent.class.getName()); + return new DefaultKafkaConsumerFactory<>(props); + } + + @Bean + public ConcurrentKafkaListenerContainerFactory kafkaListenerContainerFactory() { + ConcurrentKafkaListenerContainerFactory factory = + new ConcurrentKafkaListenerContainerFactory<>(); + factory.setConsumerFactory(consumerFactory()); + factory.setConcurrency(3); + return factory; + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/config/KafkaProducerConfig.java b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/config/KafkaProducerConfig.java new file mode 100644 index 0000000000..21a2bbb7b5 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/config/KafkaProducerConfig.java @@ -0,0 +1,41 @@ +package com.yape.challenge.transaction.infrastructure.kafka.config; + +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import org.apache.kafka.clients.producer.ProducerConfig; +import org.apache.kafka.common.serialization.StringSerializer; +import org.springframework.beans.factory.annotation.Value; +import org.springframework.context.annotation.Bean; +import org.springframework.context.annotation.Configuration; +import org.springframework.kafka.core.DefaultKafkaProducerFactory; +import org.springframework.kafka.core.KafkaTemplate; +import org.springframework.kafka.core.ProducerFactory; +import org.springframework.kafka.support.serializer.JsonSerializer; + +import java.util.HashMap; +import java.util.Map; + +@Configuration +public class KafkaProducerConfig { + + @Value("${spring.kafka.bootstrap-servers}") + private String bootstrapServers; + + @Bean + public ProducerFactory producerFactory() { + Map configProps = new HashMap<>(); + configProps.put(ProducerConfig.BOOTSTRAP_SERVERS_CONFIG, bootstrapServers); + configProps.put(ProducerConfig.KEY_SERIALIZER_CLASS_CONFIG, StringSerializer.class); + configProps.put(ProducerConfig.VALUE_SERIALIZER_CLASS_CONFIG, JsonSerializer.class); + configProps.put(ProducerConfig.ACKS_CONFIG, "all"); + configProps.put(ProducerConfig.RETRIES_CONFIG, 3); + configProps.put(ProducerConfig.ENABLE_IDEMPOTENCE_CONFIG, true); + configProps.put(JsonSerializer.ADD_TYPE_INFO_HEADERS, false); + return new DefaultKafkaProducerFactory<>(configProps); + } + + @Bean + public KafkaTemplate kafkaTemplate() { + return new KafkaTemplate<>(producerFactory()); + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/config/KafkaTopicConfig.java b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/config/KafkaTopicConfig.java new file mode 100644 index 0000000000..7743a7fb3f --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/config/KafkaTopicConfig.java @@ -0,0 +1,44 @@ +package com.yape.challenge.transaction.infrastructure.kafka.config; + +import com.yape.challenge.common.kafka.KafkaTopics; +import org.apache.kafka.clients.admin.AdminClientConfig; +import org.apache.kafka.clients.admin.NewTopic; +import org.springframework.beans.factory.annotation.Value; +import org.springframework.context.annotation.Bean; +import org.springframework.context.annotation.Configuration; +import org.springframework.kafka.config.TopicBuilder; +import org.springframework.kafka.core.KafkaAdmin; + +import java.util.HashMap; +import java.util.Map; + +@Configuration +public class KafkaTopicConfig { + + @Value("${spring.kafka.bootstrap-servers}") + private String bootstrapServers; + + @Bean + public KafkaAdmin kafkaAdmin() { + Map configs = new HashMap<>(); + configs.put(AdminClientConfig.BOOTSTRAP_SERVERS_CONFIG, bootstrapServers); + return new KafkaAdmin(configs); + } + + @Bean + public NewTopic transactionCreatedTopic() { + return TopicBuilder.name(KafkaTopics.TRANSACTION_CREATED) + .partitions(3) + .replicas(1) + .build(); + } + + @Bean + public NewTopic transactionStatusUpdatedTopic() { + return TopicBuilder.name(KafkaTopics.TRANSACTION_STATUS_UPDATED) + .partitions(3) + .replicas(1) + .build(); + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/consumer/TransactionStatusConsumer.java b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/consumer/TransactionStatusConsumer.java new file mode 100644 index 0000000000..f87afd9c39 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/consumer/TransactionStatusConsumer.java @@ -0,0 +1,62 @@ +package com.yape.challenge.transaction.infrastructure.kafka.consumer; + +import com.yape.challenge.common.dto.TransactionStatusEvent; +import com.yape.challenge.transaction.application.bus.CommandBus; +import com.yape.challenge.transaction.application.command.UpdateTransactionStatusCommand; +import com.yape.challenge.common.kafka.KafkaTopics; +import io.github.resilience4j.circuitbreaker.annotation.CircuitBreaker; +import io.github.resilience4j.retry.annotation.Retry; +import lombok.RequiredArgsConstructor; +import lombok.extern.slf4j.Slf4j; +import org.springframework.kafka.annotation.KafkaListener; +import org.springframework.stereotype.Component; + +@Component +@RequiredArgsConstructor +@Slf4j +public class TransactionStatusConsumer { + + private final CommandBus commandBus; + + @KafkaListener(topics = KafkaTopics.TRANSACTION_STATUS_UPDATED, groupId = "${spring.kafka.consumer.group-id}") + @CircuitBreaker(name = "database", fallbackMethod = "consumeTransactionStatusFallback") + @Retry(name = "kafkaProducer") + public void consumeTransactionStatus(TransactionStatusEvent event) { + log.info("Received transaction status event: {}", event); + + try { + // Create command to update transaction status + UpdateTransactionStatusCommand command = UpdateTransactionStatusCommand.builder() + .externalId(event.getTransactionExternalId()) + .status(event.getStatus()) + .build(); + + // Dispatch command through command bus + commandBus.dispatch(command); + + log.info("Transaction status updated successfully for externalId: {}", + event.getTransactionExternalId()); + } catch (Exception e) { + log.error("Error updating transaction status for externalId: {}", + event.getTransactionExternalId(), e); + throw e; + } + } + + /** + * Fallback method when Circuit Breaker is open for database operations + */ + private void consumeTransactionStatusFallback(TransactionStatusEvent event, Exception ex) { + log.error("Circuit Breaker OPEN or all retries failed for transaction status update. " + + "TransactionExternalId: {}, Status: {}. Reason: {}", + event.getTransactionExternalId(), event.getStatus(), ex.getMessage()); + + // TODO: Implement fallback strategy: + // 1. Save to dead letter topic + // 2. Store in a retry queue + // 3. Send alert for manual intervention + + // Don't throw exception to avoid message reprocessing loop + log.warn("Message will be acknowledged to avoid reprocessing loop"); + } +} diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/exception/KafkaProducerException.java b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/exception/KafkaProducerException.java new file mode 100644 index 0000000000..05f92bf7ee --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/exception/KafkaProducerException.java @@ -0,0 +1,16 @@ +package com.yape.challenge.transaction.infrastructure.kafka.exception; + +/** + * Custom exception for Kafka producer errors + */ +public class KafkaProducerException extends RuntimeException { + + public KafkaProducerException(String message) { + super(message); + } + + public KafkaProducerException(String message, Throwable cause) { + super(message, cause); + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/producer/KafkaProducerService.java b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/producer/KafkaProducerService.java new file mode 100644 index 0000000000..5d28309495 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/kafka/producer/KafkaProducerService.java @@ -0,0 +1,90 @@ +package com.yape.challenge.transaction.infrastructure.kafka.producer; + +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import com.yape.challenge.transaction.infrastructure.kafka.exception.KafkaProducerException; +import io.github.resilience4j.circuitbreaker.annotation.CircuitBreaker; +import io.github.resilience4j.retry.annotation.Retry; +import lombok.RequiredArgsConstructor; +import lombok.extern.slf4j.Slf4j; +import org.springframework.kafka.core.KafkaTemplate; +import org.springframework.kafka.support.SendResult; +import org.springframework.stereotype.Service; + +import java.util.concurrent.CompletableFuture; +import java.util.concurrent.TimeUnit; + +/** + * Service for sending messages to Kafka with Circuit Breaker pattern + */ +@Service +@RequiredArgsConstructor +@Slf4j +public class KafkaProducerService { + + private final KafkaTemplate kafkaTemplate; + + /** + * Send transaction created event to Kafka with Circuit Breaker and Retry + * + * @param topic Topic name + * @param key Message key + * @param event Event to send + */ + @CircuitBreaker(name = "kafkaProducer", fallbackMethod = "sendEventFallback") + @Retry(name = "kafkaProducer") + public void sendTransactionCreatedEvent(String topic, String key, TransactionCreatedEvent event) { + log.info("Sending event to Kafka topic '{}' with key '{}': {}", topic, key, event); + + try { + CompletableFuture> future = + kafkaTemplate.send(topic, key, event); + + // Wait for the result with timeout to ensure Circuit Breaker catches exceptions + // Timeout of 5 seconds to fail fast if Kafka is unavailable + SendResult result = future.get(5, TimeUnit.SECONDS); + + if (result != null && result.getRecordMetadata() != null) { + log.info("Event sent successfully to topic '{}', partition: {}, offset: {}", + topic, + result.getRecordMetadata().partition(), + result.getRecordMetadata().offset()); + } else { + log.info("Event sent successfully to topic '{}', but no record metadata was returned", topic); + } + + } catch (InterruptedException e) { + Thread.currentThread().interrupt(); // Restore interrupt status + log.error("Thread interrupted while sending event to Kafka topic '{}': {}", topic, e.getMessage()); + throw new KafkaProducerException("Thread interrupted while sending event to Kafka", e); + } catch (Exception e) { + log.error("Failed to send event to Kafka topic '{}': {}", topic, e.getMessage()); + throw new KafkaProducerException("Error sending event to Kafka", e); + } + } + + /** + * Fallback method when Circuit Breaker is open or all retries fail + * This method logs the error and prevents cascading failures. + * In production, this could: + * - Save to a dead letter queue + * - Save to database for later retry + * - Send notification to monitoring system + */ + @SuppressWarnings("unused") // Used by Circuit Breaker via reflection + private void sendEventFallback(String topic, String key, TransactionCreatedEvent event, Exception ex) { + log.error("Circuit Breaker OPEN or all retries failed for Kafka producer. " + + "Topic: {}, Key: {}, Event: {}. Reason: {}", + topic, key, event, ex.getMessage()); + + // Strategy: Log and allow the application to continue + // The transaction is already created in the database (Event Sourcing) + // The antifraud service will not receive the notification immediately, + // but the system remains available for other operations + + log.warn("Transaction {} created but notification to antifraud service failed. " + + "Manual intervention or retry mechanism may be required.", event.getTransactionExternalId()); + + throw new KafkaProducerException("Kafka service is temporarily unavailable. Transaction created but notification failed.", ex); + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/repository/DomainEventRepository.java b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/repository/DomainEventRepository.java new file mode 100644 index 0000000000..ec731fae2b --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/repository/DomainEventRepository.java @@ -0,0 +1,50 @@ +package com.yape.challenge.transaction.infrastructure.repository; + +import com.yape.challenge.transaction.domain.entity.DomainEvent; +import org.springframework.data.jpa.repository.JpaRepository; +import org.springframework.data.jpa.repository.Query; +import org.springframework.data.repository.query.Param; +import org.springframework.stereotype.Repository; + +import java.util.List; +import java.util.Optional; +import java.util.UUID; + +/** + * Repository for DomainEvent persistence + */ +@Repository +public interface DomainEventRepository extends JpaRepository { + + /** + * Find all events for a specific aggregate, ordered by version + */ + List findByAggregateIdOrderByVersionAsc(UUID aggregateId); + + /** + * Find the last version number for an aggregate + */ + @Query("SELECT MAX(de.version) FROM DomainEvent de WHERE de.aggregateId = :aggregateId") + Optional findLastVersionByAggregateId(@Param("aggregateId") UUID aggregateId); + + /** + * Check if an aggregate has any events + */ + boolean existsByAggregateId(UUID aggregateId); + + /** + * Find events by aggregate type + */ + List findByAggregateTypeOrderByOccurredAtDesc(String aggregateType); + + /** + * Find events by event type + */ + List findByEventTypeOrderByOccurredAtDesc(String eventType); + + /** + * Count events for an aggregate + */ + long countByAggregateId(UUID aggregateId); +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/repository/TransactionRepository.java b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/repository/TransactionRepository.java new file mode 100644 index 0000000000..2a4f7bebbc --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/repository/TransactionRepository.java @@ -0,0 +1,17 @@ +package com.yape.challenge.transaction.infrastructure.repository; + +import com.yape.challenge.transaction.domain.entity.Transaction; +import org.springframework.data.jpa.repository.JpaRepository; +import org.springframework.stereotype.Repository; + +import java.util.Optional; +import java.util.UUID; + +@Repository +public interface TransactionRepository extends JpaRepository { + + Optional findByExternalId(UUID externalId); + +} + + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/repository/TransactionTypeRepository.java b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/repository/TransactionTypeRepository.java new file mode 100644 index 0000000000..f10eed88ce --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/infrastructure/repository/TransactionTypeRepository.java @@ -0,0 +1,12 @@ +package com.yape.challenge.transaction.infrastructure.repository; + +import com.yape.challenge.transaction.domain.entity.TransactionType; +import org.springframework.data.jpa.repository.JpaRepository; +import org.springframework.stereotype.Repository; + + +@Repository +public interface TransactionTypeRepository extends JpaRepository { + +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/controller/EventStoreController.java b/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/controller/EventStoreController.java new file mode 100644 index 0000000000..0b3d8e8bb4 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/controller/EventStoreController.java @@ -0,0 +1,95 @@ +package com.yape.challenge.transaction.presentation.controller; + +import com.yape.challenge.transaction.domain.event.TransactionDomainEvent; +import com.yape.challenge.transaction.infrastructure.eventstore.EventStore; +import lombok.RequiredArgsConstructor; +import lombok.extern.slf4j.Slf4j; +import org.springframework.http.ResponseEntity; +import org.springframework.web.bind.annotation.*; + +import java.util.List; +import java.util.UUID; + +/** + * REST Controller for Event Store queries (for debugging and auditing) + */ +@RestController +@RequestMapping("/api/v1/events") +@RequiredArgsConstructor +@Slf4j +public class EventStoreController { + + private final EventStore eventStore; + + /** + * Get all events for a specific transaction + * GET /api/v1/events/transaction/{transactionId} + */ + @GetMapping("/transaction/{transactionId}") + public ResponseEntity> getTransactionEvents( + @PathVariable UUID transactionId) { + log.info("Getting events for transaction: {}", transactionId); + + List events = eventStore.getEvents(transactionId); + + if (events.isEmpty()) { + return ResponseEntity.notFound().build(); + } + + return ResponseEntity.ok(events); + } + + /** + * Get event count for a specific transaction + * GET /api/v1/events/transaction/{transactionId}/count + */ + @GetMapping("/transaction/{transactionId}/count") + public ResponseEntity getTransactionEventCount(@PathVariable UUID transactionId) { + log.info("Getting event count for transaction: {}", transactionId); + + long count = eventStore.getEventCount(transactionId); + + return ResponseEntity.ok(count); + } + + /** + * Get all events by event type + * GET /api/v1/events/type/{eventType} + */ + @GetMapping("/type/{eventType}") + public ResponseEntity> getEventsByType( + @PathVariable String eventType) { + log.info("Getting events by type: {}", eventType); + + List events = eventStore.getEventsByType(eventType); + + return ResponseEntity.ok(events); + } + + /** + * Get all transaction events + * GET /api/v1/events/all + */ + @GetMapping("/all") + public ResponseEntity> getAllEvents() { + log.info("Getting all transaction events"); + + List events = eventStore.getAllTransactionEvents(); + + return ResponseEntity.ok(events); + } + + /** + * Check if transaction exists in event store + * GET /api/v1/events/transaction/{transactionId}/exists + */ + @GetMapping("/transaction/{transactionId}/exists") + public ResponseEntity transactionExists(@PathVariable UUID transactionId) { + log.info("Checking if transaction exists in event store: {}", transactionId); + + boolean exists = eventStore.aggregateExists(transactionId); + + return ResponseEntity.ok(exists); + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/controller/TransactionController.java b/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/controller/TransactionController.java new file mode 100644 index 0000000000..a9f85d868e --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/controller/TransactionController.java @@ -0,0 +1,61 @@ +package com.yape.challenge.transaction.presentation.controller; + +import com.yape.challenge.transaction.application.bus.CommandBus; +import com.yape.challenge.transaction.application.bus.QueryBus; +import com.yape.challenge.transaction.application.command.CreateTransactionCommand; +import com.yape.challenge.transaction.application.dto.request.CreateTransactionRequest; +import com.yape.challenge.transaction.application.dto.response.TransactionResponse; +import com.yape.challenge.transaction.application.query.GetTransactionQuery; +import jakarta.validation.Valid; +import lombok.RequiredArgsConstructor; +import lombok.extern.slf4j.Slf4j; +import org.springframework.http.HttpStatus; +import org.springframework.http.ResponseEntity; +import org.springframework.web.bind.annotation.*; + +import java.util.UUID; + +@RestController +@RequestMapping("/api/v1/transactions") +@RequiredArgsConstructor +@Slf4j +public class TransactionController { + + private final CommandBus commandBus; + private final QueryBus queryBus; + + @PostMapping + public ResponseEntity createTransaction( + @Valid @RequestBody CreateTransactionRequest request) { + log.info("POST /api/v1/transactions - Request: {}", request); + + // Create command from request + CreateTransactionCommand command = CreateTransactionCommand.builder() + .accountExternalIdDebit(request.getAccountExternalIdDebit()) + .accountExternalIdCredit(request.getAccountExternalIdCredit()) + .tranferTypeId(request.getTranferTypeId()) + .value(request.getValue()) + .build(); + + // Dispatch command through command bus + TransactionResponse response = commandBus.dispatch(command); + return ResponseEntity.status(HttpStatus.CREATED).body(response); + } + + @GetMapping("/{externalId}") + public ResponseEntity getTransaction( + @PathVariable UUID externalId) { + log.info("GET /api/v1/transactions/{}", externalId); + + // Create query + GetTransactionQuery query = GetTransactionQuery.builder() + .externalId(externalId) + .build(); + + // Dispatch query through query bus + TransactionResponse response = queryBus.dispatch(query); + return ResponseEntity.ok(response); + } +} + + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/exception/ErrorResponse.java b/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/exception/ErrorResponse.java new file mode 100644 index 0000000000..b1d5e6c01e --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/exception/ErrorResponse.java @@ -0,0 +1,23 @@ +package com.yape.challenge.transaction.presentation.exception; + +import lombok.AllArgsConstructor; +import lombok.Builder; +import lombok.Data; +import lombok.NoArgsConstructor; + +import java.time.LocalDateTime; +import java.util.Map; + +@Data +@Builder +@NoArgsConstructor +@AllArgsConstructor +public class ErrorResponse { + + private LocalDateTime timestamp; + private int status; + private String error; + private String message; + private Map validationErrors; +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/exception/GlobalExceptionHandler.java b/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/exception/GlobalExceptionHandler.java new file mode 100644 index 0000000000..6cd27206a6 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/exception/GlobalExceptionHandler.java @@ -0,0 +1,75 @@ +package com.yape.challenge.transaction.presentation.exception; + +import lombok.extern.slf4j.Slf4j; +import org.springframework.http.HttpStatus; +import org.springframework.http.ResponseEntity; +import org.springframework.validation.FieldError; +import org.springframework.web.bind.MethodArgumentNotValidException; +import org.springframework.web.bind.annotation.ExceptionHandler; +import org.springframework.web.bind.annotation.RestControllerAdvice; + +import java.time.LocalDateTime; +import java.util.HashMap; +import java.util.Map; + +@RestControllerAdvice +@Slf4j +public class GlobalExceptionHandler { + + @ExceptionHandler(ResourceNotFoundException.class) + public ResponseEntity handleResourceNotFoundException(ResourceNotFoundException ex) { + log.error("ResourceNotFoundException: {}", ex.getMessage()); + ErrorResponse error = ErrorResponse.builder() + .timestamp(LocalDateTime.now()) + .status(HttpStatus.NOT_FOUND.value()) + .error(HttpStatus.NOT_FOUND.getReasonPhrase()) + .message(ex.getMessage()) + .build(); + return ResponseEntity.status(HttpStatus.NOT_FOUND).body(error); + } + + @ExceptionHandler(IllegalArgumentException.class) + public ResponseEntity handleIllegalArgumentException(IllegalArgumentException ex) { + log.error("IllegalArgumentException: {}", ex.getMessage()); + ErrorResponse error = ErrorResponse.builder() + .timestamp(LocalDateTime.now()) + .status(HttpStatus.BAD_REQUEST.value()) + .error(HttpStatus.BAD_REQUEST.getReasonPhrase()) + .message(ex.getMessage()) + .build(); + return ResponseEntity.status(HttpStatus.BAD_REQUEST).body(error); + } + + @ExceptionHandler(MethodArgumentNotValidException.class) + public ResponseEntity handleValidationException(MethodArgumentNotValidException ex) { + log.error("Validation error: {}", ex.getMessage()); + Map errors = new HashMap<>(); + ex.getBindingResult().getAllErrors().forEach(error -> { + String fieldName = ((FieldError) error).getField(); + String errorMessage = error.getDefaultMessage(); + errors.put(fieldName, errorMessage); + }); + + ErrorResponse error = ErrorResponse.builder() + .timestamp(LocalDateTime.now()) + .status(HttpStatus.BAD_REQUEST.value()) + .error(HttpStatus.BAD_REQUEST.getReasonPhrase()) + .message("Validation failed") + .validationErrors(errors) + .build(); + return ResponseEntity.status(HttpStatus.BAD_REQUEST).body(error); + } + + @ExceptionHandler(Exception.class) + public ResponseEntity handleGenericException(Exception ex) { + log.error("Unexpected error: ", ex); + ErrorResponse error = ErrorResponse.builder() + .timestamp(LocalDateTime.now()) + .status(HttpStatus.INTERNAL_SERVER_ERROR.value()) + .error(HttpStatus.INTERNAL_SERVER_ERROR.getReasonPhrase()) + .message("An unexpected error occurred") + .build(); + return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR).body(error); + } +} + diff --git a/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/exception/ResourceNotFoundException.java b/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/exception/ResourceNotFoundException.java new file mode 100644 index 0000000000..721be85e97 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/challenge/transaction/presentation/exception/ResourceNotFoundException.java @@ -0,0 +1,16 @@ +package com.yape.challenge.transaction.presentation.exception; + +/** + * Exception thrown when a requested resource is not found + */ +public class ResourceNotFoundException extends RuntimeException { + + public ResourceNotFoundException(String message) { + super(message); + } + + public ResourceNotFoundException(String message, Throwable cause) { + super(message, cause); + } +} + diff --git a/transaction-service/src/main/resources/application-docker.yml b/transaction-service/src/main/resources/application-docker.yml new file mode 100644 index 0000000000..353070184a --- /dev/null +++ b/transaction-service/src/main/resources/application-docker.yml @@ -0,0 +1,63 @@ +spring: + application: + name: transaction-service + + datasource: + url: ${SPRING_DATASOURCE_URL:jdbc:postgresql://localhost:5432/yape_transactions} + username: ${SPRING_DATASOURCE_USERNAME:yapeuser} + password: ${SPRING_DATASOURCE_PASSWORD:YapePass2026} + driver-class-name: org.postgresql.Driver + hikari: + maximum-pool-size: 20 + minimum-idle: 5 + connection-timeout: 3000 # Fallar rápido si la BD no está disponible + validation-timeout: 2000 + idle-timeout: 600000 + max-lifetime: 1800000 + connection-test-query: SELECT 1 + + sql: + init: + mode: always + data-locations: classpath:db/data.sql + continue-on-error: false + + jpa: + defer-datasource-initialization: true + hibernate: + ddl-auto: update + show-sql: false + properties: + hibernate: + dialect: org.hibernate.dialect.PostgreSQLDialect + format_sql: true + use_sql_comments: true + + kafka: + bootstrap-servers: ${SPRING_KAFKA_BOOTSTRAP_SERVERS:localhost:9092} + consumer: + group-id: transaction-service-group + auto-offset-reset: earliest + key-deserializer: org.apache.kafka.common.serialization.StringDeserializer + value-deserializer: org.springframework.kafka.support.serializer.JsonDeserializer + producer: + key-serializer: org.apache.kafka.common.serialization.StringSerializer + value-serializer: org.springframework.kafka.support.serializer.JsonSerializer + +server: + port: 8080 + +management: + endpoints: + web: + exposure: + include: health,info,metrics + endpoint: + health: + show-details: always + +logging: + level: + com.yape.challenge: INFO + org.springframework.kafka: WARN + diff --git a/transaction-service/src/main/resources/application.yml b/transaction-service/src/main/resources/application.yml new file mode 100644 index 0000000000..0e5fa365c8 --- /dev/null +++ b/transaction-service/src/main/resources/application.yml @@ -0,0 +1,173 @@ +# Application name +spring: + application: + name: transaction-service + + # Database configuration + datasource: + url: ${SPRING_DATASOURCE_URL:jdbc:postgresql://localhost:5432/yape_transactions} + username: ${SPRING_DATASOURCE_USERNAME:yapeuser} + password: ${SPRING_DATASOURCE_PASSWORD:YapePass2026} + driver-class-name: org.postgresql.Driver + + # HikariCP configuration (optimized for high volume) + hikari: + maximum-pool-size: 50 + minimum-idle: 10 + connection-timeout: 3000 # Fallar rápido si la BD no está disponible + validation-timeout: 2000 + idle-timeout: 300000 + max-lifetime: 1800000 + leak-detection-threshold: 60000 + auto-commit: false + connection-test-query: SELECT 1 + data-source-properties: + cachePrepStmts: true + prepStmtCacheSize: 250 + prepStmtCacheSqlLimit: 2048 + useServerPrepStmts: true + + # Redis configuration (for distributed cache) + data: + redis: + host: ${SPRING_REDIS_HOST:localhost} + port: ${SPRING_REDIS_PORT:6379} + timeout: 2000ms + lettuce: + pool: + max-active: 20 + max-idle: 10 + min-idle: 5 + max-wait: 2000ms + shutdown-timeout: 200ms + + # Cache configuration + cache: + type: redis + redis: + time-to-live: 300000 # 5 minutes + cache-null-values: false + use-key-prefix: true + key-prefix: "yape:txn:" + + # SQL initialization - Ejecuta después de que Hibernate cree las tablas + sql: + init: + mode: always + data-locations: classpath:db/data.sql + continue-on-error: false + + # JPA/Hibernate configuration + jpa: + defer-datasource-initialization: true # Espera a que Hibernate cree las tablas primero + hibernate: + ddl-auto: update + show-sql: false + properties: + hibernate: + dialect: org.hibernate.dialect.PostgreSQLDialect + format_sql: true + use_sql_comments: true + + # Kafka configuration + kafka: + bootstrap-servers: ${SPRING_KAFKA_BOOTSTRAP_SERVERS:localhost:9092} + consumer: + group-id: transaction-service-group + auto-offset-reset: earliest + key-deserializer: org.apache.kafka.common.serialization.StringDeserializer + value-deserializer: org.springframework.kafka.support.serializer.JsonDeserializer + producer: + key-serializer: org.apache.kafka.common.serialization.StringSerializer + value-serializer: org.springframework.kafka.support.serializer.JsonSerializer + properties: + max.block.ms: 5000 # Timeout máximo para bloqueos en send/partitionsFor + request.timeout.ms: 5000 # Timeout de solicitud + delivery.timeout.ms: 10000 # Timeout total de entrega + linger.ms: 0 # No esperar para enviar + retries: 2 # Reintentos a nivel de Kafka + +# Server configuration +server: + port: ${SERVER_PORT:8080} + +# Management/Actuator configuration +management: + endpoints: + web: + exposure: + include: health,info,metrics,circuitbreakers,circuitbreakerevents + endpoint: + health: + show-details: always + health: + circuitbreakers: + enabled: true + +# Resilience4j Circuit Breaker configuration +resilience4j: + circuitbreaker: + configs: + default: + registerHealthIndicator: true + slidingWindowSize: 10 + minimumNumberOfCalls: 5 + permittedNumberOfCallsInHalfOpenState: 3 + automaticTransitionFromOpenToHalfOpenEnabled: true + waitDurationInOpenState: 10s + failureRateThreshold: 50 + eventConsumerBufferSize: 10 + recordExceptions: + - org.springframework.kafka.KafkaException + - java.util.concurrent.TimeoutException + - org.springframework.dao.DataAccessException + - org.springframework.dao.DataAccessResourceFailureException + - org.hibernate.exception.JDBCConnectionException + - java.sql.SQLTransientConnectionException + - java.sql.SQLException + - java.util.concurrent.ExecutionException + - org.apache.kafka.common.errors.TimeoutException + - org.apache.kafka.common.errors.DisconnectException + - java.net.ConnectException + - java.net.UnknownHostException + - com.yape.challenge.transaction.infrastructure.kafka.exception.KafkaProducerException + instances: + kafkaProducer: + baseConfig: default + slidingWindowSize: 10 + minimumNumberOfCalls: 3 + waitDurationInOpenState: 30s + failureRateThreshold: 50 + slowCallRateThreshold: 80 + slowCallDurationThreshold: 5s + database: + baseConfig: default + slidingWindowSize: 10 + minimumNumberOfCalls: 3 + waitDurationInOpenState: 15s + failureRateThreshold: 60 + slowCallRateThreshold: 70 + slowCallDurationThreshold: 3s + + retry: + configs: + default: + maxAttempts: 3 + waitDuration: 1s + enableExponentialBackoff: true + exponentialBackoffMultiplier: 2 + instances: + kafkaProducer: + maxAttempts: 2 + waitDuration: 1s + retryExceptions: + - org.springframework.kafka.KafkaException + - java.util.concurrent.TimeoutException + - java.util.concurrent.ExecutionException + +# Logging configuration +logging: + level: + com.yape.challenge: ${LOGGING_LEVEL:INFO} + org.springframework.kafka: WARN + diff --git a/transaction-service/src/main/resources/db/data.sql b/transaction-service/src/main/resources/db/data.sql new file mode 100644 index 0000000000..1b7689b1c5 --- /dev/null +++ b/transaction-service/src/main/resources/db/data.sql @@ -0,0 +1,12 @@ +-- PostgreSQL data initialization +-- This script inserts initial data AFTER the schema is created + +-- Insert default transaction types (only if not exists) +INSERT INTO transaction_types (name, description) +VALUES + ('TRANSFER', 'Transfer between accounts'), + ('PAYMENT', 'Payment transaction'), + ('WITHDRAWAL', 'Cash withdrawal'), + ('DEPOSIT', 'Cash deposit') +ON CONFLICT (name) DO NOTHING; + diff --git a/transaction-service/src/test/java/com/yape/challenge/transaction/TransactionServiceApplicationTest.java b/transaction-service/src/test/java/com/yape/challenge/transaction/TransactionServiceApplicationTest.java new file mode 100644 index 0000000000..e794060440 --- /dev/null +++ b/transaction-service/src/test/java/com/yape/challenge/transaction/TransactionServiceApplicationTest.java @@ -0,0 +1,92 @@ +package com.yape.challenge.transaction; + +import org.junit.jupiter.api.DisplayName; +import org.junit.jupiter.api.Test; +import org.springframework.beans.factory.annotation.Autowired; +import org.springframework.boot.test.context.SpringBootTest; +import org.springframework.context.ApplicationContext; +import org.springframework.kafka.core.KafkaTemplate; +import org.springframework.kafka.test.context.EmbeddedKafka; +import org.springframework.test.annotation.DirtiesContext; + +import static org.junit.jupiter.api.Assertions.*; + +@SpringBootTest( + properties = { + "spring.kafka.bootstrap-servers=${spring.embedded.kafka.brokers}", + "spring.kafka.consumer.group-id=test-app-group", + "spring.datasource.url=jdbc:h2:mem:testdb;DB_CLOSE_DELAY=-1;CASE_INSENSITIVE_IDENTIFIERS=TRUE;MODE=PostgreSQL;DATABASE_TO_LOWER=TRUE;DEFAULT_NULL_ORDERING=HIGH", + "spring.datasource.driver-class-name=org.h2.Driver", + "spring.sql.init.mode=never", + "spring.jpa.hibernate.ddl-auto=create-drop", + "spring.jpa.properties.hibernate.dialect=org.hibernate.dialect.H2Dialect", + "spring.cache.type=none", + "spring.data.redis.repositories.enabled=false" + } +) +@EmbeddedKafka( + partitions = 1, + topics = {"transaction-created", "transaction-status-updated"} +) +@DirtiesContext(classMode = DirtiesContext.ClassMode.AFTER_CLASS) +@DisplayName("Transaction Service Application Tests") +class TransactionServiceApplicationTest { + + @Autowired + private ApplicationContext applicationContext; + + @Test + @DisplayName("Should load application context") + void shouldLoadApplicationContext() { + assertNotNull(applicationContext); + } + + @Test + @DisplayName("Should have transaction repository bean") + void shouldHaveTransactionRepositoryBean() { + assertTrue(applicationContext.containsBean("transactionRepository")); + } + + @Test + @DisplayName("Should have event store bean") + void shouldHaveEventStoreBean() { + assertTrue(applicationContext.containsBean("eventStore")); + } + + @Test + @DisplayName("Should have transaction aggregate service bean") + void shouldHaveTransactionAggregateServiceBean() { + assertTrue(applicationContext.containsBean("transactionAggregateService")); + } + + @Test + @DisplayName("Should have kafka template bean") + void shouldHaveKafkaTemplateBean() { + assertNotNull(applicationContext.getBean(KafkaTemplate.class)); + } + + @Test + @DisplayName("Should have command bus bean") + void shouldHaveCommandBusBean() { + assertTrue(applicationContext.containsBean("commandBus")); + } + + @Test + @DisplayName("Should have query bus bean") + void shouldHaveQueryBusBean() { + assertTrue(applicationContext.containsBean("queryBus")); + } + + @Test + @DisplayName("Should have transaction status consumer bean") + void shouldHaveTransactionStatusConsumerBean() { + assertTrue(applicationContext.containsBean("transactionStatusConsumer")); + } + + @Test + @DisplayName("Should have kafka producer service bean") + void shouldHaveKafkaProducerServiceBean() { + assertTrue(applicationContext.containsBean("kafkaProducerService")); + } +} + diff --git a/transaction-service/src/test/java/com/yape/challenge/transaction/domain/entity/TransactionTest.java b/transaction-service/src/test/java/com/yape/challenge/transaction/domain/entity/TransactionTest.java new file mode 100644 index 0000000000..5f16f4463f --- /dev/null +++ b/transaction-service/src/test/java/com/yape/challenge/transaction/domain/entity/TransactionTest.java @@ -0,0 +1,113 @@ +package com.yape.challenge.transaction.domain.entity; + +import com.yape.challenge.common.dto.TransactionStatus; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.DisplayName; +import org.junit.jupiter.api.Test; + +import java.math.BigDecimal; +import java.util.UUID; + +import static org.junit.jupiter.api.Assertions.*; + +@DisplayName("Transaction Entity Tests") +class TransactionTest { + + private Transaction transaction; + + @BeforeEach + void setUp() { + transaction = new Transaction(); + } + + @Test + @DisplayName("Should generate UUID on pre-persist when external id is null") + void shouldGenerateUuidOnPrePersist() { + // When + transaction.onCreate(); + + // Then + assertNotNull(transaction.getExternalId()); + } + + @Test + @DisplayName("Should not override existing external id on pre-persist") + void shouldNotOverrideExistingExternalId() { + // Given + UUID existingId = UUID.randomUUID(); + transaction.setExternalId(existingId); + + // When + transaction.onCreate(); + + // Then + assertEquals(existingId, transaction.getExternalId()); + } + + @Test + @DisplayName("Should set default status to PENDING on pre-persist") + void shouldSetDefaultStatusToPending() { + // When + transaction.onCreate(); + + // Then + assertEquals(TransactionStatus.PENDING, transaction.getStatus()); + } + + @Test + @DisplayName("Should not override existing status on pre-persist") + void shouldNotOverrideExistingStatus() { + // Given + transaction.setStatus(TransactionStatus.APPROVED); + + // When + transaction.onCreate(); + + // Then + assertEquals(TransactionStatus.APPROVED, transaction.getStatus()); + } + + @Test + @DisplayName("Should create transaction with builder") + void shouldCreateTransactionWithBuilder() { + // Given + UUID externalId = UUID.randomUUID(); + UUID debitId = UUID.randomUUID(); + UUID creditId = UUID.randomUUID(); + BigDecimal value = new BigDecimal("100.00"); + + // When + Transaction transaction = Transaction.builder() + .externalId(externalId) + .accountExternalIdDebit(debitId) + .accountExternalIdCredit(creditId) + .transferTypeId(1) + .value(value) + .status(TransactionStatus.PENDING) + .build(); + + // Then + assertNotNull(transaction); + assertEquals(externalId, transaction.getExternalId()); + assertEquals(debitId, transaction.getAccountExternalIdDebit()); + assertEquals(creditId, transaction.getAccountExternalIdCredit()); + assertEquals(1, transaction.getTransferTypeId()); + assertEquals(value, transaction.getValue()); + assertEquals(TransactionStatus.PENDING, transaction.getStatus()); + } + + @Test + @DisplayName("Should support all transaction statuses") + void shouldSupportAllTransactionStatuses() { + // Test each status + transaction.setStatus(TransactionStatus.PENDING); + assertEquals(TransactionStatus.PENDING, transaction.getStatus()); + + transaction.setStatus(TransactionStatus.APPROVED); + assertEquals(TransactionStatus.APPROVED, transaction.getStatus()); + + transaction.setStatus(TransactionStatus.REJECTED); + assertEquals(TransactionStatus.REJECTED, transaction.getStatus()); + } +} + diff --git a/transaction-service/src/test/java/com/yape/challenge/transaction/domain/service/TransactionAggregateServiceTest.java b/transaction-service/src/test/java/com/yape/challenge/transaction/domain/service/TransactionAggregateServiceTest.java new file mode 100644 index 0000000000..f76bd1741f --- /dev/null +++ b/transaction-service/src/test/java/com/yape/challenge/transaction/domain/service/TransactionAggregateServiceTest.java @@ -0,0 +1,252 @@ +package com.yape.challenge.transaction.domain.service; + +import com.yape.challenge.common.dto.TransactionStatus; +import com.yape.challenge.transaction.domain.entity.Transaction; +import com.yape.challenge.transaction.domain.event.TransactionCreatedDomainEvent; +import com.yape.challenge.transaction.domain.event.TransactionDomainEvent; +import com.yape.challenge.transaction.domain.event.TransactionStatusChangedDomainEvent; +import com.yape.challenge.transaction.infrastructure.eventstore.EventStore; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.DisplayName; +import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.extension.ExtendWith; +import org.mockito.InjectMocks; +import org.mockito.Mock; +import org.mockito.junit.jupiter.MockitoExtension; + +import java.math.BigDecimal; +import java.time.LocalDateTime; +import java.util.Arrays; +import java.util.Collections; +import java.util.List; +import java.util.UUID; + +import static org.junit.jupiter.api.Assertions.*; +import static org.mockito.Mockito.*; + +@ExtendWith(MockitoExtension.class) +@DisplayName("Transaction Aggregate Service Tests") +class TransactionAggregateServiceTest { + + @Mock + private EventStore eventStore; + + @InjectMocks + private TransactionAggregateService transactionAggregateService; + + private UUID transactionId; + private UUID debitAccountId; + private UUID creditAccountId; + + @BeforeEach + void setUp() { + transactionId = UUID.randomUUID(); + debitAccountId = UUID.randomUUID(); + creditAccountId = UUID.randomUUID(); + } + + @Test + @DisplayName("Should rebuild transaction from creation event") + void shouldRebuildTransactionFromCreationEvent() { + // Given + TransactionCreatedDomainEvent createdEvent = TransactionCreatedDomainEvent.builder() + .aggregateId(transactionId) + .accountExternalIdDebit(debitAccountId) + .accountExternalIdCredit(creditAccountId) + .transferTypeId(1) + .value(new BigDecimal("500.00")) + .occurredAt(LocalDateTime.now()) + .build(); + + when(eventStore.getEvents(transactionId)) + .thenReturn(Collections.singletonList(createdEvent)); + + // When + Transaction transaction = transactionAggregateService.rebuildFromEvents(transactionId); + + // Then + assertNotNull(transaction); + assertEquals(transactionId, transaction.getExternalId()); + assertEquals(debitAccountId, transaction.getAccountExternalIdDebit()); + assertEquals(creditAccountId, transaction.getAccountExternalIdCredit()); + assertEquals(1, transaction.getTransferTypeId()); + assertEquals(new BigDecimal("500.00"), transaction.getValue()); + assertEquals(TransactionStatus.PENDING, transaction.getStatus()); + verify(eventStore).getEvents(transactionId); + } + + @Test + @DisplayName("Should rebuild transaction from multiple events") + void shouldRebuildTransactionFromMultipleEvents() { + // Given + LocalDateTime createdAt = LocalDateTime.now().minusMinutes(5); + LocalDateTime updatedAt = LocalDateTime.now(); + + TransactionCreatedDomainEvent createdEvent = TransactionCreatedDomainEvent.builder() + .aggregateId(transactionId) + .accountExternalIdDebit(debitAccountId) + .accountExternalIdCredit(creditAccountId) + .transferTypeId(1) + .value(new BigDecimal("500.00")) + .occurredAt(createdAt) + .build(); + + TransactionStatusChangedDomainEvent statusChangedEvent = TransactionStatusChangedDomainEvent.builder() + .aggregateId(transactionId) + .oldStatus(TransactionStatus.PENDING) + .newStatus(TransactionStatus.APPROVED) + .occurredAt(updatedAt) + .build(); + + List events = Arrays.asList(createdEvent, statusChangedEvent); + when(eventStore.getEvents(transactionId)).thenReturn(events); + + // When + Transaction transaction = transactionAggregateService.rebuildFromEvents(transactionId); + + // Then + assertNotNull(transaction); + assertEquals(transactionId, transaction.getExternalId()); + assertEquals(TransactionStatus.APPROVED, transaction.getStatus()); + assertEquals(updatedAt, transaction.getUpdatedAt()); + verify(eventStore).getEvents(transactionId); + } + + @Test + @DisplayName("Should throw exception when no events found") + void shouldThrowExceptionWhenNoEventsFound() { + // Given + when(eventStore.getEvents(transactionId)).thenReturn(Collections.emptyList()); + + // When & Then + IllegalArgumentException exception = assertThrows(IllegalArgumentException.class, () -> + transactionAggregateService.rebuildFromEvents(transactionId) + ); + + assertTrue(exception.getMessage().contains("No events found")); + verify(eventStore).getEvents(transactionId); + } + + @Test + @DisplayName("Should apply creation event correctly") + void shouldApplyCreationEventCorrectly() { + // Given + Transaction transaction = new Transaction(); + transaction.setExternalId(transactionId); + + TransactionCreatedDomainEvent event = TransactionCreatedDomainEvent.builder() + .aggregateId(transactionId) + .accountExternalIdDebit(debitAccountId) + .accountExternalIdCredit(creditAccountId) + .transferTypeId(1) + .value(new BigDecimal("500.00")) + .occurredAt(LocalDateTime.now()) + .build(); + + // When + transactionAggregateService.applyEvent(transaction, event); + + // Then + assertEquals(debitAccountId, transaction.getAccountExternalIdDebit()); + assertEquals(creditAccountId, transaction.getAccountExternalIdCredit()); + assertEquals(TransactionStatus.PENDING, transaction.getStatus()); + } + + @Test + @DisplayName("Should apply status changed event correctly") + void shouldApplyStatusChangedEventCorrectly() { + // Given + Transaction transaction = new Transaction(); + transaction.setExternalId(transactionId); + transaction.setStatus(TransactionStatus.PENDING); + + LocalDateTime updatedAt = LocalDateTime.now(); + TransactionStatusChangedDomainEvent event = TransactionStatusChangedDomainEvent.builder() + .aggregateId(transactionId) + .oldStatus(TransactionStatus.PENDING) + .newStatus(TransactionStatus.APPROVED) + .occurredAt(updatedAt) + .build(); + + // When + transactionAggregateService.applyEvent(transaction, event); + + // Then + assertEquals(TransactionStatus.APPROVED, transaction.getStatus()); + assertEquals(updatedAt, transaction.getUpdatedAt()); + } + + @Test + @DisplayName("Should check if transaction exists") + void shouldCheckIfTransactionExists() { + // Given + when(eventStore.aggregateExists(transactionId)).thenReturn(true); + + // When + boolean exists = transactionAggregateService.transactionExists(transactionId); + + // Then + assertTrue(exists); + verify(eventStore).aggregateExists(transactionId); + } + + @Test + @DisplayName("Should return false when transaction does not exist") + void shouldReturnFalseWhenTransactionDoesNotExist() { + // Given + when(eventStore.aggregateExists(transactionId)).thenReturn(false); + + // When + boolean exists = transactionAggregateService.transactionExists(transactionId); + + // Then + assertFalse(exists); + verify(eventStore).aggregateExists(transactionId); + } + + @Test + @DisplayName("Should get event count") + void shouldGetEventCount() { + // Given + when(eventStore.getEventCount(transactionId)).thenReturn(3L); + + // When + long count = transactionAggregateService.getEventCount(transactionId); + + // Then + assertEquals(3L, count); + verify(eventStore).getEventCount(transactionId); + } + + @Test + @DisplayName("Should rebuild transaction with status changes from pending to rejected") + void shouldRebuildTransactionWithStatusChangesToRejected() { + // Given + TransactionCreatedDomainEvent createdEvent = TransactionCreatedDomainEvent.builder() + .aggregateId(transactionId) + .accountExternalIdDebit(debitAccountId) + .accountExternalIdCredit(creditAccountId) + .transferTypeId(1) + .value(new BigDecimal("2000.00")) + .occurredAt(LocalDateTime.now().minusMinutes(5)) + .build(); + + TransactionStatusChangedDomainEvent statusChangedEvent = TransactionStatusChangedDomainEvent.builder() + .aggregateId(transactionId) + .oldStatus(TransactionStatus.PENDING) + .newStatus(TransactionStatus.REJECTED) + .occurredAt(LocalDateTime.now()) + .build(); + + when(eventStore.getEvents(transactionId)) + .thenReturn(Arrays.asList(createdEvent, statusChangedEvent)); + + // When + Transaction transaction = transactionAggregateService.rebuildFromEvents(transactionId); + + // Then + assertEquals(TransactionStatus.REJECTED, transaction.getStatus()); + verify(eventStore).getEvents(transactionId); + } +} + diff --git a/transaction-service/src/test/java/com/yape/challenge/transaction/infrastructure/eventstore/EventStoreTest.java b/transaction-service/src/test/java/com/yape/challenge/transaction/infrastructure/eventstore/EventStoreTest.java new file mode 100644 index 0000000000..d14c888403 --- /dev/null +++ b/transaction-service/src/test/java/com/yape/challenge/transaction/infrastructure/eventstore/EventStoreTest.java @@ -0,0 +1,203 @@ +package com.yape.challenge.transaction.infrastructure.eventstore; + +import com.fasterxml.jackson.databind.ObjectMapper; +import com.yape.challenge.transaction.domain.entity.DomainEvent; +import com.yape.challenge.transaction.domain.event.TransactionCreatedDomainEvent; +import com.yape.challenge.transaction.domain.event.TransactionStatusChangedDomainEvent; +import com.yape.challenge.transaction.infrastructure.repository.DomainEventRepository; +import com.yape.challenge.common.dto.TransactionStatus; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.DisplayName; +import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.extension.ExtendWith; +import org.mockito.ArgumentCaptor; +import org.mockito.InjectMocks; +import org.mockito.Mock; +import org.mockito.junit.jupiter.MockitoExtension; + +import java.math.BigDecimal; +import java.time.LocalDateTime; +import java.util.Optional; +import java.util.UUID; + +import static org.junit.jupiter.api.Assertions.*; +import static org.mockito.ArgumentMatchers.any; +import static org.mockito.Mockito.*; + +@ExtendWith(MockitoExtension.class) +@DisplayName("Event Store Tests") +class EventStoreTest { + + @Mock + private DomainEventRepository domainEventRepository; + + @Mock + private ObjectMapper objectMapper; + + @InjectMocks + private EventStore eventStore; + + private UUID aggregateId; + private TransactionCreatedDomainEvent createdEvent; + private TransactionStatusChangedDomainEvent statusChangedEvent; + + @BeforeEach + void setUp() { + aggregateId = UUID.randomUUID(); + + createdEvent = TransactionCreatedDomainEvent.builder() + .aggregateId(aggregateId) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("500.00")) + .occurredAt(LocalDateTime.now()) + .build(); + + statusChangedEvent = TransactionStatusChangedDomainEvent.builder() + .aggregateId(aggregateId) + .oldStatus(TransactionStatus.PENDING) + .newStatus(TransactionStatus.APPROVED) + .occurredAt(LocalDateTime.now()) + .build(); + } + + @Test + @DisplayName("Should save transaction created event") + void shouldSaveTransactionCreatedEvent() throws Exception { + // Given + when(domainEventRepository.findLastVersionByAggregateId(aggregateId)) + .thenReturn(Optional.empty()); + when(objectMapper.writeValueAsString(any())).thenReturn("{}"); + when(domainEventRepository.save(any(DomainEvent.class))) + .thenAnswer(invocation -> invocation.getArgument(0)); + + // When + eventStore.saveEvent(createdEvent); + + // Then + ArgumentCaptor eventCaptor = ArgumentCaptor.forClass(DomainEvent.class); + verify(domainEventRepository).save(eventCaptor.capture()); + + DomainEvent savedEvent = eventCaptor.getValue(); + assertEquals(aggregateId, savedEvent.getAggregateId()); + assertEquals("Transaction", savedEvent.getAggregateType()); + assertEquals(1, savedEvent.getVersion()); + } + + @Test + @DisplayName("Should increment version when saving subsequent events") + void shouldIncrementVersionWhenSavingSubsequentEvents() throws Exception { + // Given + when(domainEventRepository.findLastVersionByAggregateId(aggregateId)) + .thenReturn(Optional.of(2)); + when(objectMapper.writeValueAsString(any())).thenReturn("{}"); + when(domainEventRepository.save(any(DomainEvent.class))) + .thenAnswer(invocation -> invocation.getArgument(0)); + + // When + eventStore.saveEvent(statusChangedEvent); + + // Then + ArgumentCaptor eventCaptor = ArgumentCaptor.forClass(DomainEvent.class); + verify(domainEventRepository).save(eventCaptor.capture()); + + DomainEvent savedEvent = eventCaptor.getValue(); + assertEquals(3, savedEvent.getVersion()); + } + + @Test + @DisplayName("Should throw exception when serialization fails") + void shouldThrowExceptionWhenSerializationFails() throws Exception { + // Given + when(domainEventRepository.findLastVersionByAggregateId(aggregateId)) + .thenReturn(Optional.empty()); + when(objectMapper.writeValueAsString(any())) + .thenThrow(new RuntimeException("Serialization error")); + + // When & Then + assertThrows(RuntimeException.class, () -> + eventStore.saveEvent(createdEvent) + ); + + verify(domainEventRepository, never()).save(any()); + } + + @Test + @DisplayName("Should save event with correct event type") + void shouldSaveEventWithCorrectEventType() throws Exception { + // Given + when(domainEventRepository.findLastVersionByAggregateId(aggregateId)) + .thenReturn(Optional.empty()); + when(objectMapper.writeValueAsString(any())).thenReturn("{}"); + when(domainEventRepository.save(any(DomainEvent.class))) + .thenAnswer(invocation -> invocation.getArgument(0)); + + // When + eventStore.saveEvent(createdEvent); + + // Then + ArgumentCaptor eventCaptor = ArgumentCaptor.forClass(DomainEvent.class); + verify(domainEventRepository).save(eventCaptor.capture()); + + DomainEvent savedEvent = eventCaptor.getValue(); + assertEquals("TransactionCreatedDomainEvent", savedEvent.getEventType()); + } + + @Test + @DisplayName("Should check if aggregate exists") + void shouldCheckIfAggregateExists() { + // Given + when(domainEventRepository.existsByAggregateId(aggregateId)).thenReturn(true); + + // When + boolean exists = eventStore.aggregateExists(aggregateId); + + // Then + assertTrue(exists); + verify(domainEventRepository).existsByAggregateId(aggregateId); + } + + @Test + @DisplayName("Should return false when aggregate does not exist") + void shouldReturnFalseWhenAggregateDoesNotExist() { + // Given + when(domainEventRepository.existsByAggregateId(aggregateId)).thenReturn(false); + + // When + boolean exists = eventStore.aggregateExists(aggregateId); + + // Then + assertFalse(exists); + verify(domainEventRepository).existsByAggregateId(aggregateId); + } + + @Test + @DisplayName("Should get event count for aggregate") + void shouldGetEventCountForAggregate() { + // Given + when(domainEventRepository.countByAggregateId(aggregateId)).thenReturn(5L); + + // When + long count = eventStore.getEventCount(aggregateId); + + // Then + assertEquals(5L, count); + verify(domainEventRepository).countByAggregateId(aggregateId); + } + + @Test + @DisplayName("Should return zero when no events exist for aggregate") + void shouldReturnZeroWhenNoEventsExistForAggregate() { + // Given + when(domainEventRepository.countByAggregateId(aggregateId)).thenReturn(0L); + + // When + long count = eventStore.getEventCount(aggregateId); + + // Then + assertEquals(0L, count); + verify(domainEventRepository).countByAggregateId(aggregateId); + } +} + diff --git a/transaction-service/src/test/java/com/yape/challenge/transaction/infrastructure/kafka/consumer/TransactionStatusConsumerTest.java b/transaction-service/src/test/java/com/yape/challenge/transaction/infrastructure/kafka/consumer/TransactionStatusConsumerTest.java new file mode 100644 index 0000000000..3b61358d24 --- /dev/null +++ b/transaction-service/src/test/java/com/yape/challenge/transaction/infrastructure/kafka/consumer/TransactionStatusConsumerTest.java @@ -0,0 +1,144 @@ +package com.yape.challenge.transaction.infrastructure.kafka.consumer; + +import com.yape.challenge.common.dto.TransactionStatus; +import com.yape.challenge.common.dto.TransactionStatusEvent; +import com.yape.challenge.transaction.application.bus.CommandBus; +import com.yape.challenge.transaction.application.command.UpdateTransactionStatusCommand; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.DisplayName; +import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.extension.ExtendWith; +import org.mockito.InjectMocks; +import org.mockito.Mock; +import org.mockito.junit.jupiter.MockitoExtension; + +import java.util.UUID; + +import static org.junit.jupiter.api.Assertions.*; +import static org.mockito.ArgumentMatchers.any; +import static org.mockito.Mockito.*; + +@ExtendWith(MockitoExtension.class) +@DisplayName("Transaction Status Consumer Tests") +class TransactionStatusConsumerTest { + + @Mock + private CommandBus commandBus; + + @InjectMocks + private TransactionStatusConsumer transactionStatusConsumer; + + private UUID transactionId; + private TransactionStatusEvent statusEvent; + + @BeforeEach + void setUp() { + transactionId = UUID.randomUUID(); + statusEvent = TransactionStatusEvent.builder() + .transactionExternalId(transactionId) + .status(TransactionStatus.APPROVED) + .build(); + } + + @Test + @DisplayName("Should consume and process transaction status event successfully") + void shouldConsumeAndProcessTransactionStatusEventSuccessfully() { + // Given + when(commandBus.dispatch(any(UpdateTransactionStatusCommand.class))).thenReturn(null); + + // When + transactionStatusConsumer.consumeTransactionStatus(statusEvent); + + // Then + verify(commandBus, times(1)).dispatch(any(UpdateTransactionStatusCommand.class)); + verify(commandBus).dispatch(argThat(command -> + command instanceof UpdateTransactionStatusCommand && + ((UpdateTransactionStatusCommand) command).getExternalId().equals(transactionId) && + ((UpdateTransactionStatusCommand) command).getStatus().equals(TransactionStatus.APPROVED) + )); + } + + @Test + @DisplayName("Should consume rejected transaction status") + void shouldConsumeRejectedTransactionStatus() { + // Given + TransactionStatusEvent rejectedEvent = TransactionStatusEvent.builder() + .transactionExternalId(transactionId) + .status(TransactionStatus.REJECTED) + .build(); + + when(commandBus.dispatch(any(UpdateTransactionStatusCommand.class))).thenReturn(null); + + // When + transactionStatusConsumer.consumeTransactionStatus(rejectedEvent); + + // Then + verify(commandBus, times(1)).dispatch(any(UpdateTransactionStatusCommand.class)); + verify(commandBus).dispatch(argThat(command -> + command instanceof UpdateTransactionStatusCommand && + ((UpdateTransactionStatusCommand) command).getStatus().equals(TransactionStatus.REJECTED) + )); + } + + @Test + @DisplayName("Should propagate exception when command bus fails") + void shouldPropagateExceptionWhenCommandBusFails() { + // Given + RuntimeException exception = new RuntimeException("Command bus failed"); + doThrow(exception).when(commandBus).dispatch(any(UpdateTransactionStatusCommand.class)); + + // When & Then + RuntimeException thrown = assertThrows(RuntimeException.class, () -> + transactionStatusConsumer.consumeTransactionStatus(statusEvent) + ); + + assertEquals("Command bus failed", thrown.getMessage()); + verify(commandBus, times(1)).dispatch(any(UpdateTransactionStatusCommand.class)); + } + + @Test + @DisplayName("Should process multiple consecutive events") + void shouldProcessMultipleConsecutiveEvents() { + // Given + TransactionStatusEvent event1 = TransactionStatusEvent.builder() + .transactionExternalId(UUID.randomUUID()) + .status(TransactionStatus.APPROVED) + .build(); + + TransactionStatusEvent event2 = TransactionStatusEvent.builder() + .transactionExternalId(UUID.randomUUID()) + .status(TransactionStatus.REJECTED) + .build(); + + when(commandBus.dispatch(any(UpdateTransactionStatusCommand.class))).thenReturn(null); + + // When + transactionStatusConsumer.consumeTransactionStatus(event1); + transactionStatusConsumer.consumeTransactionStatus(event2); + + // Then + verify(commandBus, times(2)).dispatch(any(UpdateTransactionStatusCommand.class)); + } + + @Test + @DisplayName("Should handle event with correct transaction id") + void shouldHandleEventWithCorrectTransactionId() { + // Given + UUID specificTransactionId = UUID.randomUUID(); + TransactionStatusEvent specificEvent = TransactionStatusEvent.builder() + .transactionExternalId(specificTransactionId) + .status(TransactionStatus.APPROVED) + .build(); + + when(commandBus.dispatch(any(UpdateTransactionStatusCommand.class))).thenReturn(null); + + // When + transactionStatusConsumer.consumeTransactionStatus(specificEvent); + + // Then + verify(commandBus).dispatch(argThat(command -> + ((UpdateTransactionStatusCommand) command).getExternalId().equals(specificTransactionId) + )); + } +} + diff --git a/transaction-service/src/test/java/com/yape/challenge/transaction/infrastructure/kafka/producer/KafkaProducerServiceTest.java b/transaction-service/src/test/java/com/yape/challenge/transaction/infrastructure/kafka/producer/KafkaProducerServiceTest.java new file mode 100644 index 0000000000..689ea3c2f5 --- /dev/null +++ b/transaction-service/src/test/java/com/yape/challenge/transaction/infrastructure/kafka/producer/KafkaProducerServiceTest.java @@ -0,0 +1,159 @@ +package com.yape.challenge.transaction.infrastructure.kafka.producer; + +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import com.yape.challenge.common.kafka.KafkaTopics; +import com.yape.challenge.transaction.infrastructure.kafka.exception.KafkaProducerException; +import org.apache.kafka.clients.producer.RecordMetadata; +import org.apache.kafka.common.TopicPartition; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.DisplayName; +import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.extension.ExtendWith; +import org.mockito.InjectMocks; +import org.mockito.Mock; +import org.mockito.junit.jupiter.MockitoExtension; +import org.springframework.kafka.core.KafkaTemplate; +import org.springframework.kafka.support.SendResult; + +import java.math.BigDecimal; +import java.util.UUID; +import java.util.concurrent.CompletableFuture; + +import static org.junit.jupiter.api.Assertions.*; +import static org.mockito.ArgumentMatchers.*; +import static org.mockito.Mockito.*; + +@ExtendWith(MockitoExtension.class) +@DisplayName("Kafka Producer Service Tests") +class KafkaProducerServiceTest { + + @Mock + private KafkaTemplate kafkaTemplate; + + @InjectMocks + private KafkaProducerService kafkaProducerService; + + private UUID transactionId; + private TransactionCreatedEvent event; + private String topic; + private String key; + + @BeforeEach + void setUp() { + transactionId = UUID.randomUUID(); + topic = KafkaTopics.TRANSACTION_CREATED; + key = transactionId.toString(); + + event = TransactionCreatedEvent.builder() + .transactionExternalId(transactionId) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("500.00")) + .build(); + } + + @Test + @DisplayName("Should send event successfully") + void shouldSendEventSuccessfully() throws Exception { + // Given + RecordMetadata metadata = new RecordMetadata(new TopicPartition(topic, 0), 0, 0, 0, 0, 0); + SendResult sendResult = mock(SendResult.class); + when(sendResult.getRecordMetadata()).thenReturn(metadata); + + CompletableFuture> future = + CompletableFuture.completedFuture(sendResult); + + when(kafkaTemplate.send(anyString(), anyString(), any(TransactionCreatedEvent.class))) + .thenReturn(future); + + // When + kafkaProducerService.sendTransactionCreatedEvent(topic, key, event); + + // Then + verify(kafkaTemplate, times(1)).send(topic, key, event); + } + + @Test + @DisplayName("Should throw KafkaProducerException when send fails") + void shouldThrowKafkaProducerExceptionWhenSendFails() { + // Given + CompletableFuture> future = + CompletableFuture.failedFuture(new RuntimeException("Kafka unavailable")); + + when(kafkaTemplate.send(anyString(), anyString(), any(TransactionCreatedEvent.class))) + .thenReturn(future); + + // When & Then + assertThrows(KafkaProducerException.class, () -> + kafkaProducerService.sendTransactionCreatedEvent(topic, key, event) + ); + + verify(kafkaTemplate, times(1)).send(topic, key, event); + } + + @Test + @DisplayName("Should handle interrupted exception") + void shouldHandleInterruptedException() { + // Given + CompletableFuture> future = new CompletableFuture<>(); + future.completeExceptionally(new InterruptedException("Thread interrupted")); + + when(kafkaTemplate.send(anyString(), anyString(), any(TransactionCreatedEvent.class))) + .thenReturn(future); + + // When & Then + assertThrows(KafkaProducerException.class, () -> + kafkaProducerService.sendTransactionCreatedEvent(topic, key, event) + ); + + verify(kafkaTemplate, times(1)).send(topic, key, event); + } + + @Test + @DisplayName("Should use correct topic and key") + void shouldUseCorrectTopicAndKey() throws Exception { + // Given + RecordMetadata metadata = new RecordMetadata(new TopicPartition(topic, 0), 0, 0, 0, 0, 0); + SendResult sendResult = mock(SendResult.class); + when(sendResult.getRecordMetadata()).thenReturn(metadata); + + CompletableFuture> future = + CompletableFuture.completedFuture(sendResult); + + when(kafkaTemplate.send(eq(topic), eq(key), any(TransactionCreatedEvent.class))) + .thenReturn(future); + + // When + kafkaProducerService.sendTransactionCreatedEvent(topic, key, event); + + // Then + verify(kafkaTemplate).send(eq(topic), eq(key), any(TransactionCreatedEvent.class)); + } + + @Test + @DisplayName("Should send event with correct transaction data") + void shouldSendEventWithCorrectTransactionData() throws Exception { + // Given + RecordMetadata metadata = new RecordMetadata(new TopicPartition(topic, 0), 0, 0, 0, 0, 0); + SendResult sendResult = mock(SendResult.class); + when(sendResult.getRecordMetadata()).thenReturn(metadata); + + CompletableFuture> future = + CompletableFuture.completedFuture(sendResult); + + when(kafkaTemplate.send(anyString(), anyString(), any(TransactionCreatedEvent.class))) + .thenReturn(future); + + // When + kafkaProducerService.sendTransactionCreatedEvent(topic, key, event); + + // Then + verify(kafkaTemplate).send(anyString(), anyString(), argThat(evt -> + evt.getTransactionExternalId().equals(transactionId) && + evt.getValue().equals(new BigDecimal("500.00")) && + evt.getTransferTypeId().equals(1) + )); + } +} + diff --git a/transaction-service/src/test/java/com/yape/challenge/transaction/integration/TransactionServiceIntegrationTest.java b/transaction-service/src/test/java/com/yape/challenge/transaction/integration/TransactionServiceIntegrationTest.java new file mode 100644 index 0000000000..389feb611d --- /dev/null +++ b/transaction-service/src/test/java/com/yape/challenge/transaction/integration/TransactionServiceIntegrationTest.java @@ -0,0 +1,278 @@ +package com.yape.challenge.transaction.integration; + +import com.yape.challenge.common.dto.TransactionCreatedEvent; +import com.yape.challenge.common.dto.TransactionStatus; +import com.yape.challenge.common.kafka.KafkaTopics; +import com.yape.challenge.transaction.application.dto.request.CreateTransactionRequest; +import com.yape.challenge.transaction.domain.entity.Transaction; +import com.yape.challenge.transaction.domain.entity.TransactionType; +import com.yape.challenge.transaction.infrastructure.repository.TransactionRepository; +import com.yape.challenge.transaction.infrastructure.repository.TransactionTypeRepository; +import com.fasterxml.jackson.databind.ObjectMapper; +import org.apache.kafka.clients.consumer.ConsumerConfig; +import org.apache.kafka.clients.consumer.ConsumerRecord; +import org.apache.kafka.common.serialization.StringDeserializer; +import org.junit.jupiter.api.*; +import org.springframework.beans.factory.annotation.Autowired; +import org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc; +import org.springframework.boot.test.context.SpringBootTest; +import org.springframework.http.MediaType; +import org.springframework.kafka.core.DefaultKafkaConsumerFactory; +import org.springframework.kafka.listener.ContainerProperties; +import org.springframework.kafka.listener.KafkaMessageListenerContainer; +import org.springframework.kafka.listener.MessageListener; +import org.springframework.kafka.support.serializer.JsonDeserializer; +import org.springframework.kafka.test.context.EmbeddedKafka; +import org.springframework.kafka.test.utils.ContainerTestUtils; +import org.springframework.test.annotation.DirtiesContext; +import org.springframework.test.web.servlet.MockMvc; +import org.springframework.test.web.servlet.MvcResult; + +import java.math.BigDecimal; +import java.util.HashMap; +import java.util.Map; +import java.util.UUID; +import java.util.concurrent.BlockingQueue; +import java.util.concurrent.LinkedBlockingQueue; +import java.util.concurrent.TimeUnit; + +import static org.junit.jupiter.api.Assertions.*; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; +import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.*; + +@SpringBootTest( + webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT, + properties = { + "spring.kafka.bootstrap-servers=${spring.embedded.kafka.brokers}", + "spring.datasource.url=jdbc:h2:mem:testdb;DB_CLOSE_DELAY=-1;CASE_INSENSITIVE_IDENTIFIERS=TRUE;MODE=PostgreSQL;DATABASE_TO_LOWER=TRUE;DEFAULT_NULL_ORDERING=HIGH", + "spring.datasource.driver-class-name=org.h2.Driver", + "spring.sql.init.mode=never", + "spring.jpa.hibernate.ddl-auto=create-drop", + "spring.jpa.properties.hibernate.dialect=org.hibernate.dialect.H2Dialect", + "spring.jpa.show-sql=true", + "spring.cache.type=none", + "spring.data.redis.repositories.enabled=false" + } +) +@AutoConfigureMockMvc +@DirtiesContext(classMode = DirtiesContext.ClassMode.AFTER_CLASS) +@EmbeddedKafka( + partitions = 3, + topics = {KafkaTopics.TRANSACTION_CREATED, KafkaTopics.TRANSACTION_STATUS_UPDATED} +) +@DisplayName("Transaction Service Integration Tests") +@TestMethodOrder(MethodOrderer.OrderAnnotation.class) +class TransactionServiceIntegrationTest { + + @Autowired + private MockMvc mockMvc; + + @Autowired + private ObjectMapper objectMapper; + + @Autowired + private TransactionRepository transactionRepository; + + @Autowired + private TransactionTypeRepository transactionTypeRepository; + + private KafkaMessageListenerContainer container; + private BlockingQueue> records; + + @BeforeEach + void setUp() { + // Clean up database before each test + transactionRepository.deleteAll(); + + // Ensure transaction types exist + if (transactionTypeRepository.count() == 0) { + TransactionType type = new TransactionType(); + type.setName("Tipo A"); + transactionTypeRepository.save(type); + } + + // Setup Kafka consumer for integration tests + records = new LinkedBlockingQueue<>(); + + Map consumerProps = new HashMap<>(); + consumerProps.put(ConsumerConfig.BOOTSTRAP_SERVERS_CONFIG, System.getProperty("spring.embedded.kafka.brokers")); + consumerProps.put(ConsumerConfig.GROUP_ID_CONFIG, "test-integration-group-" + System.currentTimeMillis()); + consumerProps.put(ConsumerConfig.KEY_DESERIALIZER_CLASS_CONFIG, StringDeserializer.class); + consumerProps.put(ConsumerConfig.VALUE_DESERIALIZER_CLASS_CONFIG, JsonDeserializer.class); + consumerProps.put(ConsumerConfig.AUTO_OFFSET_RESET_CONFIG, "earliest"); + consumerProps.put(JsonDeserializer.TRUSTED_PACKAGES, "*"); + consumerProps.put(JsonDeserializer.VALUE_DEFAULT_TYPE, TransactionCreatedEvent.class.getName()); + + DefaultKafkaConsumerFactory consumerFactory = + new DefaultKafkaConsumerFactory<>(consumerProps); + + ContainerProperties containerProperties = new ContainerProperties(KafkaTopics.TRANSACTION_CREATED); + container = new KafkaMessageListenerContainer<>(consumerFactory, containerProperties); + container.setupMessageListener((MessageListener) records::add); + container.start(); + + ContainerTestUtils.waitForAssignment(container, 3); + } + + @AfterEach + void tearDown() { + if (container != null) { + container.stop(); + } + } + + @Test + @Order(1) + @DisplayName("Should create transaction and publish to Kafka") + void shouldCreateTransactionAndPublishToKafka() throws Exception { + // Given + CreateTransactionRequest request = CreateTransactionRequest.builder() + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .tranferTypeId(1) + .value(new BigDecimal("500.00")) + .build(); + + // When + MvcResult result = mockMvc.perform(post("/api/v1/transactions") + .contentType(MediaType.APPLICATION_JSON) + .content(objectMapper.writeValueAsString(request))) + .andExpect(status().isCreated()) + .andExpect(jsonPath("$.transactionExternalId").exists()) + .andExpect(jsonPath("$.transactionStatus.name").value("PENDING")) + .andExpect(jsonPath("$.value").value(500.00)) + .andReturn(); + + // Then - Verify database + String response = result.getResponse().getContentAsString(); + String transactionId = objectMapper.readTree(response).get("transactionExternalId").asText(); + + Transaction savedTransaction = transactionRepository + .findByExternalId(UUID.fromString(transactionId)) + .orElse(null); + + assertNotNull(savedTransaction); + assertEquals(TransactionStatus.PENDING, savedTransaction.getStatus()); + + // Then - Verify Kafka event + ConsumerRecord received = records.poll(10, TimeUnit.SECONDS); + assertNotNull(received, "Should receive a Kafka event"); + + TransactionCreatedEvent kafkaEvent = received.value(); + assertEquals(transactionId, kafkaEvent.getTransactionExternalId().toString()); + assertEquals(new BigDecimal("500.00"), kafkaEvent.getValue()); + } + + @Test + @Order(2) + @DisplayName("Should get transaction by external id") + void shouldGetTransactionByExternalId() throws Exception { + // Given - Create a transaction first + Transaction transaction = Transaction.builder() + .externalId(UUID.randomUUID()) + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .transferTypeId(1) + .value(new BigDecimal("300.00")) + .status(TransactionStatus.PENDING) + .build(); + transaction = transactionRepository.save(transaction); + + // When & Then + mockMvc.perform(get("/api/v1/transactions/{externalId}", transaction.getExternalId())) + .andExpect(status().isOk()) + .andExpect(jsonPath("$.transactionExternalId").value(transaction.getExternalId().toString())) + .andExpect(jsonPath("$.transactionStatus.name").value("PENDING")) + .andExpect(jsonPath("$.value").value(300.00)); + } + + @Test + @Order(3) + @DisplayName("Should return 404 when transaction not found") + void shouldReturn404WhenTransactionNotFound() throws Exception { + // Given + UUID nonExistentId = UUID.randomUUID(); + + // When & Then + mockMvc.perform(get("/api/v1/transactions/{externalId}", nonExistentId)) + .andExpect(status().isNotFound()); + } + + @Test + @Order(4) + @DisplayName("Should persist transaction in event store and read model") + void shouldPersistTransactionInEventStoreAndReadModel() throws Exception { + // Given + CreateTransactionRequest request = CreateTransactionRequest.builder() + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .tranferTypeId(1) + .value(new BigDecimal("750.00")) + .build(); + + // When + MvcResult result = mockMvc.perform(post("/api/v1/transactions") + .contentType(MediaType.APPLICATION_JSON) + .content(objectMapper.writeValueAsString(request))) + .andExpect(status().isCreated()) + .andReturn(); + + // Then + String response = result.getResponse().getContentAsString(); + String transactionId = objectMapper.readTree(response).get("transactionExternalId").asText(); + + // Verify transaction exists in read model (database) + Transaction transaction = transactionRepository + .findByExternalId(UUID.fromString(transactionId)) + .orElse(null); + + assertNotNull(transaction); + assertEquals(new BigDecimal("750.00"), transaction.getValue()); + assertEquals(TransactionStatus.PENDING, transaction.getStatus()); + } + + @Test + @Order(5) + @DisplayName("Should create multiple transactions successfully") + void shouldCreateMultipleTransactionsSuccessfully() throws Exception { + // Given + long initialCount = transactionRepository.count(); + + CreateTransactionRequest request1 = CreateTransactionRequest.builder() + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .tranferTypeId(1) + .value(new BigDecimal("100.00")) + .build(); + + CreateTransactionRequest request2 = CreateTransactionRequest.builder() + .accountExternalIdDebit(UUID.randomUUID()) + .accountExternalIdCredit(UUID.randomUUID()) + .tranferTypeId(1) + .value(new BigDecimal("200.00")) + .build(); + + // When + mockMvc.perform(post("/api/v1/transactions") + .contentType(MediaType.APPLICATION_JSON) + .content(objectMapper.writeValueAsString(request1))) + .andExpect(status().isCreated()); + + mockMvc.perform(post("/api/v1/transactions") + .contentType(MediaType.APPLICATION_JSON) + .content(objectMapper.writeValueAsString(request2))) + .andExpect(status().isCreated()); + + // Then + assertEquals(initialCount + 2, transactionRepository.count()); + + // Verify both Kafka events + ConsumerRecord event1 = records.poll(10, TimeUnit.SECONDS); + ConsumerRecord event2 = records.poll(10, TimeUnit.SECONDS); + + assertNotNull(event1); + assertNotNull(event2); + } +} + diff --git a/transaction-service/src/test/java/com/yape/challenge/transaction/presentation/controller/TransactionControllerTest.java b/transaction-service/src/test/java/com/yape/challenge/transaction/presentation/controller/TransactionControllerTest.java new file mode 100644 index 0000000000..446b489a7f --- /dev/null +++ b/transaction-service/src/test/java/com/yape/challenge/transaction/presentation/controller/TransactionControllerTest.java @@ -0,0 +1,221 @@ +package com.yape.challenge.transaction.presentation.controller; + +import com.fasterxml.jackson.databind.ObjectMapper; +import com.yape.challenge.transaction.application.bus.CommandBus; +import com.yape.challenge.transaction.application.bus.QueryBus; +import com.yape.challenge.transaction.application.command.CreateTransactionCommand; +import com.yape.challenge.transaction.application.dto.request.CreateTransactionRequest; +import com.yape.challenge.transaction.application.dto.response.TransactionResponse; +import com.yape.challenge.transaction.application.query.GetTransactionQuery; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.DisplayName; +import org.junit.jupiter.api.Test; +import org.springframework.beans.factory.annotation.Autowired; +import org.springframework.boot.test.autoconfigure.web.servlet.WebMvcTest; +import org.springframework.boot.test.mock.mockito.MockBean; +import org.springframework.http.MediaType; +import org.springframework.test.web.servlet.MockMvc; + +import java.math.BigDecimal; +import java.time.LocalDateTime; +import java.util.UUID; + +import static org.mockito.ArgumentMatchers.any; +import static org.mockito.Mockito.*; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post; +import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.*; + +@WebMvcTest(TransactionController.class) +@DisplayName("Transaction Controller Tests") +class TransactionControllerTest { + + @Autowired + private MockMvc mockMvc; + + @Autowired + private ObjectMapper objectMapper; + + @MockBean + private CommandBus commandBus; + + @MockBean + private QueryBus queryBus; + + private UUID transactionId; + private UUID debitAccountId; + private UUID creditAccountId; + private CreateTransactionRequest createRequest; + private TransactionResponse transactionResponse; + + @BeforeEach + void setUp() { + transactionId = UUID.randomUUID(); + debitAccountId = UUID.randomUUID(); + creditAccountId = UUID.randomUUID(); + + createRequest = CreateTransactionRequest.builder() + .accountExternalIdDebit(debitAccountId) + .accountExternalIdCredit(creditAccountId) + .tranferTypeId(1) + .value(new BigDecimal("500.00")) + .build(); + + transactionResponse = TransactionResponse.builder() + .transactionExternalId(transactionId) + .transactionType(TransactionResponse.TransactionTypeDto.builder() + .name("Tipo A") + .build()) + .transactionStatus(TransactionResponse.TransactionStatusDto.builder() + .name("PENDING") + .build()) + .value(new BigDecimal("500.00")) + .createdAt(LocalDateTime.now()) + .build(); + } + + @Test + @DisplayName("Should create transaction successfully") + void shouldCreateTransactionSuccessfully() throws Exception { + // Given + when(commandBus.dispatch(any(CreateTransactionCommand.class))) + .thenReturn(transactionResponse); + + // When & Then + mockMvc.perform(post("/api/v1/transactions") + .contentType(MediaType.APPLICATION_JSON) + .content(objectMapper.writeValueAsString(createRequest))) + .andExpect(status().isCreated()) + .andExpect(content().contentType(MediaType.APPLICATION_JSON)) + .andExpect(jsonPath("$.transactionExternalId").value(transactionId.toString())) + .andExpect(jsonPath("$.transactionStatus.name").value("PENDING")) + .andExpect(jsonPath("$.value").value(500.00)) + .andExpect(jsonPath("$.transactionType.name").value("Tipo A")); + + verify(commandBus, times(1)).dispatch(any(CreateTransactionCommand.class)); + } + + @Test + @DisplayName("Should return bad request when request is invalid") + void shouldReturnBadRequestWhenRequestIsInvalid() throws Exception { + // Given + CreateTransactionRequest invalidRequest = CreateTransactionRequest.builder() + .accountExternalIdDebit(null) // Required field missing + .accountExternalIdCredit(creditAccountId) + .tranferTypeId(1) + .value(new BigDecimal("500.00")) + .build(); + + // When & Then + mockMvc.perform(post("/api/v1/transactions") + .contentType(MediaType.APPLICATION_JSON) + .content(objectMapper.writeValueAsString(invalidRequest))) + .andExpect(status().isBadRequest()); + + verify(commandBus, never()).dispatch(any(CreateTransactionCommand.class)); + } + + @Test + @DisplayName("Should get transaction by external id") + void shouldGetTransactionByExternalId() throws Exception { + // Given + when(queryBus.dispatch(any(GetTransactionQuery.class))) + .thenReturn(transactionResponse); + + // When & Then + mockMvc.perform(get("/api/v1/transactions/{externalId}", transactionId)) + .andExpect(status().isOk()) + .andExpect(content().contentType(MediaType.APPLICATION_JSON)) + .andExpect(jsonPath("$.transactionExternalId").value(transactionId.toString())) + .andExpect(jsonPath("$.transactionStatus.name").value("PENDING")) + .andExpect(jsonPath("$.value").value(500.00)); + + verify(queryBus, times(1)).dispatch(any(GetTransactionQuery.class)); + } + + @Test + @DisplayName("Should handle valid UUID path variable") + void shouldHandleValidUuidPathVariable() throws Exception { + // Given + when(queryBus.dispatch(any(GetTransactionQuery.class))) + .thenReturn(transactionResponse); + + // When & Then + mockMvc.perform(get("/api/v1/transactions/{externalId}", transactionId.toString())) + .andExpect(status().isOk()); + + verify(queryBus, times(1)).dispatch(any(GetTransactionQuery.class)); + } + + @Test + @DisplayName("Should create transaction with minimum valid value") + void shouldCreateTransactionWithMinimumValidValue() throws Exception { + // Given + CreateTransactionRequest minValueRequest = CreateTransactionRequest.builder() + .accountExternalIdDebit(debitAccountId) + .accountExternalIdCredit(creditAccountId) + .tranferTypeId(1) + .value(new BigDecimal("0.01")) + .build(); + + TransactionResponse minValueResponse = TransactionResponse.builder() + .transactionExternalId(transactionId) + .transactionType(TransactionResponse.TransactionTypeDto.builder() + .name("Tipo A") + .build()) + .transactionStatus(TransactionResponse.TransactionStatusDto.builder() + .name("PENDING") + .build()) + .value(new BigDecimal("0.01")) + .createdAt(LocalDateTime.now()) + .build(); + + when(commandBus.dispatch(any(CreateTransactionCommand.class))) + .thenReturn(minValueResponse); + + // When & Then + mockMvc.perform(post("/api/v1/transactions") + .contentType(MediaType.APPLICATION_JSON) + .content(objectMapper.writeValueAsString(minValueRequest))) + .andExpect(status().isCreated()) + .andExpect(jsonPath("$.value").value(0.01)); + + verify(commandBus, times(1)).dispatch(any(CreateTransactionCommand.class)); + } + + @Test + @DisplayName("Should create transaction with large value") + void shouldCreateTransactionWithLargeValue() throws Exception { + // Given + CreateTransactionRequest largeValueRequest = CreateTransactionRequest.builder() + .accountExternalIdDebit(debitAccountId) + .accountExternalIdCredit(creditAccountId) + .tranferTypeId(1) + .value(new BigDecimal("99999.99")) + .build(); + + TransactionResponse largeValueResponse = TransactionResponse.builder() + .transactionExternalId(transactionId) + .transactionType(TransactionResponse.TransactionTypeDto.builder() + .name("Tipo A") + .build()) + .transactionStatus(TransactionResponse.TransactionStatusDto.builder() + .name("PENDING") + .build()) + .value(new BigDecimal("99999.99")) + .createdAt(LocalDateTime.now()) + .build(); + + when(commandBus.dispatch(any(CreateTransactionCommand.class))) + .thenReturn(largeValueResponse); + + // When & Then + mockMvc.perform(post("/api/v1/transactions") + .contentType(MediaType.APPLICATION_JSON) + .content(objectMapper.writeValueAsString(largeValueRequest))) + .andExpect(status().isCreated()) + .andExpect(jsonPath("$.value").value(99999.99)); + + verify(commandBus, times(1)).dispatch(any(CreateTransactionCommand.class)); + } +} diff --git a/transaction-service/src/test/resources/application.yml b/transaction-service/src/test/resources/application.yml new file mode 100644 index 0000000000..2ca6791b2a --- /dev/null +++ b/transaction-service/src/test/resources/application.yml @@ -0,0 +1,68 @@ +spring: + kafka: + bootstrap-servers: localhost:9092 + consumer: + group-id: transaction-test-group + auto-offset-reset: earliest + key-deserializer: org.apache.kafka.common.serialization.StringDeserializer + value-deserializer: org.springframework.kafka.support.serializer.JsonDeserializer + properties: + spring.json.trusted.packages: "*" + producer: + key-serializer: org.apache.kafka.common.serialization.StringSerializer + value-serializer: org.springframework.kafka.support.serializer.JsonSerializer + + datasource: + url: jdbc:h2:mem:testdb;DB_CLOSE_DELAY=-1;CASE_INSENSITIVE_IDENTIFIERS=TRUE;MODE=PostgreSQL;DATABASE_TO_LOWER=TRUE;DEFAULT_NULL_ORDERING=HIGH + driverClassName: org.h2.Driver + username: sa + password: + + jpa: + hibernate: + ddl-auto: create-drop + show-sql: true + properties: + hibernate: + dialect: org.hibernate.dialect.H2Dialect + format_sql: true + + h2: + console: + enabled: true + + cache: + type: none + +resilience4j: + circuitbreaker: + instances: + database: + register-health-indicator: true + sliding-window-size: 10 + minimum-number-of-calls: 5 + permitted-number-of-calls-in-half-open-state: 3 + wait-duration-in-open-state: 10s + failure-rate-threshold: 50 + kafkaProducer: + register-health-indicator: true + sliding-window-size: 10 + minimum-number-of-calls: 5 + permitted-number-of-calls-in-half-open-state: 3 + wait-duration-in-open-state: 5s + failure-rate-threshold: 50 + + retry: + instances: + kafkaProducer: + max-attempts: 3 + wait-duration: 1s + enable-exponential-backoff: true + exponential-backoff-multiplier: 2 + +logging: + level: + com.yape.challenge: DEBUG + org.springframework.kafka: DEBUG + org.hibernate.SQL: DEBUG +