From a3ea7a58dfcae6d3d07da30b3ddb1885f682b6f3 Mon Sep 17 00:00:00 2001 From: "Bradley M. Kuhn" Date: Sun, 26 Nov 2017 14:30:44 -0800 Subject: [PATCH] Tests: clearly should be ok(), not is(). This was a Boolean test, so it clearly should be ok(), not an is(). --- Supporters/t/Supporters.t | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Supporters/t/Supporters.t b/Supporters/t/Supporters.t index 4b35714..78899da 100644 --- a/Supporters/t/Supporters.t +++ b/Supporters/t/Supporters.t @@ -772,7 +772,7 @@ lives_ok { $newFRID = $sp->fulfillRequest( { donorId => $drapperId, how => "mailed" }); } "fulfillRequest: succeeds once the request is no longer on hold..."; -is( (defined $newFRID and looks_like_number($newFRID) and $newFRID > 0 and ($newFRID != $fulfillRequestId)), +ok( (defined $newFRID and looks_like_number($newFRID) and $newFRID > 0 and ($newFRID != $fulfillRequestId)), "....id returned on successful fulfillRequest() is a number and is not the one returned by previous"); lives_ok { $tt = $sp->getRequest({donorId => $drapperId, requestTypeId => $tShirt0RequestTypeId } ); }