| 
      
            Lines 1640-1646
          subtest "Bug 13841 - Do not create new 0 amount fines" => sub {
      
      
        Link Here
      
     | 
  
        
          | 1640 | 
          };  | 
          1640 | 
          };  | 
        
        
          | 1641 | 
           | 
          1641 | 
           | 
        
        
          | 1642 | 
          subtest "AllowRenewalIfOtherItemsAvailable tests" => sub { | 
          1642 | 
          subtest "AllowRenewalIfOtherItemsAvailable tests" => sub { | 
        
          
            
              | 1643 | 
                  plan tests => 13;  | 
              1643 | 
                  plan tests => 12;  | 
            
        
          | 1644 | 
              my $biblio = $builder->build_sample_biblio();  | 
          1644 | 
              my $biblio = $builder->build_sample_biblio();  | 
        
        
          | 1645 | 
              my $item_1 = $builder->build_sample_item(  | 
          1645 | 
              my $item_1 = $builder->build_sample_item(  | 
        
        
          | 1646 | 
                  { | 
          1646 | 
                  { | 
        
  
    | 
      
            Lines 1816-1822
          subtest "AllowRenewalIfOtherItemsAvailable tests" => sub {
      
      
        Link Here
      
     | 
  
        
          | 1816 | 
           | 
          1816 | 
           | 
        
        
          | 1817 | 
              my $mock_circ = Test::MockModule->new("C4::Circulation"); | 
          1817 | 
              my $mock_circ = Test::MockModule->new("C4::Circulation"); | 
        
        
          | 1818 | 
              $mock_circ->mock( CanItemBeReserved => sub { | 
          1818 | 
              $mock_circ->mock( CanItemBeReserved => sub { | 
        
            
              | 1819 | 
                      warn "Checked";  | 
               | 
               | 
            
        
          | 1820 | 
                  return { status => 'no' } | 
          1819 | 
                  return { status => 'no' } | 
        
        
          | 1821 | 
              } );  | 
          1820 | 
              } );  | 
        
        
          | 1822 | 
           | 
          1821 | 
           | 
        
  
    | 
      
            Lines 1824-1832
          subtest "AllowRenewalIfOtherItemsAvailable tests" => sub {
      
      
        Link Here
      
     | 
  
        
          | 1824 | 
              $item_3->delete();  | 
          1823 | 
              $item_3->delete();  | 
        
        
          | 1825 | 
              # Two items total, one item available, one issued, two holds on record  | 
          1824 | 
              # Two items total, one item available, one issued, two holds on record  | 
        
        
          | 1826 | 
           | 
          1825 | 
           | 
        
          
            
              | 1827 | 
                  warnings_are{ | 
              1826 | 
                  ( $renewokay, $error ) = CanBookBeRenewed( $borrowernumber1, $item_1->itemnumber );  | 
            
            
              | 1828 | 
                     ( $renewokay, $error ) = CanBookBeRenewed( $borrowernumber1, $item_1->itemnumber );  | 
               | 
               | 
            
            
              | 1829 | 
                  } [], "CanItemBeReserved not called when there are more possible holds than available items";  | 
            
        
          | 1830 | 
              is( $renewokay, 0, 'Borrower cannot renew when there are more holds than available items' );  | 
          1827 | 
              is( $renewokay, 0, 'Borrower cannot renew when there are more holds than available items' );  | 
        
        
          | 1831 | 
           | 
          1828 | 
           | 
        
        
          | 1832 | 
              $item_3 = $builder->build_sample_item(  | 
          1829 | 
              $item_3 = $builder->build_sample_item(  | 
        
  
    | 
      
            Lines 1848-1853
          subtest "AllowRenewalIfOtherItemsAvailable tests" => sub {
      
      
        Link Here
      
     | 
  
        
          | 1848 | 
                  }  | 
          1845 | 
                  }  | 
        
        
          | 1849 | 
              );  | 
          1846 | 
              );  | 
        
        
          | 1850 | 
           | 
          1847 | 
           | 
        
            
               | 
               | 
              1848 | 
                  $mock_circ->mock( CanItemBeReserved => sub { | 
            
            
              | 1849 | 
                      warn "Checked";  | 
            
            
              | 1850 | 
                      return { status => 'no' } | 
            
            
              | 1851 | 
                  } );  | 
            
            
              | 1852 | 
               | 
            
        
          | 1851 | 
              warnings_are{ | 
          1853 | 
              warnings_are{ | 
        
        
          | 1852 | 
                 ( $renewokay, $error ) = CanBookBeRenewed( $borrowernumber1, $item_1->itemnumber );  | 
          1854 | 
                 ( $renewokay, $error ) = CanBookBeRenewed( $borrowernumber1, $item_1->itemnumber );  | 
        
        
          | 1853 | 
              } ["Checked","Checked"], "CanItemBeReserved only called once per available item if it returns a negative result for all items for a borrower";  | 
          1855 | 
              } ["Checked","Checked"], "CanItemBeReserved only called once per available item if it returns a negative result for all items for a borrower";  | 
        
            
              | 1854 | 
              -   | 
               | 
               |