From 003af81ce72a1e5f21b3b59b869f18bded7ee791 Mon Sep 17 00:00:00 2001 From: "Bradley M. Kuhn" Date: Thu, 7 Jan 2016 13:06:40 -0800 Subject: [PATCH] donorTotalGaveInPeriod: improve tests. Actually, the amount on that one test was wrong, and I think we actually do want to test the border cases a bit better. --- Supporters/t/Supporters.t | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Supporters/t/Supporters.t b/Supporters/t/Supporters.t index c93598b..25b04cf 100644 --- a/Supporters/t/Supporters.t +++ b/Supporters/t/Supporters.t @@ -852,14 +852,14 @@ lives_ok { $amount = $sp->donorTotalGaveInPeriod(donorId => $olsonId, startDate endDate => '2015-06-29') } "donorTotalGaveInPeriod(): check for total with both start and end date succeeds..."; -is($amount, 30.00, "donorTotalGaveInPeriod(): ...and returned value is correct. "); +is($amount, 20.00, "donorTotalGaveInPeriod(): ...and returned value is correct. "); lives_ok { $amount = $sp->donorTotalGaveInPeriod(donorId => $harrisId, startDate => '2015-12-04'); } "donorTotalGaveInPeriod(): check for total with just a start date succeeds..."; is($amount, 120.00, "donorTotalGaveInPeriod(): ...and returned value is correct. "); -lives_ok { $amount = $sp->donorTotalGaveInPeriod(donorId => $olsonId, endDate => '2015-02-17'); } +lives_ok { $amount = $sp->donorTotalGaveInPeriod(donorId => $olsonId, endDate => '2015-02-16'); } "donorTotalGaveInPeriod(): check for total with just a end date succeeds..."; is($amount, 10.00, "donorTotalGaveInPeriod(): ...and returned value is correct. ");