Bug 15221: (QA follow-up) Fix Message.t
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Mon, 27 May 2019 09:39:34 +0000 (09:39 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Thu, 30 May 2019 19:55:38 +0000 (20:55 +0100)
As Jonathan reported on comment11, SIP/Message.t fails now.

This is a trivial fix for that. In the first case we should just no longer
expect the alert flag. In the second case it makes more sense to toggle
the value of the checked_in_ok setting.

Test plan:
Run t/db_dependent/SIP/Message.t

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit 533ff7ab29a1ca6d35d0e306dd97273838ad3064)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

t/db_dependent/SIP/Message.t

index f837fb4..0ee1a39 100755 (executable)
@@ -358,7 +358,7 @@ sub test_checkin_v2 {
     $msg = C4::SIP::Sip::MsgType->new( $siprequest, 0 );
     $msg->handle_checkin( $server );
     is( substr($response,2,1), '1', 'OK flag is true now with checked_in_ok flag set when checking in an item that was not checked out' );
-    is( substr($response,5,1), 'Y', 'Alert flag is set' );
+    is( substr($response,5,1), 'N', 'Alert flag no longer set' );
     check_field( $respcode, $response, FID_SCREEN_MSG, undef, 'No screen msg' );
     $server->{account}->{checked_in_ok} = 0;