From a94d6f5267696e4f36804483bd7eed005dd4ea02 Mon Sep 17 00:00:00 2001 From: Ivan Perez <¨ivansperezt@gmail.com¨> Date: Fri, 27 Feb 2026 23:53:37 -0500 Subject: [PATCH 1/7] feat(transaction): set up onion architecture - define Transaction domain model with approve/reject behavior - add TransactionStatus and TransferType enums - create TransactionRepository interface in domain layer - implement TransactionRepositoryImpl with JPA in infrastructure layer - add TransactionEntity with JPA mapping - scaffold use case interfaces in application abstractions --- .gitignore | 124 ++------ transaction-service/.gitattributes | 2 + transaction-service/.gitignore | 33 ++ .../.mvn/wrapper/maven-wrapper.properties | 3 + transaction-service/mvnw | 295 ++++++++++++++++++ transaction-service/mvnw.cmd | 189 +++++++++++ transaction-service/pom.xml | 121 +++++++ .../entities/TransactionEntity.java | 37 +++ .../TransactionJpaRepository.java | 11 + .../TransactionRepositoryImpl.java | 27 ++ .../TransactionServiceApplication.java | 13 + .../CreateTransactionUseCase.java | 8 + .../abstractions/GetTransactionUseCase.java | 8 + .../CreateTransactionHandler.java | 16 + .../CreateTransactionRequest.java | 4 + .../CreateTransactionResponse.java | 4 + .../gettransaction/GetTransactionHandler.java | 12 + .../gettransaction/GetTransactionRequest.java | 4 + .../GetTransactionResponse.java | 4 + .../domain/enums/TransactionStatus.java | 5 + .../domain/enums/TransferType.java | 24 ++ .../domain/models/Transaction.java | 29 ++ .../repostories/TransactionRepository.java | 15 + .../controllers/HealthController.java | 14 + .../controllers/TransactionController.java | 32 ++ .../src/main/resources/application.properties | 11 + .../TransactionServiceApplicationTests.java | 13 + 27 files changed, 965 insertions(+), 93 deletions(-) create mode 100644 transaction-service/.gitattributes create mode 100644 transaction-service/.gitignore create mode 100644 transaction-service/.mvn/wrapper/maven-wrapper.properties create mode 100644 transaction-service/mvnw create mode 100644 transaction-service/mvnw.cmd create mode 100644 transaction-service/pom.xml create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/entities/TransactionEntity.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionJpaRepository.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionRepositoryImpl.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/TransactionServiceApplication.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/CreateTransactionUseCase.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetTransactionUseCase.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionRequest.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionResponse.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionHandler.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionRequest.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionResponse.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/domain/enums/TransactionStatus.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/domain/enums/TransferType.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/domain/repostories/TransactionRepository.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/HealthController.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/TransactionController.java create mode 100644 transaction-service/src/main/resources/application.properties create mode 100644 transaction-service/src/test/java/com/yape/transactionservice/TransactionServiceApplicationTests.java diff --git a/.gitignore b/.gitignore index 67045665db..c576586027 100644 --- a/.gitignore +++ b/.gitignore @@ -1,104 +1,42 @@ -# Logs -logs -*.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 +# .gitignore para Java + Spring Boot -# Dependency directories -node_modules/ -jspm_packages/ +``` +# Compiled files +*.class -# TypeScript v1 declaration files -typings/ - -# TypeScript cache -*.tsbuildinfo +# Logs +*.log -# Optional npm cache directory -.npm +# Maven +target/ -# Optional eslint cache -.eslintcache +# Gradle +.gradle/ +build/ -# Microbundle cache -.rpt2_cache/ -.rts2_cache_cjs/ -.rts2_cache_es/ -.rts2_cache_umd/ +# IntelliJ +.idea/ +*.iml +out/ -# Optional REPL history -.node_repl_history +# Eclipse +.project +.classpath +.settings/ +bin/ -# Output of 'npm pack' -*.tgz +# VSCode +.vscode/ -# Yarn Integrity file -.yarn-integrity +# OS files +.DS_Store +Thumbs.db -# dotenv environment variables file +# Env files .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 +# Temp +*.tmp +*.swp +*~ +``` diff --git a/transaction-service/.gitattributes b/transaction-service/.gitattributes new file mode 100644 index 0000000000..3b41682ac5 --- /dev/null +++ b/transaction-service/.gitattributes @@ -0,0 +1,2 @@ +/mvnw text eol=lf +*.cmd text eol=crlf diff --git a/transaction-service/.gitignore b/transaction-service/.gitignore new file mode 100644 index 0000000000..667aaef0c8 --- /dev/null +++ b/transaction-service/.gitignore @@ -0,0 +1,33 @@ +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/ diff --git a/transaction-service/.mvn/wrapper/maven-wrapper.properties b/transaction-service/.mvn/wrapper/maven-wrapper.properties new file mode 100644 index 0000000000..8dea6c227c --- /dev/null +++ b/transaction-service/.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/transaction-service/mvnw b/transaction-service/mvnw new file mode 100644 index 0000000000..bd8896bf22 --- /dev/null +++ b/transaction-service/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/transaction-service/mvnw.cmd b/transaction-service/mvnw.cmd new file mode 100644 index 0000000000..92450f9327 --- /dev/null +++ b/transaction-service/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/transaction-service/pom.xml b/transaction-service/pom.xml new file mode 100644 index 0000000000..f6d82f2976 --- /dev/null +++ b/transaction-service/pom.xml @@ -0,0 +1,121 @@ + + + 4.0.0 + + org.springframework.boot + spring-boot-starter-parent + 4.0.3 + + + com.yape + transaction-service + 0.0.1-SNAPSHOT + transaction-service + service of transaction + + + + + + + + + + + + + + + 21 + + + + org.mapstruct + mapstruct + 1.5.5.Final + + + org.springframework.boot + spring-boot-starter-data-jpa + + + org.springframework.boot + spring-boot-starter-kafka + + + org.springframework.boot + spring-boot-starter-validation + + + org.springframework.boot + spring-boot-starter-webmvc + + + + org.postgresql + postgresql + runtime + + + org.projectlombok + lombok + true + + + org.springframework.boot + spring-boot-starter-data-jpa-test + test + + + org.springframework.boot + spring-boot-starter-kafka-test + test + + + org.springframework.boot + spring-boot-starter-validation-test + test + + + org.springframework.boot + spring-boot-starter-webmvc-test + test + + + + + + + org.apache.maven.plugins + maven-compiler-plugin + + + + org.projectlombok + lombok + + + org.mapstruct + mapstruct-processor + 1.5.5.Final + + + + + + org.springframework.boot + spring-boot-maven-plugin + + + + org.projectlombok + lombok + + + + + + + + diff --git a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/entities/TransactionEntity.java b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/entities/TransactionEntity.java new file mode 100644 index 0000000000..ecf3eeffb2 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/entities/TransactionEntity.java @@ -0,0 +1,37 @@ +package com.yape.transactionservice.Infrastructure.persistence.entities; + +import com.yape.transactionservice.domain.enums.TransactionStatus; +import jakarta.persistence.*; +import lombok.AllArgsConstructor; +import lombok.Builder; +import lombok.Getter; +import lombok.NoArgsConstructor; + +import java.math.BigDecimal; +import java.time.LocalDateTime; +import java.util.UUID; + +@Entity +@Table(name = "transactions") +@Getter +@Builder +@NoArgsConstructor +@AllArgsConstructor +public class TransactionEntity { + @Id + @GeneratedValue(strategy = GenerationType.UUID) + private UUID transactionExternalId; + @Column(nullable = false) + private UUID accountExternalIdDebit; + @Column(nullable = false) + private UUID accountExternalIdCredit; + @Column(nullable = false) + private Integer transferTypeId; + @Column(nullable = false, precision = 15, scale = 2) + private BigDecimal value; + @Enumerated(EnumType.STRING) + @Column(nullable = false) + private TransactionStatus status; + @Column(nullable = false) + private LocalDateTime createdAt; +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionJpaRepository.java b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionJpaRepository.java new file mode 100644 index 0000000000..21ffbb8c0b --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionJpaRepository.java @@ -0,0 +1,11 @@ +package com.yape.transactionservice.Infrastructure.persistence.repositories; + +import com.yape.transactionservice.Infrastructure.persistence.entities.TransactionEntity; +import org.springframework.data.jpa.repository.JpaRepository; +import org.springframework.stereotype.Repository; + +import java.util.UUID; + +@Repository +public interface TransactionJpaRepository extends JpaRepository { +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionRepositoryImpl.java b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionRepositoryImpl.java new file mode 100644 index 0000000000..a97bc184cb --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionRepositoryImpl.java @@ -0,0 +1,27 @@ +package com.yape.transactionservice.Infrastructure.persistence.repositories; + +import com.yape.transactionservice.domain.models.Transaction; +import com.yape.transactionservice.domain.repostories.TransactionRepository; +import org.springframework.stereotype.Repository; + +import java.util.List; +import java.util.Optional; +import java.util.UUID; + +@Repository +public class TransactionRepositoryImpl implements TransactionRepository { + @Override + public Optional GetById(UUID id) { + return Optional.empty(); + } + + @Override + public Transaction Create(Transaction transaction) { + return null; + } + + @Override + public List GetAll() { + return List.of(); + } +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/TransactionServiceApplication.java b/transaction-service/src/main/java/com/yape/transactionservice/TransactionServiceApplication.java new file mode 100644 index 0000000000..2835f524f6 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/TransactionServiceApplication.java @@ -0,0 +1,13 @@ +package com.yape.transactionservice; + +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/transactionservice/application/abstractions/CreateTransactionUseCase.java b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/CreateTransactionUseCase.java new file mode 100644 index 0000000000..c59ea44e8f --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/CreateTransactionUseCase.java @@ -0,0 +1,8 @@ +package com.yape.transactionservice.application.abstractions; + +import com.yape.transactionservice.application.usecases.createtransaction.CreateTransactionRequest; +import com.yape.transactionservice.application.usecases.createtransaction.CreateTransactionResponse; + +public interface CreateTransactionUseCase { + CreateTransactionResponse execute(CreateTransactionRequest request); +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetTransactionUseCase.java b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetTransactionUseCase.java new file mode 100644 index 0000000000..ed6c2dc9c1 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetTransactionUseCase.java @@ -0,0 +1,8 @@ +package com.yape.transactionservice.application.abstractions; + +import com.yape.transactionservice.application.usecases.gettransaction.GetTransactionRequest; +import com.yape.transactionservice.application.usecases.gettransaction.GetTransactionResponse; + +public interface GetTransactionUseCase { + GetTransactionResponse execute(GetTransactionRequest request); +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java new file mode 100644 index 0000000000..29cbfea03b --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java @@ -0,0 +1,16 @@ +package com.yape.transactionservice.application.usecases.createtransaction; + +import com.yape.transactionservice.application.abstractions.CreateTransactionUseCase; +import com.yape.transactionservice.domain.enums.TransferType; +import org.springframework.stereotype.Service; + +@Service +public class CreateTransactionHandler implements CreateTransactionUseCase { + @Override + public CreateTransactionResponse execute(CreateTransactionRequest request) { + var ex = TransferType.EXTERNAL.getId(); + + + return null; + } +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionRequest.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionRequest.java new file mode 100644 index 0000000000..9a60fe7b9b --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionRequest.java @@ -0,0 +1,4 @@ +package com.yape.transactionservice.application.usecases.createtransaction; + +public class CreateTransactionRequest { +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionResponse.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionResponse.java new file mode 100644 index 0000000000..ef0772542b --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionResponse.java @@ -0,0 +1,4 @@ +package com.yape.transactionservice.application.usecases.createtransaction; + +public class CreateTransactionResponse { +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionHandler.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionHandler.java new file mode 100644 index 0000000000..3a9198755a --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionHandler.java @@ -0,0 +1,12 @@ +package com.yape.transactionservice.application.usecases.gettransaction; + +import com.yape.transactionservice.application.abstractions.GetTransactionUseCase; +import org.springframework.stereotype.Service; + +@Service +public class GetTransactionHandler implements GetTransactionUseCase { + @Override + public GetTransactionResponse execute(GetTransactionRequest request) { + return null; + } +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionRequest.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionRequest.java new file mode 100644 index 0000000000..ff0332956c --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionRequest.java @@ -0,0 +1,4 @@ +package com.yape.transactionservice.application.usecases.gettransaction; + +public class GetTransactionRequest { +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionResponse.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionResponse.java new file mode 100644 index 0000000000..9b3ac69a99 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionResponse.java @@ -0,0 +1,4 @@ +package com.yape.transactionservice.application.usecases.gettransaction; + +public class GetTransactionResponse { +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/enums/TransactionStatus.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/enums/TransactionStatus.java new file mode 100644 index 0000000000..2ac6ff6767 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/enums/TransactionStatus.java @@ -0,0 +1,5 @@ +package com.yape.transactionservice.domain.enums; + +public enum TransactionStatus { + PENDING, APPROVED, REJECTED +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/enums/TransferType.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/enums/TransferType.java new file mode 100644 index 0000000000..384eb8426c --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/enums/TransferType.java @@ -0,0 +1,24 @@ +package com.yape.transactionservice.domain.enums; + +import lombok.Getter; + +@Getter +public enum TransferType { + INTERNAL(1, "Internal Transfer"), + EXTERNAL(2, "External Transfer"); + + private final Integer id; + private final String name; + + TransferType(Integer id, String name) { + this.id = id; + this.name = name; + } + + public static TransferType getById(Integer id) { + for (TransferType type : values()) { + if (type.id.equals(id)) return type; + } + throw new IllegalArgumentException("TransferType not found: " + id); + } +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java new file mode 100644 index 0000000000..e4bc33cca2 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java @@ -0,0 +1,29 @@ +package com.yape.transactionservice.domain.models; + +import com.yape.transactionservice.domain.enums.TransactionStatus; +import lombok.Builder; +import lombok.Setter; + +import java.math.BigDecimal; +import java.time.LocalDateTime; +import java.util.UUID; + +@Setter +@Builder +public class Transaction { + private UUID transactionExternalId; + private UUID accountExternalIdDebit; + private UUID accountExternalIdCredit; + private Integer transferTypeId; + private BigDecimal value; + private TransactionStatus status; + private LocalDateTime createdAt; + + public void approve() { + this.status = TransactionStatus.APPROVED; + } + + public void reject() { + this.status = TransactionStatus.REJECTED; + } +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/repostories/TransactionRepository.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/repostories/TransactionRepository.java new file mode 100644 index 0000000000..4defdfcf00 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/repostories/TransactionRepository.java @@ -0,0 +1,15 @@ +package com.yape.transactionservice.domain.repostories; + +import com.yape.transactionservice.domain.models.Transaction; + +import java.util.List; +import java.util.Optional; +import java.util.UUID; + +public interface TransactionRepository { + Optional GetById(UUID id); + + Transaction Create(Transaction transaction); + + List GetAll(); +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/HealthController.java b/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/HealthController.java new file mode 100644 index 0000000000..d4b86441f4 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/HealthController.java @@ -0,0 +1,14 @@ +package com.yape.transactionservice.presentation.controllers; + +import org.springframework.web.bind.annotation.GetMapping; +import org.springframework.web.bind.annotation.RequestMapping; +import org.springframework.web.bind.annotation.RestController; + +@RestController +@RequestMapping("/api") +public class HealthController { + @GetMapping("/health") + public String health() { + return "Transaction Service is running!"; + } +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/TransactionController.java b/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/TransactionController.java new file mode 100644 index 0000000000..ae52b0784a --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/TransactionController.java @@ -0,0 +1,32 @@ +package com.yape.transactionservice.presentation.controllers; + +import com.yape.transactionservice.application.abstractions.CreateTransactionUseCase; +import com.yape.transactionservice.application.abstractions.GetTransactionUseCase; +import com.yape.transactionservice.application.usecases.createtransaction.CreateTransactionRequest; +import com.yape.transactionservice.application.usecases.createtransaction.CreateTransactionResponse; +import com.yape.transactionservice.application.usecases.gettransaction.GetTransactionRequest; +import com.yape.transactionservice.application.usecases.gettransaction.GetTransactionResponse; +import org.springframework.web.bind.annotation.*; + +@RestController +@RequestMapping("/transactions") +public class TransactionController { + private final CreateTransactionUseCase createTransactionUseCase; + private final GetTransactionUseCase getTransactionUseCase; + + public TransactionController(CreateTransactionUseCase createTransactionUseCase, + GetTransactionUseCase getTransactionUseCase) { + this.createTransactionUseCase = createTransactionUseCase; + this.getTransactionUseCase = getTransactionUseCase; + } + + @PostMapping + public CreateTransactionResponse CreateTransaction(@RequestBody CreateTransactionRequest request) { + return createTransactionUseCase.execute(request); + } + + @GetMapping + public GetTransactionResponse GetTransaction() { + return getTransactionUseCase.execute(new GetTransactionRequest()); + } +} diff --git a/transaction-service/src/main/resources/application.properties b/transaction-service/src/main/resources/application.properties new file mode 100644 index 0000000000..256aa3de02 --- /dev/null +++ b/transaction-service/src/main/resources/application.properties @@ -0,0 +1,11 @@ +spring.application.name=transaction-service +server.port=8080 +spring.datasource.url=jdbc:postgresql://localhost:5432/financial_db +spring.datasource.username=admin +spring.datasource.password=admin +# JPA / Hibernate +spring.jpa.hibernate.ddl-auto=update +spring.jpa.show-sql=true +spring.jpa.properties.hibernate.dialect=org.hibernate.dialect.PostgreSQLDialect +spring.kafka.bootstrap-servers=localhost:9092 +spring.autoconfigure.exclude=org.springframework.boot.autoconfigure.kafka.KafkaAutoConfiguration \ No newline at end of file diff --git a/transaction-service/src/test/java/com/yape/transactionservice/TransactionServiceApplicationTests.java b/transaction-service/src/test/java/com/yape/transactionservice/TransactionServiceApplicationTests.java new file mode 100644 index 0000000000..33748584c9 --- /dev/null +++ b/transaction-service/src/test/java/com/yape/transactionservice/TransactionServiceApplicationTests.java @@ -0,0 +1,13 @@ +package com.yape.transactionservice; + +import org.junit.jupiter.api.Test; +import org.springframework.boot.test.context.SpringBootTest; + +@SpringBootTest +class TransactionServiceApplicationTests { + + @Test + void contextLoads() { + } + +} From 3462db08ea083875e72f6229c113ab86e0001391 Mon Sep 17 00:00:00 2001 From: Ivan Perez <¨ivansperezt@gmail.com¨> Date: Sat, 28 Feb 2026 15:52:15 -0500 Subject: [PATCH 2/7] feat(transaction): get by id and create transaction - complete get by id transaction - complete create transaction with mapppers --- transaction-service/pom.xml | 6 +++ .../entities/TransactionEntity.java | 2 +- .../mappers/TransactionEntityMapper.java | 17 ++++++++ .../TransactionRepositoryImpl.java | 27 +++++++++--- .../GetByIdTransactionUseCase.java | 11 +++++ .../abstractions/GetTransactionUseCase.java | 4 +- .../CreateTransactionHandler.java | 23 +++++++++-- .../CreateTransactionMapper.java | 15 +++++++ .../CreateTransactionRequest.java | 27 +++++++++++- .../CreateTransactionResponse.java | 10 ++++- .../GetByIdTransactionHandler.java | 24 +++++++++++ .../GetByIdTransactionMapper.java | 14 +++++++ .../GetByIdTransactionRequest.java | 6 +++ .../GetByIdTransactionResponse.java | 19 +++++++++ .../gettransaction/GetTransactionHandler.java | 16 +++++++- .../gettransaction/GetTransactionMapper.java | 19 +++++++++ .../GetTransactionResponse.java | 20 ++++++++- .../domain/enums/TransferType.java | 9 ++++ .../domain/models/Transaction.java | 20 ++++++++- .../repostories/TransactionRepository.java | 6 +-- .../controllers/HealthController.java | 8 ++++ .../controllers/TransactionController.java | 26 ++++++++++-- .../GlobalExceptionHandler.java | 41 +++++++++++++++++++ .../src/main/resources/application.properties | 4 +- 24 files changed, 347 insertions(+), 27 deletions(-) create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/mappers/TransactionEntityMapper.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetByIdTransactionUseCase.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionMapper.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionHandler.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionMapper.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionRequest.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionResponse.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionMapper.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/presentation/exceptionhandlers/GlobalExceptionHandler.java diff --git a/transaction-service/pom.xml b/transaction-service/pom.xml index f6d82f2976..5e94a4232e 100644 --- a/transaction-service/pom.xml +++ b/transaction-service/pom.xml @@ -35,6 +35,12 @@ mapstruct 1.5.5.Final + + org.mapstruct + mapstruct-processor + 1.5.5.Final + provided + org.springframework.boot spring-boot-starter-data-jpa diff --git a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/entities/TransactionEntity.java b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/entities/TransactionEntity.java index ecf3eeffb2..27e6099191 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/entities/TransactionEntity.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/entities/TransactionEntity.java @@ -19,7 +19,7 @@ @AllArgsConstructor public class TransactionEntity { @Id - @GeneratedValue(strategy = GenerationType.UUID) + // @GeneratedValue(strategy = GenerationType.UUID) private UUID transactionExternalId; @Column(nullable = false) private UUID accountExternalIdDebit; diff --git a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/mappers/TransactionEntityMapper.java b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/mappers/TransactionEntityMapper.java new file mode 100644 index 0000000000..a160cb71f4 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/mappers/TransactionEntityMapper.java @@ -0,0 +1,17 @@ +package com.yape.transactionservice.Infrastructure.persistence.mappers; + +import com.yape.transactionservice.Infrastructure.persistence.entities.TransactionEntity; +import com.yape.transactionservice.domain.models.Transaction; +import org.mapstruct.Mapper; + +import java.util.List; + +@Mapper(componentModel = "spring") +public interface TransactionEntityMapper { + + TransactionEntity toEntity(Transaction transaction); + + Transaction toModel(TransactionEntity entity); + + List toModels(List transactionEntities); +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionRepositoryImpl.java b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionRepositoryImpl.java index a97bc184cb..64dd28fa40 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionRepositoryImpl.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionRepositoryImpl.java @@ -1,5 +1,6 @@ package com.yape.transactionservice.Infrastructure.persistence.repositories; +import com.yape.transactionservice.Infrastructure.persistence.mappers.TransactionEntityMapper; import com.yape.transactionservice.domain.models.Transaction; import com.yape.transactionservice.domain.repostories.TransactionRepository; import org.springframework.stereotype.Repository; @@ -10,18 +11,32 @@ @Repository public class TransactionRepositoryImpl implements TransactionRepository { + private final TransactionEntityMapper mapper; + private final TransactionJpaRepository transactionJpaRepository; + + public TransactionRepositoryImpl(TransactionEntityMapper mapper, TransactionJpaRepository transactionJpaRepository) { + this.mapper = mapper; + this.transactionJpaRepository = transactionJpaRepository; + } + @Override - public Optional GetById(UUID id) { - return Optional.empty(); + public Optional getById(UUID id) { + return transactionJpaRepository.findById(id) + .map(mapper::toModel); + } @Override - public Transaction Create(Transaction transaction) { - return null; + public Transaction create(Transaction transaction) { + var entity = mapper.toEntity(transaction); + var Transaction = transactionJpaRepository.save(entity); + + return mapper.toModel(Transaction); } @Override - public List GetAll() { - return List.of(); + public List getAll() { + var transactionEntities = transactionJpaRepository.findAll(); + return mapper.toModels(transactionEntities); } } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetByIdTransactionUseCase.java b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetByIdTransactionUseCase.java new file mode 100644 index 0000000000..8df9a374de --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetByIdTransactionUseCase.java @@ -0,0 +1,11 @@ +package com.yape.transactionservice.application.abstractions; + +import com.yape.transactionservice.application.usecases.getbyidtransaction.GetByIdTransactionRequest; +import com.yape.transactionservice.application.usecases.getbyidtransaction.GetByIdTransactionResponse; + +import java.util.Optional; + +public interface GetByIdTransactionUseCase { + + Optional execute(GetByIdTransactionRequest request); +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetTransactionUseCase.java b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetTransactionUseCase.java index ed6c2dc9c1..cbb3562f86 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetTransactionUseCase.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetTransactionUseCase.java @@ -3,6 +3,8 @@ import com.yape.transactionservice.application.usecases.gettransaction.GetTransactionRequest; import com.yape.transactionservice.application.usecases.gettransaction.GetTransactionResponse; +import java.util.List; + public interface GetTransactionUseCase { - GetTransactionResponse execute(GetTransactionRequest request); + List execute(GetTransactionRequest request); } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java index 29cbfea03b..5016230ad4 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java @@ -1,16 +1,31 @@ package com.yape.transactionservice.application.usecases.createtransaction; import com.yape.transactionservice.application.abstractions.CreateTransactionUseCase; -import com.yape.transactionservice.domain.enums.TransferType; +import com.yape.transactionservice.domain.models.Transaction; +import com.yape.transactionservice.domain.repostories.TransactionRepository; import org.springframework.stereotype.Service; @Service public class CreateTransactionHandler implements CreateTransactionUseCase { + private final CreateTransactionMapper mapper; + private final TransactionRepository repository; + + public CreateTransactionHandler(CreateTransactionMapper mapper, + TransactionRepository repository) { + this.mapper = mapper; + this.repository = repository; + } + @Override public CreateTransactionResponse execute(CreateTransactionRequest request) { - var ex = TransferType.EXTERNAL.getId(); - + Transaction transaction = Transaction.Create( + request.accountExternalIdDebit(), + request.accountExternalIdCredit(), + request.transferTypeId(), + request.value() + ); - return null; + var transactionCreated = repository.create(transaction); + return mapper.ToDto(transactionCreated); } } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionMapper.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionMapper.java new file mode 100644 index 0000000000..b59d3062f0 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionMapper.java @@ -0,0 +1,15 @@ +package com.yape.transactionservice.application.usecases.createtransaction; + +import com.yape.transactionservice.domain.models.Transaction; +import org.mapstruct.Mapper; +import org.mapstruct.Mapping; + +@Mapper(componentModel = "spring") +public interface CreateTransactionMapper { + @Mapping(target = "transactionExternalId", ignore = true) + @Mapping(target = "status", ignore = true) + @Mapping(target = "createdAt", ignore = true) + Transaction toModel(CreateTransactionRequest request); + + CreateTransactionResponse ToDto(Transaction model); +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionRequest.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionRequest.java index 9a60fe7b9b..8158f23dc9 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionRequest.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionRequest.java @@ -1,4 +1,29 @@ package com.yape.transactionservice.application.usecases.createtransaction; -public class CreateTransactionRequest { +import jakarta.validation.constraints.NotNull; +import jakarta.validation.constraints.Positive; + +import java.math.BigDecimal; +import java.util.UUID; + +public record CreateTransactionRequest(@NotNull(message = "accountExternalIdDebit is required") + UUID accountExternalIdDebit, + @NotNull(message = "accountExternalIdCredit is required") + UUID accountExternalIdCredit, + @NotNull(message = "tranferTypeId is required") + Integer transferTypeId, + @NotNull(message = "value is required") + @Positive(message = "value must be greater than 0") + BigDecimal value +) { } + + + + + + + + + + diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionResponse.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionResponse.java index ef0772542b..1b0ee84b5c 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionResponse.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionResponse.java @@ -1,4 +1,12 @@ package com.yape.transactionservice.application.usecases.createtransaction; -public class CreateTransactionResponse { +import java.math.BigDecimal; +import java.util.UUID; + +public record CreateTransactionResponse( + UUID transactionExternalId, + UUID accountExternalIdDebit, + UUID accountExternalIdCredit, + Integer transferTypeId, + BigDecimal value) { } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionHandler.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionHandler.java new file mode 100644 index 0000000000..c5f5b7b432 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionHandler.java @@ -0,0 +1,24 @@ +package com.yape.transactionservice.application.usecases.getbyidtransaction; + +import com.yape.transactionservice.application.abstractions.GetByIdTransactionUseCase; +import com.yape.transactionservice.domain.repostories.TransactionRepository; +import org.springframework.stereotype.Service; + +import java.util.Optional; + +@Service +public class GetByIdTransactionHandler implements GetByIdTransactionUseCase { + private final GetByIdTransactionMapper mapper; + private final TransactionRepository repository; + + public GetByIdTransactionHandler(GetByIdTransactionMapper mapper, TransactionRepository repository) { + this.mapper = mapper; + this.repository = repository; + } + + @Override + public Optional execute(GetByIdTransactionRequest request) { + var transaction = repository.getById(request.id()); + return transaction.map(mapper::toDTO); + } +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionMapper.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionMapper.java new file mode 100644 index 0000000000..c35c1feaa0 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionMapper.java @@ -0,0 +1,14 @@ +package com.yape.transactionservice.application.usecases.getbyidtransaction; + +import com.yape.transactionservice.domain.enums.TransferType; +import com.yape.transactionservice.domain.models.Transaction; +import org.mapstruct.Mapper; +import org.mapstruct.Mapping; + +@Mapper(componentModel = "spring", imports = {TransferType.class}) +public interface GetByIdTransactionMapper { + @Mapping(target = "transactionType", expression = "java(new GetTransactionResponse.TransactionTypeDto(TransferType.getNameById(transaction.getTransferTypeId())))") + @Mapping(target = "transactionStatus", expression = "java(new GetTransactionResponse.TransactionStatusDto(transaction.getStatus().name()))") + GetByIdTransactionResponse toDTO(Transaction transaction); + +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionRequest.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionRequest.java new file mode 100644 index 0000000000..216bf6c5d8 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionRequest.java @@ -0,0 +1,6 @@ +package com.yape.transactionservice.application.usecases.getbyidtransaction; + +import java.util.UUID; + +public record GetByIdTransactionRequest(UUID id) { +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionResponse.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionResponse.java new file mode 100644 index 0000000000..c38e12206f --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionResponse.java @@ -0,0 +1,19 @@ +package com.yape.transactionservice.application.usecases.getbyidtransaction; + +import com.yape.transactionservice.application.usecases.gettransaction.GetTransactionResponse; + +import java.math.BigDecimal; +import java.util.UUID; + +public record GetByIdTransactionResponse(UUID transactionExternalId, + UUID accountExternalIdDebit, + UUID accountExternalIdCredit, + GetTransactionResponse.TransactionTypeDto transactionType, + GetTransactionResponse.TransactionStatusDto transactionStatus, + BigDecimal value) { + public record TransactionTypeDto(String name) { + } + + public record TransactionStatusDto(String name) { + } +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionHandler.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionHandler.java index 3a9198755a..1696b3ff33 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionHandler.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionHandler.java @@ -1,12 +1,24 @@ package com.yape.transactionservice.application.usecases.gettransaction; import com.yape.transactionservice.application.abstractions.GetTransactionUseCase; +import com.yape.transactionservice.domain.repostories.TransactionRepository; import org.springframework.stereotype.Service; +import java.util.List; + @Service public class GetTransactionHandler implements GetTransactionUseCase { + private final GetTransactionMapper mapper; + private final TransactionRepository repository; + + public GetTransactionHandler(GetTransactionMapper mapper, TransactionRepository repository) { + this.mapper = mapper; + this.repository = repository; + } + @Override - public GetTransactionResponse execute(GetTransactionRequest request) { - return null; + public List execute(GetTransactionRequest request) { + var transactions = repository.getAll(); + return mapper.ToDTOs(transactions); } } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionMapper.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionMapper.java new file mode 100644 index 0000000000..05ebe2072c --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionMapper.java @@ -0,0 +1,19 @@ +package com.yape.transactionservice.application.usecases.gettransaction; + +import com.yape.transactionservice.domain.enums.TransferType; +import com.yape.transactionservice.domain.models.Transaction; +import org.mapstruct.Mapper; +import org.mapstruct.Mapping; + +import java.util.List; + +@Mapper(componentModel = "spring", imports = {TransferType.class}) +public interface GetTransactionMapper { + + @Mapping(target = "transactionType", expression = "java(new GetTransactionResponse.TransactionTypeDto(TransferType.getNameById(transaction.getTransferTypeId())))") + @Mapping(target = "transactionStatus", expression = "java(new GetTransactionResponse.TransactionStatusDto(transaction.getStatus().name()))") + GetTransactionResponse ToDTO(Transaction transaction); + + + List ToDTOs(List transactions); +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionResponse.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionResponse.java index 9b3ac69a99..7480342e45 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionResponse.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionResponse.java @@ -1,4 +1,22 @@ package com.yape.transactionservice.application.usecases.gettransaction; -public class GetTransactionResponse { +import java.math.BigDecimal; +import java.util.UUID; + +public record GetTransactionResponse(UUID transactionExternalId, + UUID accountExternalIdDebit, + UUID accountExternalIdCredit, + TransactionTypeDto transactionType, + TransactionStatusDto transactionStatus, + BigDecimal value) { + + + public record TransactionTypeDto(String name) { + } + + public record TransactionStatusDto(String name) { + } } + + + diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/enums/TransferType.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/enums/TransferType.java index 384eb8426c..2de7968588 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/domain/enums/TransferType.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/enums/TransferType.java @@ -21,4 +21,13 @@ public static TransferType getById(Integer id) { } throw new IllegalArgumentException("TransferType not found: " + id); } + + public static String getNameById(Integer id) { + for (TransferType type : values()) { + if (type.id.equals(id)) return type.name; + } + throw new IllegalArgumentException("TransferType not found: " + id); + } + + } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java index e4bc33cca2..6b4cb2eebb 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java @@ -1,15 +1,19 @@ package com.yape.transactionservice.domain.models; import com.yape.transactionservice.domain.enums.TransactionStatus; +import lombok.AllArgsConstructor; import lombok.Builder; -import lombok.Setter; +import lombok.Getter; +import lombok.NoArgsConstructor; import java.math.BigDecimal; import java.time.LocalDateTime; import java.util.UUID; -@Setter +@Getter @Builder +@NoArgsConstructor +@AllArgsConstructor public class Transaction { private UUID transactionExternalId; private UUID accountExternalIdDebit; @@ -19,6 +23,18 @@ public class Transaction { private TransactionStatus status; private LocalDateTime createdAt; + public static Transaction Create(UUID idDebit, UUID idCredit, Integer transferTypeId, BigDecimal value) { + return Transaction.builder().transactionExternalId(UUID.randomUUID()) + .accountExternalIdCredit(idCredit) + .accountExternalIdDebit(idDebit) + .transferTypeId(transferTypeId) + .value(value) + .status(TransactionStatus.PENDING) + .createdAt(LocalDateTime.now()) + .build(); + + } + public void approve() { this.status = TransactionStatus.APPROVED; } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/repostories/TransactionRepository.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/repostories/TransactionRepository.java index 4defdfcf00..05651c632a 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/domain/repostories/TransactionRepository.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/repostories/TransactionRepository.java @@ -7,9 +7,9 @@ import java.util.UUID; public interface TransactionRepository { - Optional GetById(UUID id); + Optional getById(UUID id); - Transaction Create(Transaction transaction); + Transaction create(Transaction transaction); - List GetAll(); + List getAll(); } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/HealthController.java b/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/HealthController.java index d4b86441f4..6b9a4e66f6 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/HealthController.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/HealthController.java @@ -1,8 +1,10 @@ package com.yape.transactionservice.presentation.controllers; +import org.springframework.http.HttpStatus; import org.springframework.web.bind.annotation.GetMapping; import org.springframework.web.bind.annotation.RequestMapping; import org.springframework.web.bind.annotation.RestController; +import org.springframework.web.server.ResponseStatusException; @RestController @RequestMapping("/api") @@ -11,4 +13,10 @@ public class HealthController { public String health() { return "Transaction Service is running!"; } + + @GetMapping("/test") + public String test() { + throw new ResponseStatusException(HttpStatus.NOT_FOUND, "NO VALIDO"); + + } } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/TransactionController.java b/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/TransactionController.java index ae52b0784a..a87748d1c9 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/TransactionController.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/TransactionController.java @@ -1,32 +1,50 @@ package com.yape.transactionservice.presentation.controllers; import com.yape.transactionservice.application.abstractions.CreateTransactionUseCase; +import com.yape.transactionservice.application.abstractions.GetByIdTransactionUseCase; import com.yape.transactionservice.application.abstractions.GetTransactionUseCase; import com.yape.transactionservice.application.usecases.createtransaction.CreateTransactionRequest; import com.yape.transactionservice.application.usecases.createtransaction.CreateTransactionResponse; +import com.yape.transactionservice.application.usecases.getbyidtransaction.GetByIdTransactionRequest; +import com.yape.transactionservice.application.usecases.getbyidtransaction.GetByIdTransactionResponse; import com.yape.transactionservice.application.usecases.gettransaction.GetTransactionRequest; import com.yape.transactionservice.application.usecases.gettransaction.GetTransactionResponse; +import jakarta.validation.Valid; +import org.springframework.http.HttpStatus; import org.springframework.web.bind.annotation.*; +import java.util.List; +import java.util.Optional; +import java.util.UUID; + @RestController -@RequestMapping("/transactions") +@RequestMapping("/api/transactions") public class TransactionController { private final CreateTransactionUseCase createTransactionUseCase; private final GetTransactionUseCase getTransactionUseCase; + private final GetByIdTransactionUseCase getByIdTransactionUseCase; public TransactionController(CreateTransactionUseCase createTransactionUseCase, - GetTransactionUseCase getTransactionUseCase) { + GetTransactionUseCase getTransactionUseCase, + GetByIdTransactionUseCase getByIdTransactionUseCase) { this.createTransactionUseCase = createTransactionUseCase; this.getTransactionUseCase = getTransactionUseCase; + this.getByIdTransactionUseCase = getByIdTransactionUseCase; } @PostMapping - public CreateTransactionResponse CreateTransaction(@RequestBody CreateTransactionRequest request) { + @ResponseStatus(HttpStatus.CREATED) + public CreateTransactionResponse createTransaction(@RequestBody @Valid CreateTransactionRequest request) { return createTransactionUseCase.execute(request); } @GetMapping - public GetTransactionResponse GetTransaction() { + public List getAll() { return getTransactionUseCase.execute(new GetTransactionRequest()); } + + @GetMapping("/{id}") + public Optional getById(@PathVariable UUID id) { + return getByIdTransactionUseCase.execute(new GetByIdTransactionRequest(id)); + } } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/presentation/exceptionhandlers/GlobalExceptionHandler.java b/transaction-service/src/main/java/com/yape/transactionservice/presentation/exceptionhandlers/GlobalExceptionHandler.java new file mode 100644 index 0000000000..6a16db9c63 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/presentation/exceptionhandlers/GlobalExceptionHandler.java @@ -0,0 +1,41 @@ +package com.yape.transactionservice.presentation.exceptionhandlers; + +import org.springframework.core.Ordered; +import org.springframework.core.annotation.Order; +import org.springframework.http.HttpStatus; +import org.springframework.http.ProblemDetail; +import org.springframework.http.ResponseEntity; +import org.springframework.http.converter.HttpMessageNotReadableException; +import org.springframework.web.bind.MethodArgumentNotValidException; +import org.springframework.web.bind.annotation.ExceptionHandler; +import org.springframework.web.bind.annotation.RestControllerAdvice; + +import java.util.HashMap; +import java.util.Map; + +@RestControllerAdvice +@Order(Ordered.HIGHEST_PRECEDENCE) +public class GlobalExceptionHandler { + @ExceptionHandler(MethodArgumentNotValidException.class) + public ProblemDetail handleValidationErrors( + MethodArgumentNotValidException ex) { + + Map errors = new HashMap<>(); + ex.getBindingResult().getFieldErrors() + .forEach(error -> errors.put(error.getField(), error.getDefaultMessage())); + + ProblemDetail problem = ProblemDetail.forStatus(HttpStatus.BAD_REQUEST); + problem.setTitle("Validation failed"); + problem.setDetail("One or more fields are invalid"); + problem.setProperty("errors", errors); + + return problem; + } + + @ExceptionHandler(HttpMessageNotReadableException.class) + public ResponseEntity> handleInvalidFormat(HttpMessageNotReadableException ex) { + + return ResponseEntity.badRequest() + .body(Map.of("error", "El JSON tiene un tipo de dato inválido")); + } +} diff --git a/transaction-service/src/main/resources/application.properties b/transaction-service/src/main/resources/application.properties index 256aa3de02..252093f191 100644 --- a/transaction-service/src/main/resources/application.properties +++ b/transaction-service/src/main/resources/application.properties @@ -7,5 +7,7 @@ spring.datasource.password=admin spring.jpa.hibernate.ddl-auto=update spring.jpa.show-sql=true spring.jpa.properties.hibernate.dialect=org.hibernate.dialect.PostgreSQLDialect +#kafka spring.kafka.bootstrap-servers=localhost:9092 -spring.autoconfigure.exclude=org.springframework.boot.autoconfigure.kafka.KafkaAutoConfiguration \ No newline at end of file +spring.autoconfigure.exclude=org.springframework.boot.autoconfigure.kafka.KafkaAutoConfiguration +spring.mvc.problemdetails.enabled=true From 20ade542827187d6e35c9a65307d8f58036644be Mon Sep 17 00:00:00 2001 From: Ivan Perez <¨ivansperezt@gmail.com¨> Date: Sat, 28 Feb 2026 19:21:26 -0500 Subject: [PATCH 3/7] feat(transaction): event implementation - emit event when creating transaction - interfaces for publishing and consuming events --- transaction-service/pom.xml | 4 ++-- .../TransactionServiceApplication.java | 2 ++ .../abstractions/TransactionEventListener.java | 10 ++++++++++ .../abstractions/TransactionEventPublisher.java | 8 ++++++++ .../CreateTransactionHandler.java | 15 ++++++++++++--- .../TransactionCreatedEventHandler.java | 15 +++++++++++++++ .../domain/events/TransactionCreatedEvent.java | 8 ++++++++ .../src/main/resources/application.properties | 11 +++++++---- 8 files changed, 64 insertions(+), 9 deletions(-) create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/TransactionEventListener.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/TransactionEventPublisher.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/TransactionCreatedEventHandler.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/domain/events/TransactionCreatedEvent.java diff --git a/transaction-service/pom.xml b/transaction-service/pom.xml index 5e94a4232e..88b76e8a72 100644 --- a/transaction-service/pom.xml +++ b/transaction-service/pom.xml @@ -46,8 +46,8 @@ spring-boot-starter-data-jpa - org.springframework.boot - spring-boot-starter-kafka + org.springframework.kafka + spring-kafka org.springframework.boot diff --git a/transaction-service/src/main/java/com/yape/transactionservice/TransactionServiceApplication.java b/transaction-service/src/main/java/com/yape/transactionservice/TransactionServiceApplication.java index 2835f524f6..5bc576e30d 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/TransactionServiceApplication.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/TransactionServiceApplication.java @@ -2,8 +2,10 @@ import org.springframework.boot.SpringApplication; import org.springframework.boot.autoconfigure.SpringBootApplication; +import org.springframework.scheduling.annotation.EnableAsync; @SpringBootApplication +@EnableAsync public class TransactionServiceApplication { public static void main(String[] args) { diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/TransactionEventListener.java b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/TransactionEventListener.java new file mode 100644 index 0000000000..dd9a921c90 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/TransactionEventListener.java @@ -0,0 +1,10 @@ +package com.yape.transactionservice.application.abstractions; + +import com.yape.transactionservice.domain.models.Transaction; + +public interface TransactionEventListener { + + void onTransactionStatusApproved(Transaction transaction); + + void onTransactionStatusRejected(Transaction transaction); +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/TransactionEventPublisher.java b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/TransactionEventPublisher.java new file mode 100644 index 0000000000..084ec8897e --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/TransactionEventPublisher.java @@ -0,0 +1,8 @@ +package com.yape.transactionservice.application.abstractions; + +import com.yape.transactionservice.domain.models.Transaction; + +public interface TransactionEventPublisher { + void publishTransactionCreated(Transaction transaction); + +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java index 5016230ad4..0649e08970 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java @@ -1,19 +1,24 @@ package com.yape.transactionservice.application.usecases.createtransaction; import com.yape.transactionservice.application.abstractions.CreateTransactionUseCase; +import com.yape.transactionservice.domain.events.TransactionCreatedEvent; import com.yape.transactionservice.domain.models.Transaction; import com.yape.transactionservice.domain.repostories.TransactionRepository; +import org.springframework.context.ApplicationEventPublisher; import org.springframework.stereotype.Service; @Service public class CreateTransactionHandler implements CreateTransactionUseCase { private final CreateTransactionMapper mapper; private final TransactionRepository repository; + private final ApplicationEventPublisher eventPublisher; public CreateTransactionHandler(CreateTransactionMapper mapper, - TransactionRepository repository) { + TransactionRepository repository, + ApplicationEventPublisher eventPublisher) { this.mapper = mapper; this.repository = repository; + this.eventPublisher = eventPublisher; } @Override @@ -25,7 +30,11 @@ public CreateTransactionResponse execute(CreateTransactionRequest request) { request.value() ); - var transactionCreated = repository.create(transaction); - return mapper.ToDto(transactionCreated); + var saved = repository.create(transaction); + + eventPublisher.publishEvent(new TransactionCreatedEvent( + saved.getTransactionExternalId(), + saved.getValue())); + return mapper.ToDto(saved); } } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/TransactionCreatedEventHandler.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/TransactionCreatedEventHandler.java new file mode 100644 index 0000000000..474bd20622 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/TransactionCreatedEventHandler.java @@ -0,0 +1,15 @@ +package com.yape.transactionservice.application.usecases.createtransaction; + +import com.yape.transactionservice.domain.events.TransactionCreatedEvent; +import org.springframework.context.event.EventListener; +import org.springframework.scheduling.annotation.Async; +import org.springframework.stereotype.Component; + +@Component +public class TransactionCreatedEventHandler { + @Async + @EventListener + public void handle(TransactionCreatedEvent event) { + System.out.println("Transaction created: " + event.transactionId()); + } +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/events/TransactionCreatedEvent.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/events/TransactionCreatedEvent.java new file mode 100644 index 0000000000..8b2cc4b5f7 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/events/TransactionCreatedEvent.java @@ -0,0 +1,8 @@ +package com.yape.transactionservice.domain.events; + +import java.math.BigDecimal; +import java.util.UUID; + +public record TransactionCreatedEvent(UUID transactionId, + BigDecimal value) { +} diff --git a/transaction-service/src/main/resources/application.properties b/transaction-service/src/main/resources/application.properties index 252093f191..32acaa47ef 100644 --- a/transaction-service/src/main/resources/application.properties +++ b/transaction-service/src/main/resources/application.properties @@ -1,13 +1,16 @@ spring.application.name=transaction-service server.port=8080 -spring.datasource.url=jdbc:postgresql://localhost:5432/financial_db -spring.datasource.username=admin -spring.datasource.password=admin +spring.datasource.url=jdbc:postgresql://localhost:5432/postgres +spring.datasource.username=postgres +spring.datasource.password=postgres # JPA / Hibernate spring.jpa.hibernate.ddl-auto=update spring.jpa.show-sql=true spring.jpa.properties.hibernate.dialect=org.hibernate.dialect.PostgreSQLDialect +spring.datasource.hikari.maximum-pool-size=20 +spring.datasource.hikari.minimum-idle=5 #kafka spring.kafka.bootstrap-servers=localhost:9092 -spring.autoconfigure.exclude=org.springframework.boot.autoconfigure.kafka.KafkaAutoConfiguration +#kspring.autoconfigure.exclude=org.springframework.boot.autoconfigure.kafka.KafkaAutoConfiguration spring.mvc.problemdetails.enabled=true + From 2bb5ffe68aaea528bb148fd4a371df5dc968b70c Mon Sep 17 00:00:00 2001 From: Ivan Perez Date: Sat, 28 Feb 2026 23:54:40 -0500 Subject: [PATCH 4/7] feat(kafka): publisher and consumer complete - config consumer and producer en properties - publisher transaction created implemented - consumer transaction validation implemented --- transaction-service/pom.xml | 12 +++++++-- .../KafkaTransactionEventListener.java | 27 +++++++++++++++++++ .../KafkaTransactionEventPublisher.java | 21 +++++++++++++++ .../TransactionRepositoryImpl.java | 8 +++--- .../TransactionServiceApplication.java | 1 - .../TransactionEventListener.java | 3 +-- .../CreateTransactionHandler.java | 9 +++---- .../CreateTransactionMapper.java | 5 ---- .../TransactionCreatedEventHandler.java | 9 ++++++- .../events/TransactionCreatedEvent.java | 6 ++--- .../domain/models/Transaction.java | 18 ++++++++----- .../repostories/TransactionRepository.java | 2 +- .../src/main/resources/application.properties | 13 +++++++-- 13 files changed, 101 insertions(+), 33 deletions(-) create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/kafka/consumers/KafkaTransactionEventListener.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/kafka/producers/KafkaTransactionEventPublisher.java diff --git a/transaction-service/pom.xml b/transaction-service/pom.xml index 88b76e8a72..01bcee8a8c 100644 --- a/transaction-service/pom.xml +++ b/transaction-service/pom.xml @@ -46,8 +46,16 @@ spring-boot-starter-data-jpa - org.springframework.kafka - spring-kafka + com.fasterxml.jackson.core + jackson-databind + + + com.fasterxml.jackson.datatype + jackson-datatype-jsr310 + + + org.springframework.boot + spring-boot-starter-kafka org.springframework.boot diff --git a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/kafka/consumers/KafkaTransactionEventListener.java b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/kafka/consumers/KafkaTransactionEventListener.java new file mode 100644 index 0000000000..22185c5c6d --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/kafka/consumers/KafkaTransactionEventListener.java @@ -0,0 +1,27 @@ +package com.yape.transactionservice.Infrastructure.kafka.consumers; + +import com.yape.transactionservice.application.abstractions.TransactionEventListener; +import com.yape.transactionservice.domain.models.Transaction; +import com.yape.transactionservice.domain.repostories.TransactionRepository; +import org.springframework.kafka.annotation.KafkaListener; +import org.springframework.stereotype.Component; + +@Component +public class KafkaTransactionEventListener implements TransactionEventListener { + private final TransactionRepository transactionRepository; + + public KafkaTransactionEventListener(TransactionRepository transactionRepository) { + this.transactionRepository = transactionRepository; + } + + @Override + @KafkaListener(topics = "transaction.status") + public void onTransactionStatusUpdated(Transaction transaction) { + transactionRepository + .getById(transaction.getTransactionExternalId()) + .ifPresent(existing -> { + existing.changeStatus(transaction.getStatus()); + transactionRepository.save(existing); + }); + } +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/kafka/producers/KafkaTransactionEventPublisher.java b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/kafka/producers/KafkaTransactionEventPublisher.java new file mode 100644 index 0000000000..91c1ca984e --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/kafka/producers/KafkaTransactionEventPublisher.java @@ -0,0 +1,21 @@ +package com.yape.transactionservice.Infrastructure.kafka.producers; + +import com.yape.transactionservice.application.abstractions.TransactionEventPublisher; +import com.yape.transactionservice.domain.models.Transaction; +import org.springframework.kafka.core.KafkaTemplate; +import org.springframework.stereotype.Component; + +@Component +public class KafkaTransactionEventPublisher implements TransactionEventPublisher { + + private final KafkaTemplate kafkaTemplate; + + public KafkaTransactionEventPublisher(KafkaTemplate kafkaTemplate) { + this.kafkaTemplate = kafkaTemplate; + } + + @Override + public void publishTransactionCreated(Transaction transaction) { + kafkaTemplate.send("transaction.created", transaction.getTransactionExternalId().toString(), transaction); + } +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionRepositoryImpl.java b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionRepositoryImpl.java index 64dd28fa40..4d5b874135 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionRepositoryImpl.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/persistence/repositories/TransactionRepositoryImpl.java @@ -14,7 +14,9 @@ public class TransactionRepositoryImpl implements TransactionRepository { private final TransactionEntityMapper mapper; private final TransactionJpaRepository transactionJpaRepository; - public TransactionRepositoryImpl(TransactionEntityMapper mapper, TransactionJpaRepository transactionJpaRepository) { + public TransactionRepositoryImpl(TransactionEntityMapper mapper, + TransactionJpaRepository transactionJpaRepository + ) { this.mapper = mapper; this.transactionJpaRepository = transactionJpaRepository; } @@ -22,12 +24,12 @@ public TransactionRepositoryImpl(TransactionEntityMapper mapper, TransactionJpaR @Override public Optional getById(UUID id) { return transactionJpaRepository.findById(id) - .map(mapper::toModel); + .map(mapper::toModel); } @Override - public Transaction create(Transaction transaction) { + public Transaction save(Transaction transaction) { var entity = mapper.toEntity(transaction); var Transaction = transactionJpaRepository.save(entity); diff --git a/transaction-service/src/main/java/com/yape/transactionservice/TransactionServiceApplication.java b/transaction-service/src/main/java/com/yape/transactionservice/TransactionServiceApplication.java index 5bc576e30d..47aa9b442c 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/TransactionServiceApplication.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/TransactionServiceApplication.java @@ -7,7 +7,6 @@ @SpringBootApplication @EnableAsync public class TransactionServiceApplication { - public static void main(String[] args) { SpringApplication.run(TransactionServiceApplication.class, args); } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/TransactionEventListener.java b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/TransactionEventListener.java index dd9a921c90..fda21c3438 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/TransactionEventListener.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/TransactionEventListener.java @@ -4,7 +4,6 @@ public interface TransactionEventListener { - void onTransactionStatusApproved(Transaction transaction); + void onTransactionStatusUpdated(Transaction transaction); - void onTransactionStatusRejected(Transaction transaction); } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java index 0649e08970..e481bb50b9 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionHandler.java @@ -15,7 +15,8 @@ public class CreateTransactionHandler implements CreateTransactionUseCase { public CreateTransactionHandler(CreateTransactionMapper mapper, TransactionRepository repository, - ApplicationEventPublisher eventPublisher) { + ApplicationEventPublisher eventPublisher + ) { this.mapper = mapper; this.repository = repository; this.eventPublisher = eventPublisher; @@ -30,11 +31,9 @@ public CreateTransactionResponse execute(CreateTransactionRequest request) { request.value() ); - var saved = repository.create(transaction); + var saved = repository.save(transaction); - eventPublisher.publishEvent(new TransactionCreatedEvent( - saved.getTransactionExternalId(), - saved.getValue())); + eventPublisher.publishEvent(new TransactionCreatedEvent(saved)); return mapper.ToDto(saved); } } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionMapper.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionMapper.java index b59d3062f0..fd7cba26ef 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionMapper.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionMapper.java @@ -2,14 +2,9 @@ import com.yape.transactionservice.domain.models.Transaction; import org.mapstruct.Mapper; -import org.mapstruct.Mapping; @Mapper(componentModel = "spring") public interface CreateTransactionMapper { - @Mapping(target = "transactionExternalId", ignore = true) - @Mapping(target = "status", ignore = true) - @Mapping(target = "createdAt", ignore = true) - Transaction toModel(CreateTransactionRequest request); CreateTransactionResponse ToDto(Transaction model); } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/TransactionCreatedEventHandler.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/TransactionCreatedEventHandler.java index 474bd20622..aad0e4f587 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/TransactionCreatedEventHandler.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/TransactionCreatedEventHandler.java @@ -1,5 +1,6 @@ package com.yape.transactionservice.application.usecases.createtransaction; +import com.yape.transactionservice.application.abstractions.TransactionEventPublisher; import com.yape.transactionservice.domain.events.TransactionCreatedEvent; import org.springframework.context.event.EventListener; import org.springframework.scheduling.annotation.Async; @@ -7,9 +8,15 @@ @Component public class TransactionCreatedEventHandler { + private final TransactionEventPublisher transactionEventPublisher; + + public TransactionCreatedEventHandler(TransactionEventPublisher transactionEventPublisher) { + this.transactionEventPublisher = transactionEventPublisher; + } + @Async @EventListener public void handle(TransactionCreatedEvent event) { - System.out.println("Transaction created: " + event.transactionId()); + transactionEventPublisher.publishTransactionCreated(event.transaction()); } } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/events/TransactionCreatedEvent.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/events/TransactionCreatedEvent.java index 8b2cc4b5f7..8ce057b0e7 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/domain/events/TransactionCreatedEvent.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/events/TransactionCreatedEvent.java @@ -1,8 +1,6 @@ package com.yape.transactionservice.domain.events; -import java.math.BigDecimal; -import java.util.UUID; +import com.yape.transactionservice.domain.models.Transaction; -public record TransactionCreatedEvent(UUID transactionId, - BigDecimal value) { +public record TransactionCreatedEvent(Transaction transaction) { } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java index 6b4cb2eebb..2e85d5465b 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java @@ -25,13 +25,13 @@ public class Transaction { public static Transaction Create(UUID idDebit, UUID idCredit, Integer transferTypeId, BigDecimal value) { return Transaction.builder().transactionExternalId(UUID.randomUUID()) - .accountExternalIdCredit(idCredit) - .accountExternalIdDebit(idDebit) - .transferTypeId(transferTypeId) - .value(value) - .status(TransactionStatus.PENDING) - .createdAt(LocalDateTime.now()) - .build(); + .accountExternalIdCredit(idCredit) + .accountExternalIdDebit(idDebit) + .transferTypeId(transferTypeId) + .value(value) + .status(TransactionStatus.PENDING) + .createdAt(LocalDateTime.now()) + .build(); } @@ -42,4 +42,8 @@ public void approve() { public void reject() { this.status = TransactionStatus.REJECTED; } + + public void changeStatus(TransactionStatus status) { + this.status = status; + } } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/repostories/TransactionRepository.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/repostories/TransactionRepository.java index 05651c632a..abd945d98d 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/domain/repostories/TransactionRepository.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/repostories/TransactionRepository.java @@ -9,7 +9,7 @@ public interface TransactionRepository { Optional getById(UUID id); - Transaction create(Transaction transaction); + Transaction save(Transaction transaction); List getAll(); } diff --git a/transaction-service/src/main/resources/application.properties b/transaction-service/src/main/resources/application.properties index 32acaa47ef..bf4bdb64fa 100644 --- a/transaction-service/src/main/resources/application.properties +++ b/transaction-service/src/main/resources/application.properties @@ -9,8 +9,17 @@ spring.jpa.show-sql=true spring.jpa.properties.hibernate.dialect=org.hibernate.dialect.PostgreSQLDialect spring.datasource.hikari.maximum-pool-size=20 spring.datasource.hikari.minimum-idle=5 -#kafka +# kafka spring.kafka.bootstrap-servers=localhost:9092 -#kspring.autoconfigure.exclude=org.springframework.boot.autoconfigure.kafka.KafkaAutoConfiguration +# producer +spring.kafka.producer.key-serializer=org.apache.kafka.common.serialization.StringSerializer +spring.kafka.producer.value-serializer=org.springframework.kafka.support.serializer.JsonSerializer +# consumer +spring.kafka.consumer.group-id=transaction-service +spring.kafka.consumer.key-deserializer=org.apache.kafka.common.serialization.StringDeserializer +spring.kafka.consumer.value-deserializer=org.springframework.kafka.support.serializer.JsonDeserializer +spring.kafka.consumer.properties.spring.json.trusted.packages=* +spring.kafka.consumer.properties.spring.json.value.default.type=com.yape.transactionservice.domain.models.Transaction +#problem spring.mvc.problemdetails.enabled=true From 0f0e731c790dfc9b4d1f2a368e6c2729f8896a1d Mon Sep 17 00:00:00 2001 From: Ivan Perez Date: Sun, 1 Mar 2026 09:14:42 -0500 Subject: [PATCH 5/7] feat: anti-fraud service configured --- anti-fraud-service/.gitattributes | 2 + anti-fraud-service/.gitignore | 33 ++ .../.mvn/wrapper/maven-wrapper.properties | 3 + anti-fraud-service/mvnw | 295 ++++++++++++++++++ anti-fraud-service/mvnw.cmd | 189 +++++++++++ anti-fraud-service/pom.xml | 62 ++++ .../AntiFraudServiceApplication.java | 13 + .../consumers/TransactionCreatedConsumer.java | 35 +++ .../enums/TransactionStatus.java | 5 + .../models/TransactionEvent.java | 11 + .../src/main/resources/application.properties | 12 + .../AntiFraudServiceApplicationTests.java | 13 + .../KafkaTransactionEventPublisher.java | 2 +- .../src/main/resources/application.properties | 1 + 14 files changed, 675 insertions(+), 1 deletion(-) create mode 100644 anti-fraud-service/.gitattributes create mode 100644 anti-fraud-service/.gitignore create mode 100644 anti-fraud-service/.mvn/wrapper/maven-wrapper.properties create mode 100644 anti-fraud-service/mvnw create mode 100644 anti-fraud-service/mvnw.cmd create mode 100644 anti-fraud-service/pom.xml create mode 100644 anti-fraud-service/src/main/java/com/yape/antifraudservice/AntiFraudServiceApplication.java create mode 100644 anti-fraud-service/src/main/java/com/yape/antifraudservice/consumers/TransactionCreatedConsumer.java create mode 100644 anti-fraud-service/src/main/java/com/yape/antifraudservice/enums/TransactionStatus.java create mode 100644 anti-fraud-service/src/main/java/com/yape/antifraudservice/models/TransactionEvent.java create mode 100644 anti-fraud-service/src/main/resources/application.properties create mode 100644 anti-fraud-service/src/test/java/com/yape/antifraudservice/AntiFraudServiceApplicationTests.java diff --git a/anti-fraud-service/.gitattributes b/anti-fraud-service/.gitattributes new file mode 100644 index 0000000000..3b41682ac5 --- /dev/null +++ b/anti-fraud-service/.gitattributes @@ -0,0 +1,2 @@ +/mvnw text eol=lf +*.cmd text eol=crlf diff --git a/anti-fraud-service/.gitignore b/anti-fraud-service/.gitignore new file mode 100644 index 0000000000..667aaef0c8 --- /dev/null +++ b/anti-fraud-service/.gitignore @@ -0,0 +1,33 @@ +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/ diff --git a/anti-fraud-service/.mvn/wrapper/maven-wrapper.properties b/anti-fraud-service/.mvn/wrapper/maven-wrapper.properties new file mode 100644 index 0000000000..8dea6c227c --- /dev/null +++ b/anti-fraud-service/.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/anti-fraud-service/mvnw b/anti-fraud-service/mvnw new file mode 100644 index 0000000000..bd8896bf22 --- /dev/null +++ b/anti-fraud-service/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/anti-fraud-service/mvnw.cmd b/anti-fraud-service/mvnw.cmd new file mode 100644 index 0000000000..92450f9327 --- /dev/null +++ b/anti-fraud-service/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/anti-fraud-service/pom.xml b/anti-fraud-service/pom.xml new file mode 100644 index 0000000000..7f52a4df66 --- /dev/null +++ b/anti-fraud-service/pom.xml @@ -0,0 +1,62 @@ + + + 4.0.0 + + org.springframework.boot + spring-boot-starter-parent + 4.0.3 + + + com.yape + anti-fraud-service + 0.0.1-SNAPSHOT + anti-fraud-service + anti-fraud-service + + + + + + + + + + + + + + + 21 + + + + com.fasterxml.jackson.core + jackson-databind + + + com.fasterxml.jackson.datatype + jackson-datatype-jsr310 + + + org.springframework.boot + spring-boot-starter-kafka + + + + org.springframework.boot + spring-boot-starter-kafka-test + test + + + + + + + org.springframework.boot + spring-boot-maven-plugin + + + + + diff --git a/anti-fraud-service/src/main/java/com/yape/antifraudservice/AntiFraudServiceApplication.java b/anti-fraud-service/src/main/java/com/yape/antifraudservice/AntiFraudServiceApplication.java new file mode 100644 index 0000000000..e070376356 --- /dev/null +++ b/anti-fraud-service/src/main/java/com/yape/antifraudservice/AntiFraudServiceApplication.java @@ -0,0 +1,13 @@ +package com.yape.antifraudservice; + +import org.springframework.boot.SpringApplication; +import org.springframework.boot.autoconfigure.SpringBootApplication; + +@SpringBootApplication +public class AntiFraudServiceApplication { + + public static void main(String[] args) { + SpringApplication.run(AntiFraudServiceApplication.class, args); + } + +} diff --git a/anti-fraud-service/src/main/java/com/yape/antifraudservice/consumers/TransactionCreatedConsumer.java b/anti-fraud-service/src/main/java/com/yape/antifraudservice/consumers/TransactionCreatedConsumer.java new file mode 100644 index 0000000000..b52664ccae --- /dev/null +++ b/anti-fraud-service/src/main/java/com/yape/antifraudservice/consumers/TransactionCreatedConsumer.java @@ -0,0 +1,35 @@ +package com.yape.antifraudservice.consumers; + +import com.yape.antifraudservice.enums.TransactionStatus; +import com.yape.antifraudservice.models.TransactionEvent; +import org.springframework.kafka.annotation.KafkaListener; +import org.springframework.kafka.core.KafkaTemplate; +import org.springframework.stereotype.Component; + +import java.math.BigDecimal; + +@Component +public class TransactionCreatedConsumer { + private final KafkaTemplate kafkaTemplate; + + public TransactionCreatedConsumer(KafkaTemplate kafkaTemplate) { + this.kafkaTemplate = kafkaTemplate; + } + + @KafkaListener(topics = "transaction.created") + public void onTransactionCreated(TransactionEvent transaction) { + TransactionStatus status = transaction.value().compareTo(new BigDecimal("1000")) > 0 + ? TransactionStatus.REJECTED + : TransactionStatus.APPROVED; + + TransactionEvent response = new TransactionEvent( + transaction.transactionExternalId(), + status, + transaction.value() + ); + + kafkaTemplate.send("transaction.status", transaction.transactionExternalId().toString(), response); + + } + +} diff --git a/anti-fraud-service/src/main/java/com/yape/antifraudservice/enums/TransactionStatus.java b/anti-fraud-service/src/main/java/com/yape/antifraudservice/enums/TransactionStatus.java new file mode 100644 index 0000000000..d0d0d042bc --- /dev/null +++ b/anti-fraud-service/src/main/java/com/yape/antifraudservice/enums/TransactionStatus.java @@ -0,0 +1,5 @@ +package com.yape.antifraudservice.enums; + +public enum TransactionStatus { + PENDING, APPROVED, REJECTED +} diff --git a/anti-fraud-service/src/main/java/com/yape/antifraudservice/models/TransactionEvent.java b/anti-fraud-service/src/main/java/com/yape/antifraudservice/models/TransactionEvent.java new file mode 100644 index 0000000000..0fa3ec256b --- /dev/null +++ b/anti-fraud-service/src/main/java/com/yape/antifraudservice/models/TransactionEvent.java @@ -0,0 +1,11 @@ +package com.yape.antifraudservice.models; + +import com.yape.antifraudservice.enums.TransactionStatus; +import org.apache.kafka.common.protocol.types.Field; + +import java.math.BigDecimal; +import java.util.UUID; + + +public record TransactionEvent(UUID transactionExternalId, TransactionStatus status, BigDecimal value) { +} diff --git a/anti-fraud-service/src/main/resources/application.properties b/anti-fraud-service/src/main/resources/application.properties new file mode 100644 index 0000000000..de1be6a8f7 --- /dev/null +++ b/anti-fraud-service/src/main/resources/application.properties @@ -0,0 +1,12 @@ +spring.application.name=anti-fraud-service +#kafka +spring.kafka.bootstrap-servers=localhost:9092 +spring.kafka.consumer.group-id=anti-fraud-service +spring.kafka.consumer.key-deserializer=org.apache.kafka.common.serialization.StringDeserializer +spring.kafka.consumer.value-deserializer=org.springframework.kafka.support.serializer.JsonDeserializer +spring.kafka.consumer.properties.spring.json.trusted.packages=* +spring.kafka.producer.key-serializer=org.apache.kafka.common.serialization.StringSerializer +spring.kafka.producer.value-serializer=org.springframework.kafka.support.serializer.JsonSerializer +#kafka headers +spring.kafka.consumer.properties.spring.json.value.default.type=com.yape.antifraudservice.models.TransactionEvent +spring.kafka.consumer.properties.spring.json.use.type.headers=false \ No newline at end of file diff --git a/anti-fraud-service/src/test/java/com/yape/antifraudservice/AntiFraudServiceApplicationTests.java b/anti-fraud-service/src/test/java/com/yape/antifraudservice/AntiFraudServiceApplicationTests.java new file mode 100644 index 0000000000..7a99f1ddc7 --- /dev/null +++ b/anti-fraud-service/src/test/java/com/yape/antifraudservice/AntiFraudServiceApplicationTests.java @@ -0,0 +1,13 @@ +package com.yape.antifraudservice; + +import org.junit.jupiter.api.Test; +import org.springframework.boot.test.context.SpringBootTest; + +@SpringBootTest +class AntiFraudServiceApplicationTests { + + @Test + void contextLoads() { + } + +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/kafka/producers/KafkaTransactionEventPublisher.java b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/kafka/producers/KafkaTransactionEventPublisher.java index 91c1ca984e..aa2583d996 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/kafka/producers/KafkaTransactionEventPublisher.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/Infrastructure/kafka/producers/KafkaTransactionEventPublisher.java @@ -7,7 +7,7 @@ @Component public class KafkaTransactionEventPublisher implements TransactionEventPublisher { - + private final KafkaTemplate kafkaTemplate; public KafkaTransactionEventPublisher(KafkaTemplate kafkaTemplate) { diff --git a/transaction-service/src/main/resources/application.properties b/transaction-service/src/main/resources/application.properties index bf4bdb64fa..5f1870ca60 100644 --- a/transaction-service/src/main/resources/application.properties +++ b/transaction-service/src/main/resources/application.properties @@ -20,6 +20,7 @@ spring.kafka.consumer.key-deserializer=org.apache.kafka.common.serialization.Str spring.kafka.consumer.value-deserializer=org.springframework.kafka.support.serializer.JsonDeserializer spring.kafka.consumer.properties.spring.json.trusted.packages=* spring.kafka.consumer.properties.spring.json.value.default.type=com.yape.transactionservice.domain.models.Transaction +spring.kafka.consumer.properties.spring.json.use.type.headers=false #problem spring.mvc.problemdetails.enabled=true From abad5dd6c34b9bb09cb9ae4800c32d612cfacd07 Mon Sep 17 00:00:00 2001 From: Ivan Perez Date: Sun, 1 Mar 2026 10:42:13 -0500 Subject: [PATCH 6/7] feat: add swagger and config - readme documentation --- README.md | 142 +++++++++++------- .../consumers/TransactionCreatedConsumer.java | 2 +- transaction-service/pom.xml | 5 + .../GetByIdTransactionUseCase.java | 2 +- .../CreateTransactionRequest.java | 7 +- .../GetByIdTransactionHandler.java | 11 +- .../GetByIdTransactionMapper.java | 4 +- .../GetByIdTransactionResponse.java | 4 +- .../gettransaction/GetTransactionRequest.java | 2 +- .../domain/exceptions/DomainException.java | 7 + .../TransactionNotFoundException.java | 9 ++ .../domain/models/Transaction.java | 23 ++- .../controllers/HealthController.java | 6 - .../controllers/TransactionController.java | 2 +- .../GlobalExceptionHandler.java | 21 +++ .../src/main/resources/application.properties | 1 + 16 files changed, 171 insertions(+), 77 deletions(-) create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/domain/exceptions/DomainException.java create mode 100644 transaction-service/src/main/java/com/yape/transactionservice/domain/exceptions/TransactionNotFoundException.java diff --git a/README.md b/README.md index b067a71026..303dca34e7 100644 --- a/README.md +++ b/README.md @@ -1,82 +1,112 @@ -# Yape Code Challenge :rocket: +# Yape Transaction Service -Our code challenge will let you marvel us with your Jedi coding skills :smile:. +## Overview -Don't forget that the proper way to submit your work is to fork the repo and create a PR :wink: ... have fun !! +Event-driven microservices system for processing financial transactions with anti-fraud validation using Apache Kafka. -- [Problem](#problem) -- [Tech Stack](#tech_stack) -- [Send us your challenge](#send_us_your_challenge) +## Architecture -# Problem +The project consists of two microservices: -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: +**transaction-service** — Core service built with Onion Architecture (DDD). Handles transaction creation, persistence, and status updates. -
    -
  1. pending
    2. -
  2. approved
    3. -
  3. rejected
    4. -
