release notes for Koha 3.16.0-beta

[koha_fer] / t / Budgets / CanUserUseBudget.t

diff --git a/t/Budgets/CanUserUseBudget.t b/t/Budgets/CanUserUseBudget.t
index 8219197..ca94065 100644 (file)
--- a/t/Budgets/CanUserUseBudget.t
+++ b/t/Budgets/CanUserUseBudget.t
@@ -182,12 +182,12 @@ ok (CanUserUseBudget($borrower2, $budget11, $flags));
 ok (CanUserUseBudget($borrower2, $budget12, $flags));
 
 # Restriction is 'owner and users'
-ok (CanUserUseBudget($borrower1, $budget13, $flags));
+ok (!CanUserUseBudget($borrower1, $budget13, $flags)); # No user, no owner
 ok (CanUserUseBudget($borrower1, $budget14, $flags));
 ok (CanUserUseBudget($borrower1, $budget15, $flags));
 ok (CanUserUseBudget($borrower1, $budget16, $flags));
-ok (CanUserUseBudget($borrower2, $budget13, $flags));
-ok (CanUserUseBudget($borrower2, $budget14, $flags));
+ok (!CanUserUseBudget($borrower2, $budget13, $flags)); # No user, no owner
+ok (!CanUserUseBudget($borrower2, $budget14, $flags)); # No owner and user list contains borrower1
 ok (CanUserUseBudget($borrower2, $budget15, $flags));
 ok (!CanUserUseBudget($borrower2, $budget16, $flags));
 
@@ -220,17 +220,17 @@ ok (CanUserUseBudget($borrower1, $budget10, $flags));
 ok (CanUserUseBudget($borrower1, $budget11, $flags));
 ok (CanUserUseBudget($borrower1, $budget12, $flags));
 ok (CanUserUseBudget($borrower2, $budget9, $flags));
-ok (CanUserUseBudget($borrower2, $budget10, $flags));
+ok (!CanUserUseBudget($borrower2, $budget10, $flags)); # Limited to library B1
 ok (CanUserUseBudget($borrower2, $budget11, $flags));
 ok (!CanUserUseBudget($borrower2, $budget12, $flags));
 
 # Restriction is 'owner and users'
-ok (CanUserUseBudget($borrower1, $budget13, $flags));
+ok (!CanUserUseBudget($borrower1, $budget13, $flags)); # No owner, no user
 ok (CanUserUseBudget($borrower1, $budget14, $flags));
 ok (CanUserUseBudget($borrower1, $budget15, $flags));
 ok (CanUserUseBudget($borrower1, $budget16, $flags));
-ok (CanUserUseBudget($borrower2, $budget13, $flags));
-ok (CanUserUseBudget($borrower2, $budget14, $flags));
+ok (!CanUserUseBudget($borrower2, $budget13, $flags)); # No owner, no user
+ok (!CanUserUseBudget($borrower2, $budget14, $flags)); # No owner and user list contains borrower1
 ok (CanUserUseBudget($borrower2, $budget15, $flags));
 ok (!CanUserUseBudget($borrower2, $budget16, $flags));