From fe0ae583c1aa8b7734f1eabfa13c0b4065d13c6e Mon Sep 17 00:00:00 2001
From: Philip Riecks <mail@philipriecks.de>
Date: Fri, 14 Feb 2025 08:24:12 +0100
Subject: [PATCH] add PIT

---
 .../de/rieckpil/blog/pit/FraudDetector.java   |  8 +++----
 .../rieckpil/blog/pit/FraudDetectorTests.java | 23 +++++++++++--------
 2 files changed, 17 insertions(+), 14 deletions(-)

diff --git a/spring-boot-example/src/main/java/de/rieckpil/blog/pit/FraudDetector.java b/spring-boot-example/src/main/java/de/rieckpil/blog/pit/FraudDetector.java
index 5337e2a..49bded5 100644
--- a/spring-boot-example/src/main/java/de/rieckpil/blog/pit/FraudDetector.java
+++ b/spring-boot-example/src/main/java/de/rieckpil/blog/pit/FraudDetector.java
@@ -8,13 +8,13 @@ public class FraudDetector {
     Set.of("WONDERLAND", "MARS", "URANUS");
 
   public boolean isFraudulentTransaction(double amount, String country) {
-    // Any transaction above 10k is flagged
-    if (amount > 10_000) {
+    // Transactions from high-risk countries are flagged
+    if (HIGH_RISK_COUNTRIES.contains(country)) {
       return true;
     }
 
-    // Transactions from high-risk countries are flagged
-    if (HIGH_RISK_COUNTRIES.contains(country)) {
+    // Any transaction above 10k is flagged
+    if (amount > 10_000) {
       return true;
     }
 
diff --git a/spring-boot-example/src/test/java/de/rieckpil/blog/pit/FraudDetectorTests.java b/spring-boot-example/src/test/java/de/rieckpil/blog/pit/FraudDetectorTests.java
index f2bc08f..24c937b 100644
--- a/spring-boot-example/src/test/java/de/rieckpil/blog/pit/FraudDetectorTests.java
+++ b/spring-boot-example/src/test/java/de/rieckpil/blog/pit/FraudDetectorTests.java
@@ -13,23 +13,26 @@
 class FraudDetectorTest {
 
   @Test
-  void testHighAmountTransaction() {
+  void highAmountTransactionShouldBeFraudulent() {
     FraudDetector detector = new FraudDetector();
-
     assertTrue(detector.isFraudulentTransaction(15_000, "USA"));
   }
 
   @Test
-  void testHighRiskCountryTransaction() {
+  void highRiskCountryTransactionShouldBeFraudulent() {
     FraudDetector detector = new FraudDetector();
-
     assertTrue(detector.isFraudulentTransaction(500, "MARS"));
   }
 
-  @Test
-  void testSafeTransaction() {
-    FraudDetector cut = new FraudDetector();
-
-    assertFalse(cut.isFraudulentTransaction(200, "USA"));
-  }
+//  @Test
+//  void lowAmountTransactionShouldNotBeFraudulent() {
+//    FraudDetector detector = new FraudDetector();
+//    assertFalse(detector.isFraudulentTransaction(1_000, "USA"));
+//  }
+//
+//  @Test
+//  void thresholdAmountTransactionShouldNotBeFraudulent() {
+//    FraudDetector detector = new FraudDetector();
+//    assertFalse(detector.isFraudulentTransaction(10_000, "GER"));
+//  }
 }