+**anti-fraud-service** — Lightweight worker service that consumes transaction events, validates fraud rules, and publishes the result. -Every transaction with a value greater than 1000 should be rejected. +### Event Flow -```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)] +``` +1. Client creates a transaction via REST API +2. Transaction is saved in PostgreSQL with status PENDING +3. transaction-service publishes event to Kafka topic "transaction.created" +4. anti-fraud-service consumes the event and validates: + - Amount <= 1000 → APPROVED + - Amount > 1000 → REJECTED +5. anti-fraud-service publishes result to Kafka topic "transaction.status" +6. transaction-service consumes the event and updates the status in the database ``` -# Tech Stack +### Project Structure -
    -
  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/ → Onion Architecture (DDD) +├── domain/ → Entities, enums, repository interfaces +├── application/ → Use cases, ports (interfaces), DTOs +└── infrastructure/ → JPA, Kafka producers/consumers +└── Presentation Layer/ → REST API, Spring controllers, exceptions handlers + +anti-fraud-service/ → Simple structure +├── models/ → TransactionEvent, TransactionStatus enum +└── consumers/ → Kafka consumer + producer logic +``` -We do provide a `Dockerfile` to help you get started with a dev environment. +## Tech Stack -You must have two resources: +- Java 21 +- Spring Boot 4.x +- Apache Kafka (Confluent 5.5.3) +- PostgreSQL 14 +- MapStruct +- Lombok +- Springdoc OpenAPI (Swagger) -1. Resource to create a transaction that must containt: +## Prerequisites -```json -{ - "accountExternalIdDebit": "Guid", - "accountExternalIdCredit": "Guid", - "tranferTypeId": 1, - "value": 120 -} +- Java 21 +- Docker & Docker Compose +- Maven + +## Getting Started + +### 1. Start infrastructure + +```bash +docker-compose up -d ``` -2. Resource to retrieve a transaction +This starts PostgreSQL, Zookeeper, and Kafka. + +### 2. Run transaction-service + +```bash +cd transaction-service +mvn spring-boot:run +``` + +Runs on `http://localhost:8080` + +### 3. Run anti-fraud-service + +```bash +cd anti-fraud-service +mvn spring-boot:run +``` + +Runs as a Kafka worker (no HTTP server). + +## API Documentation + +Swagger UI available at: `http://localhost:8080/` + +### Endpoints + +**POST** `/transactions` — Create a new transaction ```json { - "transactionExternalId": "Guid", - "transactionType": { - "name": "" - }, - "transactionStatus": { - "name": "" - }, - "value": 120, - "createdAt": "Date" + "accountExternalIdDebit": "uuid", + "accountExternalIdCredit": "uuid", + "transferTypeId": 1, + "value": 500 } ``` -## Optional - -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? - -You can use Graphql; +**GET** `/transactions/{id}` — Get transaction by external ID -# Send us your challenge +Returns transaction with current status (PENDING, APPROVED, or REJECTED). -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. +## Kafka Topics -If you have any questions, please let us know. +| Topic | Producer | Consumer | Description | +|---|---|---|---| +| `transaction.created` | transaction-service | anti-fraud-service | New transaction event | +| `transaction.status` | anti-fraud-service | transaction-service | Fraud validation result | \ No newline at end of file diff --git a/anti-fraud-service/src/main/java/com/yape/antifraudservice/consumers/TransactionCreatedConsumer.java b/anti-fraud-service/src/main/java/com/yape/antifraudservice/consumers/TransactionCreatedConsumer.java index b52664ccae..87f06c2027 100644 --- a/anti-fraud-service/src/main/java/com/yape/antifraudservice/consumers/TransactionCreatedConsumer.java +++ b/anti-fraud-service/src/main/java/com/yape/antifraudservice/consumers/TransactionCreatedConsumer.java @@ -18,7 +18,7 @@ public TransactionCreatedConsumer(KafkaTemplate kafkaT @KafkaListener(topics = "transaction.created") public void onTransactionCreated(TransactionEvent transaction) { - TransactionStatus status = transaction.value().compareTo(new BigDecimal("1000")) > 0 + TransactionStatus status = transaction.value().compareTo(BigDecimal.valueOf(1000)) > 0 ? TransactionStatus.REJECTED : TransactionStatus.APPROVED; diff --git a/transaction-service/pom.xml b/transaction-service/pom.xml index 01bcee8a8c..dd88c69d2c 100644 --- a/transaction-service/pom.xml +++ b/transaction-service/pom.xml @@ -30,6 +30,11 @@ 21 + + org.springdoc + springdoc-openapi-starter-webmvc-ui + 2.8.6 + org.mapstruct mapstruct diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetByIdTransactionUseCase.java b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetByIdTransactionUseCase.java index 8df9a374de..eaa00d562e 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetByIdTransactionUseCase.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/abstractions/GetByIdTransactionUseCase.java @@ -7,5 +7,5 @@ public interface GetByIdTransactionUseCase { - Optional execute(GetByIdTransactionRequest request); + GetByIdTransactionResponse execute(GetByIdTransactionRequest request); } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionRequest.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionRequest.java index 8158f23dc9..c8fb985a69 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionRequest.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/createtransaction/CreateTransactionRequest.java @@ -1,5 +1,7 @@ package com.yape.transactionservice.application.usecases.createtransaction; +import jakarta.validation.constraints.Max; +import jakarta.validation.constraints.Min; import jakarta.validation.constraints.NotNull; import jakarta.validation.constraints.Positive; @@ -10,7 +12,9 @@ public record CreateTransactionRequest(@NotNull(message = "accountExternalIdDebi UUID accountExternalIdDebit, @NotNull(message = "accountExternalIdCredit is required") UUID accountExternalIdCredit, - @NotNull(message = "tranferTypeId is required") + @NotNull(message = "transferTypeId is required") + @Min(value = 1, message = "transferTypeId must be 1 or 2") + @Max(value = 2, message = "transferTypeId must be 1 or 2") Integer transferTypeId, @NotNull(message = "value is required") @Positive(message = "value must be greater than 0") @@ -27,3 +31,4 @@ public record CreateTransactionRequest(@NotNull(message = "accountExternalIdDebi + diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionHandler.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionHandler.java index c5f5b7b432..5ffbcde084 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionHandler.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionHandler.java @@ -1,8 +1,12 @@ package com.yape.transactionservice.application.usecases.getbyidtransaction; import com.yape.transactionservice.application.abstractions.GetByIdTransactionUseCase; +import com.yape.transactionservice.domain.exceptions.TransactionNotFoundException; +import com.yape.transactionservice.domain.models.Transaction; import com.yape.transactionservice.domain.repostories.TransactionRepository; +import org.springframework.http.HttpStatus; import org.springframework.stereotype.Service; +import org.springframework.web.server.ResponseStatusException; import java.util.Optional; @@ -17,8 +21,9 @@ public GetByIdTransactionHandler(GetByIdTransactionMapper mapper, TransactionRep } @Override - public Optional execute(GetByIdTransactionRequest request) { - var transaction = repository.getById(request.id()); - return transaction.map(mapper::toDTO); + public GetByIdTransactionResponse execute(GetByIdTransactionRequest request) { + Transaction transaction = repository.getById(request.id()) + .orElseThrow(() -> new TransactionNotFoundException(request.id())); + return mapper.toDTO(transaction); } } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionMapper.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionMapper.java index c35c1feaa0..1869c25625 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionMapper.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionMapper.java @@ -7,8 +7,8 @@ @Mapper(componentModel = "spring", imports = {TransferType.class}) public interface GetByIdTransactionMapper { - @Mapping(target = "transactionType", expression = "java(new GetTransactionResponse.TransactionTypeDto(TransferType.getNameById(transaction.getTransferTypeId())))") - @Mapping(target = "transactionStatus", expression = "java(new GetTransactionResponse.TransactionStatusDto(transaction.getStatus().name()))") + @Mapping(target = "transactionType", expression = "java(new GetByIdTransactionResponse.TransactionTypeDto(TransferType.getNameById(transaction.getTransferTypeId())))") + @Mapping(target = "transactionStatus", expression = "java(new GetByIdTransactionResponse.TransactionStatusDto(transaction.getStatus().name()))") GetByIdTransactionResponse toDTO(Transaction transaction); } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionResponse.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionResponse.java index c38e12206f..0dcc08046d 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionResponse.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/getbyidtransaction/GetByIdTransactionResponse.java @@ -8,8 +8,8 @@ public record GetByIdTransactionResponse(UUID transactionExternalId, UUID accountExternalIdDebit, UUID accountExternalIdCredit, - GetTransactionResponse.TransactionTypeDto transactionType, - GetTransactionResponse.TransactionStatusDto transactionStatus, + TransactionTypeDto transactionType, + TransactionStatusDto transactionStatus, BigDecimal value) { public record TransactionTypeDto(String name) { } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionRequest.java b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionRequest.java index ff0332956c..50157ecf7b 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionRequest.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/application/usecases/gettransaction/GetTransactionRequest.java @@ -1,4 +1,4 @@ package com.yape.transactionservice.application.usecases.gettransaction; -public class GetTransactionRequest { +public record GetTransactionRequest(){ } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/exceptions/DomainException.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/exceptions/DomainException.java new file mode 100644 index 0000000000..cb7b0159fd --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/exceptions/DomainException.java @@ -0,0 +1,7 @@ +package com.yape.transactionservice.domain.exceptions; + +public class DomainException extends RuntimeException { + public DomainException(String message) { + super(message); + } +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/exceptions/TransactionNotFoundException.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/exceptions/TransactionNotFoundException.java new file mode 100644 index 0000000000..024c6cb007 --- /dev/null +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/exceptions/TransactionNotFoundException.java @@ -0,0 +1,9 @@ +package com.yape.transactionservice.domain.exceptions; + +import java.util.UUID; + +public class TransactionNotFoundException extends RuntimeException { + public TransactionNotFoundException(UUID id) { + super("Transaction not found with id: " + id); + } +} diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java index 2e85d5465b..5b37c5fbb5 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java @@ -1,6 +1,7 @@ package com.yape.transactionservice.domain.models; import com.yape.transactionservice.domain.enums.TransactionStatus; +import com.yape.transactionservice.domain.exceptions.DomainException; import lombok.AllArgsConstructor; import lombok.Builder; import lombok.Getter; @@ -23,10 +24,26 @@ public class Transaction { private TransactionStatus status; private LocalDateTime createdAt; - public static Transaction Create(UUID idDebit, UUID idCredit, Integer transferTypeId, BigDecimal value) { + public static Transaction Create(UUID debitId, UUID creditId, Integer transferTypeId, BigDecimal value) { + + if (value.compareTo(BigDecimal.ZERO) <= 0) { + throw new DomainException("Value must be greater than zero"); + } + + if (debitId == null || debitId.equals(new UUID(0L,0L))) { + throw new DomainException("credit id can't be empty "); + } + + if (creditId == null || creditId.equals(new UUID(0L,0L))) { + throw new DomainException("credit id can't be empty "); + } + if (transferTypeId.equals(0)) { + throw new DomainException("transferTypeId must be greater than zero"); + } + return Transaction.builder().transactionExternalId(UUID.randomUUID()) - .accountExternalIdCredit(idCredit) - .accountExternalIdDebit(idDebit) + .accountExternalIdCredit(creditId) + .accountExternalIdDebit(debitId) .transferTypeId(transferTypeId) .value(value) .status(TransactionStatus.PENDING) diff --git a/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/HealthController.java b/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/HealthController.java index 6b9a4e66f6..da5216e1a1 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/HealthController.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/HealthController.java @@ -13,10 +13,4 @@ public class HealthController { public String health() { return "Transaction Service is running!"; } - - @GetMapping("/test") - public String test() { - throw new ResponseStatusException(HttpStatus.NOT_FOUND, "NO VALIDO"); - - } } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/TransactionController.java b/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/TransactionController.java index a87748d1c9..07997b4cd7 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/TransactionController.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/presentation/controllers/TransactionController.java @@ -44,7 +44,7 @@ public List getAll() { } @GetMapping("/{id}") - public Optional getById(@PathVariable UUID id) { + public GetByIdTransactionResponse getById(@PathVariable UUID id) { return getByIdTransactionUseCase.execute(new GetByIdTransactionRequest(id)); } } diff --git a/transaction-service/src/main/java/com/yape/transactionservice/presentation/exceptionhandlers/GlobalExceptionHandler.java b/transaction-service/src/main/java/com/yape/transactionservice/presentation/exceptionhandlers/GlobalExceptionHandler.java index 6a16db9c63..239e362f47 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/presentation/exceptionhandlers/GlobalExceptionHandler.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/presentation/exceptionhandlers/GlobalExceptionHandler.java @@ -1,8 +1,11 @@ package com.yape.transactionservice.presentation.exceptionhandlers; +import com.yape.transactionservice.domain.exceptions.DomainException; +import com.yape.transactionservice.domain.exceptions.TransactionNotFoundException; import org.springframework.core.Ordered; import org.springframework.core.annotation.Order; import org.springframework.http.HttpStatus; +import org.springframework.http.HttpStatusCode; import org.springframework.http.ProblemDetail; import org.springframework.http.ResponseEntity; import org.springframework.http.converter.HttpMessageNotReadableException; @@ -38,4 +41,22 @@ public ResponseEntity> handleInvalidFormat(HttpMessageNotRea return ResponseEntity.badRequest() .body(Map.of("error", "El JSON tiene un tipo de dato inválido")); } + + @ExceptionHandler(TransactionNotFoundException.class) + public ProblemDetail handleNotFound(TransactionNotFoundException ex) { + ProblemDetail problem = ProblemDetail.forStatus(HttpStatus.NOT_FOUND); + problem.setTitle("Resource not found"); + problem.setDetail(ex.getMessage()); + + return problem; + } + @ExceptionHandler(DomainException.class) + public ProblemDetail handleDomain(DomainException ex) { + ProblemDetail problem = ProblemDetail.forStatus(HttpStatusCode.valueOf(422)); + problem.setTitle("Unprocessable Entity"); + problem.setDetail(ex.getMessage()); + + return problem; + } + } diff --git a/transaction-service/src/main/resources/application.properties b/transaction-service/src/main/resources/application.properties index 5f1870ca60..f38629e120 100644 --- a/transaction-service/src/main/resources/application.properties +++ b/transaction-service/src/main/resources/application.properties @@ -23,4 +23,5 @@ spring.kafka.consumer.properties.spring.json.value.default.type=com.yape.transac spring.kafka.consumer.properties.spring.json.use.type.headers=false #problem spring.mvc.problemdetails.enabled=true +springdoc.swagger-ui.path=/ From b7454e3920671884c8df5845674d8f301a65df6a Mon Sep 17 00:00:00 2001 From: Ivan Perez Date: Sun, 1 Mar 2026 11:06:32 -0500 Subject: [PATCH 7/7] feat: unit test transaction model --- .../domain/models/Transaction.java | 14 ++-- .../transactionservice/TransactionTest.java | 79 +++++++++++++++++++ 2 files changed, 85 insertions(+), 8 deletions(-) create mode 100644 transaction-service/src/test/java/com/yape/transactionservice/TransactionTest.java diff --git a/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java b/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java index 5b37c5fbb5..94a9f3e5ba 100644 --- a/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java +++ b/transaction-service/src/main/java/com/yape/transactionservice/domain/models/Transaction.java @@ -52,15 +52,13 @@ public static Transaction Create(UUID debitId, UUID creditId, Integer transferTy } - public void approve() { - this.status = TransactionStatus.APPROVED; - } - - public void reject() { - this.status = TransactionStatus.REJECTED; - } - public void changeStatus(TransactionStatus status) { + if(status == null) { + throw new DomainException("status can't be null"); + } + if (this.status != TransactionStatus.PENDING) { + throw new DomainException("You cannot change a state that is not pending"); + } this.status = status; } } diff --git a/transaction-service/src/test/java/com/yape/transactionservice/TransactionTest.java b/transaction-service/src/test/java/com/yape/transactionservice/TransactionTest.java new file mode 100644 index 0000000000..baf32de34c --- /dev/null +++ b/transaction-service/src/test/java/com/yape/transactionservice/TransactionTest.java @@ -0,0 +1,79 @@ +package com.yape.transactionservice; + +import com.yape.transactionservice.domain.enums.TransactionStatus; +import com.yape.transactionservice.domain.exceptions.DomainException; +import com.yape.transactionservice.domain.models.Transaction; +import org.junit.jupiter.api.Test; + +import java.math.BigDecimal; +import java.util.UUID; + +import static org.junit.jupiter.api.Assertions.*; + +public class TransactionTest { + + @Test + void shouldCreateTransactionWithPendingStatus() { + Transaction transaction = Transaction.Create( + UUID.randomUUID(), + UUID.randomUUID(), + 1, + new BigDecimal("500") + ); + + assertNotNull(transaction.getTransactionExternalId()); + assertEquals(TransactionStatus.PENDING, transaction.getStatus()); + assertEquals(new BigDecimal("500"), transaction.getValue()); + assertNotNull(transaction.getCreatedAt()); + } + + @Test + void shouldThrowExceptionWhenValueIsZeroOrNegative() { + assertThrows(DomainException.class, () -> + Transaction.Create(UUID.randomUUID(), UUID.randomUUID(), 1, BigDecimal.ZERO) + ); + + assertThrows(DomainException.class, () -> + Transaction.Create(UUID.randomUUID(), UUID.randomUUID(), 1, new BigDecimal("-100")) + ); + } + + @Test + void shouldThrowExceptionWhenDebitIdIsNull() { + assertThrows(DomainException.class, () -> + Transaction.Create(null, UUID.randomUUID(), 1, new BigDecimal("500")) + ); + } + + @Test + void shouldThrowExceptionWhenDebitIdIsEmpty() { + assertThrows(DomainException.class, () -> + Transaction.Create(new UUID(0L, 0L), UUID.randomUUID(), 1, new BigDecimal("500")) + ); + } + + @Test + void shouldThrowExceptionWhenCreditIdIsNull() { + assertThrows(DomainException.class, () -> + Transaction.Create(UUID.randomUUID(), null, 1, new BigDecimal("500")) + ); + } + + @Test + void shouldThrowExceptionWhenTransferTypeIdIsZero() { + assertThrows(DomainException.class, () -> + Transaction.Create(UUID.randomUUID(), UUID.randomUUID(), 0, new BigDecimal("500")) + ); + } + + @Test + void shouldChangeStatus() { + Transaction transaction = Transaction.Create( + UUID.randomUUID(), UUID.randomUUID(), 1, new BigDecimal("500") + ); + + transaction.changeStatus(TransactionStatus.APPROVED); + + assertEquals(TransactionStatus.APPROVED, transaction.getStatus()); + } +}