From 90374a81e8df17415a53d09be4fc9cef83a32a3c Mon Sep 17 00:00:00 2001 From: record-bot Date: Wed, 22 Apr 2026 12:11:43 -0700 Subject: [PATCH 1/2] record/test_fail: intro failure --- src/main/java/com/phalanx/calc/MathOps.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/main/java/com/phalanx/calc/MathOps.java b/src/main/java/com/phalanx/calc/MathOps.java index 16d072d..32081f8 100644 --- a/src/main/java/com/phalanx/calc/MathOps.java +++ b/src/main/java/com/phalanx/calc/MathOps.java @@ -20,7 +20,8 @@ public static double subtract(double a, double b) { } public static double multiply(double a, double b) { - return a * b; + // BUG (intentional testbed failure): multiply is using add's operator. + return a + b; } public static double divide(double a, double b) { From b23a179d3d4e955906ea13623fc3c4f97a0a7a8d Mon Sep 17 00:00:00 2001 From: Phalanx Date: Wed, 22 Apr 2026 19:18:52 +0000 Subject: [PATCH 2/2] Fix multiply implementation causing failing tests Restore MathOps.multiply to use multiplication instead of addition. This matches the failing JUnit assertions in MathOpsTest and makes `mvn -B verify` pass again, including the coverage gate. --- src/main/java/com/phalanx/calc/MathOps.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/main/java/com/phalanx/calc/MathOps.java b/src/main/java/com/phalanx/calc/MathOps.java index 32081f8..16d072d 100644 --- a/src/main/java/com/phalanx/calc/MathOps.java +++ b/src/main/java/com/phalanx/calc/MathOps.java @@ -20,8 +20,7 @@ public static double subtract(double a, double b) { } public static double multiply(double a, double b) { - // BUG (intentional testbed failure): multiply is using add's operator. - return a + b; + return a * b; } public static double divide(double a, double b) {