https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23964 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #11 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- 725 my $hold_1 = place_item_hold( $patron, $item, $library, $priority ); 726 $hold = place_item_hold( $patron_2, $item, $library, $priority ); 727 is( $hold->priority, 1, 'If ReservesNeedReturns is 0 but item already on hold priority must be set to 1' ); What about $hold_1->priority? Should not we have $hold_1->priority == 1 instead of $hold->priority == 1? 737 isnt( $hold->priority, 0, 'If ReservesNeedReturns is 0 but item in transit the hold must not be set to waiting' ); Test's description does not match the test. -- You are receiving this mail because: You are watching all bug changes.